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

libsepol update #3559

Merged
merged 2 commits into from
Sep 18, 2024
Merged

libsepol update #3559

merged 2 commits into from
Sep 18, 2024

Conversation

EbonJaeger
Copy link
Member

Summary

  • libsepol: Update to 3.3
  • libselinux: Rebuild for libsepol

While newer versions than 3.3 are available, our libselinux is only at 3.3, so I opted to use the same version here. Updating that sounds like a can of worms that I don't want to open.

Part of #3554

Test Plan

Updated using the two packages, rebooted, ran sudo, saw that my system did not explode. Since we don't actually use SELinux in the kernel, I'm not sure how else exactly to test this.

Checklist

  • Package was built and tested against unstable

**Summary**
- `sepol_polcap_getnum`/`name()` are now exported as part of the shared libsepol interface
- `sepol/policydb/flask.h` was removed
- Removed restrictions in libsepol and checkpolicy that required all declared initial SIDs to be assigned a context
- libsepol implemented a new, more space-efficient form of storing filename transitions in the binary policy and reduced the size of the binary policy
- sepolgen sorts extended rules like normal ones
- libsepol and libsemanage dropped old and deprecated symbols and functions
- libsepol version was bumped to libsepol.so.2
- Add `monitoring.yml`
- Add homepage
- Fix license

Part of #3554

**Packager note**

While newer versions than 3.3 are available, our `libselinux` is only at 3.3, so I opted to use the same version here. Updating that sounds like a can of worms that I don't want to open.

Signed-off-by: Evan Maddock <maddock.evan@vivaldi.net>
**Summary**
- Add `monitoring.yml`
- Add missing builddep

Signed-off-by: Evan Maddock <maddock.evan@vivaldi.net>
Copy link
Contributor

@TraceyC77 TraceyC77 left a comment

Choose a reason for hiding this comment

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

Looks good here. The packages build and install.
@ReillyBrogan and/or @silkeh , did you want to take a look?

@silkeh
Copy link
Member

silkeh commented Aug 18, 2024

Should be fine to merge this and see what breaks, I think. I've made #3618 as a follow-up.

@ReillyBrogan
Copy link
Contributor

TBH we don't use selinux so I don't really see how updating the libselinux library would cause problems. It pretty much only exists in the repo because it's a build/run dep of other packages, and I can't imagine they use it beyond "is selinux enabled? no? then skip that part of the code".

@davidjharder davidjharder merged commit 6684aa2 into main Sep 18, 2024
1 check passed
@davidjharder davidjharder deleted the libsepol-update branch September 18, 2024 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

5 participants