Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Computer Before Delegation #1690

Merged
merged 1 commit into from
Feb 8, 2024
Merged

Add Computer Before Delegation #1690

merged 1 commit into from
Feb 8, 2024

Conversation

gjhami
Copy link
Contributor

@gjhami gjhami commented Jan 24, 2024

Reorder attack components so a computer account is created, if specified, before the escalation through resource-based constrained delegation attack is attempted.

This allows users to create a relay and specify both --add-computer and --delegate-access --escalate-user to create a computer account and delegate access to the account in a single relay. Currently, if this is attempted, the delegation attack is performed first and it results in an error that the computer to delegate to does not exist.

What I'd like to be able to do is add a computer, escalate access, and setup a DNS record all in a single relay. For example

impacket-ntlmrelayx -t ldaps://<DC.DOMAIN.TLD> --add-computer '<COMPUTER NAME>$' '<PASSWORD>' --add-dns-record '<COMPUTER NAME>' '<COMPUTER IP>' --delegate-access --escalate-user '<COMPUTER NAME>$'

Reorder attack components so a computer account is created before the delegate attack is attempted.
@anadrianmanrique anadrianmanrique added the bug Unexpected problem or unintended behavior label Feb 1, 2024
@anadrianmanrique anadrianmanrique self-assigned this Feb 6, 2024
@anadrianmanrique anadrianmanrique merged commit 63438ae into fortra:master Feb 8, 2024
9 checks passed
anadrianmanrique added a commit that referenced this pull request Feb 22, 2024
@anadrianmanrique
Copy link
Contributor

Thanks for the PR. A bug was fixed 337d50d, after merging this changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Unexpected problem or unintended behavior
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants