[PATCH 2/3] drm/omap: fix possible NULL ref issue in tiler_reserve_2d

[Laurent Pinchart]

Hi Tomi,

Thank you for the patch.

On Thursday, 29 March 2018 13:40:37 EEST Tomi Valkeinen wrote:
> tiler_reserve_2d allocates memory but does not check if it got the
> memory. Add the check and return ENOMEM on failure.
> 
> Signed-off-by: Tomi Valkeinen <tomi.valkeinen at ti.com>
> ---
>  drivers/gpu/drm/omapdrm/omap_dmm_tiler.c | 3 +++
>  1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c
> b/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c index c40f90d2db82..28d93ece191e
> 100644
> --- a/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c
> +++ b/drivers/gpu/drm/omapdrm/omap_dmm_tiler.c
> @@ -532,6 +532,9 @@ struct tiler_block *tiler_reserve_2d(enum tiler_fmt fmt,
> u16 w, unsigned long flags;
>  	u32 slot_bytes;
> 
> +	if (!block)
> +		return ERR_PTR(-ENOMEM);
> +

I'd write this as

	struct tiler_block *block;
	...

	block = kzalloc(sizeof(*block), GFP_KERNEL);
	if (!block)
		return ERR_PTR(-ENOMEM);

to make it clearer. I'll leave it up to you though, so in any case

Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>

>  	BUG_ON(!validfmt(fmt));
> 
>  	/* convert width/height to slots */

-- 
Regards,

Laurent Pinchart