From 55d3a20bb9a79da86f738d403861e64a9abc47cc Mon Sep 17 00:00:00 2001 From: Kiran Venkatappa Date: Tue, 20 Dec 2016 11:29:34 +0530 Subject: [PATCH] qcacmn: Fix for crash during unload of driver modules Change ce_cancel API to return error status in srng implementation. Since this feature is not yet added, it should return error status, zero or success return value will result in a infinite loop causing access to invalid memory. Change-Id: I871cc16e2854aeca694c7538cc1e99ab24b04de6 CRs-Fixed: 1097695 --- hif/src/ce/ce_service_srng.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/hif/src/ce/ce_service_srng.c b/hif/src/ce/ce_service_srng.c index a5df6b759b6d..a8e96abaf0f4 100644 --- a/hif/src/ce/ce_service_srng.c +++ b/hif/src/ce/ce_service_srng.c @@ -535,7 +535,7 @@ ce_cancel_send_next_srng(struct CE_handle *copyeng, unsigned int *transfer_idp, uint32_t *toeplitz_hash_result) { - return 0; + return QDF_STATUS_E_INVAL; } /* Shift bits to convert IS_*_RING_*_WATERMARK_MASK to CE_WM_FLAG_*_* */