From: Niels Möller <nisse@lysator.liu.se>
Date: Sun, 22 Jun 2025 19:07:55 +0000 (+0200)
Subject: Improve failure messages for test_mac.
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=670b9134adb63b5561b9dbdb567357ae37ce1d45;p=thirdparty%2Fnettle.git

Improve failure messages for test_mac.
---

diff --git a/ChangeLog b/ChangeLog
index fc08f527..4cfe7966 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,8 @@
 2025-06-22  Niels Möller  <nisse@lysator.liu.se>
 
+	* testsuite/testutils.c (test_mac): Print name of mac algorithm on
+	failure.
+
 	* gosthash94.c (gosthash94_init): Trim memset call, no need to
 	clear block buffer.
 	* md2.c (md2_init): Likewise.
diff --git a/testsuite/testutils.c b/testsuite/testutils.c
index a60e20d6..9ab354a3 100644
--- a/testsuite/testutils.c
+++ b/testsuite/testutils.c
@@ -1295,7 +1295,7 @@ test_mac(const struct nettle_mac *mac,
 
   if (!MEMEQ (digest->length, digest->data, hash))
     {
-      fprintf (stderr, "test_mac failed, msg: ");
+      fprintf (stderr, "test_mac %s failed, msg: ", mac->name);
       print_hex (msg->length, msg->data);
       fprintf(stderr, "Output:");
       print_hex (mac->digest_size, hash);
@@ -1310,7 +1310,7 @@ test_mac(const struct nettle_mac *mac,
   mac->digest (ctx, hash);
   if (!MEMEQ (digest->length, digest->data, hash))
     {
-      fprintf (stderr, "test_mac: failed on re-use, msg: ");
+      fprintf (stderr, "test_mac %s failed on re-use, msg: ", mac->name);
       print_hex (msg->length, msg->data);
       fprintf(stderr, "Output:");
       print_hex (mac->digest_size, hash);
@@ -1326,7 +1326,7 @@ test_mac(const struct nettle_mac *mac,
   mac->digest (ctx, hash);
   if (!MEMEQ (digest->length, digest->data, hash))
     {
-      fprintf (stderr, "test_mac failed on byte-by-byte, msg: ");
+      fprintf (stderr, "test_mac %s failed on byte-by-byte, msg: ", mac->name);
       print_hex (msg->length, msg->data);
       fprintf(stderr, "Output:");
       print_hex (16, hash);