Btrfs: bail out on error during replay_dir_deletes
authorLiu Bo <bo.liu@linux.alibaba.com>
Mon, 2 Apr 2018 17:59:48 +0000 (01:59 +0800)
committerDavid Sterba <dsterba@suse.com>
Thu, 5 Apr 2018 17:22:26 +0000 (19:22 +0200)
If errors were returned by btrfs_next_leaf(), replay_dir_deletes needs
to bail out, otherwise @ret would be forced to be 0 after 'break;' and
the caller won't be aware of it.

Fixes: e02119d5a7b4 ("Btrfs: Add a write ahead tree log to optimize synchronous operations")
Reviewed-by: Nikolay Borisov <nborisov@suse.com>
Signed-off-by: Liu Bo <bo.liu@linux.alibaba.com>
Signed-off-by: David Sterba <dsterba@suse.com>
fs/btrfs/tree-log.c

index c6bfa86e2f228ef27965e78faabd2d44151425b9..678154d4b78f27f80193cdd87fa395ec8df8f382 100644 (file)
@@ -2352,8 +2352,10 @@ again:
                        nritems = btrfs_header_nritems(path->nodes[0]);
                        if (path->slots[0] >= nritems) {
                                ret = btrfs_next_leaf(root, path);
-                               if (ret)
+                               if (ret == 1)
                                        break;
+                               else if (ret < 0)
+                                       goto out;
                        }
                        btrfs_item_key_to_cpu(path->nodes[0], &found_key,
                                              path->slots[0]);