Re: [PATCH] libmultipath: Fix MAPINFO_CHECK_UUID with partitions

[Benjamin Marzinski <bmarzins@redhat.com>]

On Thu, Nov 21, 2024 at 06:05:48PM +0100, Martin Wilck wrote:
> On Wed, 2024-11-20 at 19:17 -0500, Benjamin Marzinski wrote:
> > In libmp_mapinfo__(), if is_mpath_part_uuid() succeeded, instead of
> > ending the 'if' statement, is_mpath_uuid() would be called because of
> > the OR operator.  This would always fail if is_mpath_part_uuid()
> > passed.
> > This meant that libmp_mapinfo__() could never match partitions with
> > MAPINFO_CHECK_UUID.
> > 
> > Fix that by not calling is_mpath_uuid() if MAPINFO_PART_ONLY is set.
> > 
> > Fixes: c1aa0285 ("libmultipath: make MAPINFO_CHECK_UUID work with
> > partitions")
> > Signed-off-by: Benjamin Marzinski <bmarzins@redhat.com>
> 
> Ouch. Thanks!!
> Reviewed-by: Martin Wilck <mwilck@suse.com>

So, in theory the next step would be to open a PR for the upstream
stable branch, if that existed, right?

-Ben