xfs: check xfarray iteration errors when committing unlinked inode lists

LOLLM noticed that we neglect to check for xfarray_iter itself returning
errors when writing a new AGI.  Fix that.

Cc: stable@vger.kernel.org # v6.10
Fixes: ab97f4b1c0 ("xfs: repair AGI unlinked inode bucket lists")
Signed-off-by: Darrick J. Wong <djwong@kernel.org>
Assisted-by: LOLLM # finding obvious bugs
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Carlos Maiolino <cem@kernel.org>
This commit is contained in:
Darrick J. Wong
2026-07-26 22:25:50 -07:00
committed by Carlos Maiolino
parent 5644fab990
commit 2daf3ed5d0

View File

@@ -1675,6 +1675,8 @@ xrep_iunlink_commit(
if (error)
return error;
}
if (error < 0)
return error;
/* Fix all the back links */
idx = XFARRAY_CURSOR_INIT;
@@ -1683,6 +1685,8 @@ xrep_iunlink_commit(
if (error)
return error;
}
if (error < 0)
return error;
/* Copy the staged iunlink buckets to the new AGI. */
for (i = 0; i < XFS_AGI_UNLINKED_BUCKETS; i++) {