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.

Sign up for GitHub

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

Jump to bottom

Add safe wrapper for PTRACE_INTERRUPT #1279

Closed
wants to merge 1 commit into from
Closed

Add safe wrapper for PTRACE_INTERRUPT #1279

wants to merge 1 commit into from

Conversation

GabrielMajeri
Copy link
Contributor

Adds a new ptrace:interrupt function for stopping the running traced process. Also exposes the PTRACE_EVENT_STOP variant from the updated libc.

@ranweiler ranweiler mentioned this pull request Nov 15, 2020
Closed
@ranweiler
Copy link
Contributor

It looks like this has been pending review for a while. The PTRACE_EVENT_STOP change seems independent of the rest, and is a change I'd like to see land. @GabrielMajeri, would you consider splitting it off into a separate PR? I'd be happy to for you, if that's easier.

@GabrielMajeri GabrielMajeri mentioned this pull request Nov 16, 2020
Merged
@GabrielMajeri
Copy link
Contributor Author

@ranweiler I've created separate PR #1335 for the PTRACE_EVENT_STOP change.

@ranweiler ranweiler mentioned this pull request Jun 19, 2021
Merged
@asomers
Copy link
Member

asomers commented Aug 9, 2021

Closing in favor of #1422.

@asomers asomers closed this Aug 9, 2021
@GabrielMajeri GabrielMajeri deleted the ptrace-interrupt branch August 11, 2021 08:12
bors bot added a commit that referenced this pull request Aug 21, 2021
@bors @GabrielMajeri
Merge #1335
8866df6 
1335: Add `PTRACE_EVENT_STOP` enum variant r=asomers a=GabrielMajeri

Adds the `PTRACE_EVENT_STOP` variant to the `ptrace::Event` enum.

Smaller part of #1279

Closes #1334 

Co-authored-by: Gabriel Majeri <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@GabrielMajeri @ranweiler @asomers