diff options
| author | Kyle McMartin <kyle@shortfin.cabal.ca> | 2007-10-20 13:23:00 -0700 |
|---|---|---|
| committer | Kyle McMartin <kyle@shortfin.cabal.ca> | 2007-10-20 13:23:00 -0700 |
| commit | 7210c678202bb3107085bffeb63f66a9b8ba1c85 (patch) | |
| tree | 522752089cb90b8aa6fc92d9c82caca8b7946810 /include | |
| parent | 562d139cc587fc7f7fb8e07aad13cdd2a412c9a7 (diff) | |
| download | blackbird-op-linux-7210c678202bb3107085bffeb63f66a9b8ba1c85.tar.gz blackbird-op-linux-7210c678202bb3107085bffeb63f66a9b8ba1c85.zip | |
[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.h | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/include/asm-parisc/rtc.h b/include/asm-parisc/rtc.h index f4ebff11dcbd..099d641a42c2 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. |
