bcachefs: .get_parent() should return an error pointer
authorKent Overstreet <kent.overstreet@linux.dev>
Thu, 2 Nov 2023 20:54:00 +0000 (16:54 -0400)
committerKent Overstreet <kent.overstreet@linux.dev>
Sun, 5 Nov 2023 02:19:13 +0000 (22:19 -0400)
Delete the useless check for inum == 0; we'll return -ENOENT without it,
which is what we want.

Signed-off-by: Kent Overstreet <kent.overstreet@linux.dev>
fs/bcachefs/fs.c

index 6642b88c41a0e27379d189372d9a369d282c338d..a425c7783cda08e9c75fa8a6567e2e93c850b29a 100644 (file)
@@ -1213,9 +1213,6 @@ static struct dentry *bch2_get_parent(struct dentry *child)
                .inum = inode->ei_inode.bi_dir,
        };
 
-       if (!parent_inum.inum)
-               return NULL;
-
        return d_obtain_alias(bch2_vfs_inode_get(c, parent_inum));
 }