diff options
| author | Brian Starkey <brian.starkey@arm.com> | 2017-02-17 15:26:54 +0000 |
|---|---|---|
| committer | Petri Latvala <petri.latvala@intel.com> | 2017-02-20 12:41:52 +0200 |
| commit | 5ef31ea1c82c2571e7f65f11fddf99fd199d6314 (patch) | |
| tree | 6393561a943a6f366f26ee4ba79a64e531929b01 /tests/kms_atomic_transition.c | |
| parent | 50c4f0237183fcd35a0bbb93b5c60d69217f04b4 (diff) | |
kms_atomic_transition: Initialize pipe_crcs to avoid segfault
igt_pipe_crc_new() is skipped for non-Intel devices, but the later calls
to collect_crcs_mask() will attempt to retrieve CRC values if the
pipe_crcs pointers are non-NULL.
Zero-initialise pipe_crcs to avoid accessing garbage pointers on
non-Intel devices.
Signed-off-by: Brian Starkey <brian.starkey@arm.com>
Reviewed-by: Petri Latvala <petri.latvala@intel.com>
Diffstat (limited to 'tests/kms_atomic_transition.c')
| -rw-r--r-- | tests/kms_atomic_transition.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/tests/kms_atomic_transition.c b/tests/kms_atomic_transition.c index 797134c0..b0dce6c2 100644 --- a/tests/kms_atomic_transition.c +++ b/tests/kms_atomic_transition.c @@ -607,7 +607,7 @@ static void run_modeset_tests(igt_display_t *display, int howmany, bool nonblock struct igt_fb fbs[2]; int i, j; unsigned iter_max = 1 << display->n_pipes; - igt_pipe_crc_t *pipe_crcs[I915_MAX_PIPES]; + igt_pipe_crc_t *pipe_crcs[I915_MAX_PIPES] = { 0 }; igt_output_t *output; unsigned width = 0, height = 0; bool skip_test = false; |