btrfs: fix an error handling path in btrfs_defrag_leaves()
authorChristophe JAILLET <christophe.jaillet@wanadoo.fr>
Mon, 12 Dec 2022 20:01:43 +0000 (21:01 +0100)
committerDavid Sterba <dsterba@suse.com>
Thu, 15 Dec 2022 18:16:50 +0000 (19:16 +0100)
All error handling paths end to 'out', except this memory allocation
failure.

This is spurious. So branch to the error handling path also in this case.
It will add a call to:

memset(&root->defrag_progress, 0,
       sizeof(root->defrag_progress));

Fixes: 6702ed490ca0 ("Btrfs: Add run time btree defrag, and an ioctl to force btree defrag")
Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/defrag.c

index 0a3c261..d81b764 100644 (file)
@@ -358,8 +358,10 @@ int btrfs_defrag_leaves(struct btrfs_trans_handle *trans,
                goto out;
 
        path = btrfs_alloc_path();
-       if (!path)
-               return -ENOMEM;
+       if (!path) {
+               ret = -ENOMEM;
+               goto out;
+       }
 
        level = btrfs_header_level(root->node);