linux-ext4 - Re: [bug report] ext4: fast commit recovery path

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]

Date:   Wed, 1 Dec 2021 08:35:34 -0800
From:   harshad shirwadkar <harshadshirwadkar@...il.com>
To:     Dan Carpenter <dan.carpenter@...cle.com>
Cc:     Ext4 Developers List <linux-ext4@...r.kernel.org>
Subject: Re: [bug report] ext4: fast commit recovery path

Thanks for the report Dan. This patch -
https://patchwork.ozlabs.org/project/linux-ext4/patch/20211201163421.2631661-1-harshads@google.com/
should take care of this report.

- Harshad

On Tue, Nov 30, 2021 at 1:40 AM Dan Carpenter <dan.carpenter@...cle.com> wrote:
>
> Hello Harshad Shirwadkar,
>
> The patch 8016e29f4362: "ext4: fast commit recovery path" from Oct
> 15, 2020, leads to the following Smatch static checker warnings:
>
>         fs/ext4/inode.c:4533 __ext4_get_inode_loc_noinmem()
>         error: uninitialized symbol 'err_blk'.
>
>         fs/ext4/inode.c:4548 ext4_get_inode_loc()
>         error: uninitialized symbol 'err_blk'.
>
> fs/ext4/inode.c
>     4523 static int __ext4_get_inode_loc_noinmem(struct inode *inode,
>     4524                                         struct ext4_iloc *iloc)
>     4525 {
>     4526         ext4_fsblk_t err_blk;
>     4527         int ret;
>     4528
>     4529         ret = __ext4_get_inode_loc(inode->i_sb, inode->i_ino, NULL, iloc,
>     4530                                         &err_blk);
>     4531
>     4532         if (ret == -EIO)
> --> 4533                 ext4_error_inode_block(inode, err_blk, EIO,
>
> Only the last return -EIO sets err_blk.  The first return -EIO leaves it
> uninitialized.
>
>     4534                                         "unable to read itable block");
>     4535
>     4536         return ret;
>     4537 }
>
> regards,
> dan carpenter