From 4833caed466ce8b7ae1d5fbafc23740885dbd9cc Mon Sep 17 00:00:00 2001 From: Pauli Date: Mon, 10 Apr 2017 13:38:24 +1000 Subject: [PATCH] Remove fprintfs from the poly1305 internal test but keep the test number information. The framework will display the non-matching memory. Reviewed-by: Matt Caswell Reviewed-by: Richard Levitte (Merged from https://github.com/openssl/openssl/pull/3156) --- test/poly1305_internal_test.c | 39 +++++------------------------------ 1 file changed, 5 insertions(+), 34 deletions(-) diff --git a/test/poly1305_internal_test.c b/test/poly1305_internal_test.c index af54ac3e6c..03b0fa91df 100644 --- a/test/poly1305_internal_test.c +++ b/test/poly1305_internal_test.c @@ -35,15 +35,6 @@ typedef struct { * ***/ -/* TODO : hex decoder / encoder should be implemented in testutil.c */ -static void hexdump(const unsigned char *a, size_t len) -{ - size_t i; - - for (i = 0; i < len; i++) - fprintf(stderr, "%02x", a[i]); -} - static void benchmark_poly1305() { # ifdef OPENSSL_CPUID_OBJ @@ -1573,12 +1564,7 @@ static int test_poly1305(int idx) Poly1305_Final(&poly1305, out); if (!TEST_mem_eq(out, expectedlen, expected, expectedlen)) { - fprintf(stderr, "Poly1305 test #%d failed.\n", idx); - fprintf(stderr, "got: "); - hexdump(out, sizeof(out)); - fprintf(stderr, "\nexpected: "); - hexdump(expected, expectedlen); - fprintf(stderr, "\n"); + TEST_info("Poly1305 test #%d failed.", idx); return 0; } @@ -1589,12 +1575,7 @@ static int test_poly1305(int idx) Poly1305_Final(&poly1305, out); if (!TEST_mem_eq(out, expectedlen, expected, expectedlen)) { - fprintf(stderr, "Poly1305 test #%d/1+(N-1) failed.\n", idx); - fprintf(stderr, "got: "); - hexdump(out, sizeof(out)); - fprintf(stderr, "\nexpected: "); - hexdump(expected, expectedlen); - fprintf(stderr, "\n"); + TEST_info("Poly1305 test #%d/1+(N-1) failed.", idx); return 0; } } @@ -1608,12 +1589,7 @@ static int test_poly1305(int idx) Poly1305_Final(&poly1305, out); if (!TEST_mem_eq(out, expectedlen, expected, expectedlen)) { - fprintf(stderr, "Poly1305 test #%d/2 failed.\n", idx); - fprintf(stderr, "got: "); - hexdump(out, sizeof(out)); - fprintf(stderr, "\nexpected: "); - hexdump(expected, expectedlen); - fprintf(stderr, "\n"); + TEST_info("Poly1305 test #%d/2 failed.", idx); return 0; } @@ -1624,13 +1600,8 @@ static int test_poly1305(int idx) Poly1305_Final(&poly1305, out); if (!TEST_mem_eq(out, expectedlen, expected, expectedlen)) { - fprintf(stderr, "Poly1305 test #%d/%" OSSLzu "+%" OSSLzu " failed.\n", - idx, half, inlen-half); - fprintf(stderr, "got: "); - hexdump(out, sizeof(out)); - fprintf(stderr, "\nexpected: "); - hexdump(expected, expectedlen); - fprintf(stderr, "\n"); + TEST_info("Poly1305 test #%d/%" OSSLzu "+%" OSSLzu " failed.", + idx, half, inlen-half); return 0; } } -- 2.25.1