diff options
| author | Al Viro <viro@ftp.linux.org.uk> | 2007-07-23 18:43:54 -0700 |
|---|---|---|
| committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-07-24 12:24:59 -0700 |
| commit | d316ee5787153765980ca0f55ee864ec883979ab (patch) | |
| tree | 2fb761e5b33f63e80bc018cd19027ec88626e035 /fs/gfs2/ops_super.c | |
| parent | c51aea80a9fbb423b49d432e89e515257bb2b983 (diff) | |
Fix failure exits in asus-laptop
Fallout from f8a7c6fe14f556ca8eeddce258cb21392d0c3a2f. However, looking
at it shows that checks done in ASUS_LED_UNREGISTER() can't trigger
at all (we never get to asus_led_exit() if registration fails) and
if that registration fails, we actually leak stuff. IOW, it's worse
than just replacing class_dev with dev in there - the tests themselves
had been papering over the lousy cleanup logics.
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Michal Piotrowski <michal.k.k.piotrowski@gmail.com>
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>
Diffstat (limited to 'fs/gfs2/ops_super.c')
0 files changed, 0 insertions, 0 deletions