A-Team_Digital_Solutions/android_kernel_motorola_sm6375
Watch
2
0
Fork
You've already forked android_kernel_motorola_sm6375
 0
mirror of https://github.com/BobTheBlinker/android_kernel_motorola_sm6375.git synced 2026-10-12 07:37:31 -04:00
Code Issues Projects Releases Packages Wiki Activity

qcacld-3.0: Fix the incorrect usage of node in wma_cleanup_hold_req

Browse source 
Fix the usage of list node which was deleted before the usage in
wma_cleanup_hold_req function.

Change-Id: I8ee243fda511de71083ce7e631a21497f4f153c1
CRs-Fixed: 937267
This commit is contained in:
Krishna Kumaar Natarajan 2015-11-09 16:57:40 -08:00 • committed by Prakash Dhavali
commit 9355399b46
1 changed files with 8 additions and 13 deletions
Download patch file Download diff file Expand all files Collapse all files

21
core/wma/src/wma_main.c
View file

@ -2757,19 +2757,18 @@ end:
static void wma_cleanup_hold_req(tp_wma_handle wma)
{
struct wma_target_req *req_msg = NULL;
cdf_list_node_t *node1 = NULL, *node2 = NULL;
cdf_list_node_t *node1 = NULL;
CDF_STATUS status;
cdf_spin_lock_bh(&wma->wma_hold_req_q_lock);
if (CDF_STATUS_SUCCESS != cdf_list_peek_front(&wma->wma_hold_req_queue,
&node2)) {
if (cdf_list_empty(&wma->wma_hold_req_queue)) {
cdf_spin_unlock_bh(&wma->wma_hold_req_q_lock);
WMA_LOGI(FL("request queue maybe empty"));
WMA_LOGI(FL("request queue is empty"));
return;
}
do {
node1 = node2;
while (CDF_STATUS_SUCCESS !=
cdf_list_peek_front(&wma->wma_hold_req_queue, &node1)) {
req_msg = cdf_container_of(node1, struct wma_target_req, node);
status = cdf_list_remove_node(&wma->wma_hold_req_queue, node1);
if (CDF_STATUS_SUCCESS != status) {
@ -2777,14 +2776,10 @@ static void wma_cleanup_hold_req(tp_wma_handle wma)
WMA_LOGE(FL("Failed to remove request for vdev_id %d type %d"),
req_msg->vdev_id, req_msg->type);
return;
} else {
cdf_mc_timer_destroy(&req_msg->event_timeout);
cdf_mem_free(req_msg);
}
} while (CDF_STATUS_SUCCESS ==
cdf_list_peek_next(&wma->wma_hold_req_queue, node1,
&node2));
cdf_mc_timer_destroy(&req_msg->event_timeout);
cdf_mem_free(req_msg);
}
cdf_spin_unlock_bh(&wma->wma_hold_req_q_lock);
}