summaryrefslogtreecommitdiffstats
path: root/include
diff options
context:
space:
mode:
authorKyle McMartin <kyle@shortfin.cabal.ca>2007-10-20 13:23:00 -0700
committerKyle McMartin <kyle@shortfin.cabal.ca>2007-10-20 13:23:00 -0700
commit7210c678202bb3107085bffeb63f66a9b8ba1c85 (patch)
tree522752089cb90b8aa6fc92d9c82caca8b7946810 /include
parent562d139cc587fc7f7fb8e07aad13cdd2a412c9a7 (diff)
downloadop-kernel-dev-7210c678202bb3107085bffeb63f66a9b8ba1c85.zip
op-kernel-dev-7210c678202bb3107085bffeb63f66a9b8ba1c85.tar.gz
[PARISC] fix uninitialized variable warning in asm/rtc.h
get_rtc_time, in the case that PDC returns that the battery is bad, returns an unmodified rtc_time arg to the caller, which then uses uninitialized values. Fix this by memset-ing the arg with zeroes, so it will at least be cleared if we return failure. Spotted by John David Anglin. Signed-off-by: Kyle McMartin <kyle@mcmartin.ca>
Diffstat (limited to 'include')
-rw-r--r--include/asm-parisc/rtc.h4
1 files changed, 2 insertions, 2 deletions
diff --git a/include/asm-parisc/rtc.h b/include/asm-parisc/rtc.h
index f4ebff1..099d641 100644
--- a/include/asm-parisc/rtc.h
+++ b/include/asm-parisc/rtc.h
@@ -50,10 +50,10 @@ static inline unsigned int get_rtc_time(struct rtc_time *wtime)
long int days, rem, y;
const unsigned short int *ip;
- if(pdc_tod_read(&tod_data) < 0)
+ memset(wtime, 0, sizeof(*wtime));
+ if (pdc_tod_read(&tod_data) < 0)
return RTC_24H | RTC_BATT_BAD;
-
// most of the remainder of this function is:
// Copyright (C) 1991, 1993, 1997, 1998 Free Software Foundation, Inc.
// This was originally a part of the GNU C Library.
OpenPOWER on IntegriCloud