btrfs: tree-checker: fix bounds check in check_inode_extref()

commit e92c2941204de7b62e9c2deecfeb9eaefe54a22a upstream.

The parentheses for the unlikely() annotation were put in the wrong
place so it means that the condition is basically never true and the
bounds checking is skipped.

Fixes: aab9458b9f00 ("btrfs: tree-checker: add inode extref checks")
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Reviewed-by: Qu Wenruo <wqu@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/fs/btrfs/tree-checker.c b/fs/btrfs/tree-checker.c
index a83e455f813bf26c5775e3661728a753f6c06342..30bc8eb28005cd988de35f4962be0d439ba749ce 100644 (file)
--- a/fs/btrfs/tree-checker.c
+++ b/fs/btrfs/tree-checker.c
@@ -1797,7 +1797,7 @@ static int check_inode_extref(struct extent_buffer *leaf,
                struct btrfs_inode_extref *extref = (struct btrfs_inode_extref *)ptr;
                u16 namelen;
 
-               if (unlikely(ptr + sizeof(*extref)) > end) {
+               if (unlikely(ptr + sizeof(*extref) > end)) {
                        inode_ref_err(leaf, slot,
                        "inode extref overflow, ptr %lu end %lu inode_extref size %zu",
                                      ptr, end, sizeof(*extref));