| 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
| ||
|
Message-ID: <307a2ff6-eeb7-4b86-a5c6-c438dcec618e@mandelbit.com> Date: Mon, 18 Nov 2024 11:02:27 +0100 From: Antonio Quartulli <antonio@...delbit.com> To: Zhihao Cheng <chengzhihao1@...wei.com>, linux-mtd@...ts.infradead.org Cc: Richard Weinberger <richard@....at>, linux-kernel@...r.kernel.org Subject: Re: [PATCH] ubifs: fix uninitialized variable usage On 16/11/2024 03:02, Zhihao Cheng wrote: > 在 2024/11/16 6:26, Antonio Quartulli 写道: >> In ubifs_jnl_write_inode(), when an inode cannot be deleted >> due to too many xattrs, err is passed to ubifs_ro_mode() >> uninitialized, thus leading to bogus error reporting. >> >> Fix this case by passing -EPERM, which is the same value that >> ubifs_jnl_write_inode() is going to return to the caller. >> >> This fixes 1 UNINIT issue reported by Coverity >> Report: CID 1601860: Uninitialized scalar variable (UNINIT) >> >> Cc: Richard Weinberger <richard@....at> >> Cc: Zhihao Cheng <chengzhihao1@...wei.com> >> Cc: linux-kernel@...r.kernel.org (open list) >> Signed-off-by: Antonio Quartulli <antonio@...delbit.com> >> --- >> fs/ubifs/journal.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) > > Hi Antonio, thanks for the patch. Nathan has sent a patch to fix it. > https://lore.kernel.org/linux-mtd/ > b560f413-70f1-8ebb-7403-34591658ca86@...wei.com/T/#t Perfect! Thanks for letting me know. Regards, -- Antonio Quartulli CEO and Co-Founder Mandelbit Srl https://www.mandelbit.com
Powered by blists - more mailing lists