On Wed, Sep 25, 2024 at 10:52:06AM +0200, Jan Beulich wrote:
> On 25.09.2024 10:42, Roger Pau Monne wrote:
> > The current check for duplicated sections in a payload is not effective.
> > Such
> > check is done inside a loop that iterates over the sections names, it's
> > logically impossible for the bitmap to be set more than once.
> >
> > The usage of a bitmap in check_patching_sections() has been replaced with a
> > boolean, since the function just cares that at least one of the special
> > sections is present.
> >
> > No functional change intended, as the check was useless.
> >
> > Fixes: 29f4ab0b0a4f ('xsplice: Implement support for
> > applying/reverting/replacing patches.')
> > Fixes: 76b3d4098a92 ('livepatch: Do not enforce ELF_LIVEPATCH_FUNC section
> > presence')
> > Signed-off-by: Roger Pau Monné <roger....@citrix.com>
>
> Just to clarify for my eventual picking up for backporting: Despite
> the Fixes: tags there's no actual bug being fixed; it's merely code
> simplification. So no need to backport.
Indeed, no strict bug, just useless code (unless my analysis is wrong).
Thanks, Roger.
