commit 5b4c821911604d3850f5a2caf17c22416537fd60
author Sushant Kaushik <skaushik@qti.qualcomm.com> Fri Feb 20 14:54:25 2015 +0530
committer Satyanarayana Dash <c_sadash@qti.qualcomm.com> Tue Feb 24 23:28:27 2015 +0530
tree cdaab4b2b81946db562f91fa8b95e76ffb29b6ec
parent 89064d7542ec109a71ef0d2417a5029579f4ebeb

wlan: Fix static analysis error.

Currently numChan is initialized to WNI_CFG_VALID_CHANNEL_LIST_LEN.
Then a list of valid channels are populated. But if valid channels
are not obtained still it proceeds further resulting in buffer
corruption. As a part of fix make sure to retun if valid channels
are not properly obtained.

Change-Id: I436b5c93919db297662ff7960d309ffe88d41599
CRs-fixed: 798061

CORE/MAC/src/pe/lim/limProcessTdls.c

diff --git a/CORE/MAC/src/pe/lim/limProcessTdls.c b/CORE/MAC/src/pe/lim/limProcessTdls.c
index ffae803..8a166a0 100644
--- a/CORE/MAC/src/pe/lim/limProcessTdls.c
+++ b/CORE/MAC/src/pe/lim/limProcessTdls.c
@@ -5076,6 +5076,7 @@
          * Log error.
          */
          limLog(pMac, LOGE, FL("could not retrieve valid channel list"));
+         return;
     }
     suppChannels->num_bands = (tANI_U8) numChans;