diff options
author | Bhanuprakash Modem <bhanuprakash.modem@intel.com> | 2022-07-06 15:02:53 +0530 |
---|---|---|
committer | Bhanuprakash Modem <bhanuprakash.modem@intel.com> | 2022-07-08 10:55:34 +0530 |
commit | cd21565a5fffc451e2cae5f1a0fe20104dc2a2cf (patch) | |
tree | ae140fa94e89d7bb06ba0f6b721b932c00c61adc | |
parent | d001f24bac01f1ab77f68912ec5baaa21196b828 (diff) |
tests/kms_color: Fix memory leaks
Free allocated memory before return.
Fixes: d61e4598
Signed-off-by: Bhanuprakash Modem <bhanuprakash.modem@intel.com>
Reviewed-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>
-rw-r--r-- | tests/kms_color.c | 11 |
1 files changed, 10 insertions, 1 deletions
diff --git a/tests/kms_color.c b/tests/kms_color.c index 0b03fb9ad..6ea721f14 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; } |