[igt-dev] [i-g-t V2] tests/kms_color: Fix memory leaks

[Kamil Konieczny]

On 2022-07-06 at 15:02:53 +0530, Bhanuprakash Modem wrote:
> Free allocated memory before return.
> 
> Fixes: d61e4598
> Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem at intel.com>

Reviewed-by: Kamil Konieczny <kamil.konieczny at linux.intel.com>

> ---
>  tests/kms_color.c | 11 ++++++++++-
>  1 file changed, 10 insertions(+), 1 deletion(-)
> 
> diff --git a/tests/kms_color.c b/tests/kms_color.c
> index 0b03fb9a..6ea721f1 100644
> --- a/tests/kms_color.c
> +++ b/tests/kms_color.c
> @@ -397,8 +397,13 @@ static bool test_pipe_legacy_gamma_reset(data_t *data,
>  	drmModeFreeCrtc(kms_crtc);
>  
>  	red_lut = malloc(sizeof(uint16_t) * legacy_lut_size);
> +	igt_assert(red_lut);
> +
>  	green_lut = malloc(sizeof(uint16_t) * legacy_lut_size);
> +	igt_assert(green_lut);
> +
>  	blue_lut = malloc(sizeof(uint16_t) * legacy_lut_size);
> +	igt_assert(blue_lut);
>  
>  	for (i = 0; i < legacy_lut_size; i++)
>  		red_lut[i] = green_lut[i] = blue_lut[i] = 0xffff;
> @@ -427,13 +432,17 @@ static bool test_pipe_legacy_gamma_reset(data_t *data,
>  			   lut[i].blue == 0xffff);
>  	drmModeFreePropertyBlob(blob);
>  
> +	free(red_lut);
> +	free(green_lut);
> +	free(blue_lut);
> +
> +end:
>  	igt_plane_set_fb(primary, NULL);
>  	igt_output_set_pipe(output, PIPE_NONE);
>  	igt_display_commit(&data->display);
>  
>  	free_lut(degamma_linear);
>  	free_lut(gamma_zero);
> -end:
>  	return ret;
>  }
>  
> -- 
> 2.35.1
>