mirror of
https://github.com/torvalds/linux.git
synced 2026-05-30 18:13:41 +02:00
xfs: check return value of xchk_scrub_create_subord
Fix this function to return NULL instead of a mangled ENOMEM, then fix
the callers to actually check for a null pointer and return ENOMEM.
Most of the corrections here are for code merged between 6.2 and 6.10.
Cc: r772577952@gmail.com
Cc: <stable@vger.kernel.org> # v6.12
Fixes: 1a5f6e08d4 ("xfs: create subordinate scrub contexts for xchk_metadata_inode_subtype")
Signed-off-by: "Darrick J. Wong" <djwong@kernel.org>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Tested-by: Jiaming Zhang <r772577952@gmail.com>
This commit is contained in:
parent
ba408d299a
commit
ca27313fb3
|
|
@ -1399,6 +1399,9 @@ xchk_metadata_inode_subtype(
|
|||
int error;
|
||||
|
||||
sub = xchk_scrub_create_subord(sc, scrub_type);
|
||||
if (!sub)
|
||||
return -ENOMEM;
|
||||
|
||||
error = sub->sc.ops->scrub(&sub->sc);
|
||||
xchk_scrub_free_subord(sub);
|
||||
return error;
|
||||
|
|
|
|||
|
|
@ -1136,6 +1136,9 @@ xrep_metadata_inode_subtype(
|
|||
* setup/teardown routines.
|
||||
*/
|
||||
sub = xchk_scrub_create_subord(sc, scrub_type);
|
||||
if (!sub)
|
||||
return -ENOMEM;
|
||||
|
||||
error = sub->sc.ops->scrub(&sub->sc);
|
||||
if (error)
|
||||
goto out;
|
||||
|
|
|
|||
|
|
@ -634,7 +634,7 @@ xchk_scrub_create_subord(
|
|||
|
||||
sub = kzalloc(sizeof(*sub), XCHK_GFP_FLAGS);
|
||||
if (!sub)
|
||||
return ERR_PTR(-ENOMEM);
|
||||
return NULL;
|
||||
|
||||
sub->old_smtype = sc->sm->sm_type;
|
||||
sub->old_smflags = sc->sm->sm_flags;
|
||||
|
|
|
|||
Loading…
Reference in New Issue
Block a user