aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorHyogi Gim <hyogi.gim@lge.com>2014-08-08 17:20:11 -0400
committerLinus Torvalds <torvalds@linux-foundation.org>2014-08-08 18:57:19 -0400
commitca6dc2dab97133b874c2f6a76b6125497b67b429 (patch)
tree82bcedccb2fa224f4d00b9155325699f678f16a8
parent6e85bab6bc1019f9b87c53b32da3ad7791e7ddf9 (diff)
drivers/rtc/interface.c: check the error after __rtc_read_time()
In __rtc_set_alarm(), the error after __rtc_read_time() is not checked. If rtc device fail to read time, we cannot guarantee the following process. Add the verification code for returned __rtc_read_time() error. Signed-off-by: Hyogi Gim <hyogi.gim@lge.com> Acked-by: Alessandro Zummo <a.zummo@towertech.it> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
-rw-r--r--drivers/rtc/interface.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/rtc/interface.c b/drivers/rtc/interface.c
index 5813fa52c3d4..5b2717f5dafa 100644
--- a/drivers/rtc/interface.c
+++ b/drivers/rtc/interface.c
@@ -348,6 +348,8 @@ static int __rtc_set_alarm(struct rtc_device *rtc, struct rtc_wkalrm *alarm)
348 348
349 /* Make sure we're not setting alarms in the past */ 349 /* Make sure we're not setting alarms in the past */
350 err = __rtc_read_time(rtc, &tm); 350 err = __rtc_read_time(rtc, &tm);
351 if (err)
352 return err;
351 rtc_tm_to_time(&tm, &now); 353 rtc_tm_to_time(&tm, &now);
352 if (scheduled <= now) 354 if (scheduled <= now)
353 return -ETIME; 355 return -ETIME;