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.

Sign up for GitHub

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

Jump to bottom

Add an example of IMA policy #536

Merged
merged 1 commit into from
Jul 7, 2023
Merged

Add an example of IMA policy #536

merged 1 commit into from
Jul 7, 2023

Conversation

aplanas
Copy link
Contributor

@aplanas aplanas commented Mar 15, 2023

Add a SELinux extended IMA policy as an example, and some documentation about how to use and extend it.

@aplanas
Copy link
Contributor Author

aplanas commented Mar 15, 2023

This IMA policy is using the SELinux types available in openSUSE, but this is the same one that is used in Fedora. Both are instances of the SELinux reference policy, so the types referenced on it are quite generic.

ueno
ueno approved these changes Mar 17, 2023
View reviewed changes
ima-policy/README Outdated Show resolved Hide resolved
ima-policy/README Outdated Show resolved Hide resolved
ima-policy/ima-policy Outdated Show resolved Hide resolved
@aplanas aplanas force-pushed the fix_ima-policy branch 2 times, most recently from 8c92297 to 6b2d07c Compare March 17, 2023 08:05
@THS-on
Copy link
Member

THS-on commented Mar 17, 2023

We also have some IMA policies here: https://github.com/keylime/keylime/tree/master/demo/ima-policies

Maybe we should unify them in one place and then point the documentation to it.

@aplanas
Copy link
Contributor Author

aplanas commented Mar 17, 2023

Maybe we should unify them in one place and then point the documentation to it.

From a brief comparison, seems that the overlayfs fs ID was missing, and I added it now.

@codecov
Copy link

codecov bot commented Jul 6, 2023 •
edited
Loading

Codecov Report

Merging #536 (9553c56) into master (636c80a) will increase coverage by 0.14%.
The diff coverage is 100.00%.

Additional details and impacted files
Flag Coverage Δ
e2e-testsuite 59.65% <100.00%> (+0.17%) ⬆️
upstream-unit-tests 59.92% <0.00%> (-0.14%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
keylime-agent/src/main.rs 37.04% <100.00%> (+1.23%) ⬆️

... and 1 file with indirect coverage changes

@aplanas @ansasaki
Add an example of IMA policy
9553c56 
Add a SELinux extended IMA policy as an example, and some documentation
about how to use and extend it.

Signed-off-by: Alberto Planas <[email protected]>
@ansasaki ansasaki merged commit 381faac into keylime:master Jul 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ueno ueno ueno approved these changes

@THS-on THS-on Awaiting requested review from THS-on

@maugustosilva maugustosilva Awaiting requested review from maugustosilva

Assignees

@maugustosilva maugustosilva

@THS-on THS-on

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@aplanas @THS-on @ueno @maugustosilva @ansasaki