The simpletlv_* functions return lengths as an int, but the test is casting it to a size_t before comparing it against -1. This is not giving the expected result on i686, causing the test to fail. The variables used in the test do not need to be size_t, and can be changed to int to match the return value and avoid the failure. Signed-off-by: Christophe Fergeau <cfergeau@xxxxxxxxxx> --- tests/simpletlv.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tests/simpletlv.c b/tests/simpletlv.c index cd0cd69..3e5958e 100644 --- a/tests/simpletlv.c +++ b/tests/simpletlv.c @@ -48,7 +48,7 @@ static void test_length_simple(void) static void test_length_nested(void) { - size_t length = 0; + int length = 0; unsigned char simple_value[] = "\x12\x14"; static struct simpletlv_member simple[1] = { {0x25, 2, {/*.value = simple_value*/}, SIMPLETLV_TYPE_LEAF} @@ -102,7 +102,7 @@ static void test_length_skipped(void) static void test_encode_simple(void) { unsigned char *result = NULL; - size_t result_len = 0; + int result_len = 0; unsigned char simple_value[] = "\x10\x11"; unsigned char simple_encoded[] = "\x25\x02\x10\x11"; unsigned char long_value[256] = "Long data value"; -- 2.17.1 _______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel