diff options
author | Maciej W. Rozycki <macro@linux-mips.org> | 2008-05-12 17:02:35 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2008-05-13 11:02:26 -0400 |
commit | 417607d05f41dbe2acccdb7a298a81d30ba1d22b (patch) | |
tree | 8e9b8fd1140aa9876a76da235e3fcbd7e6519682 /drivers/rtc | |
parent | 6c82c4150910dedd449194cb6d286b80478f3542 (diff) |
RTC/watchdog: M41T80: fix a potential use of unitialized data
Watchdog handlers within the driver make use of "save_client" -- make sure it
has been initalized before the handlers are registered.
Signed-off-by: Maciej W. Rozycki <macro@linux-mips.org>
Cc: Alessandro Zummo <a.zummo@towertech.it>
Cc: Alexander Bigga <ab@mycable.de>
Cc: Wim Van Sebroeck <wim@iguana.be>
Cc: David Brownell <david-b@pacbell.net>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
Diffstat (limited to 'drivers/rtc')
-rw-r--r-- | drivers/rtc/rtc-m41t80.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/rtc/rtc-m41t80.c b/drivers/rtc/rtc-m41t80.c index 316bfaa8087..3d451ece253 100644 --- a/drivers/rtc/rtc-m41t80.c +++ b/drivers/rtc/rtc-m41t80.c | |||
@@ -803,6 +803,7 @@ static int m41t80_probe(struct i2c_client *client, | |||
803 | 803 | ||
804 | #ifdef CONFIG_RTC_DRV_M41T80_WDT | 804 | #ifdef CONFIG_RTC_DRV_M41T80_WDT |
805 | if (clientdata->features & M41T80_FEATURE_HT) { | 805 | if (clientdata->features & M41T80_FEATURE_HT) { |
806 | save_client = client; | ||
806 | rc = misc_register(&wdt_dev); | 807 | rc = misc_register(&wdt_dev); |
807 | if (rc) | 808 | if (rc) |
808 | goto exit; | 809 | goto exit; |
@@ -811,7 +812,6 @@ static int m41t80_probe(struct i2c_client *client, | |||
811 | misc_deregister(&wdt_dev); | 812 | misc_deregister(&wdt_dev); |
812 | goto exit; | 813 | goto exit; |
813 | } | 814 | } |
814 | save_client = client; | ||
815 | } | 815 | } |
816 | #endif | 816 | #endif |
817 | return 0; | 817 | return 0; |