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

New elevated privilege system #484

Closed
wants to merge 3 commits into from
Closed

New elevated privilege system #484

wants to merge 3 commits into from

Commits on May 4, 2023

  1. Unify perf binary path handling 

    For running perf with pkexec we need to get the full path to the binary,
so handle this consistently. This has the nice side effect of showing
clearly which perf binary is being used in the output console.
    @zeno-endemann-kdab
    zeno-endemann-kdab committed May 4, 2023
    Configuration menu
    Copy the full SHA
    34079cb View commit details
    Browse the repository at this point in the history

  2. Remove KAuth support 

    The new previlige handling will call perf directly without a helper, so
KAuth does not make sense for that anymore.
    @zeno-endemann-kdab
    zeno-endemann-kdab committed May 4, 2023
    Configuration menu
    Copy the full SHA
    b2ee046 View commit details
    Browse the repository at this point in the history

  3. Implement new perf elevated privilege system 

    Instead of using a script that changes system configuration temporarily,
just run perf as root.

To properly synchronize with a launched app (that should not run as root),
launch the app in a separate, initially stopped process, and use the
control fifo feature of perf to properly synchronize with it.

The control fifos are also needed to be able to stop sudo-perf, as one
does not have permission to SIGINT it anymore.
    @zeno-endemann-kdab
    zeno-endemann-kdab committed May 4, 2023
    Configuration menu
    Copy the full SHA
    bdb1ebd View commit details
    Browse the repository at this point in the history