All of lore.kernel.org
 help / color / mirror / Atom feed
From: Ondrej Moris <omoris@redhat.com>
To: Steve Grubb <sgrubb@redhat.com>, linux-audit@redhat.com
Subject: Re: /var/log/audit ownership/permissions
Date: Thu, 21 Jul 2016 16:12:48 +0200	[thread overview]
Message-ID: <5790D860.8060508@redhat.com> (raw)
In-Reply-To: <12890758.RtUGNIL9cO@x2>

On 07/21/2016 03:55 PM, Steve Grubb wrote:
> On Thursday, July 21, 2016 11:48:04 AM EDT Ondrej Moris wrote:
>> Hi, I noticed that in 2.6.5 /var/log/audit permission were dropped from
>> 750 to 600. 
> 
> The directory should be 0750 or 0700 depending on your config. 0600 would be a 
> mistake.

Sorry, it was a typo - it should be 0700 (not 0600).

> 
> 
>> I am fine with that but while I see the motivation [1], I
>> just cannot find where is that happening in the code. 
> 
> https://fedorahosted.org/audit/browser/trunk/src/auditd-event.c#L886

Thanks, now it is clear. You one thing - line 903 suggests that it is
either 0700 or 0770 which I can confirm by testing:

# # log_group = root
# ls -ld /var/log/audit/
drwx------. 2 root root 4096 Jul 21 09:56 /var/log/audit/

# # log_group = input
# ls -ld /var/log/audit/
drwxrwx---. 2 root input 4096 Jul 21 09:56 /var/log/audit/

> 
>> Besides, specfile
>> still contains:
>>
>> %attr(750,root,root) %dir %{_var}/log/audit
> 
> Maybe I should take the attr away or modify it to (-,root,-). The group can 
> change. For example, I have wheel allowed to run audit reports on my system.
> 
>> and hence 'rpm -V audit' obviously fails.
> 
> Yeah. Hmm.

Yes, change you mentioned would solve 'rpm -V' problem. It sounds very
reasonable since both group ownership and permission are configurable
via auditd.conf.

> 
> -Steve
> 
>> [1]
>> http://post-office.corp.redhat.com/archives/tech-list/2016-May/msg00468.html
>>
>> --
>> Ondrej
>>
>> --
>> Linux-audit mailing list
>> Linux-audit@redhat.com
>> https://www.redhat.com/mailman/listinfo/linux-audit
> 
> 
> --
> Linux-audit mailing list
> Linux-audit@redhat.com
> https://www.redhat.com/mailman/listinfo/linux-audit
> 

  reply	other threads:[~2016-07-21 14:12 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-21  9:48 /var/log/audit ownership/permissions Ondrej Moris
2016-07-21 13:55 ` Steve Grubb
2016-07-21 14:12   ` Ondrej Moris [this message]
2016-07-21 14:31     ` Steve Grubb

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=5790D860.8060508@redhat.com \
    --to=omoris@redhat.com \
    --cc=linux-audit@redhat.com \
    --cc=sgrubb@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body. 
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.