HomePhabricator

Add test case for CheckSignatureEncoding for proper DER encoding

Description

Add test case for CheckSignatureEncoding for proper DER encoding

Summary:
This ensures IsValidSignatureEncoding is properly checked.

The test time is atrocious because IsValidSignatureEncoding assumes the presence of the sighash type which causes a combinotary explosion, and also uncover a SRP violation. I would like to refactor these to detangle the two, but obviously doing so safely require a test so here we are.

Depends on D1570

Test Plan:

make check

Reviewers: #bitcoin_abc, jasonbcox

Reviewed By: #bitcoin_abc, jasonbcox

Subscribers: jasonbcox, teamcity

Differential Revision: https://reviews.bitcoinabc.org/D1571

Details

Provenance
deadalnixAuthored on Jul 15 2018, 18:38
deadalnixPushed on Jul 23 2018, 10:50
Reviewer
Restricted Project
Differential Revision
D1571: Add test case for CheckSignatureEncoding for proper DER encoding
Parents
rSTAGING184cf2684319: Create a typedef for handling mapDeltas in the mempool.
Branches
Unknown
Tags
Unknown
References
tag: phabricator/base/4370