Bug 452733

Summary: Incorrect file labels for RealPlayer and HelixPlayer
Product: [Fedora] Fedora Reporter: Carl Roth <roth>
Component: selinux-policyAssignee: Daniel Walsh <dwalsh>
Status: CLOSED CURRENTRELEASE QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: low Docs Contact:
Priority: low    
Version: 9CC: rvokal
Target Milestone: ---   
Target Release: ---   
Hardware: All   
OS: Linux   
Whiteboard:
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2008-11-17 22:04:47 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Carl Roth 2008-06-24 17:46:32 UTC
Description of problem:

RealPlayer and HelixPlayer don't work in selinux enforcing mode because its
binaries and libraries are mislabeled under the current policy.

Also, the labels that are there in the targeted policy appear to have regex bugs
that will prevent them from being applied.

Version-Release number of selected component (if applicable):

HelixPlayer-1.1.0.4327-20080518.amd64
RealPlayer-10.1.0.4326-20080518.amd64
selinux-policy-targeted-3.3.1-64.fc9.noarch

How reproducible:


Steps to Reproduce:
1.
2.
3.
  
Actual results:


Expected results:


Additional info:

The RPMs available from the RealPlayer and HelixPlayer download sites install by
default into /opt; this should be reflected in the default selinux policy.

AFAIK the correct labels are as follows:

/opt/real/(.*/)?.+\.so(\.[^/]*)* --
gen_context(system_u:object_r:textrel_shlib_t,s0)
/opt/helix/(.*/)?.+\.so(\.[^/]*)* --
gen_context(system_u:object_r:textrel_shlib_t,s0)

/opt/real/(.*/)?realplay[.]bin --
gen_context(system_u:object_r:unconfined_execmem_exec_t,s0)

The /opt/helix/HelixPlayer/hxplay.bin may also need a label too, though I
haven't tested this.

The labels that *do* exist in the policy have a regex bug:

/usr/(.*/)?/RealPlayer/.+\.so(\.[^/]*)*	--	system_u:object_r:textrel_shlib_t:s0
/usr/(.*/)?/HelixPlayer/.+\.so(\.[^/]*)*	--	system_u:object_r:textrel_shlib_t:s0

If you look carefully (and test it with matchpathcon) you can see that this
won't match '/usr/lib/RealPlayer/foo.so' like it should.  A corrected regex would be

/usr/(.*/)?RealPlayer/.+\.so(\.[^/]*)*

Comment 1 Daniel Walsh 2008-06-25 11:14:55 UTC
Fixed in selinux-policy-3.3.1-72.fc9.noarch

Comment 2 Daniel Walsh 2008-11-17 22:04:47 UTC
Closing all bugs that have been in modified for over a month.  Please reopen if the bug is not actually fixed.