commit 8097632475b8093f669ef33fa24414fc36e51eea
author Adam Langley <alangley@gmail.com> Tue Sep 29 10:55:55 2020 -0700
committer Adam Langley <alangley@gmail.com> Tue Sep 29 20:35:50 2020 +0000
tree c26b8195c0b13b24ee59cf16fa89fbeb70ef0ac7
parent 3ff161cc0876443bfa54c511b37d033312c81328

modulewrapper: fix unknown functions.

Due to a typo, if an unknown function was requested it was previously
ignored.

Change-Id: Id815c4b7e80d1452034b10bf9c7beb80a5ac3ed2
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/43125
Reviewed-by: David Benjamin <davidben@google.com>

util/fipstools/acvp/modulewrapper/modulewrapper.cc

diff --git a/util/fipstools/acvp/modulewrapper/modulewrapper.cc b/util/fipstools/acvp/modulewrapper/modulewrapper.cc
index 219fe7e..6ea5143 100644
--- a/util/fipstools/acvp/modulewrapper/modulewrapper.cc
+++ b/util/fipstools/acvp/modulewrapper/modulewrapper.cc
@@ -703,7 +703,7 @@
       offset += nums[i + 1];
     }
 
-    bool found = true;
+    bool found = false;
     for (const auto &func : kFunctions) {
       if (args[0].size() == strlen(func.name) &&
           memcmp(args[0].data(), func.name, args[0].size()) == 0) {
@@ -715,6 +715,7 @@
         }
 
         if (!func.handler(&args[1])) {
+          fprintf(stderr, "\'%s\' operation failed.\n", func.name);
           return 4;
         }