diff options
| author | Al Viro <viro@zeniv.linux.org.uk> | 2024-05-14 08:48:58 -0600 |
|---|---|---|
| committer | Al Viro <viro@zeniv.linux.org.uk> | 2025-06-11 18:00:31 -0400 |
| commit | 2c58d42de71f9c73e40afacc9d062892d2cc8862 (patch) | |
| tree | e29faa87924819c287dd10d7fb1d472480fe359a /drivers/virt | |
| parent | e4de72650202e3da56b107f8e537d2b4a0341e40 (diff) | |
fix locking in efi_secret_unlink()
We used to need securityfs_remove() to undo simple_pin_fs() done when
the file had been created and to drop the second extra reference
taken at the same time. Now that neither is needed (or done by
securityfs_remove()), we can simply call simple_unlink() and be done
with that - the broken games with locking had been there only for the
sake of securityfs_remove().
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Diffstat (limited to 'drivers/virt')
| -rw-r--r-- | drivers/virt/coco/efi_secret/efi_secret.c | 10 |
1 files changed, 1 insertions, 9 deletions
diff --git a/drivers/virt/coco/efi_secret/efi_secret.c b/drivers/virt/coco/efi_secret/efi_secret.c index 1864f9f80617..f2da4819ec3b 100644 --- a/drivers/virt/coco/efi_secret/efi_secret.c +++ b/drivers/virt/coco/efi_secret/efi_secret.c @@ -136,15 +136,7 @@ static int efi_secret_unlink(struct inode *dir, struct dentry *dentry) if (s->fs_files[i] == dentry) s->fs_files[i] = NULL; - /* - * securityfs_remove tries to lock the directory's inode, but we reach - * the unlink callback when it's already locked - */ - inode_unlock(dir); - securityfs_remove(dentry); - inode_lock(dir); - - return 0; + return simple_unlink(inode, dentry); } static const struct inode_operations efi_secret_dir_inode_operations = { |