All of lore.kernel.org
 help / color / mirror / Atom feed
From: Steve Grubb <sgrubb@redhat.com>
To: burn@swtf.dyndns.org
Cc: Richard Guy Briggs <rgb@redhat.com>,
	Linux Audit <linux-audit@redhat.com>
Subject: Re: Occasional delayed output of events
Date: Tue, 26 Jan 2021 15:42:34 -0500	[thread overview]
Message-ID: <5679566.lOV4Wx5bFT@x2> (raw)
In-Reply-To: <c878d2aa65ba99579c800a67c54dccea24b9b03d.camel@iinet.net.au>

On Tuesday, January 26, 2021 6:53:31 AM EST Burn Alting wrote:
> On Tue, 2021-01-26 at 11:29 +1100, Burn Alting wrote:
> > On Mon, 2021-01-25 at 19:20 -0500, Steve Grubb wrote:
> > > On Monday, January 25, 2021 7:11:45 PM EST Burn Alting wrote:
> > > > On Mon, 2021-01-25 at 18:53 -0500, Steve Grubb wrote:
> > > > > On Saturday, January 23, 2021 5:55:44 PM EST Burn Alting wrote:
> > > > > > > > How is the following for a way forward.a. I will author a
> > > > > > > > patch to the
> > > > > > > > user space code to correctly parsethiscondition and submit it
> > > > > > > > on the
> > > > > > > > weekend. It will be via a newconfiguration item to
> > > > > > > > auditd.conf just in
> > > > > > > > case placing a fixedextended timeout (15-20 secs) affects
> > > > > > > > memory usage
> > > > > > > > for users of theauparse library. This solves the initial
> > > > > > > > problem
> > > > > > > > ofausearch/auparsefailing to parse generated audit.b. I am
> > > > > > > > happy to
> > > > > > > > instrument whateveris recommended on my hosts at home (vm's
> > > > > > > > and bare
> > > > > > > > metal) to providemore information, should we want to
> > > > > > > > 'explain' the
> > > > > > > > occurrence, givenIsee this every week or two and report back.
> > > > > > > 
> > > > > > > Seems reasonable to me.
> > > > > > 
> > > > > > I can implement the 'end_of_event_timeout' change either asi. a
> > > > > > command
> > > > > > line argument to ausearch/aureport (say --eoetmo secs) andanew
> > > > > > pair of
> > > > > > library functions within the  auparse() stable
> > > > > > (sayauparse_set_eoe_timeout() and auparse_get_eoe_timeout())orii.
> > > > > > a
> > > > > > configuration item in /etc/audit/auditd.conf, or
> > > > > > 
> > > > > > Which is your preference? Mine is i. as this is a user space
> > > > > > processingchange, not a demon change.
> > > > > 
> > > > > To be honest, I'm not entirely sure what we're seeing. I run some
> > > > > teststoday on my system. It's seeing issues also. I'd still like
> > > > > to treat theroot cause of this. But we do need to change the
> > > > > default. That I whatI'm trying to figure out.
> > > > > Back to your question, I'm wondering if we should do both? A
> > > > > changeabledefault in auditd.conf and an override on the command
> > > > > line.
> > > > 
> > > > So far, all items in /etc/audit/auditd.conf appear to only affect
> > > > thedaemon. Is this the right location to start adding
> > > > non-daemonconfiguration items? (I accept there is no other place).
> > > 
> > > ausearch/report/auparse all read the auditd.conf to find the canonical
> > > location for where the logs are supposed to be. So, they already read
> > > this file. I'd rather keep it there than make yet another config. The
> > > only drawback it that it might again confuse people that auditd really
> > > doesn't do anything with the records but just some light processing.
> > 
> > OK. I will put it in /etc/audit/auditd.conf
> 
> One question with this solution. If the user does not have read permission
> to /etc/audit/auditd.conf, then any change cannot take effect. The default
> mode for this file is 640 to root, so a non-root user could never change
> the timeout.

Right, but since they cannot access the logs, it's not a problem in general.  
But if they so happen to have a local copy of logs, then the command line 
override should allow them to correct this. I am also reviewing things to see 
if a better default can be picked.

> Should I also add - a command line argument to ausearch/aureport (say --
> eoetmo secs) and, - a pair of new auparse() functions -
> auparse_set_eoe_timeout() and  auparse_get_eoe_timeout()
> so that non root users can make use of the new configuration item.

Yes, that is what I meant by doing both. We have default in auditd.conf that 
works for everyone with direct audit access. We have a commandline option for 
overriding the auditd.conf value.

Although, I don't know why we would want to get the eoe_timeout value? I 
can't imagine a use for it right now. 

As for ausearch/report, let's just make a long option --eoe-timeout

-Steve

> Also, do you want the default timeout to be 2 seconds or should I make it
> higher.

I'm likely to adjust it, but I'm still looking to see what is happening. Just 
go with the 2 second default for now.

Thanks,
-Steve


--
Linux-audit mailing list
Linux-audit@redhat.com
https://www.redhat.com/mailman/listinfo/linux-audit


  reply	other threads:[~2021-01-26 20:50 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-01-01 21:22 Occasional delayed output of events Burn Alting
2021-01-03 15:41 ` Steve Grubb
2021-01-04  7:55   ` Burn Alting
2021-01-04 14:46     ` Steve Grubb
2021-01-04 20:12       ` Burn Alting
2021-01-10  4:39         ` Burn Alting
2021-01-15 22:18           ` Burn Alting
2021-01-16  0:35             ` Richard Guy Briggs
2021-01-16  2:42               ` Burn Alting
2021-01-17 14:07                 ` Paul Moore
2021-01-17 21:12                   ` Steve Grubb
2021-01-18 13:54                     ` Paul Moore
2021-01-18 14:31                       ` Steve Grubb
2021-01-18 20:34                         ` Burn Alting
2021-01-18 20:36                         ` Paul Moore
2021-01-19  8:18                           ` Burn Alting
2021-01-19 15:31                             ` Lenny Bruzenak
2021-01-19 19:11                             ` Paul Moore
2021-01-19 19:38                               ` Burn Alting
2021-01-19 20:26                                 ` Paul Moore
2021-01-19 21:51                                   ` Steve Grubb
2021-01-20  6:38                                     ` Burn Alting
2021-01-20 22:50                                       ` Paul Moore
2021-01-23 22:55                                         ` Burn Alting
2021-01-25 23:53                                           ` Steve Grubb
2021-01-26  0:11                                             ` Burn Alting
2021-01-26  0:20                                               ` Steve Grubb
2021-01-26  0:29                                                 ` Burn Alting
2021-01-26 11:53                                                   ` Burn Alting
2021-01-26 20:42                                                     ` Steve Grubb [this message]
2021-01-27 12:12                                                       ` Burn Alting
2021-01-19 20:42                               ` Richard Guy Briggs

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=5679566.lOV4Wx5bFT@x2 \
    --to=sgrubb@redhat.com \
    --cc=burn@swtf.dyndns.org \
    --cc=linux-audit@redhat.com \
    --cc=rgb@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.