[PATCH] dma-buf: Fix static checker warning

[Christian König]

Am 03.11.20 um 04:51 schrieb Jianxin Xiong:
> Here is the warning message:
>
> 	drivers/dma-buf/dma-buf.c:917 dma_buf_map_attachment()
> 	error: 'sg_table' dereferencing possible ERR_PTR()
>
> Fix by adding error checking before dereferencing the pointer.
>
> Fixes: ac80cd17a615 ("dma-buf: Clarify that dma-buf sg lists are page aligned")
> Reported-by: Dan Carpenter <dan.carpenter at oracle.com>
> Signed-off-by: Jianxin Xiong <jianxin.xiong at intel.com>

Reviewed-by: Christian König <christian.koenig at amd.com>

Do you have commit access to drm-misc-next or should I push it?

> ---
>   drivers/dma-buf/dma-buf.c | 2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/dma-buf/dma-buf.c b/drivers/dma-buf/dma-buf.c
> index 556f62e..0eb80c1 100644
> --- a/drivers/dma-buf/dma-buf.c
> +++ b/drivers/dma-buf/dma-buf.c
> @@ -908,7 +908,7 @@ struct sg_table *dma_buf_map_attachment(struct dma_buf_attachment *attach,
>   	}
>   
>   #ifdef CONFIG_DMA_API_DEBUG
> -	{
> +	if (!IS_ERR(sg_table)) {
>   		struct scatterlist *sg;
>   		u64 addr;
>   		int len;