Bug 782492

Summary:	Propose that you turn on PrivateTmp=true in service file for corosync
Product:	[Fedora] Fedora	Reporter:	Daniel Walsh <dwalsh>
Component:	corosync	Assignee:	Steven Dake <sdake>
Status:	CLOSED NOTABUG	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	rawhide	CC:	agk, fdinitto, jfriesse, sdake
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:	2012-01-17 16:45:44 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:
Bug Depends On:
Bug Blocks:	782466

Description Daniel Walsh 2012-01-17 15:32:55 UTC

I would like to propose using PrivateTmp for corosync, to make it more secure
and avoid users from being able to potentially effect it.

http://fedoraproject.org/wiki/Features/ServicesPrivateTmp

Comment 1 Steven Dake 2012-01-17 16:21:30 UTC

Dan,

Corosync doesn't create/open/delete any files in /tmp or /var/tmp on Fedora.  The cman plugins may do this, so I've copied Fabio (cman maintainer), but we are removing cman in Fedora 17.  On Fedora, we use /var/lib/corosync and /dev/shm exclusively.  If you can show me evidence of the corosync process using /var/tmp we can consider this feature.  Searching the code base doesn't show any code using this directory and running through strace doesn't demonstrate access to these directories either.

Thanks
-steve

Comment 2 Daniel Walsh 2012-01-17 16:32:10 UTC

Steve, all I did was search the SELinux policy for domains that used tmp, if corosync no longer uses /tmp then you can close this bug.

One other thing that could cause this would be bash redirection.

command << _EOF
STUFF
__EOF

Anyways if you don't think you use /tmp then close the bug.

Comment 3 Steven Dake 2012-01-17 16:45:44 UTC

We did use /var/tmp at one time but changed to shm_open for more secure operation.