]> git.ipfire.org Git - thirdparty/linux.git/commitdiff
xfs: fix stupid compiler warning
authorDarrick J. Wong <djwong@kernel.org>
Thu, 4 Dec 2025 21:44:15 +0000 (13:44 -0800)
committerCarlos Maiolino <cem@kernel.org>
Tue, 16 Dec 2025 07:50:07 +0000 (08:50 +0100)
gcc 14.2 warns about:

xfs_attr_item.c: In function ‘xfs_attr_recover_work’:
xfs_attr_item.c:785:9: warning: ‘ip’ may be used uninitialized [-Wmaybe-uninitialized]
  785 |         xfs_trans_ijoin(tp, ip, 0);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~
xfs_attr_item.c:740:42: note: ‘ip’ was declared here
  740 |         struct xfs_inode                *ip;
      |                                          ^~

I think this is bogus since xfs_attri_recover_work either returns a real
pointer having initialized ip or an ERR_PTR having not touched it, but
the tools are smarter than me so let's just null-init the variable
anyway.

Cc: stable@vger.kernel.org # v6.8
Fixes: e70fb328d52772 ("xfs: recreate work items when recovering intent items")
Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Reviewed-by: Carlos Maiolino <cmaiolino@redhat.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Carlos Maiolino <cem@kernel.org>
fs/xfs/xfs_attr_item.c

index c3a593319bee71497f1c688062f06d37d6ea20b2..e8fa326ac995bcc67e60bca05f65b8a42bda1d4a 100644 (file)
@@ -737,7 +737,7 @@ xfs_attr_recover_work(
        struct xfs_attri_log_item       *attrip = ATTRI_ITEM(lip);
        struct xfs_attr_intent          *attr;
        struct xfs_mount                *mp = lip->li_log->l_mp;
-       struct xfs_inode                *ip;
+       struct xfs_inode                *ip = NULL;
        struct xfs_da_args              *args;
        struct xfs_trans                *tp;
        struct xfs_trans_res            resv;