[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Z1MVodP9Xien/Dv8@madcap2.tricolour.ca>
Date: Fri, 6 Dec 2024 10:17:53 -0500
From: Richard Guy Briggs <rgb@...hat.com>
To: Ricardo Robaina <rrobaina@...hat.com>
Cc: Paul Moore <paul@...l-moore.com>, audit@...r.kernel.org,
linux-kernel@...r.kernel.org, eparis@...hat.com,
viro@...iv.linux.org.uk
Subject: Re: [PATCH v2] audit: fix suffixed '/' filename matching in
__audit_inode_child()
On 2024-12-06 10:00, Ricardo Robaina wrote:
> On Thu, Dec 5, 2024 at 9:22 PM Paul Moore <paul@...l-moore.com> wrote:
> >
> > Yes, Richard did provide a reviewed-by tag on the v1 patch, but v2 has
> > enough changes that I don't think we can reasonably carry that forward;
> > of course Richard re-review this iteration and provide a new tag. I'm
> > going to remove it for now.
> >
> > Al never provided an explicit reviewed-by tag; simply commenting on a
> > patch is not the same as providing a 'Reviewed-by', the reviewer will
> > provide an explicit 'Reviewed-by' tag in their email. I'm going to
> > remove Al's tag too.
>
> I'm sorry about that, I was just trying to give credit to the ones who
> helped. Thanks for clarifying the expected review process, I
> appreciate it!
Please re-add my reviewed-by tag.
> > Other than those issues, I think this looks much better than v1, I'm
> > going to merge this into audit/dev now, thanks!
>
> Thanks, Paul!
>
> - Ricardo
- RGB
--
Richard Guy Briggs <rgb@...hat.com>
Sr. S/W Engineer, Kernel Security, Base Operating Systems
Remote, Ottawa, Red Hat Canada
Upstream IRC: SunRaycer
Voice: +1.613.860 2354 SMS: +1.613.518.6570
Powered by blists - more mailing lists