Commit 37a6547d authored by Darrick J. Wong's avatar Darrick J. Wong

xfs: xrep_reap_extents should not destroy the bitmap

Remove the xfs_bitmap_destroy call from the end of xrep_reap_extents
because this sort of violates our rule that the function initializing a
structure should destroy it.
Signed-off-by: default avatarDarrick J. Wong <darrick.wong@oracle.com>
Reviewed-by: default avatarBrian Foster <bfoster@redhat.com>
parent 77ca1eed
...@@ -696,7 +696,7 @@ xrep_agfl( ...@@ -696,7 +696,7 @@ xrep_agfl(
goto err; goto err;
/* Dump any AGFL overflow. */ /* Dump any AGFL overflow. */
return xrep_reap_extents(sc, &agfl_extents, &XFS_RMAP_OINFO_AG, error = xrep_reap_extents(sc, &agfl_extents, &XFS_RMAP_OINFO_AG,
XFS_AG_RESV_AGFL); XFS_AG_RESV_AGFL);
err: err:
xfs_bitmap_destroy(&agfl_extents); xfs_bitmap_destroy(&agfl_extents);
......
...@@ -617,11 +617,9 @@ xrep_reap_extents( ...@@ -617,11 +617,9 @@ xrep_reap_extents(
error = xrep_reap_block(sc, fsbno, oinfo, type); error = xrep_reap_block(sc, fsbno, oinfo, type);
if (error) if (error)
goto out; break;
} }
out:
xfs_bitmap_destroy(bitmap);
return error; return error;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment