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

sepolicy: Cache the value of is_selinux_enabled() to work around bug #815

Closed
wants to merge 2 commits into from

Conversation

cgwalters
Copy link
Member

This fixes a regression from:
#797
which is really due to an underlying bug in libselinux which
we're working around:
http://marc.info/?l=selinux&m=149323809332417&w=2

We drop the per-policy instance variable, since the SELinux state
is really per-kernel.

I have a fix I want to make here and it's really hard to write
the old style code now.
This fixes a regression from:
ostreedev#797
which is really due to an underlying bug in libselinux which
we're working around:
http://marc.info/?l=selinux&m=149323809332417&w=2

We drop the per-policy instance variable, since the SELinux state
is *really* per-kernel.

Closes: ostreedev#814
@jlebon
Copy link
Member

jlebon commented Apr 27, 2017

Ahh, good catch.
@rh-atomic-bot r+ 9fa082a

@rh-atomic-bot
Copy link

⌛ Testing commit 9fa082a with merge 90b24a8...

rh-atomic-bot pushed a commit that referenced this pull request Apr 27, 2017
This fixes a regression from:
#797
which is really due to an underlying bug in libselinux which
we're working around:
http://marc.info/?l=selinux&m=149323809332417&w=2

We drop the per-policy instance variable, since the SELinux state
is *really* per-kernel.

Closes: #814

Closes: #815
Approved by: jlebon
@rh-atomic-bot
Copy link

☀️ Test successful - status-atomicjenkins
Approved by: jlebon
Pushing 90b24a8 to master...

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