Skip to content

Explain NNP transitions - #53

Closed
WOnder93 wants to merge 1 commit into
SELinuxProject:mainfrom
WOnder93:document-nnp-transition
Closed

Explain NNP transitions#53
WOnder93 wants to merge 1 commit into
SELinuxProject:mainfrom
WOnder93:document-nnp-transition

Conversation

@WOnder93

@WOnder93 WOnder93 commented Sep 8, 2025

Copy link
Copy Markdown
Member

Provide a brief explanation of the NNP concept and how it realtes to SELinux transitions to help guide people deciding whether to allow the nnp_transtion permission or not.


It has been pointed out to me that the semantics behind the nnp_transition permission are not obvious and that it would help to have some documentation on it, so I tried to to add it here. I'm not quite sure if it's sufficiently clear and correct, so would welcome feedback.

Cc @zpytela @stephensmalley

Provide a brief explanation of the NNP concept and how it realtes to
SELinux transitions to help guide people deciding whether to allow the
nnp_transtion permission or not.

Signed-off-by: Ondrej Mosnacek <omosnace@redhat.com>
@stephensmalley
stephensmalley self-requested a review September 10, 2025 15:01

@stephensmalley stephensmalley left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. Most common use case is allowing daemons to still transition to their own domains when executed with NNP set or from a nosuid mount. SELinux domain transitions are rarely if ever exact subsets of the calling domain's permissions since they at least differ wrt their permissions to their own executable, tmp, and other derived types.

@dburgener dburgener left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM as well

@dburgener

Copy link
Copy Markdown
Member

Merged via manual push. Thanks!

@dburgener dburgener closed this Oct 22, 2025
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