diff options
| author |   Josef Bacik <josef@toxicpanda.com> | 2020-01-24 09:32:50 -0500 |
|---|---|---|
| committer |   David Sterba <dsterba@suse.com> | 2020-03-23 17:01:31 +0100 |
| commit | 5119cfc36f6da62ee7c8f38208afece006a27fcb (patch) | |
| tree | 7483174c656f165685ec002957194faacbe53a19 /fs/btrfs/transaction.c | |
| parent | btrfs: hold a ref on the root in get_subvol_name_from_objectid (diff) | |
| download | linux-dev-5119cfc36f6da62ee7c8f38208afece006a27fcb.tar.xz linux-dev-5119cfc36f6da62ee7c8f38208afece006a27fcb.zip | |
btrfs: hold a ref on the root in create_pending_snapshot
We create the snapshot and then use it for a bunch of things, we need to
hold a ref on it while we're messing with it.
Signed-off-by: Josef Bacik <josef@toxicpanda.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/transaction.c')
| -rw-r--r-- | fs/btrfs/transaction.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/fs/btrfs/transaction.c b/fs/btrfs/transaction.c index bcf23b06e67f..3fa2e7d52eda 100644 --- a/fs/btrfs/transaction.c +++ b/fs/btrfs/transaction.c @@ -1639,6 +1639,12 @@ static noinline int create_pending_snapshot(struct btrfs_trans_handle *trans, btrfs_abort_transaction(trans, ret); goto fail; } + if (!btrfs_grab_fs_root(pending->snap)) { + ret = -ENOENT; + pending->snap = NULL; + btrfs_abort_transaction(trans, ret); + goto fail; + } ret = btrfs_reloc_post_snapshot(trans, pending); if (ret) { |