diff options
author | Miao Xie <miaox@cn.fujitsu.com> | 2012-11-01 03:34:54 -0400 |
---|---|---|
committer | Josef Bacik <jbacik@fusionio.com> | 2012-12-12 17:15:22 -0500 |
commit | bbe1426764e5dfaa57e7b12cc954acdb3fb7f94b (patch) | |
tree | c6f5fea584e15ccbfa56308fdac998ef4ffacd52 /fs | |
parent | 315a9850da2b89c83971b26fe54a60f22bdd91ad (diff) |
Btrfs: fix unprotected extent map operation when logging file extents
We forget to protect the modified_extents list, fix it.
Signed-off-by: Miao Xie <miaox@cn.fujitsu.com>
Reviewed-by: Liu Bo <bo.li.liu@oracle.com>
Signed-off-by: Chris Mason <chris.mason@fusionio.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/btrfs/tree-log.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/fs/btrfs/tree-log.c b/fs/btrfs/tree-log.c index bcf0e48b1932..d1947af67bcd 100644 --- a/fs/btrfs/tree-log.c +++ b/fs/btrfs/tree-log.c | |||
@@ -3526,8 +3526,10 @@ next_slot: | |||
3526 | struct extent_map_tree *tree = &BTRFS_I(inode)->extent_tree; | 3526 | struct extent_map_tree *tree = &BTRFS_I(inode)->extent_tree; |
3527 | struct extent_map *em, *n; | 3527 | struct extent_map *em, *n; |
3528 | 3528 | ||
3529 | write_lock(&tree->lock); | ||
3529 | list_for_each_entry_safe(em, n, &tree->modified_extents, list) | 3530 | list_for_each_entry_safe(em, n, &tree->modified_extents, list) |
3530 | list_del_init(&em->list); | 3531 | list_del_init(&em->list); |
3532 | write_unlock(&tree->lock); | ||
3531 | } | 3533 | } |
3532 | 3534 | ||
3533 | if (inode_only == LOG_INODE_ALL && S_ISDIR(inode->i_mode)) { | 3535 | if (inode_only == LOG_INODE_ALL && S_ISDIR(inode->i_mode)) { |