fs: btrfs: fix btrfs_search_tree invalid results
authorPierre Bourdon <delroth@gmail.com>
Tue, 16 Apr 2019 00:47:14 +0000 (02:47 +0200)
committerTom Rini <trini@konsulko.com>
Fri, 26 Apr 2019 21:53:16 +0000 (17:53 -0400)
btrfs_search_tree should return the first item in the tree that is
greater or equal to the searched item.

The search algorithm did not properly handle the edge case where the
searched item is higher than the last item of the node but lower than
the first item of the next node. Instead of properly returning the first
item of the next node, it was returning an invalid path pointer
(pointing to a non-existent item after the last item of the node + 1).

This fixes two issues in the btrfs driver:
  - Looking for a ROOT_ITEM could fail if it was the first item of its
    leaf node.
  - Iterating through DIR_INDEX entries (for readdir) could fail if the
    first DIR_INDEX entry was the first item of a leaf node.

Signed-off-by: Pierre Bourdon <delroth@gmail.com>
Cc: Marek Behun <marek.behun@nic.cz>
fs/btrfs/ctree.c

index d248d79932753f3e7e2d9bd9e32554309b7403dc..7fae383f1509b3d079f082cc841dbbab153846d9 100644 (file)
@@ -185,10 +185,20 @@ int btrfs_search_tree(const struct btrfs_root *root, struct btrfs_key *key,
                p->slots[lvl] = slot;
                p->nodes[lvl] = buf;
 
-               if (lvl)
+               if (lvl) {
                        logical = buf->node.ptrs[slot].blockptr;
-               else
+               } else {
+                       /*
+                        * The path might be invalid if:
+                        *   cur leaf max < searched value < next leaf min
+                        *
+                        * Jump to the next valid element if it exists.
+                        */
+                       if (slot >= buf->header.nritems)
+                               if (btrfs_next_slot(p) < 0)
+                                       goto err;
                        break;
+               }
        }
 
        return 0;