[PATCH][V2][next] drm/vmwgfx: Fix memory allocation check and a leak of object fifo

Zack Rusin zackr at vmware.com
Fri May 14 18:32:21 UTC 2021


On 5/14/21 10:49 AM, Colin King wrote:
> From: Colin Ian King <colin.king at canonical.com>
> 
> The allocation of fifo is lacking an allocation failure check, so
> fix this by adding one.
> 
> In the case where fifo->static_buffer fails to be allocated the
> error return path neglects to kfree the fifo object. Fix this by
> adding in the missing kfree.
> 
> Kudos to Dan Carpenter for spotting the missing kzalloc failure
> check.
> 
> Addresses-Coverity: ("Resource leak")
> Fixes: 2cd80dbd3551 ("drm/vmwgfx: Add basic support for SVGA3")
> Signed-off-by: Colin Ian King <colin.king at canonical.com>
> ---
> 
> V2: Add missing allocation failure check
>      Update $SUBJECT to reflect this extra change

Looks good. Thanks. I'll push it through drm-misc-next.

z


More information about the dri-devel mailing list