From: Mateusz Polrola <mateuszx.potrola@xxxxxxxxx> Added error checking on the result of function call, dma_buf_map_attachment Signed-off-by: Dongwon Kim <dongwon.kim@xxxxxxxxx> --- 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@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/dri-devel