summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBhanuprakash Modem <bhanuprakash.modem@intel.com>2022-07-06 15:02:53 +0530
committerBhanuprakash Modem <bhanuprakash.modem@intel.com>2022-07-08 10:55:34 +0530
commitcd21565a5fffc451e2cae5f1a0fe20104dc2a2cf (patch)
treeae140fa94e89d7bb06ba0f6b721b932c00c61adc
parentd001f24bac01f1ab77f68912ec5baaa21196b828 (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.c11
1 files changed, 10 insertions, 1 deletions
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;
}