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

Include sys_ptrace capability information in the error #121

Merged
merged 1 commit into from Feb 26, 2018

Conversation

jmeagher
Copy link
Contributor

@jmeagher jmeagher commented Feb 26, 2018

Add a more helpful message when permission denied errors arise. See #119 for a little background.

Note: This is a Linux specific error that came up for me under docker. Fix it in docker with docker run --cap-add SYS_PTRACE

Note to future me: To see the currently allowed capabilities use capsh --print and for this issue look for cap_sys_ptrace. If it isn't there this error message should happen.

@jvns
Copy link
Collaborator

jvns commented Feb 26, 2018

thanks, this is a better error message!

@jvns jvns merged commit 2ad0502 into rbspy:master Feb 26, 2018
@jmeagher jmeagher deleted the sys-ptrace-capability-error-message branch February 26, 2018 05:22
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.

None yet

2 participants