diff options
| author | Mukul Sharma <mukul@codeaurora.org> | 2017-09-22 16:45:37 +0530 |
|---|---|---|
| committer | snandini <snandini@codeaurora.org> | 2017-09-25 06:24:53 -0700 |
| commit | b58d860155aaa2755a9349ca54e08328b89756b9 (patch) | |
| tree | f2cca9924db6de9e4eb207fd55da0abb02e5fe31 | |
| parent | 4ae2484358a4dcea42e9d35be60ee2ccf2108f4b (diff) | |
qcacld-2.0: Prevent memory leak in wlan_hdd_cfg80211_ocb_set_config
Currently, Host performs validation of NLA attribute
QCA_WLAN_VENDOR_ATTR_OCB_SET_CONFIG_CHANNEL_ARRAY twice
which leads to memory leak.
As a part of fix, Check for SET_CONFIG_CHANNEL_ARRAY
only once and free memory in error scenario.
Change-Id: I5c90c937682417b8181fda7d499908b9425d4c6a
CRs-Fixed: 2114111
| -rw-r--r-- | CORE/HDD/src/wlan_hdd_ocb.c | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/CORE/HDD/src/wlan_hdd_ocb.c b/CORE/HDD/src/wlan_hdd_ocb.c index 668986910b74..a49851e0110e 100644 --- a/CORE/HDD/src/wlan_hdd_ocb.c +++ b/CORE/HDD/src/wlan_hdd_ocb.c @@ -1042,10 +1042,6 @@ static int __wlan_hdd_cfg80211_ocb_set_config(struct wiphy *wiphy, config->def_tx_param_size = def_tx_param_size; /* Read the channel array */ - if (!tb[QCA_WLAN_VENDOR_ATTR_OCB_SET_CONFIG_CHANNEL_ARRAY]) { - hddLog(LOGE, FL("CHANNEL_ARRAY is not present")); - return -EINVAL; - } channel_array = tb[QCA_WLAN_VENDOR_ATTR_OCB_SET_CONFIG_CHANNEL_ARRAY]; if (!channel_array) { hddLog(LOGE, FL("No channel present")); |