Re: [PATCH v2] audit: fix suffixed '/' filename matching in __audit_inode_child()
From: Richard Guy Briggs
Date: Fri Dec 06 2024 - 10:18:16 EST
On 2024-12-06 10:00, Ricardo Robaina wrote:
> On Thu, Dec 5, 2024 at 9:22 PM Paul Moore <paul@xxxxxxxxxxxxxx> 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@xxxxxxxxxx>
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