Bug 887010 – CVE-2012-5638 sanlock world writable /var/log/sanlock.log

Bugzilla will be upgraded to version 5.0 on a still to be determined date in the near future. The original upgrade date has been delayed.

Bug 887010 - (CVE-2012-5638) CVE-2012-5638 sanlock world writable /var/log/sanlock.log

Summary:	CVE-2012-5638 sanlock world writable /var/log/sanlock.log

Status:	CLOSED ERRATA

Aliases:	CVE-2012-5638

Product:	Security Response
Classification:	Other
Component:	vulnerability (Show other bugs)
Sub Component:
Version:	unspecified
Hardware:	All Linux

Priority	low Severity low
Target Milestone:	---
Target Release:	---
Assigned To:	Red Hat Product Security
QA Contact:
Docs Contact:

URL:
Whiteboard:	impact=low,public=20120822,reported=2...
Keywords:	Security

Depends On:	849181 887114
Blocks:	886367 887012
	Show dependency tree / graph

Reported:	2012-12-13 15:08 EST by Kurt Seifried
Modified:	2014-04-07 05:20 EDT (History)
CC List:	10 users (show)

See Also:
Fixed In Version:
Doc Type:	Bug Fix
Doc Text:
Story Points:	---
Clone Of:
Environment:
Last Closed:	2013-08-21 22:49:24 EDT
Type:	---
Regression:	---
Mount Type:	---
Documentation:	---
CRM:
Verified Versions:
Category:	---
oVirt Team:	---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---

Attachments	(Terms of Use)
proposed patch (1.39 KB, patch) 2012-12-24 16:28 EST, Martin Quinson	no flags	Details \| Diff
Add an attachment (proposed patch, testcase, etc.)

External Trackers
Tracker	ID	Priority	Status	Summary	Last Updated
Red Hat Product Errata	RHSA-2013:0691	normal	SHIPPED_LIVE	Important: Red Hat Storage 2.0 security, bug fix, and enhancement update #4	2013-03-28 22:21:19 EDT

Groups: None (edit)

Description Kurt Seifried 2012-12-13 15:08:03 EST

The sanlock server creates the /var/log/sanlock.log world writable allowing 
any one on the system to wipe the contents of the log file or to store data 
within the log file (bypassing any quotas applied to their account). The 
affected code is:

src/log.h

int setup_logging(void)
{
        int fd, rv;
        snprintf(logfile_path, PATH_MAX, "%s/%s", SANLK_LOG_DIR,
                 SANLK_LOGFILE_NAME);
        logfile_fp = fopen(logfile_path, "a+");

Comment 2 Martin Quinson 2012-12-24 16:28:22 EST

Created attachment 668606 [details]
proposed patch

This is just a starting
point, as I was not able to test the patch myself. Also, I used 660 as
permissions to the file, I'm not sure of whether it's sensible or not.

Please review and test before applying.

Comment 3 David Teigland 2013-01-02 11:28:29 EST

This problem was fixed in bug 849181.

Comment 4 Murray McAllister 2013-03-27 22:03:57 EDT

Acknowledgements:

This issue was discovered by Red Hat.

Comment 5 errata-xmlrpc 2013-03-28 18:28:15 EDT

This issue has been addressed in following products:

  Red Hat Storage 2.0
  Red Hat Storage 2.0 Console
  Native Client for RHEL 5 for Red Hat Storage
  Native Client for RHEL 6 for Red Hat Storage

Via RHSA-2013:0691 https://rhn.redhat.com/errata/RHSA-2013-0691.html

Note You need to log in before you can comment on or make changes to this bug.