On Thu, Jan 17, 2019 at 4:33 AM Steve Grubb <sgr...@redhat.com> wrote: > On Mon, 14 Jan 2019 17:58:58 -0500 > Paul Moore <p...@paul-moore.com> wrote: > > > On Mon, Dec 10, 2018 at 5:18 PM Richard Guy Briggs <r...@redhat.com> > > wrote: > > > > > > Tie syscall information to all CONFIG_CHANGE calls since they are > > > all a result of user actions. > > Please don't tie syscall information to this. The syscall will be > sendto. We don't need that information, its implicit. Also, doing this > will possibly wreck things in libauparse. Please test the events with > ausearch --format csv and --format text. IFF the event looks better or > the same should we do this. If stuff disappears, the patch is > breaking things
We've discussed this quite a bit already; connecting associated records into a single event is something that should happen, needs to happen, and will happen. Conceptually it makes no sense to record the syscall (and any other associated records) which triggers the audit configuration change, and the configuration change record itself as two distinct events - they are the same event. We've also heard from a prominent user that associating records in this way is desirable. If the ausearch csv and text audit log transformations can't handle this particular change, I would consider that a shortcoming of that code. We have multi-record events now, and this is only going to increase in the future. Richard, if you can't make the requested changes to this patch and resubmit by ... let's say the middle of next week? that should be enough time, yes? ... please let me know and I'll make the changes and get this merged. -- paul moore www.paul-moore.com
