wlan: hdd: fix logging format/argument mismatch
Source code analysis uncovered mismatches between format and argument
in invocations of logging APIs in the hdd code, so fix them.
Change-Id: Iacb0175ade2e255c3b5fcc852ab5d4ff8f901826
CRs-fixed: 570511
diff --git a/CORE/HDD/src/wlan_hdd_cfg80211.c b/CORE/HDD/src/wlan_hdd_cfg80211.c
index 729033e..dae506f 100644
--- a/CORE/HDD/src/wlan_hdd_cfg80211.c
+++ b/CORE/HDD/src/wlan_hdd_cfg80211.c
@@ -815,7 +815,7 @@
if (eHAL_STATUS_SUCCESS != sme_GetDefaultCountryCodeFrmNv(pHddCtx->hHal,
&defaultCountryCode[0]))
{
- hddLog(LOGE, FL("%s Failed to get default country code from NV"));
+ hddLog(LOGE, FL("Failed to get default country code from NV"));
}
if ((defaultCountryCode[0]== 'U') && (defaultCountryCode[1]=='S'))
{
@@ -3100,7 +3100,7 @@
VOS_TRACE(VOS_MODULE_ID_HDD, TDLS_LOG_LEVEL,
"params->capability: %0x",StaParams->capability);
VOS_TRACE(VOS_MODULE_ID_HDD, TDLS_LOG_LEVEL,
- "params->ext_capab_len: %0x",StaParams->extn_capability);
+ "params->ext_capab_len: %0x",StaParams->extn_capability[0]);
if(StaParams->vhtcap_present)
{
VOS_TRACE(VOS_MODULE_ID_HDD, TDLS_LOG_LEVEL,
@@ -3438,7 +3438,7 @@
#endif
default:
- hddLog(VOS_TRACE_LEVEL_ERROR, "%s: unsupported cipher type %lu",
+ hddLog(VOS_TRACE_LEVEL_ERROR, "%s: unsupported cipher type %u",
__func__, params->cipher);
return -EOPNOTSUPP;
}
@@ -5835,7 +5835,7 @@
#ifdef FEATURE_WLAN_WAPI
case WLAN_EID_WAPI:
pAdapter->wapi_info.nWapiMode = 1; //Setting WAPI Mode to ON=1
- hddLog(VOS_TRACE_LEVEL_INFO,"WAPI MODE IS %lu \n",
+ hddLog(VOS_TRACE_LEVEL_INFO, "WAPI MODE IS %u",
pAdapter->wapi_info.nWapiMode);
tmp = (u16 *)ie;
tmp = tmp + 2; // Skip element Id and Len, Version
@@ -7455,7 +7455,8 @@
tANI_U8 BSSIDMatched = 0;
hdd_context_t *pHddCtx;
- ENTER();
+ hddLog(VOS_TRACE_LEVEL_DEBUG, "%s: set PMKSA for " MAC_ADDRESS_STR,
+ __func__, MAC_ADDR_ARRAY(pmksa->bssid));
// Validate pAdapter
if ( NULL == pAdapter )
@@ -7544,8 +7545,8 @@
hdd_context_t *pHddCtx;
int result = 0;
- hddLog(VOS_TRACE_LEVEL_DEBUG, "%s: deleting PMKSA with PMKSA_ID %d .",
- __func__,pmksa->pmkid);
+ hddLog(VOS_TRACE_LEVEL_DEBUG, "%s: deleting PMKSA for " MAC_ADDRESS_STR,
+ __func__, MAC_ADDR_ARRAY(pmksa->bssid));
/* Validate pAdapter */
if (NULL == pAdapter)
@@ -7624,8 +7625,9 @@
/* we compare all entries,but cannot find matching entry */
if (j == MAX_PMKSAIDS_IN_CACHE && !BSSIDMatched)
{
- hddLog(VOS_TRACE_LEVEL_FATAL, "%s: No such PMKSA entry existed %d.",
- __func__,pmksa->bssid);
+ hddLog(VOS_TRACE_LEVEL_FATAL,
+ "%s: No such PMKSA entry existed " MAC_ADDRESS_STR,
+ __func__, MAC_ADDR_ARRAY(pmksa->bssid));
dump_bssid(pmksa->bssid);
dump_pmkid(halHandle, pmksa->pmkid);
return -EINVAL;
@@ -8250,7 +8252,7 @@
if ((rc <= 0) || (TRUE != pAdapter->mgmtTxCompletionStatus))
{
VOS_TRACE(VOS_MODULE_ID_HDD, VOS_TRACE_LEVEL_ERROR,
- "%s: Mgmt Tx Completion failed status %ld TxCompletion %lu",
+ "%s: Mgmt Tx Completion failed status %ld TxCompletion %u",
__func__, rc, pAdapter->mgmtTxCompletionStatus);
pAdapter->mgmtTxCompletionStatus = FALSE;
wlan_hdd_tdls_check_bmps(pAdapter);