Add extra SID presence checks (GPO case)

aniqfakhrul / powerview.py

Just another Powerview alternative

MIT License

427 stars 47 forks source link

Add extra SID presence checks (GPO case) #69

Closed snovvcrash closed 9 months ago

snovvcrash commented 9 months ago

Hey @aniqfakhrul!

Spotted a bug when trying to remove an object ACL in case of -TargetIdentity being a GPO (SID value is absent), so there're a few extra SID presence checks in this PR.

aniqfakhrul commented 9 months ago

I have verified this on my end. Thanks for the PR, appreciate it!