diff options
author | Alexander Block <ablock84@googlemail.com> | 2012-07-28 06:51:32 -0400 |
---|---|---|
committer | Chris Mason <chris.mason@fusionio.com> | 2012-10-01 15:18:50 -0400 |
commit | adbe7fb6c4750621a56867d9bb1980da3a4b8f33 (patch) | |
tree | 9acf6c9421aa9593c0647cd5b0d45b033d1cac25 | |
parent | 52f9e53ede8e1b261e68216c6c2f32bb3f26c795 (diff) |
Btrfs: don't break in the final loop of find_extent_clone
If we break, we may miss the clone from send_root which we prefer
over all other clones.
Commit is a result of Arne's review.
Reported-by: Arne Jansen <sensille@gmx.net>
Signed-off-by: Alexander Block <ablock84@googlemail.com>
-rw-r--r-- | fs/btrfs/send.c | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/fs/btrfs/send.c b/fs/btrfs/send.c index d2a4ee9125df..68b2543e5d6c 100644 --- a/fs/btrfs/send.c +++ b/fs/btrfs/send.c | |||
@@ -1257,7 +1257,6 @@ verbose_printk(KERN_DEBUG "btrfs: find_extent_clone: data_offset=%llu, " | |||
1257 | else if (sctx->clone_roots[i].root == sctx->send_root) | 1257 | else if (sctx->clone_roots[i].root == sctx->send_root) |
1258 | /* prefer clones from send_root over others */ | 1258 | /* prefer clones from send_root over others */ |
1259 | cur_clone_root = sctx->clone_roots + i; | 1259 | cur_clone_root = sctx->clone_roots + i; |
1260 | break; | ||
1261 | } | 1260 | } |
1262 | 1261 | ||
1263 | } | 1262 | } |