From 9ffbe1e0a6aebb1f7ca989c59bf70e7e8bd6981d Mon Sep 17 00:00:00 2001 From: Simon McVittie Date: Fri, 6 Sep 2019 11:42:47 +0100 Subject: tests: Force defined behaviour for 24-bit left-shifts When r is 128 or more, running tests compiled with the undefined behaviour sanitizer (ubsan) reports: test-utils.c:312:45: runtime error: left shift of 128 by 24 places cannot be represented in type 'int' which indeed it cannot. Force the type to be unsigned 32-bit so that we get defined behaviour. Similarly, in test-atlas-migration, the left-shifted guint8 is promoted to int, which again does not have enough non-sign bits available to left-shift a value >= 128 by 24 bits. Again, force the shift to be done in unsigned 32-bit space. Signed-off-by: Simon McVittie --- test-fixtures/test-utils.c | 12 +++++++++++- tests/conform/test-atlas-migration.c | 6 +++--- 2 files changed, 14 insertions(+), 4 deletions(-) diff --git a/test-fixtures/test-utils.c b/test-fixtures/test-utils.c index 59e3fd8c..a08fda0e 100644 --- a/test-fixtures/test-utils.c +++ b/test-fixtures/test-utils.c @@ -298,7 +298,17 @@ void test_utils_check_pixel_rgb (CoglFramebuffer *test_fb, int x, int y, int r, int g, int b) { - test_utils_check_pixel (test_fb, x, y, (r << 24) | (g << 16) | (b << 8)); + g_return_if_fail (r >= 0); + g_return_if_fail (g >= 0); + g_return_if_fail (b >= 0); + g_return_if_fail (r <= 0xFF); + g_return_if_fail (g <= 0xFF); + g_return_if_fail (b <= 0xFF); + + test_utils_check_pixel (test_fb, x, y, + (((guint32) r) << 24) | + (((guint32) g) << 16) | + (((guint32) b) << 8)); } void diff --git a/tests/conform/test-atlas-migration.c b/tests/conform/test-atlas-migration.c index 39e8a3c1..60502760 100644 --- a/tests/conform/test-atlas-migration.c +++ b/tests/conform/test-atlas-migration.c @@ -98,9 +98,9 @@ verify_texture (CoglTexture *texture, int size) }; test_utils_compare_pixel (p, - (real_color.red << 24) | - (real_color.green << 16) | - (real_color.blue << 8) | + (((guint32) real_color.red) << 24) | + (((guint32) real_color.green) << 16) | + (((guint32) real_color.blue) << 8) | opacity); g_assert_cmpint (p[3], ==, opacity); -- cgit v1.2.1