Skip to content

Conversation

tomato42
Copy link
Member

all curves we have defined, with exception of NIST P-521, have
bit length of order that's multiple of 8, and there are no hashes
that are longer than 512 bits, so NIST P-521 is not affected either,
but do fix this issue in case user provides raw input longer than
512 bits or uses custom curves

fix #197

@tomato42 tomato42 added the bug unintended behaviour in ecdsa code label Nov 11, 2020
@tomato42 tomato42 self-assigned this Nov 11, 2020
all curves we have defined, with exception of NIST P-521, have
bit length of order that's multiple of 8, and there are no hashes
that are longer than 512 bits, so NIST P-521 is not affected either,
but do fix this issue in case user provides raw input longer than
512 bits or uses custom curves
@coveralls
Copy link

Coverage Status

Coverage decreased (-2.8%) to 95.195% when pulling ccff823 on tomato42:fix-truncation into 73f3d3e on warner:master.

@coveralls
Copy link

coveralls commented Nov 11, 2020

Coverage Status

Coverage increased (+0.001%) to 97.997% when pulling ccff823 on tomato42:fix-truncation into 73f3d3e on warner:master.

@tomato42
Copy link
Member Author

Copy link

@The-Mule The-Mule left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug unintended behaviour in ecdsa code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Digest truncation should happen according to bitlength not bytelength
3 participants