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.

Sign up for GitHub

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

Jump to bottom

Fix #221: ecdsa-modified: fix bias and omission of zero in getBigRandom() #631

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Fix #221: ecdsa-modified: fix bias and omission of zero in getBigRandom() #631

wants to merge 1 commit into from

Commits on Nov 14, 2024

  1. ecdsa-modified: fix getBigRandom() 

    this replaces the previously remainder-based limiting of the random number
which caused bias toward small numbers and excluded zero altogether by
simple filtering as proposed frequently in
kjur#221
and because the performance in most cases is actually faster than in the
present implementation;

also, an adaptation of swiftlang/swift#39143 has
been considered but it performed significantly slower for large integers;
    @tvogel
    tvogel committed Nov 14, 2024
    Configuration menu
    Copy the full SHA
    4c12028 View commit details
    Browse the repository at this point in the history