From: Su Yue <suy.fnst@xxxxxxxxxxxxxx>
btrfs_unlink() uses btrfs_lookup_inode_ref() to look up inode_ref
but forget inode_extref case.
Let btrfs_unlink() call btrfs_lookup_inode_extref() if inode_ref is
found and EXTENDED_IREF feature is enabled.
Fixes: 0cc75eddd093 ("btrfs-progs: Add btrfs_unlink() and btrfs_add_link() functions.")
Signed-off-by: Su Yue <suy.fnst@xxxxxxxxxxxxxx>
---
inode.c | 14 +++++++++++++-
1 file changed, 13 insertions(+), 1 deletion(-)
diff --git a/inode.c b/inode.c
index 2398bca4a109..598ad0ab6b4c 100644
--- a/inode.c
+++ b/inode.c
@@ -277,6 +277,7 @@ int btrfs_unlink(struct btrfs_trans_handle *trans, struct btrfs_root *root,
struct btrfs_key key;
struct btrfs_inode_item *inode_item;
struct btrfs_inode_ref *inode_ref;
+ struct btrfs_inode_extref *inode_extref = NULL;
struct btrfs_dir_item *dir_item;
u64 inode_size;
u32 nlinks;
@@ -296,7 +297,18 @@ int btrfs_unlink(struct btrfs_trans_handle *trans, struct btrfs_root *root,
ret = PTR_ERR(inode_ref);
goto out;
}
- if (inode_ref)
+
+ if (!inode_ref && btrfs_fs_incompat(root->fs_info, EXTENDED_IREF)) {
+ btrfs_release_path(path);
+ inode_extref = btrfs_lookup_inode_extref(trans, root, path,
+ name, namelen, ino, parent_ino, 0);
+ if (IS_ERR(inode_extref)) {
+ ret = PTR_ERR(inode_extref);
+ goto out;
+ }
+ }
+
+ if (inode_ref || inode_extref)
del_inode_ref = 1;
btrfs_release_path(path);
--
2.18.0
