Skip to content

Allow to grind ECDSA signature to have low-R - #54

Closed
dgpv wants to merge 14 commits into
v1.1.2-devfrom
grind-ecdsa-sig-size
Closed

dgpv wants to merge 14 commits into
v1.1.2-devfrom
grind-ecdsa-sig-size

Conversation

@dgpv

@dgpv dgpv commented Oct 21, 2020

Copy link
Copy Markdown

No description provided.

dgpv and others added 13 commits October 17, 2020 19:42
This built-in function is faster, and using it, we can avoid
copying as much data around, since each slice will be copying.
It also is easier to read.
@classmethod parameter is always a type, so the check was meaningless
This also removes the need for the stream to be seekable
f.tell() was used to get the stream position for better error message,
but this means the stream has to be seekable. Instead of this
constraint, just check if the stream has 'tell' attribute and provide
a lambda that return '<untracked>' for the error message instead of the
position
Previously the empty witness that was supplied to CTransaction.__init__
was not checked for compatibility with currently selected witness
class for mutable transaction, it just replaced with new CTxWitness() instance.
After this change, it is checked that the supplied instance was compatible
with new instance
@dgpv
dgpv force-pushed the grind-ecdsa-sig-size branch from fdc1531 to ec87cb1 Compare October 21, 2020 18:15
@dgpv dgpv changed the title Allow to grind ECDSA signature to have certain size Allow to grind ECDSA signature to have low-R Oct 21, 2020
@dgpv
dgpv force-pushed the grind-ecdsa-sig-size branch from ec87cb1 to e0d48b6 Compare October 21, 2020 18:18
@dgpv

dgpv commented Oct 28, 2020

Copy link
Copy Markdown
Author

Included in v1.1.2-dev now, closing

@dgpv dgpv closed this Oct 28, 2020
@dgpv
dgpv deleted the grind-ecdsa-sig-size branch October 28, 2020 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants