Btrfs: fix check for changed extent in is_extent_unchanged
authorAlexander Block <ablock84@googlemail.com>
Wed, 1 Aug 2012 10:46:05 +0000 (12:46 +0200)
committerChris Mason <chris.mason@fusionio.com>
Mon, 1 Oct 2012 19:18:57 +0000 (15:18 -0400)
The previous check was working fine, but this check should be
easier to read. Also, we could theoritically have some exotic
bugs with the previous checks.

Signed-off-by: Alexander Block <ablock84@googlemail.com>
fs/btrfs/send.c

index cfbe987f854bddeef04d3ff774c4562f1a404b78..35222d5420f61ff64f1e13ac0a92a2c4b8894c76 100644 (file)
@@ -3819,8 +3819,8 @@ static int is_extent_unchanged(struct send_ctx *sctx,
                /*
                 * Check if we have the same extent.
                 */
-               if (left_disknr + left_offset_fixed !=
-                               right_disknr + right_offset) {
+               if (left_disknr != right_disknr ||
+                   left_offset_fixed != right_offset) {
                        ret = 0;
                        goto out;
                }