> -----Original Message----- > From: Subash Patel [mailto:subashrp@xxxxxxxxx] > Sent: Tuesday, June 26, 2012 3:23 AM > To: dri-devel@xxxxxxxxxxxxxxxxxxxxx; linux-samsung-soc@xxxxxxxxxxxxxxx; > linaro-mm-sig@xxxxxxxxxxxxxxxx > Cc: olofj@xxxxxxxxxxxx; inki.dae@xxxxxxxxxxx; airlied@xxxxxxxxxx; Subash > Patel; Subash Patel > Subject: [PATCH 2/2] DRM: Exynos: check for null in return value of > dma_buf_map_attachment() > > From: Subash Patel <subash.rp@xxxxxxxxxxx> > > dma_buf_map_attachment() can return NULL and valid sg as return > value. Hence the check for the returned scatter-gather must be using > the inline function IS_ERR_OR_NULL() in place of IS_ERR() > > Change-Id: I33218480e220f6a26a1e726b336bf533a95363de > Signed-off-by: Subash Patel <subash.ramaswamy@xxxxxxxxxx> > CC: dri-devel@xxxxxxxxxxxxxxxxxxxxx > CC: linux-samsung-soc@xxxxxxxxxxxxxxx > CC: linaro-mm-sig@xxxxxxxxxxxxxxxx > CC: inki.dae@xxxxxxxxxxx > CC: airlied@xxxxxxxxxx > CC: olofj@xxxxxxxxxxxx > --- > drivers/gpu/drm/exynos/exynos_drm_dmabuf.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/gpu/drm/exynos/exynos_drm_dmabuf.c > b/drivers/gpu/drm/exynos/exynos_drm_dmabuf.c > index 52cf761..c908a29 100644 > --- a/drivers/gpu/drm/exynos/exynos_drm_dmabuf.c > +++ b/drivers/gpu/drm/exynos/exynos_drm_dmabuf.c > @@ -245,7 +245,7 @@ struct drm_gem_object > *exynos_dmabuf_prime_import(struct drm_device *drm_dev, > > > sgt = dma_buf_map_attachment(attach, DMA_BIDIRECTIONAL); > - if (IS_ERR(sgt)) { > + if (IS_ERR_OR_NULL(sgt)) { > ret = PTR_ERR(sgt); > goto err_buf_detach; > } > -- > 1.7.9.5 Applied. Thanks, Inki Dae -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
 |