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

Allow fapolicyd to watch /run/netns #859

Merged

Conversation

dsugar100
Copy link
Contributor

/run/netns is a tmpfs mountpoint

node=localhost type=AVC msg=audit(1738868630.348:1695): avc: denied { watch_sb watch_with_perm } for pid=967 comm="fapolicyd" path="/run/netns" dev="tmpfs" ino=1954

Signed-of-by: Dave Sugar [email protected]

@dsugar100 dsugar100 force-pushed the fapolicyd_watch_network_namespace branch from 6745c3f to c013927 Compare February 6, 2025 20:38
@dsugar100
Copy link
Contributor Author

I'm not sure the interface name is really the best, while it is a directory, maybe we should be specific that it is a mountpoint? Thoughts?

@pebenito
Copy link
Member

I'm not sure the interface name is really the best, while it is a directory, maybe we should be specific that it is a mountpoint? Thoughts?

The naming seems ok to me, since it expresses the intent; however, it should probably be 2 interfaces.

/run/netns is a tmpfs mountpoint

node=localhost type=AVC msg=audit(1738868630.348:1695): avc:  denied  { watch_sb watch_with_perm } for  pid=967 comm="fapolicyd" path="/run/netns" dev="tmpfs" ino=1954

Signed-off-by: Dave Sugar <[email protected]>
@dsugar100 dsugar100 force-pushed the fapolicyd_watch_network_namespace branch from c013927 to fad6401 Compare February 12, 2025 03:34
@dsugar100
Copy link
Contributor Author

Ok, please let me know if you want any changes.

@pebenito pebenito merged commit 5b13e7f into SELinuxProject:main Feb 12, 2025
118 checks passed
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.

2 participants