From 652e5441504ddded9d6b138cd6df3447ed9b2e74 Mon Sep 17 00:00:00 2001 From: Daniel Vetter Date: Fri, 13 Jun 2014 15:19:09 +0200 Subject: [PATCH] tests/pm_psr_sink_crc: Fix longjmp fun igt_fixture and igt_subtests use longjmp/setjmp internally, which means local variables at the same stack frame are at risk. Best practice is to move them out right in front of the igt_main block. It would be awesome if someone could come up with a cocci patch to auto-fix this, but unfortunately my attempts failed. Signed-off-by: Daniel Vetter --- tests/kms_psr_sink_crc.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/kms_psr_sink_crc.c b/tests/kms_psr_sink_crc.c index 999a0bfd..c2c12d11 100644 --- a/tests/kms_psr_sink_crc.c +++ b/tests/kms_psr_sink_crc.c @@ -587,11 +587,11 @@ static void run_test(data_t *data) } } +data_t data = {}; +enum tests test; + igt_main { - data_t data = {}; - enum tests test; - igt_skip_on_simulation(); igt_fixture {