@path is always NULL when it comes to the if branch. Signed-off-by: Liu Bo <bo....@linux.alibaba.com> --- fs/btrfs/inode.c | 17 +++++++---------- 1 file changed, 7 insertions(+), 10 deletions(-)
diff --git a/fs/btrfs/inode.c b/fs/btrfs/inode.c index 8b135a46835f..4b79916472fb 100644 --- a/fs/btrfs/inode.c +++ b/fs/btrfs/inode.c @@ -6825,18 +6825,15 @@ struct extent_map *btrfs_get_extent(struct btrfs_inode *inode, em->len = (u64)-1; em->block_len = (u64)-1; + path = btrfs_alloc_path(); if (!path) { - path = btrfs_alloc_path(); - if (!path) { - err = -ENOMEM; - goto out; - } - /* - * Chances are we'll be called again, so go ahead and do - * readahead - */ - path->reada = READA_FORWARD; + err = -ENOMEM; + goto out; } + /* + * Chances are we'll be called again, so go ahead and do readahead. + */ + path->reada = READA_FORWARD; ret = btrfs_lookup_file_extent(NULL, root, path, objectid, start, 0); if (ret < 0) { -- 1.8.3.1