btrfs: handle btrfs_record_root_in_trans failure in create_subvol
authorJosef Bacik <josef@toxicpanda.com>
Fri, 12 Mar 2021 20:25:06 +0000 (15:25 -0500)
committerDavid Sterba <dsterba@suse.com>
Mon, 19 Apr 2021 15:25:20 +0000 (17:25 +0200)
btrfs_record_root_in_trans will return errors in the future, so handle
the error properly in create_subvol.

Signed-off-by: Josef Bacik <josef@toxicpanda.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/ioctl.c

index 3415a9f..37c92a9 100644 (file)
@@ -721,7 +721,12 @@ static noinline int create_subvol(struct inode *dir,
        /* Freeing will be done in btrfs_put_root() of new_root */
        anon_dev = 0;
 
-       btrfs_record_root_in_trans(trans, new_root);
+       ret = btrfs_record_root_in_trans(trans, new_root);
+       if (ret) {
+               btrfs_put_root(new_root);
+               btrfs_abort_transaction(trans, ret);
+               goto fail;
+       }
 
        ret = btrfs_create_subvol_root(trans, new_root, root);
        btrfs_put_root(new_root);