wlan: Logging enhancement for connect/Disconnect in SAP mode
Added logging improvements in connect and disconnect path
of SAP mode
Change-Id: I8982693d731e0709bbec73bbbe915a26979be48c
CRs-Fixed: 773065
diff --git a/CORE/MAC/src/pe/lim/limProcessAssocReqFrame.c b/CORE/MAC/src/pe/lim/limProcessAssocReqFrame.c
index ea5076b..00cc2dc 100644
--- a/CORE/MAC/src/pe/lim/limProcessAssocReqFrame.c
+++ b/CORE/MAC/src/pe/lim/limProcessAssocReqFrame.c
@@ -303,7 +303,7 @@
if (status != eSIR_SUCCESS)
{
- limLog(pMac, LOGW, FL("Parse error AssocRequest, length=%d from "MAC_ADDRESS_STR),
+ limLog(pMac, LOGE, FL("Parse error AssocRequest, length=%d from "MAC_ADDRESS_STR),
framelen, MAC_ADDR_ARRAY(pHdr->sa));
limSendAssocRspMgmtFrame(pMac, eSIR_MAC_UNSPEC_FAILURE_STATUS, 1, pHdr->sa, subType, 0, psessionEntry);
goto error;
@@ -332,8 +332,8 @@
pAssocReq,
&localCapabilities,psessionEntry) == false)
{
- limLog(pMac, LOGW, FL("local caps mismatch received caps"));
- limLog(pMac, LOGW, FL("Received %s Req with unsupported "
+ limLog(pMac, LOGE, FL("local caps mismatch received caps"));
+ limLog(pMac, LOGE, FL("Received %s Req with unsupported "
"capabilities from"MAC_ADDRESS_STR),
(LIM_ASSOC == subType) ? "Assoc" : "ReAssoc",
MAC_ADDR_ARRAY(pHdr->sa));
@@ -356,7 +356,7 @@
if (limCmpSSid(pMac, &pAssocReq->ssId, psessionEntry) == false)
{
- limLog(pMac, LOGW, FL("Received %s Req with unmatched ssid ( Received"
+ limLog(pMac, LOGE, FL("Received %s Req with unmatched ssid ( Received"
" SSID: %.*s current SSID: %.*s ) from "MAC_ADDRESS_STR),
(LIM_ASSOC == subType) ? "Assoc" : "ReAssoc", pAssocReq->ssId.length,
pAssocReq->ssId.ssId, psessionEntry->ssId.length,
@@ -397,7 +397,7 @@
}
if (limCheckRxBasicRates(pMac, basicRates, psessionEntry) == false)
{
- limLog(pMac, LOGW, FL("Received %s Req with unsupported "
+ limLog(pMac, LOGE, FL("Received %s Req with unsupported "
"rates from"MAC_ADDRESS_STR),
(LIM_ASSOC == subType) ? "Assoc" : "ReAssoc",
MAC_ADDR_ARRAY(pHdr->sa));
@@ -509,7 +509,7 @@
if ( (pAssocReq->HTCaps.present) && (limCheckMCSSet(pMac, pAssocReq->HTCaps.supportedMCSSet) == false))
{
- limLog(pMac, LOGW, FL("received %s req with unsupported"
+ limLog(pMac, LOGE, FL("received %s req with unsupported"
"MCS Rate Set from "MAC_ADDRESS_STR),
(LIM_ASSOC == subType) ? "Assoc" : "ReAssoc",
MAC_ADDR_ARRAY(pHdr->sa));
@@ -540,7 +540,7 @@
if (!pAssocReq->extendedRatesPresent && val)
{
- limLog(pMac, LOGW, FL("Rejecting Re/Assoc req from 11b STA: "
+ limLog(pMac, LOGE, FL("Rejecting Re/Assoc req from 11b STA: "
MAC_ADDRESS_STR),MAC_ADDR_ARRAY(pHdr->sa));
/**
* Received Re/Association Request from
@@ -662,7 +662,7 @@
&& psessionEntry->pLimStartBssReq->privacy
&& psessionEntry->pLimStartBssReq->rsnIE.length)
{
- limLog(pMac, LOGE,
+ limLog(pMac, LOG1,
FL("RSN enabled auth, Re/Assoc req from STA: "MAC_ADDRESS_STR),
MAC_ADDR_ARRAY(pHdr->sa));
if(pAssocReq->rsnPresent)
@@ -682,7 +682,7 @@
if(eSIR_SUCCESS != (status = limCheckRxRSNIeMatch(pMac, Dot11fIERSN, psessionEntry,
pAssocReq->HTCaps.present, &pmfConnection)))
{
- limLog(pMac, LOGW, FL("Rejecting Re/Assoc req from "
+ limLog(pMac, LOGE, FL("RSN Mismatch. Rejecting Re/Assoc req from "
"STA: "MAC_ADDRESS_STR),
MAC_ADDR_ARRAY(pHdr->sa));
@@ -701,7 +701,8 @@
}
else
{
- limLog(pMac, LOGW, FL("Rejecting Re/Assoc req from "
+ limLog(pMac, LOGE, FL("RSN Version mismatch. "
+ "Rejecting Re/Assoc req from "
"STA: "MAC_ADDRESS_STR),
MAC_ADDR_ARRAY(pHdr->sa));
@@ -718,8 +719,10 @@
}
else
{
- limLog(pMac, LOGW, FL("Rejecting Re/Assoc req from STA:"
- MAC_ADDRESS_STR),MAC_ADDR_ARRAY(pHdr->sa));
+ limLog(pMac, LOGW, FL("RSN length not correct. "
+ "Rejecting Re/Assoc req from STA:"
+ MAC_ADDRESS_STR),
+ MAC_ADDR_ARRAY(pHdr->sa));
/* received Association req frame with RSN IE but length is 0 */
limSendAssocRspMgmtFrame(
pMac,
@@ -744,8 +747,10 @@
/* check the groupwise and pairwise cipher suites */
if(eSIR_SUCCESS != (status = limCheckRxWPAIeMatch(pMac, Dot11fIEWPA, psessionEntry, pAssocReq->HTCaps.present)))
{
- limLog(pMac, LOGW, FL("Rejecting Re/Assoc req from "
- "STA: "MAC_ADDRESS_STR),MAC_ADDR_ARRAY(pHdr->sa));
+ limLog(pMac, LOGW, FL("WPA IE mismatch. "
+ "Rejecting Re/Assoc req from "
+ "STA: "MAC_ADDRESS_STR),
+ MAC_ADDR_ARRAY(pHdr->sa));
/* received Association req frame with WPA IE but mismatch */
limSendAssocRspMgmtFrame(
pMac,
@@ -759,8 +764,10 @@
}
else
{
- limLog(pMac, LOGW, FL("Rejecting Re/Assoc req from STA: "
- MAC_ADDRESS_STR),MAC_ADDR_ARRAY(pHdr->sa));
+ limLog(pMac, LOGW, FL("WPA len incorrect. "
+ "Rejecting Re/Assoc req from STA: "
+ MAC_ADDRESS_STR),
+ MAC_ADDR_ARRAY(pHdr->sa));
/* received Association req frame with invalid WPA IE */
limSendAssocRspMgmtFrame(
pMac,
@@ -795,6 +802,8 @@
* Maximum number of STAs that AP can handle reached.
* Send Association response to peer MAC entity
*/
+ limLog(pMac, LOGE, FL("Max Sta count reached : %d"),
+ pMac->lim.maxStation);
limRejectAssociation(pMac, pHdr->sa,
subType, false,
(tAniAuthType) 0, 0,
@@ -823,7 +832,7 @@
eSIR_MAC_STA_NOT_PRE_AUTHENTICATED_REASON, //=9
pHdr->sa, psessionEntry, FALSE);
- limLog(pMac, LOGW, FL("received %s req on sessionid: %d from STA "
+ limLog(pMac, LOGE, FL("received %s req on sessionid: %d from STA "
"that does not have pre-auth context"MAC_ADDRESS_STR),
(LIM_ASSOC == subType) ? "Assoc" : "ReAssoc",
psessionEntry->peSessionId,
@@ -855,7 +864,7 @@
#ifdef WLAN_DEBUG
pMac->lim.gLimNumAssocReqDropInvldState++;
#endif
- limLog(pMac, LOG1, FL("received Assoc req in state "
+ limLog(pMac, LOGE, FL("received Assoc req in state "
"%d from "), pStaDs->mlmStaContext.mlmState);
}
else
@@ -863,7 +872,7 @@
#ifdef WLAN_DEBUG
pMac->lim.gLimNumReassocReqDropInvldState++;
#endif
- limLog(pMac, LOG1, FL("received ReAssoc req in state %d"
+ limLog(pMac, LOGE, FL("received ReAssoc req in state %d"
" from "), pStaDs->mlmStaContext.mlmState);
}
limPrintMacAddr(pMac, pHdr->sa, LOG1);
@@ -944,6 +953,8 @@
* 'associated' state. Update STA capabilities and
* send Association response frame with same AID
*/
+ limLog(pMac, LOG1, FL("Received Assoc req from STA already connected"
+ " UpdateConext"));
pStaDs->mlmStaContext.capabilityInfo = pAssocReq->capabilityInfo;
if (pStaPreAuthContext &&
(pStaPreAuthContext->mlmState ==
@@ -983,7 +994,7 @@
if (limAdmitControlAddTS(pMac, pHdr->sa, &(pAssocReq->addtsReq),
&(pAssocReq->qosCapability), 0, false, NULL, &tspecIdx, psessionEntry) != eSIR_SUCCESS)
{
- limLog(pMac, LOGW, FL("AdmitControl: TSPEC rejected"));
+ limLog(pMac, LOGE, FL("AdmitControl: TSPEC rejected"));
limSendAssocRspMgmtFrame(
pMac,
eSIR_MAC_QAP_NO_BANDWIDTH_REASON,
@@ -999,7 +1010,7 @@
else if (limAdmitControlAddSta(pMac, pHdr->sa, false)
!= eSIR_SUCCESS)
{
- limLog(pMac, LOGW, FL("AdmitControl: Sta rejected"));
+ limLog(pMac, LOGE, FL("AdmitControl: Sta rejected"));
limSendAssocRspMgmtFrame(
pMac,
eSIR_MAC_QAP_NO_BANDWIDTH_REASON,
@@ -1035,6 +1046,8 @@
{
// Could not assign AID
// Reject association
+ limLog(pMac, LOGE, FL("PeerIdx not avaialble. Reject associaton"));
+
limRejectAssociation(pMac, pHdr->sa,
subType, true, authType,
peerIdx, false,
@@ -1366,8 +1379,8 @@
/* use the same AID, already allocated */
if (limAddSta(pMac, pStaDs, false, psessionEntry) != eSIR_SUCCESS)
{
- limLog( pMac, LOGE, FL( "AP do not support UPASD "
- "REASSOC Failed"));
+ limLog(pMac, LOGE, FL( "Could not AddSta with assocId= %d staId %d"),
+ pStaDs->assocId, pStaDs->staIndex);
limRejectAssociation( pMac, pStaDs->staAddr, pStaDs->mlmStaContext.subType, true, pStaDs->mlmStaContext.authType,
pStaDs->assocId, true,(tSirResultCodes) eSIR_MAC_WME_REFUSED_STATUS, psessionEntry);