diff options
| author | Karthikeyan Periasamy <kperiasa@codeaurora.org> | 2016-10-11 12:21:42 -0700 |
|---|---|---|
| committer | Karthikeyan Periasamy <kperiasa@codeaurora.org> | 2016-10-17 11:17:37 -0700 |
| commit | 0dc00b42d29993a6c083e42d60db82c647efe001 (patch) | |
| tree | 09cb483f56297ae5a9f045019c5b770e70fd4e84 | |
| parent | 67598d84280a9a9f3d2770379627fec1f1762297 (diff) | |
msm: vidc: Check ion_handle for errors
When ion imports dma buf, it will return negative error number
in the case of failure like bad file number or invalid dma buf file.
check ion_handle for error numbers.
CRs-Fixed: 1071602
Change-Id: I1ea93161b85deb667cbb6f8515ff7c6943da6e3d
Signed-off-by: Karthikeyan Periasamy <kperiasa@codeaurora.org>
| -rw-r--r-- | drivers/media/platform/msm/vidc/msm_smem.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/media/platform/msm/vidc/msm_smem.c b/drivers/media/platform/msm/vidc/msm_smem.c index bb2715654f45..90047a608984 100644 --- a/drivers/media/platform/msm/vidc/msm_smem.c +++ b/drivers/media/platform/msm/vidc/msm_smem.c @@ -475,7 +475,7 @@ bool msm_smem_compare_buffers(void *clt, int fd, void *priv) } handle = ion_import_dma_buf(client->clnt, fd); ret = handle == priv; - handle ? ion_free(client->clnt, handle) : 0; + (!IS_ERR_OR_NULL(handle)) ? ion_free(client->clnt, handle) : 0; return ret; } |