diff options
author | YoungJun Cho <yj44.cho@samsung.com> | 2013-06-26 10:21:40 +0900 |
---|---|---|
committer | Inki Dae <inki.dae@samsung.com> | 2014-10-15 20:05:11 +0900 |
commit | e9a61d842dc76d9413d9c44a0639184d6fb2e787 (patch) | |
tree | 2b83e773c34ea85cc4f1dc8671959ca68b4a3d5e | |
parent | 09e30e8f9b5f3a99cda8ca7da4bc04e15ccd1b8e (diff) | |
download | linux-3.10-e9a61d842dc76d9413d9c44a0639184d6fb2e787.tar.gz linux-3.10-e9a61d842dc76d9413d9c44a0639184d6fb2e787.tar.bz2 linux-3.10-e9a61d842dc76d9413d9c44a0639184d6fb2e787.zip |
drm/prime: fix to put an exported dma_buf for adding handle failure
When drm_prime_add_buf_handle() returns failure for an exported
dma_buf, the dma_buf was already allocated and its refcount was
increased, so it needs to be put.
Signed-off-by: YoungJun Cho <yj44.cho@samsung.com>
Signed-off-by: Seung-Woo Kim <sw0312.kim@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Signed-off-by: Dave Airlie <airlied@redhat.com>
-rw-r--r-- | drivers/gpu/drm/drm_prime.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/drivers/gpu/drm/drm_prime.c b/drivers/gpu/drm/drm_prime.c index a61266bc79f..a633b14e33d 100644 --- a/drivers/gpu/drm/drm_prime.c +++ b/drivers/gpu/drm/drm_prime.c @@ -307,7 +307,7 @@ int drm_gem_prime_handle_to_fd(struct drm_device *dev, ret = drm_prime_add_buf_handle(&file_priv->prime, obj->export_dma_buf, handle); if (ret) - goto out; + goto fail_put_dmabuf; *prime_fd = dma_buf_fd(buf, flags); mutex_unlock(&file_priv->prime.lock); @@ -316,6 +316,12 @@ int drm_gem_prime_handle_to_fd(struct drm_device *dev, out_have_obj: get_dma_buf(dmabuf); *prime_fd = dma_buf_fd(dmabuf, flags); + goto out; + +fail_put_dmabuf: + /* clear NOT to be checked when releasing dma_buf */ + obj->export_dma_buf = NULL; + dma_buf_put(buf); out: drm_gem_object_unreference_unlocked(obj); mutex_unlock(&file_priv->prime.lock); |