From: Mateusz Polrola <mateuszx.potr...@intel.com>

Added error checking on the result of function call,
dma_buf_map_attachment

Signed-off-by: Dongwon Kim <dongwon....@intel.com>
---
 drivers/xen/hyper_dmabuf/hyper_dmabuf_ioctl.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/drivers/xen/hyper_dmabuf/hyper_dmabuf_ioctl.c 
b/drivers/xen/hyper_dmabuf/hyper_dmabuf_ioctl.c
index c0048d9..476c0d7 100644
--- a/drivers/xen/hyper_dmabuf/hyper_dmabuf_ioctl.c
+++ b/drivers/xen/hyper_dmabuf/hyper_dmabuf_ioctl.c
@@ -149,6 +149,11 @@ static int hyper_dmabuf_export_remote(struct file *filp, 
void *data)
 
        sgt = dma_buf_map_attachment(attachment, DMA_BIDIRECTIONAL);
 
+       if (IS_ERR(sgt)) {
+               dev_err(hyper_dmabuf_private.device, "Cannot map attachment\n");
+               return PTR_ERR(sgt);
+       }
+
        sgt_info = kcalloc(1, sizeof(*sgt_info), GFP_KERNEL);
 
        if(!sgt_info) {
-- 
2.7.4

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to