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

Move /var/run fc entries to /run #61

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

Conversation

ca-hu
Copy link

@ca-hu ca-hu commented Apr 11, 2024

/var/run is a symlink to /run in microos.

Currently the selinux-policy package has a rule for file contexts from /run/ use to the same file context specified in /var/run/.

The upcoming main selinux-policy package update will change the direction of that "forwarding", so /var/run will follow the rules in /run. This way, the file context entries will match the actual filesystem path.

All existing file context entries in custom modules based on the /var/run path need to change to /run, otherwise there is no rule for /var/run entries to "be forwarded to" in /run.

@ca-hu ca-hu force-pushed the master branch 2 times, most recently from b7d69e4 to 93d3c8d Compare April 11, 2024 10:31
@ca-hu
Copy link
Author

ca-hu commented Apr 11, 2024

This is also happening in fedora, see: fedora-selinux/selinux-policy@1f76e52

I did not add a commit for fedora/coreos since I don't really know your setup there.

/var/run is a symlink to /run in microos.

Currently the selinux-policy package has a rule for file contexts
from /run/<file> to use the same file context specified in /var/run/<file>.

The upcoming main selinux-policy package update will change the direction
of that "forwarding", so /var/run will follow the rules in /run.
This way, the file context entries will match the actual filesystem path.

All existing file context entries in custom modules
based on the /var/run path need to change to /run, otherwise there is no rule
for /var/run entries to "be forwarded to" in /run.

Signed-off-by: Cathy Hu <[email protected]>
@jsegitz
Copy link

jsegitz commented May 6, 2024

This is blocking our policy update ATM. Can you please work on this? We'll wait until next week, but then we'll update our package, which will break your build

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.

3 participants