evie/android_kernel_oneplus_msm8998
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

memshare: Fix possible memory leak

Browse source 
This patch frees up the dynamically allotted memory
when met with error condition and function return.

CRs-Fixed: 1106180
Change-Id: I01446fbdbb422dcb025ee50929b32f40a5a78681
Signed-off-by: Manoj Prabhu B <bmanoj@codeaurora.org>
This commit is contained in:
Manoj Prabhu B 2016-12-29 13:19:48 +05:30
parent 55e8426a19
commit ec046f26dd
1 changed files with 9 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
drivers/soc/qcom/memshare/msm_memshare.c
View file

@ -511,6 +511,8 @@ static int handle_alloc_generic_req(void *req_h, void *req, void *conn_h)
pr_err("memshare: %s client not found, requested client: %d, proc_id: %d\n",
__func__, alloc_req->client_id,
alloc_req->proc_id);
kfree(alloc_resp);
alloc_resp = NULL;
return -EINVAL;
}
@ -555,6 +557,9 @@ static int handle_alloc_generic_req(void *req_h, void *req, void *conn_h)
if (rc < 0)
pr_err("In %s, Error sending the alloc request: %d\n",
__func__, rc);
kfree(alloc_resp);
alloc_resp = NULL;
return rc;
}
@ -672,6 +677,8 @@ static int handle_query_size_req(void *req_h, void *req, void *conn_h)
pr_err("memshare: %s client not found, requested client: %d, proc_id: %d\n",
__func__, query_req->client_id,
query_req->proc_id);
kfree(query_resp);
query_resp = NULL;
return -EINVAL;
}
@ -697,6 +704,8 @@ static int handle_query_size_req(void *req_h, void *req, void *conn_h)
pr_err("In %s, Error sending the query request: %d\n",
__func__, rc);
kfree(query_resp);
query_resp = NULL;
return rc;
}