diff options
| author | Matthew Garrett <mjg59@srcf.ucam.org> | 2008-08-20 22:50:16 +0000 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@suse.de> | 2008-09-08 04:44:16 -0700 |
| commit | e76d1b317c210f0d9508f2b0bf3204b6849d3da6 (patch) | |
| tree | a603f745f090c7977d8411b47061c77094db8e5d | |
| parent | 045eff1504ce63a080b79f931ec3d8e136fb65cd (diff) | |
| download | kernel-common-e76d1b317c210f0d9508f2b0bf3204b6849d3da6.tar.gz kernel-common-e76d1b317c210f0d9508f2b0bf3204b6849d3da6.tar.bz2 kernel-common-e76d1b317c210f0d9508f2b0bf3204b6849d3da6.zip | |
eeepc-laptop: fix use after free
commit f14413184b1de4dcbd5ec3e7c129c3ce2079f543 upstream
Date: Wed, 20 Aug 2008 14:08:57 -0700
Subject: eeepc-laptop: fix use after free
eeepc-laptop uses the hwmon struct after unregistering the device, causing
an oops on module unload. Flip the ordering to fix.
Signed-off-by: Matthew Garrett <mjg@redhat.com>
Cc: Henrique de Moraes Holschuh <hmh@hmh.eng.br>
Cc: Corentin Chary <corentincj@iksaif.net>
Cc: Karol Kozimor <sziwan@users.sourceforge.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
| -rw-r--r-- | drivers/misc/eeepc-laptop.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/misc/eeepc-laptop.c b/drivers/misc/eeepc-laptop.c index 6d727609097f..3f3abf941f17 100644 --- a/drivers/misc/eeepc-laptop.c +++ b/drivers/misc/eeepc-laptop.c @@ -553,9 +553,9 @@ static void eeepc_hwmon_exit(void) hwmon = eeepc_hwmon_device; if (!hwmon) return ; - hwmon_device_unregister(hwmon); sysfs_remove_group(&hwmon->kobj, &hwmon_attribute_group); + hwmon_device_unregister(hwmon); eeepc_hwmon_device = NULL; } |