linux-kernel - Re: [PATCH 6.1 000/600] 6.1.53-rc1 review

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] [thread-next>] [day] [month] [year] [list]

Message-ID: <2023091217-reflux-playroom-017a@gregkh>
Date:   Tue, 12 Sep 2023 11:03:50 +0200
From:   Greg Kroah-Hartman <gregkh@...uxfoundation.org>
To:     Naresh Kamboju <naresh.kamboju@...aro.org>
Cc:     jack@...e.cz, stable@...r.kernel.org, patches@...ts.linux.dev,
        linux-kernel@...r.kernel.org, torvalds@...ux-foundation.org,
        akpm@...ux-foundation.org, linux@...ck-us.net, shuah@...nel.org,
        patches@...nelci.org, lkft-triage@...ts.linaro.org, pavel@...x.de,
        jonathanh@...dia.com, f.fainelli@...il.com,
        sudipm.mukherjee@...il.com, srw@...dewatkins.net, rwarsow@....de,
        conor@...nel.org,
        Daniel Díaz <daniel.diaz@...aro.org>,
        Tom Rix <trix@...hat.com>,
        Nathan Chancellor <nathan@...nel.org>
Subject: Re: [PATCH 6.1 000/600] 6.1.53-rc1 review

On Tue, Sep 12, 2023 at 02:19:34PM +0530, Naresh Kamboju wrote:
> On Tue, 12 Sept 2023 at 07:55, Daniel Díaz <daniel.diaz@...aro.org> wrote:
> >
> > Hello!
> >
> > On Mon, 11 Sept 2023 at 14:58, Daniel Díaz <daniel.diaz@...aro.org> wrote:
> > > On 11/09/23 7:40 a. m., Greg Kroah-Hartman wrote:
> > > > This is the start of the stable review cycle for the 6.1.53 release.
> > > > There are 600 patches in this series, all will be posted as a response
> > > > to this one.  If anyone has any issues with these being applied, please
> > > > let me know.
> > > >
> > > > Responses should be made by Wed, 13 Sep 2023 13:44:56 +0000.
> > > > Anything received after that time might be too late.
> > > >
> > > > The whole patch series can be found in one patch at:
> > > >       https://www.kernel.org/pub/linux/kernel/v6.x/stable-review/patch-6.1.53-rc1.gz
> > > > or in the git tree and branch at:
> > > >       git://git.kernel.org/pub/scm/linux/kernel/git/stable/linux-stable-rc.git linux-6.1.y
> > > > and the diffstat can be found below.
> > > >
> > > > thanks,
> > > >
> > > > greg k-h
> > >
> > > We're seeing this new warning:
> > > -----8<-----
> > >    /builds/linux/fs/udf/inode.c:892:6: warning: variable 'newblock' is used uninitialized whenever 'if' condition is true [-Wsometimes-uninitialized]
> > >      892 |         if (*err < 0)
> > >          |             ^~~~~~~~
> > >    /builds/linux/fs/udf/inode.c:914:9: note: uninitialized use occurs here
> > >      914 |         return newblock;
> > >          |                ^~~~~~~~
> > >    /builds/linux/fs/udf/inode.c:892:2: note: remove the 'if' if its condition is always false
> > >      892 |         if (*err < 0)
> > >          |         ^~~~~~~~~~~~~
> > >      893 |                 goto out_free;
> > >          |                 ~~~~~~~~~~~~~
> > >    /builds/linux/fs/udf/inode.c:699:34: note: initialize the variable 'newblock' to silence this warning
> > >      699 |         udf_pblk_t newblocknum, newblock;
> > >          |                                         ^
> > >          |                                          = 0
> > >    1 warning generated.
> > > ----->8-----
> > >
> > > That's with Clang 17 (and nightly) on:
> > > * arm
> > > * powerpc
> > > * s390
> >
> > For what it's worth, bisection points to 903b487b5ba6 ("udf: Handle
> > error when adding extent to a file").
> 
> I see the following commit is fixing the reported problem.
> 
> commit 6d5ab7c2f7cf90877dab8f2bb06eb5ca8edc73ef
> Author: Tom Rix <trix@...hat.com>
> Date:   Fri Dec 30 12:53:41 2022 -0500
> 
>     udf: initialize newblock to 0
> 
>     The clang build reports this error
>     fs/udf/inode.c:805:6: error: variable 'newblock' is used
> uninitialized whenever 'if' condition is true
> [-Werror,-Wsometimes-uninitialized]
>             if (*err < 0)
>                 ^~~~~~~~
>     newblock is never set before error handling jump.
>     Initialize newblock to 0 and remove redundant settings.
> 
>     Fixes: d8b39db5fab8 ("udf: Handle error when adding extent to a file")

Ah, the Fixes: tag lied!  There is no such git id in Linus's tree
anywhere, so our scripts couldn't match this up at all.

I'll go queue this fix up, thanks for digging it out.

greg k-h