Bug 759898

Summary: selinux-policy has a typo in package scripts
Product: [Fedora] Fedora Reporter: Michal Jaegermann <michal>
Component: selinux-policyAssignee: Miroslav Grepl <mgrepl>
Status: CLOSED DUPLICATE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: rawhideCC: dominick.grift, dwalsh, mgrepl
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2011-12-04 21:02:43 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Michal Jaegermann 2011-12-04 18:46:25 UTC
Description of problem:

At least in selinux-policy-targeted-3.10.0-64.fc17 (and earlier too) packages there is the following line:

[ "${SELINUXTYPE}" == "targeted" ] && [ selinuxenabled ] && load_policy;

If ${SELINUXTYPE} happens to be "targeted" then '[ selinuxenabled ]' always
evaluates to "true" and if it was not really enabled then 'load_policy'
will complain loudly:

Could not open policy file <= /etc/selinux/targeted/policy/policy.26:  No such file or directory

even if such file actually exists.

I believe that the following was intended:

[ "${SELINUXTYPE}" == "targeted" ] && selinuxenabled && load_policy;


Version-Release number of selected component (if applicable):
selinux-policy-3.10.0-64.fc17

How reproducible:
always with SELinux disabled

Additional info:
I did NOT look at scripts in other 'selinux-policy...' packages.

Comment 1 Miroslav Grepl 2011-12-04 19:51:54 UTC
Yes, we know about. But good catch.

Comment 2 Miroslav Grepl 2011-12-04 21:02:43 UTC

*** This bug has been marked as a duplicate of bug 747549 ***