diff options
| author | Mingcheng Zhu <mingchen@qca.qualcomm.com> | 2014-08-05 23:37:34 -0700 |
|---|---|---|
| committer | Pitani Venkata Rajesh Kumar <c_vpitan@qti.qualcomm.com> | 2014-08-07 12:46:38 +0530 |
| commit | ae4448218fbc93f26e6983867003bf4ebfa24a5e (patch) | |
| tree | e12000ba7e33a69d31797633d2592dd905957bee | |
| parent | 8f96e6b77a9496bef3fcd1fcbe4ac023df85b071 (diff) | |
WLAN: Fix static analysis errors
Fix static analysis errors
CRs-Fixed: 704236
Change-Id: I1385a022ab1eff796df3570042278bed38896f21
| -rw-r--r-- | CORE/EPPING/src/epping_helper.c | 10 | ||||
| -rw-r--r-- | CORE/EPPING/src/epping_txrx.c | 3 |
2 files changed, 11 insertions, 2 deletions
diff --git a/CORE/EPPING/src/epping_helper.c b/CORE/EPPING/src/epping_helper.c index 193dd3dc6c4f..353b4bdef823 100644 --- a/CORE/EPPING/src/epping_helper.c +++ b/CORE/EPPING/src/epping_helper.c @@ -273,7 +273,15 @@ void epping_register_tx_copier(HTC_ENDPOINT_ID eid, epping_context_t *pEpping_ct } void epping_unregister_tx_copier(HTC_ENDPOINT_ID eid, epping_context_t *pEpping_ctx) { - epping_poll_t *epping_poll = &pEpping_ctx->epping_poll[eid]; + epping_poll_t *epping_poll; + + if (eid < 0 || eid >= EPPING_MAX_NUM_EPIDS ) { + EPPING_LOG(VOS_TRACE_LEVEL_FATAL, "%s: invalid eid = %d", + __func__, eid); + return; + } + + epping_poll = &pEpping_ctx->epping_poll[eid]; epping_poll->done = true; if (epping_poll->inited) { diff --git a/CORE/EPPING/src/epping_txrx.c b/CORE/EPPING/src/epping_txrx.c index 7c1a92feac6b..cf2a7d4074d3 100644 --- a/CORE/EPPING/src/epping_txrx.c +++ b/CORE/EPPING/src/epping_txrx.c @@ -324,7 +324,8 @@ void epping_destroy_adapter(epping_adapter_t *pAdapter) while (adf_nbuf_queue_len(&pAdapter->nodrop_queue)) { adf_nbuf_t tmp_nbuf = NULL; tmp_nbuf = adf_nbuf_queue_remove(&pAdapter->nodrop_queue); - adf_nbuf_free(tmp_nbuf); + if (tmp_nbuf) + adf_nbuf_free(tmp_nbuf); } free_netdev(dev); |