qcacld-3.0: Sanitize the sme_get_link_speed() API
Currently sme_get_link_speed() requires that the caller pass an
allocated buffer as the lsReq param. This violates the general
concensus that SME should make no assumptions about the buffers passed
to it by HDD.
Update the implementation such that SME allocates the buffer that will
be passed in the message, and update HDD to use the stack.
Note that this is a prerequisite for moving the linkspeed request
processing to the new HDD Request Management infrastructure.
Change-Id: I09257855c6c260bb4a6ca91eb79bb24813cc5efa
CRs-Fixed: 2000023
diff --git a/core/hdd/src/wlan_hdd_wext.c b/core/hdd/src/wlan_hdd_wext.c
index 588a037..e6e8116 100644
--- a/core/hdd/src/wlan_hdd_wext.c
+++ b/core/hdd/src/wlan_hdd_wext.c
@@ -2055,28 +2055,22 @@
QDF_STATUS status;
unsigned long rc;
static struct linkspeedContext context;
- tSirLinkSpeedInfo *linkspeed_req;
+ tSirLinkSpeedInfo linkspeed_req;
if (NULL == pAdapter) {
hdd_err("pAdapter is NULL");
return QDF_STATUS_E_FAULT;
}
- linkspeed_req = qdf_mem_malloc(sizeof(*linkspeed_req));
- if (NULL == linkspeed_req) {
- hdd_err("Request Buffer Alloc Fail");
- return QDF_STATUS_E_NOMEM;
- }
init_completion(&context.completion);
context.pAdapter = pAdapter;
context.magic = LINK_CONTEXT_MAGIC;
- qdf_copy_macaddr(&linkspeed_req->peer_macaddr, &macAddress);
+ qdf_copy_macaddr(&linkspeed_req.peer_macaddr, &macAddress);
status = sme_get_link_speed(WLAN_HDD_GET_HAL_CTX(pAdapter),
- linkspeed_req,
+ &linkspeed_req,
&context, hdd_get_link_speed_cb);
if (QDF_STATUS_SUCCESS != status) {
hdd_err("Unable to retrieve statistics for link speed");
- qdf_mem_free(linkspeed_req);
} else {
rc = wait_for_completion_timeout
(&context.completion,