Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ECDSA verification: Refactor a*g + b*p multiplication #1757

Merged
merged 4 commits into from
Oct 18, 2023

P-256 ECDSA verification: Clarify multiplication.

107c046
Select commit
Loading
Failed to load commit list.
Merged

ECDSA verification: Refactor a*g + b*p multiplication #1757

P-256 ECDSA verification: Clarify multiplication.
107c046
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Oct 18, 2023 in 0s

95.98% (+0.04%) compared to 2a0e495

View this Pull Request on Codecov

95.98% (+0.04%) compared to 2a0e495

Details

Codecov Report

Merging #1757 (107c046) into main (2a0e495) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1757      +/-   ##
==========================================
+ Coverage   95.93%   95.98%   +0.04%     
==========================================
  Files         138      138              
  Lines       20610    20691      +81     
  Branches      221      226       +5     
==========================================
+ Hits        19772    19860      +88     
+ Misses        800      792       -8     
- Partials       38       39       +1     
Files Coverage Δ
crypto/fipsmodule/ec/p256-nistz.c 96.73% <100.00%> (+0.84%) ⬆️
src/ec/suite_b/ecdsa/verification.rs 98.52% <100.00%> (-0.12%) ⬇️
src/ec/suite_b/ops.rs 98.08% <100.00%> (+0.03%) ⬆️
src/ec/suite_b/ops/p256.rs 100.00% <100.00%> (ø)
src/ec/suite_b/ops/p384.rs 100.00% <100.00%> (ø)

... and 3 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more