Re: [PATCH v6 3/3] fanotify, audit: Allow audit to use the full permission event response

2023-01-27 Thread Steve Grubb
On Friday, January 27, 2023 3:00:37 PM EST Paul Moore wrote: > On Wed, Jan 25, 2023 at 5:06 PM Richard Guy Briggs wrote: > > On 2023-01-20 13:52, Paul Moore wrote: > > > On Wed, Jan 18, 2023 at 1:34 PM Steve Grubb wrote: > > > > Hello Richard, > > > > > > > > I built a new kernel and tested

Re: [PATCH v6 3/3] fanotify,audit: Allow audit to use the full permission event response

2023-01-27 Thread Paul Moore
On Wed, Jan 25, 2023 at 5:11 PM Richard Guy Briggs wrote: > > On 2023-01-20 13:58, Paul Moore wrote: > > On Tue, Jan 17, 2023 at 4:14 PM Richard Guy Briggs wrote: > > > > > > This patch passes the full response so that the audit function can use all > > > of it. The audit function was updated to

Re: [PATCH v6 3/3] fanotify,audit: Allow audit to use the full permission event response

2023-01-27 Thread Paul Moore
On Wed, Jan 25, 2023 at 5:06 PM Richard Guy Briggs wrote: > On 2023-01-20 13:52, Paul Moore wrote: > > On Wed, Jan 18, 2023 at 1:34 PM Steve Grubb wrote: > > > Hello Richard, > > > > > > I built a new kernel and tested this with old and new user space. It is > > > working as advertised. The only

Re: [PATCH v6 3/3] fanotify,audit: Allow audit to use the full permission event response

2023-01-25 Thread Richard Guy Briggs
On 2023-01-20 13:58, Paul Moore wrote: > On Tue, Jan 17, 2023 at 4:14 PM Richard Guy Briggs wrote: > > > > This patch passes the full response so that the audit function can use all > > of it. The audit function was updated to log the additional information in > > the AUDIT_FANOTIFY record. > > >

Re: [PATCH v6 3/3] fanotify,audit: Allow audit to use the full permission event response

2023-01-25 Thread Richard Guy Briggs
On 2023-01-20 13:52, Paul Moore wrote: > On Wed, Jan 18, 2023 at 1:34 PM Steve Grubb wrote: > > Hello Richard, > > > > I built a new kernel and tested this with old and new user space. It is > > working as advertised. The only thing I'm wondering about is why we have 3F > > as the default value

Re: [PATCH v6 3/3] fanotify,audit: Allow audit to use the full permission event response

2023-01-20 Thread Paul Moore
On Tue, Jan 17, 2023 at 4:14 PM Richard Guy Briggs wrote: > > This patch passes the full response so that the audit function can use all > of it. The audit function was updated to log the additional information in > the AUDIT_FANOTIFY record. > > Currently the only type of fanotify info that is

Re: [PATCH v6 3/3] fanotify,audit: Allow audit to use the full permission event response

2023-01-20 Thread Paul Moore
On Wed, Jan 18, 2023 at 1:34 PM Steve Grubb wrote: > > Hello Richard, > > I built a new kernel and tested this with old and new user space. It is > working as advertised. The only thing I'm wondering about is why we have 3F > as the default value when no additional info was sent? Would it be

Re: [PATCH v6 3/3] fanotify, audit: Allow audit to use the full permission event response

2023-01-18 Thread Steve Grubb
Hello Richard, I built a new kernel and tested this with old and new user space. It is working as advertised. The only thing I'm wondering about is why we have 3F as the default value when no additional info was sent? Would it be better to just make it 0? Btw, the change to %X makes life

[PATCH v6 3/3] fanotify, audit: Allow audit to use the full permission event response

2023-01-17 Thread Richard Guy Briggs
This patch passes the full response so that the audit function can use all of it. The audit function was updated to log the additional information in the AUDIT_FANOTIFY record. Currently the only type of fanotify info that is defined is an audit rule number, but convert it to hex encoding to