diff options
author | Armin Wolf <W_Armin@gmx.de> | 2023-09-25 17:28:18 +0300 |
---|---|---|
committer | Hans de Goede <hdegoede@redhat.com> | 2023-10-04 12:52:17 +0300 |
commit | 528ab3e605cabf2f9c9bd5944d3bfe15f6e94f81 (patch) | |
tree | cfff5f66036d5c7aaee06ba85b9da0d1f64ad4e9 /scripts/gdb/linux/proc.py | |
parent | bc3b6f59463ba9f4367a80331213db491766b5a1 (diff) | |
download | linux-528ab3e605cabf2f9c9bd5944d3bfe15f6e94f81.tar.xz |
platform/x86: think-lmi: Fix reference leak
If a duplicate attribute is found using kset_find_obj(), a reference
to that attribute is returned which needs to be disposed accordingly
using kobject_put(). Move the setting name validation into a separate
function to allow for this change without having to duplicate the
cleanup code for this setting.
As a side note, a very similar bug was fixed in
commit 7295a996fdab ("platform/x86: dell-sysman: Fix reference leak"),
so it seems that the bug was copied from that driver.
Compile-tested only.
Fixes: 1bcad8e510b2 ("platform/x86: think-lmi: Fix issues with duplicate attributes")
Reviewed-by: Mark Pearson <mpearson-lenovo@squebb.ca>
Reviewed-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Signed-off-by: Armin Wolf <W_Armin@gmx.de>
Link: https://lore.kernel.org/r/20230925142819.74525-2-W_Armin@gmx.de
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Diffstat (limited to 'scripts/gdb/linux/proc.py')
0 files changed, 0 insertions, 0 deletions