samsung-sm8650/android_kernel_samsung_sm8650-modules
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

qcacmn: Fix tlv formation of arp ns offload command

Browse Source 
Fix arp ns offload command formation by removing the double
free of arp ns offload parameter structure.

Change-Id: I38d11c770f8e79d39f7593bcea3712ab07d34660
CRs-Fixed: 987362
This commit is contained in:
Govind Singh
2016-03-28 22:07:13 +05:30
parent 3419aea3a4
commit 17c39b0457
1 changed files with 1 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
wmi_unified_tlv.c
View File

@@ -9319,7 +9319,6 @@ QDF_STATUS send_enable_arp_ns_offload_cmd_tlv(wmi_unified_t wmi_handle,
buf = wmi_buf_alloc(wmi_handle, len); buf = wmi_buf_alloc(wmi_handle, len);
if (!buf) { if (!buf) {
WMI_LOGE("%s: wmi_buf_alloc failed", __func__); WMI_LOGE("%s: wmi_buf_alloc failed", __func__);
qdf_mem_free(param);
return QDF_STATUS_E_NOMEM; return QDF_STATUS_E_NOMEM;
} }
@@ -9340,7 +9339,7 @@ QDF_STATUS send_enable_arp_ns_offload_cmd_tlv(wmi_unified_t wmi_handle,
* both ARP and NS info in single cmd */ * both ARP and NS info in single cmd */
if (arp_only) if (arp_only)
qdf_mem_copy(&wmi_handle->arp_info, param, qdf_mem_copy(&wmi_handle->arp_info, param,
sizeof(tSirHostOffloadReq)); sizeof(struct host_offload_req_param));
buf_ptr += sizeof(WMI_SET_ARP_NS_OFFLOAD_CMD_fixed_param); buf_ptr += sizeof(WMI_SET_ARP_NS_OFFLOAD_CMD_fixed_param);
WMITLV_SET_HDR(buf_ptr, WMITLV_TAG_ARRAY_STRUC, WMITLV_SET_HDR(buf_ptr, WMITLV_TAG_ARRAY_STRUC,
@@ -9462,11 +9461,9 @@ QDF_STATUS send_enable_arp_ns_offload_cmd_tlv(wmi_unified_t wmi_handle,
if (res) { if (res) {
WMI_LOGE("Failed to enable ARP NDP/NSffload"); WMI_LOGE("Failed to enable ARP NDP/NSffload");
wmi_buf_free(buf); wmi_buf_free(buf);
qdf_mem_free(param);
return QDF_STATUS_E_FAILURE; return QDF_STATUS_E_FAILURE;
} }
qdf_mem_free(param);
return QDF_STATUS_SUCCESS; return QDF_STATUS_SUCCESS;
} }