diff options
author | Ian Kent <raven@themaw.net> | 2010-11-21 21:21:38 -0500 |
---|---|---|
committer | Chris Mason <chris.mason@oracle.com> | 2010-11-27 13:37:44 -0500 |
commit | 619c8c763928841b1112e1d417f88bc1d44daecb (patch) | |
tree | 23fc73cf043faac2cdb15a0b22e6e9e29a69797a | |
parent | bc1cbf1f86aa2501efa9ca637c736fce6bcc4b1d (diff) |
Btrfs - fix race between btrfs_get_sb() and umount
When mounting a btrfs file system btrfs_test_super() may attempt to
use sb->s_fs_info, the btrfs root, of a super block that is going away
and that has had the btrfs root set to NULL in its ->put_super(). But
if the super block is going away it cannot be an existing super block
so we can return false in this case.
Signed-off-by: Ian Kent <raven@themaw.net>
Signed-off-by: Chris Mason <chris.mason@oracle.com>
-rw-r--r-- | fs/btrfs/super.c | 6 |
1 files changed, 6 insertions, 0 deletions
diff --git a/fs/btrfs/super.c b/fs/btrfs/super.c index 66e4612a7916..141fb317d3bc 100644 --- a/fs/btrfs/super.c +++ b/fs/btrfs/super.c | |||
@@ -566,6 +566,12 @@ static int btrfs_test_super(struct super_block *s, void *data) | |||
566 | struct btrfs_fs_devices *test_fs_devices = data; | 566 | struct btrfs_fs_devices *test_fs_devices = data; |
567 | struct btrfs_root *root = btrfs_sb(s); | 567 | struct btrfs_root *root = btrfs_sb(s); |
568 | 568 | ||
569 | /* | ||
570 | * If this super block is going away, return false as it | ||
571 | * can't match as an existing super block. | ||
572 | */ | ||
573 | if (!atomic_read(&s->s_active)) | ||
574 | return 0; | ||
569 | return root->fs_info->fs_devices == test_fs_devices; | 575 | return root->fs_info->fs_devices == test_fs_devices; |
570 | } | 576 | } |
571 | 577 | ||