summaryrefslogtreecommitdiffstats
path: root/fs/efivarfs
diff options
context:
space:
mode:
authorArd Biesheuvel <ardb@kernel.org>2024-02-24 18:45:41 +0100
committerArd Biesheuvel <ardb@kernel.org>2024-02-25 09:43:38 +0100
commit9ca01c7adf3993044f59934082087ebb9f7df6d5 (patch)
tree3ebde3654d154e3d62f85476d1b8de2953d745a9 /fs/efivarfs
parentefivarfs: Request at most 512 bytes for variable names (diff)
downloadlinux-9ca01c7adf3993044f59934082087ebb9f7df6d5.tar.xz
linux-9ca01c7adf3993044f59934082087ebb9f7df6d5.zip
efivarfs: Drop redundant cleanup on fill_super() failure
Al points out that kill_sb() will be called if efivarfs_fill_super() fails and so there is no point in cleaning up the efivar entry list. Reported-by: Alexander Viro <viro@zeniv.linux.org.uk> Signed-off-by: Ard Biesheuvel <ardb@kernel.org>
Diffstat (limited to 'fs/efivarfs')
-rw-r--r--fs/efivarfs/super.c7
1 files changed, 1 insertions, 6 deletions
diff --git a/fs/efivarfs/super.c b/fs/efivarfs/super.c
index 6038dd39367a..210daac79748 100644
--- a/fs/efivarfs/super.c
+++ b/fs/efivarfs/super.c
@@ -343,12 +343,7 @@ static int efivarfs_fill_super(struct super_block *sb, struct fs_context *fc)
if (err)
return err;
- err = efivar_init(efivarfs_callback, (void *)sb, true,
- &sfi->efivarfs_list);
- if (err)
- efivar_entry_iter(efivarfs_destroy, &sfi->efivarfs_list, NULL);
-
- return err;
+ return efivar_init(efivarfs_callback, sb, true, &sfi->efivarfs_list);
}
static int efivarfs_get_tree(struct fs_context *fc)