This PR adds an exhaustive test for the ECDH module, looping over all key combinations and verifying the commutativity property (ECDH(i*G, j) == ECDH(j*G, i)) and checking against a recalculated ECDH result (by manually invoking the default ECDH hash function on the precalculated group element group[i * j]'s coordinates). The existing test coverage is already solid (including Wycheproof test vectors), but I figured it likely wouldn't hurt to add this as well.
Add exhaustive test for ECDH module #1852
pull theStack wants to merge 1 commits into bitcoin-core:master from theStack:add_ecdh_exhaustive_test changing 3 files +64 −0-
theStack commented at 4:53 PM on May 2, 2026: contributor
-
Add exhaustive test for ECDH module 6b9e4b88dc
- real-or-random added the label assurance on May 2, 2026
- real-or-random added the label feature on May 2, 2026
- real-or-random approved
-
real-or-random commented at 7:56 PM on May 2, 2026: contributor
Contributors