diff options
| author | David Lin <dtwlin@google.com> | 2017-04-30 16:50:31 -0700 |
|---|---|---|
| committer | Michael Bestas <mkbestas@lineageos.org> | 2019-12-23 23:43:30 +0200 |
| commit | 97be1f89cbc68dc4eb62c7d150933ecbd8d218f4 (patch) | |
| tree | 872281f766d4b1e6ca33a2d187619dc799d32034 | |
| parent | 265c97f85ebb33d807b78f7f62c51a53f86289c4 (diff) | |
msm: memshare: relaese mutex if req client id is not found
This fixes the smatch warning below:
drivers/soc/qcom/memshare/msm_memshare.c:570 handle_alloc_generic_req()
warn: inconsistent returns 'mutex:&memsh_drv->mem_share'.
Bug: 35898203
Change-Id: I86f3d77c9bb471b1bc26360e11e378fa3d1a3f66
Signed-off-by: David Lin <dtwlin@google.com>
| -rw-r--r-- | drivers/soc/qcom/memshare/msm_memshare.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/soc/qcom/memshare/msm_memshare.c b/drivers/soc/qcom/memshare/msm_memshare.c index 94b535111f24..cc566d6480f6 100644 --- a/drivers/soc/qcom/memshare/msm_memshare.c +++ b/drivers/soc/qcom/memshare/msm_memshare.c @@ -522,6 +522,7 @@ static int handle_alloc_generic_req(void *req_h, void *req, void *conn_h) alloc_req->proc_id); kfree(alloc_resp); alloc_resp = NULL; + mutex_unlock(&memsh_drv->mem_share); return -EINVAL; } |