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

Error out when trying to set PTRACE_O_TRACESECCOMP under ptrace emulation. #311

Merged
merged 1 commit into from
Jan 22, 2022

Conversation

jzakrzew
Copy link
Contributor

@jzakrzew jzakrzew commented Nov 8, 2021

proot currently does not even try to handle seccomp traps when running
ptrace emulation. Returning an error is better than misleading the ptracer
into expecting seccomp events.

In conjunction with PR #308, this (sort of) fixes test-230f47ch.

…tion.

proot currently does not even try to handle seccomp traps when running
ptrace emulation. Returning an error is better than misleading the ptracer
into expecting seccomp events.

This (sort of) fixes test-230f47ch.
@oxr463 oxr463 self-requested a review November 8, 2021 12:01
@oxr463 oxr463 added this to the PRoot Next milestone Nov 8, 2021
@jzakrzew
Copy link
Contributor Author

jzakrzew commented Nov 8, 2021

This also seems to fix issue #310

@oxr463 oxr463 merged commit d92431a into proot-me:master Jan 22, 2022
dna2github pushed a commit to dna2fork/proot that referenced this pull request May 1, 2023
…tion. (proot-me#311)

proot currently does not even try to handle seccomp traps when running
ptrace emulation. Returning an error is better than misleading the ptracer
into expecting seccomp events.

This (sort of) fixes test-230f47ch.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants