diff options
| author | Dhinakaran Pandiyan <dhinakaran.pandiyan@gmail.com> | 2017-09-18 18:55:09 -0700 |
|---|---|---|
| committer | Rodrigo Vivi <rodrigo.vivi@intel.com> | 2017-09-22 06:05:31 -0700 |
| commit | b94a17d13fb5e03bbc7ef50ce88352b37ad06c85 (patch) | |
| tree | c1d4404dd90c0ed5496f40eacdb68a030bd0713a /tests | |
| parent | 7c79e2ca1f78d1d2fd95f66fbf98daaf4e56de41 (diff) | |
tests/psr: Don't strcmp CRCs that are not NULL terminated.
Switched to strncmp.
Signed-off-by: Dhinakaran Pandiyan <dhinakaran.pandiyan@gmail.com>
Reviewed-by: Radhakrishna Sripada <radhakrishna.sripada@intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi@intel.com>
Diffstat (limited to 'tests')
| -rw-r--r-- | tests/kms_psr_sink_crc.c | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/tests/kms_psr_sink_crc.c b/tests/kms_psr_sink_crc.c index f023b12c..83a69f0b 100644 --- a/tests/kms_psr_sink_crc.c +++ b/tests/kms_psr_sink_crc.c @@ -34,6 +34,7 @@ bool running_with_psr_disabled; #define CRC_BLACK "000000000000" +#define CRC_LEN 12 enum operations { PAGE_FLIP, @@ -243,7 +244,7 @@ static void get_sink_crc(data_t *data, char *crc) { usleep(300000); /* Black screen is always invalid */ - igt_assert(strcmp(crc, CRC_BLACK) != 0); + igt_assert(strncmp(crc, CRC_BLACK, CRC_LEN) != 0); } static bool is_green(char *crc) @@ -298,8 +299,8 @@ static void run_test(data_t *data) uint32_t handle = data->fb_white.gem_handle; igt_plane_t *test_plane; void *ptr; - char ref_crc[12]; - char crc[12]; + char ref_crc[CRC_LEN]; + char crc[CRC_LEN]; const char *expected = ""; /* Confirm that screen became Green */ @@ -356,9 +357,9 @@ static void run_test(data_t *data) memset(ptr, 0xff, data->mod_size); get_sink_crc(data, crc); if (data->test_plane == DRM_PLANE_TYPE_PRIMARY) - assert_or_manual(strcmp(ref_crc, crc) == 0, "screen WHITE"); + assert_or_manual(strncmp(ref_crc, crc, CRC_LEN) == 0, "screen WHITE"); else - assert_or_manual(strcmp(ref_crc, crc) == 0, + assert_or_manual(strncmp(ref_crc, crc, CRC_LEN) == 0, "GREEN background with WHITE box"); igt_info("Waiting 10s...\n"); @@ -401,7 +402,7 @@ static void run_test(data_t *data) break; } get_sink_crc(data, crc); - assert_or_manual(strcmp(ref_crc, crc) != 0, expected); + assert_or_manual(strncmp(ref_crc, crc, CRC_LEN) != 0, expected); } static void test_cleanup(data_t *data) { |