Results 20 comments of BlackWasp

Hum, effectively it makes sense. It is the first time I script something with the Impacket's libraries, so I don't know all the specifies :smile: Thanks for the explain on...

> Just tested, it's working like a charm +1 > > ![2022-03-29_07-56](https://user-images.githubusercontent.com/5891788/160610620-cfdfa058-9314-4a09-b1dc-21d86a841822.png) Thanks for the feedback and the test !

> Great PR! For the password generation, imho it would be nice to implement what was done in #1063 where the password can be defined by the user, or randomly...

Two updates: - It is now possible to specify a computer name and its password, or just the computer name, or nothing and everything will be randomly generated. Also, the...

> Literally just ran into this on a gig, domain controllers without signing required! Awesome work @BlWasp Thanks for the feedback !

> Hi @BlWasp! Thanks for the PR. I left you a comment/change request we need to address before merging. Hi @0xdeaddood ! Thanks for the review. It is my first...

The script works perfectly in my lab, even in the scenario "from A to B then B to A". I haven't seen any issue for the moment: ```bash > owneredit.py...

> > The script works perfectly in my lab, even in the scenario "from A to B then B to A". I haven't seen any issue for the moment: >...

After debug and research I have finally found something that looks to be the point. According to the [Microsoft doc](https://docs.microsoft.com/en-us/windows/win32/api/iads/ne-iads-ads_rights_enum), the WriteOwner right **doesn't allow** to arbitrary change the owner...

Hi everyone. I have made some changes on this PR :) First I have resolved the different conflicts. Then I have remove the created function `connectSamr2()` from `secretsdump.py` and now...