mirrors/madwifi
1
0
Fork 0
mirror of https://github.com/proski/madwifi synced 2024-11-22 06:21:47 +03:00
Code Issues Packages Projects Releases Wiki Activity

Fix compiler errors with -O3

Browse Source 
When the compiler tries inlining static functions, it can notice that
some variables may not be initialized in the functions being inlined.
Provide initialization for output variables in all branches of
execution, no matter how anomalous.  Add asserts in some cases.


git-svn-id: http://madwifi-project.org/svn/madwifi/trunk@3988 0192ed92-7a03-0410-a25b-9323aeb14dbd
This commit is contained in:
proski 2009-04-08 09:15:22 +00:00
parent c135dc326c
commit 1c2147eb68
10 changed files with 17 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
ath_hal/ar5210/ar5210_reset.c
View File

@ -691,6 +691,7 @@ getGainF(struct ath_hal *ah, const struct tpcMap *pRD,
"%s: no valid entries in the pcdac table: %d\n", "%s: no valid entries in the pcdac table: %d\n",
__func__, pcdac); __func__, pcdac);
#endif #endif
*dBm = 0;
return 63; return 63;
} }
if (i >= AR_TP_SCALING_ENTRIES) { if (i >= AR_TP_SCALING_ENTRIES) {

3
ath_hal/ar5211/ar5211_beacon.c
View File

@ -63,8 +63,7 @@ ar5211BeaconInit(struct ath_hal *ah,
* interrupt; otherwise it sets the start of the next CFP. * interrupt; otherwise it sets the start of the next CFP.
*/ */
switch (AH_PRIVATE(ah)->ah_opmode) { switch (AH_PRIVATE(ah)->ah_opmode) {
case HAL_M_STA: default:
case HAL_M_MONITOR:
bt.bt_nextdba = 0xffff; bt.bt_nextdba = 0xffff;
bt.bt_nextswba = 0x7ffff; bt.bt_nextswba = 0x7ffff;
break; break;

3
ath_hal/ar5211/ar5211_reset.c
View File

@ -1877,6 +1877,9 @@ ar5211GetLowerUpperValues(uint16_t value,
return; return;
} }
} }
HALASSERT(AH_FALSE); /* should not reach here */
*pLowerValue = *pList;
*pUpperValue = *pList;
} }
/* /*

2
ath_hal/ar5212/ar2316.c
View File

@ -295,6 +295,8 @@ GetLowerUpperIndex(int16_t v, const uint16_t *lp, uint16_t listSize,
return; return;
} }
} }
HALASSERT(AH_FALSE); /* should not reach here */
*vlo = *vhi = 0;
} }
/* /*

2
ath_hal/ar5212/ar2317.c
View File

@ -271,6 +271,8 @@ GetLowerUpperIndex(int16_t v, const uint16_t *lp, uint16_t listSize,
return; return;
} }
} }
HALASSERT(AH_FALSE); /* should not reach here */
*vlo = *vhi = 0;
} }
/* /*

2
ath_hal/ar5212/ar2413.c
View File

@ -289,6 +289,8 @@ GetLowerUpperIndex(int16_t v, const uint16_t *lp, uint16_t listSize,
return; return;
} }
} }
HALASSERT(AH_FALSE); /* should not reach here */
*vlo = *vhi = 0;
} }
/* /*

2
ath_hal/ar5212/ar2425.c
View File

@ -290,6 +290,8 @@ GetLowerUpperIndex(int16_t v, const uint16_t *lp, uint16_t listSize,
return; return;
} }
} }
HALASSERT(AH_FALSE); /* should not reach here */
*vlo = *vhi = 0;
} }
/* /*

3
ath_hal/ar5212/ar5212_beacon.c
View File

@ -76,8 +76,7 @@ ar5212BeaconInit(struct ath_hal *ah,
* interrupt; otherwise it sets the start of the next CFP. * interrupt; otherwise it sets the start of the next CFP.
*/ */
switch (AH_PRIVATE(ah)->ah_opmode) { switch (AH_PRIVATE(ah)->ah_opmode) {
case HAL_M_STA: default:
case HAL_M_MONITOR:
bt.bt_nextdba = 0xffff; bt.bt_nextdba = 0xffff;
bt.bt_nextswba = 0x7ffff; bt.bt_nextswba = 0x7ffff;
break; break;

1
ath_hal/ar5212/ar5212_reset.c
View File

@ -2525,6 +2525,7 @@ ar5212GetLowerUpperValues(uint16_t v, uint16_t *lp, uint16_t listSize,
} }
} }
HALASSERT(AH_FALSE); /* should not reach here */ HALASSERT(AH_FALSE); /* should not reach here */
*vlo = *vhi = 0;
} }
/* /*

2
ath_hal/ar5212/ar5413.c
View File

@ -324,6 +324,8 @@ GetLowerUpperIndex(int16_t v, const uint16_t *lp, uint16_t listSize,
return; return;
} }
} }
HALASSERT(AH_FALSE); /* should not reach here */
*vlo = *vhi = 0;
} }
/* /*