linux-kernel - Re: [PATCH] MAINTAINERS: rectify radix-tree testing entry in XARRAY

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <5ac73de311101754a25aaeac3df1aca23dbaf2af.camel@perches.com>
Date:   Sat, 01 Feb 2020 15:35:23 -0800
From:   Joe Perches <joe@...ches.com>
To:     Lukas Bulwahn <lukas.bulwahn@...il.com>
Cc:     Matthew Wilcox <willy@...radead.org>,
        linux-fsdevel@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] MAINTAINERS: rectify radix-tree testing entry in XARRAY

On Sat, 2020-02-01 at 22:35 +0100, Lukas Bulwahn wrote:
> 
> On Sat, 1 Feb 2020, Lukas Bulwahn wrote:
> 
> > The initial commit 3d5bd6e1a04a ("xarray: Add MAINTAINERS entry")
> > missed a trailing slash to include all files and subdirectory files.
> > Hence, all files in tools/testing/radix-tree were not part of XARRAY,
> > but were considered to be part of "THE REST".
> > 
> > Rectify this to ensure patches reach the actual maintainer.
> > 
> > This was identified with a small script that finds all files only
> > belonging to "THE REST" according to the current MAINTAINERS file, and I
> > acted upon its output.
> > 
> > Fixes: 3d5bd6e1a04a ("xarray: Add MAINTAINERS entry")
> > Signed-off-by: Lukas Bulwahn <lukas.bulwahn@...il.com>
> > ---
> > Matthew, please pick this small fixup patch.
> > 
> > applies cleanly on current master and next-20200131
> > 
> 
> Matthew, Joe Perches just informed me that get_maintainers.pl is smart
> enough to handle the case below and does not strictly need a further 
> trailing slash.
> 
> Please ignore this patch; the commit message is actually wrong.

While the commit message may be inexact,
I think the actual change is good and
could/should still be applied.

The "Fixes:" tag is unnecessary though.