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

sesearch: Raise parse error mixing expressions unused by RBAC/MLS searches. #116

Merged
merged 3 commits into from
Dec 13, 2023

Conversation

pebenito
Copy link
Member

@pebenito pebenito commented Dec 11, 2023

Using -p, -x, and -b with RBAC or MLS searches is an error as these expressions never yield results.

Fixes #111

Fully specify --type/role/range_transition options. Python's argparse argument abbreviation feature provides backwards compatibility.

@pebenito pebenito changed the title Sesearch-no-results-opts sesearch: Short circuit incompatible RBAC/MLS search expressions. Dec 11, 2023
Use more pythonic all() rather than multiple ORs.

Signed-off-by: Chris PeBenito <[email protected]>
This will fully match the rule types, plus the ArgumentParser argument
abbreviations will enable the previous long options to continue to work.

Signed-off-by: Chris PeBenito <[email protected]>
…rches.

Using -p, -x, and -b with RBAC or MLS searches is an error as these expressions
never yield results.

Signed-off-by: Chris PeBenito <[email protected]>
@pebenito pebenito force-pushed the sesearch-no-results-opts branch from b0fe7b0 to a1e8949 Compare December 12, 2023 13:36
@pebenito pebenito changed the title sesearch: Short circuit incompatible RBAC/MLS search expressions. sesearch: Raise parse error mixing expressions unused by RBAC/MLS searches. Dec 12, 2023
@pebenito pebenito merged commit a3281e6 into SELinuxProject:main Dec 13, 2023
9 checks passed
@pebenito pebenito deleted the sesearch-no-results-opts branch December 13, 2023 18:50
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.

Hide role-allow rules while filtering for boolean
1 participant