ci: Fix leftover use of old ECMULTGENPRECISION
What changed, and why it matters
This is a one-line fix in the project's automated testing configuration. It renames a build option used only by the CI test runner from an old name (ECMULTGENPRECISION) to the current name (ECMULTGENKB). The change does not touch any cryptographic code, does not affect released binaries, and has no security relevance.
No security action needed. Treat as routine CI maintenance.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit updates .github/workflows/ci.yml, replacing the stale environment variable ECMULTGENPRECISION with ECMULTGENKB in a single matrix entry. This aligns the CI configuration with a prior rename in the build system. It is purely a CI/maintenance change; no source code, build defaults, or runtime behavior for end users is modified.
Changed components
.github/workflows/ci.ymlInspect captured patch +1 / −1
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index 5f8c973..44c2754 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -514,7 +514,7 @@ jobs:
matrix:
env_vars:
- { WIDEMUL: 'int64', RECOVERY: 'yes', ECDH: 'yes', EXTRAKEYS: 'yes', SCHNORRSIG: 'yes', MUSIG: 'yes', ELLSWIFT: 'yes' }
- - { WIDEMUL: 'int128_struct', ECMULTGENPRECISION: 2, ECMULTWINDOW: 4 }
+ - { WIDEMUL: 'int128_struct', ECMULTGENKB: 2, ECMULTWINDOW: 4 }
- { WIDEMUL: 'int128', ECDH: 'yes', EXTRAKEYS: 'yes', SCHNORRSIG: 'yes', MUSIG: 'yes', ELLSWIFT: 'yes' }
- { WIDEMUL: 'int128', RECOVERY: 'yes' }
- { WIDEMUL: 'int128', RECOVERY: 'yes', ECDH: 'yes', EXTRAKEYS: 'yes', SCHNORRSIG: 'yes', MUSIG: 'yes', ELLSWIFT: 'yes' }
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.