By comparison logic if ret => 0 -> ret=0; So lets cleanup compare logic Signed-off-by: Timofey Titovets <nefelim4ag@xxxxxxxxx> --- fs/btrfs/backref.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/fs/btrfs/backref.c b/fs/btrfs/backref.c index 897d664a9..f53045891 100644 --- a/fs/btrfs/backref.c +++ b/fs/btrfs/backref.c @@ -592,10 +592,10 @@ static int add_all_parents(struct btrfs_root *root, struct btrfs_path *path, ret = btrfs_next_old_item(root, path, time_seq); } - if (ret > 0) - ret = 0; - else if (ret < 0) + if (ret < 0) free_inode_elem_list(eie); + else + ret = 0; return ret; } -- 2.13.0 -- To unsubscribe from this list: send the line "unsubscribe linux-btrfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
