diff options
| author | Ard Biesheuvel <ardb@kernel.org> | 2020-05-22 18:15:49 +0200 |
|---|---|---|
| committer | Ard Biesheuvel <ardb@kernel.org> | 2020-05-22 18:19:06 +0200 |
| commit | d8bd8c6e2cfab8b78b537715255be8d7557791c0 (patch) | |
| tree | 2a37490a93705a81c7c17aeb75b1270715099b1f | |
| parent | efi/libstub: Use pool allocation for the command line (diff) | |
| download | linux-d8bd8c6e2cfab8b78b537715255be8d7557791c0.tar.xz linux-d8bd8c6e2cfab8b78b537715255be8d7557791c0.zip | |
efi/efivars: Add missing kobject_put() in sysfs entry creation error path
The documentation provided by kobject_init_and_add() clearly spells out
the need to call kobject_put() on the kobject if an error is returned.
Add this missing call to the error path.
Cc: <stable@vger.kernel.org>
Reported-by: 亿一 <teroincn@gmail.com>
Signed-off-by: Ard Biesheuvel <ardb@kernel.org>
| -rw-r--r-- | drivers/firmware/efi/efivars.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/firmware/efi/efivars.c b/drivers/firmware/efi/efivars.c index 78ad1ba8c987..26528a46d99e 100644 --- a/drivers/firmware/efi/efivars.c +++ b/drivers/firmware/efi/efivars.c @@ -522,8 +522,10 @@ efivar_create_sysfs_entry(struct efivar_entry *new_var) ret = kobject_init_and_add(&new_var->kobj, &efivar_ktype, NULL, "%s", short_name); kfree(short_name); - if (ret) + if (ret) { + kobject_put(&new_var->kobj); return ret; + } kobject_uevent(&new_var->kobj, KOBJ_ADD); if (efivar_entry_add(new_var, &efivar_sysfs_list)) { |