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

xserver: add xdm user with role access to system_r and xdm_r #8

Closed
wants to merge 1 commit into from

Conversation

WavyEbuilder
Copy link
Contributor

Sync with upstream's xserver changes.

Previously reverted in dada9b3. The reasoning for the revert:

This commit added the sddm user to the xserver module.
This caused problems loading the xserver module if the user did not
exist on the system.

no longer applies, as upstream hit this issue here: SELinuxProject/refpolicy#488 and resolved it here: SELinuxProject/refpolicy@b6998e4.

Fixes: #7

@WavyEbuilder
Copy link
Contributor Author

@perfinion @0xC0ncord

@WavyEbuilder
Copy link
Contributor Author

Also note the gen_user call is intentionally below our ifdef for gentoo - it needs to be the last call in the module

@WavyEbuilder
Copy link
Contributor Author

CI Fails seem unrelated?

@0xC0ncord 0xC0ncord self-assigned this Jan 6, 2025
Sync with upstream's xserver changes.

Previously reverted in dada9b3. The
reasoning for the revert:
```
This commit added the sddm user to the xserver module.
This caused problems loading the xserver module if the user did not
exist on the system.
```
no longer applies, as upstream hit this issue here:
SELinuxProject/refpolicy#488 and resolved it.

Fixes: gentoo#7

Signed-off-by: Rahul Sandhu <[email protected]>
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.

xserver: sddmv2 patchset: sync with upstream
2 participants