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

Fixes #13041 - passenger var context fix #55

Closed
wants to merge 1 commit into from

Conversation

lzap
Copy link
Member

@lzap lzap commented Jan 11, 2016

If this is confirmed to work, I will go ahead and fix this upstream:

https://github.com/TresysTechnology/refpolicy-contrib/blob/master/passenger.fc

Change in Passenger 4.0 perhaps?

@domcleal
Copy link
Contributor

It's a patch in our package (theforeman/foreman-packaging@a66b367), which I think came from either Fedora or RHSCL packages. The Fedora packages have since repackaged Passenger just as 'passenger' rather than 'rubygem-passenger', and now use /run/passenger again - so the patch is probably just needed here.

@domcleal
Copy link
Contributor

Looks fine, but could you also update the relabel script to ensure it fixes it on installation please?

# ll -dZ /var/run/*passenger/                                                                                         
drwxr-xr-x. root root system_u:object_r:var_run_t:s0   /var/run/rubygem-passenger/
# rpm -Uvh ...
# ll -dZ /var/run/*passenger/                                                                                         
drwxr-xr-x. root root system_u:object_r:var_run_t:s0   /var/run/rubygem-passenger/
# restorecon -Rvvn /var/run/*passenger/
restorecon reset /var/run/rubygem-passenger context system_u:object_r:var_run_t:s0->system_u:object_r:passenger_var_run_t:s0

@lzap
Copy link
Member Author

lzap commented Jan 27, 2016

Good catch, added.

@lzap
Copy link
Member Author

lzap commented Feb 1, 2016

If you have a minute or two, I'd love to get this in plus I found one issue I will be filing in a sec.

@domcleal
Copy link
Contributor

domcleal commented Feb 8, 2016

Merged as 86fa6e7, thanks @lzap.

@domcleal domcleal closed this Feb 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants