Bug 208834 - auditd fails to restart after log rotation with num_logs=0
Summary: auditd fails to restart after log rotation with num_logs=0
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Linux 5
Classification: Red Hat
Component: audit
Version: 5.0
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
: ---
Assignee: Steve Grubb
QA Contact: Brian Brock
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2006-10-02 10:15 UTC by Bastien Nocera
Modified: 2007-11-30 22:07 UTC (History)
1 user (show)

Fixed In Version: 5.0.0
Doc Type: Bug Fix
Doc Text:
Clone Of:
Environment:
Last Closed: 2006-12-19 15:50:33 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)
auditd.fix (560 bytes, patch)
2006-10-02 10:18 UTC, Bastien Nocera
no flags Details | Diff

Description Bastien Nocera 2006-10-02 10:15:25 UTC
audit-1.2.5-8

1. Set num_logs=0 in /etc/audit/auditd.conf
2. Restart auditd
3. run "killall -USR1 auditd" to attempt log rotation
4. Try to restart auditd:
Sep 28 11:38:18 test auditd: /var/log/audit/audit.log permissions should be 0640
Sep 28 11:38:18 test auditd: The audit daemon is exiting.

The log rotation code changes the permissions on the log file before trying to
move it. It shouldn't do that as it won't rotate anything with num_logs=0

Comment 1 Bastien Nocera 2006-10-02 10:18:30 UTC
Created attachment 137532 [details]
auditd.fix

Patch from srinivds.com to fix the problem
Will probably need indentation changes

Comment 2 Steve Grubb 2006-10-02 13:01:30 UTC
Will review problem and fix in 1.2.9. There is probably a more general solution
needed since it should never enter the rotate code if num_logs == 0.

Comment 3 Steve Grubb 2006-10-17 14:13:24 UTC
audit-1.2.8-2 was built to solve this problem. Thanks for reporting it.

Comment 4 RHEL Program Management 2006-10-26 19:40:19 UTC
This request was evaluated by Red Hat Product Management for inclusion in a Red
Hat Enterprise Linux major release.  Product Management has requested further
review of this request by Red Hat Engineering, for potential inclusion in a Red
Hat Enterprise Linux Major release.  This request is not yet committed for
inclusion.

Comment 7 James Laska 2006-12-14 13:09:14 UTC
Thank you for the detailed test case.  Marking as VERIFIED

Comment 8 Jay Turner 2006-12-19 15:50:33 UTC
audit-1.3.1-1.el5 included in 20061218.1 trees.


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