From 0101f5317ea9383b474ec7bb01e3e949d5251373 Mon Sep 17 00:00:00 2001 From: "Selvaraj, Sridhar" Date: Thu, 29 Sep 2016 11:56:45 +0530 Subject: qcacld-3.0: Send proper reason code to Supplicant on beacon miss event qcacld-2.0 to qcacld-3.0 propagation Once the beacon miss event is received in driver, sta cleanup happens and sends deauth reason code as 505 which is internal value to driver(eSIR_SME_LOST_LINK_WITH_PEER_RESULT_CODE). Send reason code as zero to Supplicant for Beacon miss kickout. Cleanup the similar handling in csr_api_roam as it is redundant. Change-Id: I6df97e05ac20f285646df3d5fe73800138a18d89 CRs-Fixed: 1025871 --- core/mac/src/pe/lim/lim_send_sme_rsp_messages.c | 20 ++++++++++++++++++++ core/sme/src/csr/csr_api_roam.c | 15 ++------------- 2 files changed, 22 insertions(+), 13 deletions(-) diff --git a/core/mac/src/pe/lim/lim_send_sme_rsp_messages.c b/core/mac/src/pe/lim/lim_send_sme_rsp_messages.c index 40993dc1f0e4..08b179b9a2d0 100644 --- a/core/mac/src/pe/lim/lim_send_sme_rsp_messages.c +++ b/core/mac/src/pe/lim/lim_send_sme_rsp_messages.c @@ -958,6 +958,16 @@ lim_send_sme_disassoc_ntf(tpAniSirGlobal pMac, sizeof(tSirMacAddr)); sir_sme_dis_ind->session_id = smesessionId; sir_sme_dis_ind->reason_code = reasonCode; + /* + * Instead of sending deauth reason code as 505 which is + * internal value(eSIR_SME_LOST_LINK_WITH_PEER_RESULT_CODE) + * Send reason code as zero to Supplicant + */ + if (reasonCode == eSIR_SME_LOST_LINK_WITH_PEER_RESULT_CODE) + sir_sme_dis_ind->reason_code = 0; + else + sir_sme_dis_ind->reason_code = reasonCode; + pMsg = (uint32_t *)sir_sme_dis_ind; break; @@ -1421,6 +1431,16 @@ lim_send_sme_deauth_ntf(tpAniSirGlobal pMac, tSirMacAddr peerMacAddr, sir_sme_dis_ind->reason_code = reasonCode; qdf_mem_copy(sir_sme_dis_ind->peer_mac, peerMacAddr, ETH_ALEN); + /* + * Instead of sending deauth reason code as 505 which is + * internal value(eSIR_SME_LOST_LINK_WITH_PEER_RESULT_CODE) + * Send reason code as zero to Supplicant + */ + if (reasonCode == eSIR_SME_LOST_LINK_WITH_PEER_RESULT_CODE) + sir_sme_dis_ind->reason_code = 0; + else + sir_sme_dis_ind->reason_code = reasonCode; + pMsg = (uint32_t *)sir_sme_dis_ind; break; diff --git a/core/sme/src/csr/csr_api_roam.c b/core/sme/src/csr/csr_api_roam.c index 2fca70196a60..9c6460205deb 100644 --- a/core/sme/src/csr/csr_api_roam.c +++ b/core/sme/src/csr/csr_api_roam.c @@ -11745,19 +11745,8 @@ QDF_STATUS csr_roam_lost_link(tpAniSirGlobal pMac, uint32_t sessionId, result = eCSR_ROAM_RESULT_DEAUTH_IND; pDeauthIndMsg = (tSirSmeDeauthInd *) pSirMsg; pSession->roamingStatusCode = pDeauthIndMsg->statusCode; - /* Convert into proper reason code */ - if ((pDeauthIndMsg->reasonCode == eSIR_BEACON_MISSED) || - (pDeauthIndMsg->reasonCode == - eSIR_MAC_DISASSOC_DUE_TO_INACTIVITY_REASON)) - pSession->joinFailStatusCode.reasonCode = 0; - else - pSession->joinFailStatusCode.reasonCode = - pDeauthIndMsg->reasonCode; - /* - * cfg layer expects 0 as reason code if - * the driver dosent know the reason code - * eSIR_BEACON_MISSED is defined as locally - */ + pSession->joinFailStatusCode.reasonCode = + pDeauthIndMsg->reasonCode; } else { sms_log(pMac, LOGW, FL("gets an unknown type (%d)"), type); result = eCSR_ROAM_RESULT_NONE; -- cgit v1.2.3