Re: [PATCH v2 6.14.y] btrfs: fix the inode leak in btrfs_iget()

[Greg KH <gregkh@linuxfoundation.org>]

On Mon, May 05, 2025 at 08:03:22AM -0700, Penglei Jiang wrote:
> The commit 48c1d1bb525b1c44b8bdc8e7ec5629cb6c2b9fc4 fails to compile
> in kernel version 6.14.
> 
> Version 6.14:
>     struct inode *btrfs_iget(u64 ino, struct btrfs_root *root)
>     {
>             struct inode *inode;
>             ~~~~~~~~~~~~~~
>             ...
>             inode = btrfs_iget_locked(ino, root);
>             ...
>     }
> 
> Version 6.15:
>     struct btrfs_inode *btrfs_iget(u64 ino, struct btrfs_root *root)
>     {
>             struct btrfs_inode *inode;
>             ~~~~~~~~~~~~~~~~~~~~
>             ...
>             inode = btrfs_iget_locked(ino, root);
>             ...
>     }
> 
> In kernel version 6.14, the function btrfs_iget_locked() returns a
> struct inode *, so the patch code adjusts to use iget_failed(inode).
> 
> Reported-by: Penglei Jiang <superman.xpt@gmail.com>
> Link: https://lore.kernel.org/linux-btrfs/20250421102425.44431-1-superman.xpt@gmail.com/
> Fixes: 7c855e16ab72 ("btrfs: remove conditional path allocation in btrfs_read_locked_inode()")
> Signed-off-by: Penglei Jiang <superman.xpt@gmail.com>
> ---
>  fs/btrfs/inode.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

This is not a v2, it is a totally different patch.

Please just fix up your original backport to build properly when you
submit it, don't send a "fix up patch" to solve a build problem a
previous patch added.

thanks,

greg k-h