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

Add file context entry for /usr/bin/tpm2-abrmd #846

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

zpytela
Copy link

@zpytela zpytela commented Jul 15, 2024

To comply with the "Unify bin and sbin" Fedora Change [1], file equivalency for the /bin, /sbin, and /usr/sbin paths are now set in selinux-policy to /usr/bin.
This requires follow-up changes in DSP modules, too.

[1] https://fedoraproject.org/wiki/Changes/Unify_bin_and_sbin

@ca-hu
Copy link

ca-hu commented Aug 13, 2024

any updates here? this is breaking the tpm2-abrmd.service on systems with selinux enabled

see https://bugzilla.suse.com/show_bug.cgi?id=1229047

@kkaarreell
Copy link

@zpytela Hi, could you please update your commit with sign-off?

Copy link
Member

@williamcroberts williamcroberts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to sign off your changes @zpytela

@williamcroberts williamcroberts added this to the next (3.0.1) milestone Nov 20, 2024
To comply with the "Unify bin and sbin" Fedora Change [1],
file equivalency for the /bin, /sbin, and /usr/sbin paths
are now set in selinux-policy to /usr/bin.
This requires follow-up changes in DSP modules, too.

[1] https://fedoraproject.org/wiki/Changes/Unify_bin_and_sbin

Signed-off-by: Zdenek Pytela <[email protected]>
@zpytela zpytela force-pushed the comply-to-bin-sbin-merge branch from 6e14a51 to 5095ace Compare November 20, 2024 21:59
@zpytela
Copy link
Author

zpytela commented Nov 20, 2024

Signed, sorry that I missed the original request.

Also note this change is temporary until the actual change is made and may be handled in a different way for different supported releases.

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.

4 participants