summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKapil Gupta <kapgupta@qti.qualcomm.com>2016-08-05 15:42:48 +0530
committerAnjaneedevi Kapparapu <akappa@codeaurora.org>2016-08-08 16:10:52 +0530
commit2070c9bf71eb372f0e13e61788e0b79fc5c95ec3 (patch)
tree17168e8b1cdadd289e7890986b1d674c76622747
parent213b1380d23f2224cade71fdef83f7fd06082a72 (diff)
qcacld-2.0: Fix memory leak issue
Memory allocated for dfs filters in dfs_attach() API does not get freed in failure case. Add changes to fix memory leak issue. Change-Id: I8c025c25089f64256539a3fa74a2e7e20edb4cc1 CRs-Fixed: 1051017
-rw-r--r--CORE/SERVICES/DFS/src/dfs.c12
1 files changed, 6 insertions, 6 deletions
diff --git a/CORE/SERVICES/DFS/src/dfs.c b/CORE/SERVICES/DFS/src/dfs.c
index 6dfae3226b4c..5a73a7cda410 100644
--- a/CORE/SERVICES/DFS/src/dfs.c
+++ b/CORE/SERVICES/DFS/src/dfs.c
@@ -466,9 +466,9 @@ dfs_attach(struct ieee80211com *ic)
return 0;
bad3:
- if (!(dfs->dfs_dc_radartable)) {
+ if (dfs->dfs_dc_radartable) {
for (n = 0; n < DFS_MAX_RADAR_TYPES; n++) {
- if (!(dfs->dfs_dc_radartable[n])) {
+ if (dfs->dfs_dc_radartable[n]) {
OS_FREE(dfs->dfs_dc_radartable[n]);
dfs->dfs_dc_radartable[n] = NULL;
}
@@ -489,7 +489,7 @@ bad2:
}
bad4:
for (n = 0; n < DFS_MAX_RADAR_TYPES; n++) {
- if (!(dfs->dfs_dc_radarf[n])) {
+ if (dfs->dfs_dc_radarf[n]) {
dfs_free_filter(dfs->dfs_dc_radarf[n]);
OS_FREE(dfs->dfs_dc_radarf[n]);
dfs->dfs_dc_radarf[n] = NULL;
@@ -584,7 +584,7 @@ dfs_detach(struct ieee80211com *ic)
}
}
for (n = 0; n < DFS_MAX_RADAR_TYPES; n++) {
- if (!(dfs->dfs_dc_radarf[n])) {
+ if (dfs->dfs_dc_radarf[n]) {
dfs_free_filter(dfs->dfs_dc_radarf[n]);
OS_FREE(dfs->dfs_dc_radarf[n]);
dfs->dfs_dc_radarf[n] = NULL;
@@ -607,9 +607,9 @@ dfs_detach(struct ieee80211com *ic)
#endif
}
- if (!(dfs->dfs_dc_radartable)) {
+ if (dfs->dfs_dc_radartable) {
for (n = 0; n < DFS_MAX_RADAR_TYPES; n++) {
- if (!(dfs->dfs_dc_radartable[n])) {
+ if (dfs->dfs_dc_radartable[n]) {
OS_FREE(dfs->dfs_dc_radartable[n]);
dfs->dfs_dc_radartable[n] = NULL;
}