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 PHC | |
Open Source and information security mailing list archives
| ||
|
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
Powered by blists - more mailing lists