Bug 1431468

Summary: openstack-ironic: /var/log/ironic is world readable
Product: Red Hat OpenStack Reporter: Summer Long <slong>
Component: openstack-ironicAssignee: Derek Higgins <derekh>
Status: CLOSED CURRENTRELEASE QA Contact: mlammon
Severity: high Docs Contact:
Priority: high    
Version: 11.0 (Ocata)CC: apevec, bfournie, derekh, eglynn, jjoyce, jruzicka, jschluet, lhh, mburns, pkilambi, rhel-osp-director-maint, srevivo, ssmolyak
Target Milestone: ---Keywords: Security, Triaged
Target Release: 11.0 (Ocata)   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: openstack-ironic-7.0.1-1.el7ost Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: 1431466
: 1431479 (view as bug list) Environment:
Last Closed: 2017-09-27 19:09:28 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Description Summer Long 2017-03-13 00:07:09 UTC
Description of problem:
The directory /var/log/ironic is world readable and contains log files that are readable, which can result in the exposure of sensitive information. The 'other readable/execute' bits need to be removed from the /var/log/ironic directory.

Because no sensitive data was found in the files, this is being raised as a hardening bug, and not a flaw.

Version-Release number of selected component (if applicable):
openstack-ironic-7.0.1-0.20170223011831.a13ea4f.el7ost

How reproducible:
List /var/log directory for openstack-ironic:
$ ls -la ironic/
total 1072
drwxr-xr-x.  2 ironic ironic     78 Mar  8 22:35 .
drwxr-xr-x. 25 root   root     4096 Mar 12 19:44 ..
-rw-r--r--.  1 ironic ironic 240138 Mar 12 19:45 ironic-api.log
-rw-r--r--.  1 ironic ironic 647216 Mar 12 20:00 ironic-conductor.log
-rw-r--r--.  1 root   root        0 Mar  8 22:15 ironic-dbsync.log

Actual results:
Directory and files are world readable.

Expected results:
Directory and files should not be world readable.

Comment 1 Derek Higgins 2017-04-04 09:16:24 UTC
This has been fixed upstream in the RDO packaging 
https://review.rdoproject.org/r/#/c/5258/

Comment 2 Derek Higgins 2017-04-04 10:10:52 UTC
An submitted to Ocata
https://review.rdoproject.org/r/#/c/6149/