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

Implement attach for Windows #4626

Closed
wants to merge 3 commits into from
Closed

Implement attach for Windows #4626

wants to merge 3 commits into from

Conversation

raefko
Copy link

@raefko raefko commented Dec 16, 2020

Hello!

I started working on this pull request to continue the -attach option.
For the moment it can attach to a process.

drrun.exe -attach <PID>

I will continue to improve it and I will create the testsuite after.

@raefko
Copy link
Author

raefko commented Dec 16, 2020

I also need some help:
I am trying to attach dynamoRIO to lsass.exe, but it looks like this line can't wait for lsass.

success = dr_inject_wait_for_child(inject_data, limit_millis);

If someone wants to try, you have to run drrun.exe with psexec -s -i.

@derekbruening
Copy link
Contributor

Thank you for the contribution. It looks like this is similar to the PR #3328, which was the source for PR #5075 which was merged as a0a44a9. I think we can close this as its functionality is now in the repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants