linux-kernel - Re: [PATCH v2] fuse: Uninitialized variable in fuse_epoch

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 for Android: free password hash cracker in your pocket

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <CAJfpegsnFYLkHP=KCwkK1UW=KJOqFPKX9H28G4BERkzebg=igQ@mail.gmail.com>
Date: Wed, 26 Nov 2025 12:47:47 +0100
From: Miklos Szeredi <miklos@...redi.hu>
To: Dan Carpenter <dan.carpenter@...aro.org>
Cc: Luis Henriques <luis@...lia.com>, linux-fsdevel@...r.kernel.org, 
	linux-kernel@...r.kernel.org, kernel-janitors@...r.kernel.org
Subject: Re: [PATCH v2] fuse: Uninitialized variable in fuse_epoch_work()

On Mon, 24 Nov 2025 at 07:05, Dan Carpenter <dan.carpenter@...aro.org> wrote:
>
> The fuse_ilookup() function only sets *fm on the success path so this
> "if (fm) {" NULL check doesn't work.  The "fm" pointer is either
> uninitialized or valid.  Check the "inode" pointer instead.
>
> Also, while it's not necessary, it is cleaner to move the iput(inode)
> under the NULL check as well.
>
> Fixes: 64becd224ff9 ("fuse: new work queue to invalidate dentries from old epochs")
> Signed-off-by: Dan Carpenter <dan.carpenter@...aro.org>
> Reviewed-by: Luis Henriques <luis@...lia.com>

Applied, thanks.

Miklos