doc/bench: Added cmake build options to bench error messages
What the developer wrote
Authored by kevkevinpal
doc/bench: Added cmake build options to bench error messages
What changed, and why it matters
This commit only changes the text of error messages in the benchmark program. It replaces specific outdated build instructions (referencing the old Autotools ./configure command) with a generic pointer to README.md, because the project now uses CMake. There is no change to cryptographic code, no security fix, and no vulnerability.
No security action needed. Treat as a normal documentation/help-text cleanup.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The patch updates four fprintf(stderr, …) strings in src/bench.c. When a benchmark is requested for a module that was not compiled in, the program now tells the user to consult README.md for configuration instructions instead of naming a specific ./configure flag. This is a documentation/help-text change in the bench utility only; it does not touch secp256k1’s cryptographic implementation, build system logic, or any runtime behavior beyond the emitted error string.
Changed components
src/bench.c (benchmark utility error messages only)Inspect captured patch +4 / −4
diff --git a/src/bench.c b/src/bench.c
index 8ba7623..04685d9 100644
--- a/src/bench.c
+++ b/src/bench.c
@@ -205,7 +205,7 @@ int main(int argc, char** argv) {
#ifndef ENABLE_MODULE_ECDH
if (have_flag(argc, argv, "ecdh")) {
fprintf(stderr, "./bench: ECDH module not enabled.\n");
- fprintf(stderr, "Use ./configure --enable-module-ecdh.\n\n");
+ fprintf(stderr, "See README.md for configuration instructions.\n\n");
return EXIT_FAILURE;
}
#endif
@@ -213,7 +213,7 @@ int main(int argc, char** argv) {
#ifndef ENABLE_MODULE_RECOVERY
if (have_flag(argc, argv, "recover") || have_flag(argc, argv, "ecdsa_recover")) {
fprintf(stderr, "./bench: Public key recovery module not enabled.\n");
- fprintf(stderr, "Use ./configure --enable-module-recovery.\n\n");
+ fprintf(stderr, "See README.md for configuration instructions.\n\n");
return EXIT_FAILURE;
}
#endif
@@ -221,7 +221,7 @@ int main(int argc, char** argv) {
#ifndef ENABLE_MODULE_SCHNORRSIG
if (have_flag(argc, argv, "schnorrsig") || have_flag(argc, argv, "schnorrsig_sign") || have_flag(argc, argv, "schnorrsig_verify")) {
fprintf(stderr, "./bench: Schnorr signatures module not enabled.\n");
- fprintf(stderr, "Use ./configure --enable-module-schnorrsig.\n\n");
+ fprintf(stderr, "See README.md for configuration instructions.\n\n");
return EXIT_FAILURE;
}
#endif
@@ -231,7 +231,7 @@ int main(int argc, char** argv) {
have_flag(argc, argv, "encode") || have_flag(argc, argv, "decode") || have_flag(argc, argv, "ellswift_keygen") ||
have_flag(argc, argv, "ellswift_ecdh")) {
fprintf(stderr, "./bench: ElligatorSwift module not enabled.\n");
- fprintf(stderr, "Use ./configure --enable-module-ellswift.\n\n");
+ fprintf(stderr, "See README.md for configuration instructions.\n\n");
return EXIT_FAILURE;
}
#endif
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.