diff options
| author | Kanchanapally, Vidyullatha <vkanchan@qti.qualcomm.com> | 2015-06-26 12:00:43 +0530 |
|---|---|---|
| committer | Anjaneedevi Kapparapu <akappa@codeaurora.org> | 2015-06-26 18:28:18 +0530 |
| commit | 1599121b98a8c1eecb2e06da9233f55ede4cf7ee (patch) | |
| tree | 3cf5e22fcb9e7b8a6cb21dc23a83f74a78c7ec8d | |
| parent | 9d0e21f23431c2ab3ba74c3ea2a40d08c3f728d8 (diff) | |
qcacld-2.0: Resolve pointer dereference issue in processCfgDownloadReq
In processCfgDownloadReq(), dst_ptr is dereferenced first and later
checked for NULL. Fix the same.
Change-Id: I48245acce5f92fecffa805b29313fef28dbe7bcb
CRs-Fixed: 862049
| -rw-r--r-- | CORE/MAC/src/cfg/cfgProcMsg.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/CORE/MAC/src/cfg/cfgProcMsg.c b/CORE/MAC/src/cfg/cfgProcMsg.c index 2f0127477d14..5f3bad83d211 100644 --- a/CORE/MAC/src/cfg/cfgProcMsg.c +++ b/CORE/MAC/src/cfg/cfgProcMsg.c @@ -2962,13 +2962,13 @@ processCfgDownloadReq(tpAniSirGlobal pMac) continue; dst_ptr = &pMac->cfg.gCfgSBuf[index]; - max_saved_len = *dst_ptr; str_cfg = (struct cfgstatic_string *)cfg_static[i].p_str_data; src_ptr = str_cfg->data; if ((dst_ptr == NULL) || (str_cfg == NULL) || (src_ptr == NULL)) continue; + max_saved_len = *dst_ptr; len = str_cfg->length; if (len > max_saved_len) continue; |