diff options
author | sam <sam@FreeBSD.org> | 2009-02-11 22:15:37 +0000 |
---|---|---|
committer | sam <sam@FreeBSD.org> | 2009-02-11 22:15:37 +0000 |
commit | f6d409ff3b78fda40089ca7199aaa36bb744cd1c (patch) | |
tree | cf0d6bceb8f86f15c4b7d5cf6572c7896cf3c29e /sys/dev/ath/ath_hal/ar5416 | |
parent | 017f8027a18008ff7e60a1b2c5ba44d94671c37b (diff) | |
download | FreeBSD-src-f6d409ff3b78fda40089ca7199aaa36bb744cd1c.zip FreeBSD-src-f6d409ff3b78fda40089ca7199aaa36bb744cd1c.tar.gz |
gcc 4.3.2 examines getLowerUpperIndex() and concludes that it's not
guaranteed to initialize its two last arguments. Therefore, there is a
warning in the subsequent caller ar5416FillVpdTable(), which doesn't
initialize those arguments.
Change getLowerUpperIndex() to assign values to indexL and indexR even
in the case of assertion failure.
Submitted by: Pavel Roskin <proski@gnu.org>
Diffstat (limited to 'sys/dev/ath/ath_hal/ar5416')
-rw-r--r-- | sys/dev/ath/ath_hal/ar5416/ar5416_reset.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/sys/dev/ath/ath_hal/ar5416/ar5416_reset.c b/sys/dev/ath/ath_hal/ar5416/ar5416_reset.c index b017017..9f3ba35 100644 --- a/sys/dev/ath/ath_hal/ar5416/ar5416_reset.c +++ b/sys/dev/ath/ath_hal/ar5416/ar5416_reset.c @@ -2710,6 +2710,7 @@ getLowerUpperIndex(uint8_t target, uint8_t *pList, uint16_t listSize, } } HALASSERT(0); + *indexL = *indexR = 0; return AH_FALSE; } |