garrettfoster13 / aced Goto Github PK

Given these two commands, the below command will fail:

python ./aced.py -hashes 'LM:NTLM' -dc-ip dc.domain.tld 'domain.tld/computer$'

The below command will succeed:

python ./aced.py -hashes 'LM:NTLM' 'domain.tld/computer$'@dc.domain.tld

This is due to the target address being required and there is no logic if the dc-ip argument is used instead. I'm requesting either method being supported to align with official Impacket examples.

[proxychains] Strict chain ... 127.0.0.1:9052 ... 10.9.15.10:389 ... OK Enter target sAMAccountName or distinguishedName: administrator Traceback (most recent call last): File "/home/test/Desktop/aced/aced.py", line 4, in <module> lib.main() File "/home/test/Desktop/aced/lib/main.py", line 791, in main ldapsearch=magic(ldap_server, ldap_session, domain, logs_dir) File "/home/test/Desktop/aced/lib/main.py", line 390, in __init__ self.WRITE_SPN = self.resolve_key("(cn=Service-Principal-Name)") File "/home/test/Desktop/aced/lib/main.py", line 494, in resolve_key return guid UnboundLocalError: local variable 'guid' referenced before assignment

This occurs when it is not able to determine both the 1) domain and 2) user. By initializing the guid variable in the beginning of the function, it would return a none value for both domain and user.