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

logind: add …WithFlags methods to policy #18933

Merged
merged 1 commit into from Mar 9, 2021

Conversation

keszybz
Copy link
Member

@keszybz keszybz commented Mar 9, 2021

Without this, privilege escalation through polkit does not work, because all
methods fail with permission errors.

Forgotten in 8885fed.
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1933335.

Without this, privilege escalation through polkit does not work, because all
methods fail with permission errors.

Forgotten in 8885fed.
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1933335.
@keszybz keszybz added this to the v248 milestone Mar 9, 2021
@yuwata
Copy link
Member

yuwata commented Mar 9, 2021

LGTM.

@yuwata yuwata added good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed login labels Mar 9, 2021
@bluca bluca merged commit 9ca6d78 into systemd:main Mar 9, 2021
@keszybz keszybz deleted the with-flags-policy branch March 9, 2021 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good-to-merge/waiting-for-ci 👍 PR is good to merge, but CI hasn't passed at time of review. Please merge if you see CI has passed login
Development

Successfully merging this pull request may close these issues.

None yet

3 participants