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

Fixes to seccomp event handling logic #266

Merged
merged 5 commits into from
Sep 7, 2021
Merged

Fixes to seccomp event handling logic #266

merged 5 commits into from
Sep 7, 2021

Commits on Aug 28, 2021

  1. Fix indentation

    @jzakrzew
    jzakrzew committed Aug 28, 2021
    Configuration menu
    Copy the full SHA
    b51a40f View commit details
    Browse the repository at this point in the history

  2. Add a testcase. 

    When running under seccomp, sometimes sysexit handlers fail to execute.
This is possible when the first syscall a process makes,
before seccomp is enabled, gets handled in the SIGTRAP path.
However the conditions for this to occur seem fairly random,
so we fork out many processes to make it likely that at least
some hit this problem.

This problem may be related to issue proot-me#106.
    @jzakrzew
    jzakrzew committed Aug 28, 2021
    Configuration menu
    Copy the full SHA
    9aa7f8b View commit details
    Browse the repository at this point in the history

  3. Fix an issue in seccomp event handling logic, that could cause 

    sysexit events handler to be missed if sysenter is handled during
a syscall-enter-stop event instead of the seccomp ptrace event.

This may be a (at least partial) fix for issue proot-me#106.
    @jzakrzew
    jzakrzew committed Aug 28, 2021
    Configuration menu
    Copy the full SHA
    1f15a5e View commit details
    Browse the repository at this point in the history

  4. Make kernel version detection work for kernels 5.0+ and fix indentation.

    @jzakrzew
    jzakrzew committed Aug 28, 2021
    Configuration menu
    Copy the full SHA
    a13cef7 View commit details
    Browse the repository at this point in the history

  5. Remove preprocessor directives and associated code, 

    that tie runtime behavior to compilation environment,
which is the wrong thing to do.
    @jzakrzew
    jzakrzew committed Aug 28, 2021
    Configuration menu
    Copy the full SHA
    6e678b7 View commit details
    Browse the repository at this point in the history