Skip to content

PROV: Ensure the AlgorithmIdentifier registers in DSA signature impl#11037

Closed
levitte wants to merge 1 commit intoopenssl:masterfrom
levitte:fix-DSA-implementation-20200207
Closed

PROV: Ensure the AlgorithmIdentifier registers in DSA signature impl#11037
levitte wants to merge 1 commit intoopenssl:masterfrom
levitte:fix-DSA-implementation-20200207

Conversation

@levitte
Copy link
Member

@levitte levitte commented Feb 7, 2020

When setting up the hash function for DSA signature, the encoded
AlgorithmIdentifier for the DSA+hash combination is queried, but not
stored, which leads to problems when signing ASN.1 items in libcrypto.

@levitte levitte added branch: master Applies to master branch approval: review pending This pull request needs review by a committer labels Feb 7, 2020
When setting up the hash function for DSA signature, the encoded
AlgorithmIdentifier for the DSA+hash combination is queried, but not
stored, which leads to problems when signing ASN.1 items in libcrypto.
@levitte levitte force-pushed the fix-DSA-implementation-20200207 branch from a8e153c to 312b421 Compare February 7, 2020 10:03
@slontis slontis added approval: done This pull request has the required number of approvals and removed approval: review pending This pull request needs review by a committer labels Feb 8, 2020
@iamamoose iamamoose added approval: ready to merge The 24 hour grace period has passed, ready to merge and removed approval: done This pull request has the required number of approvals labels Feb 10, 2020
@iamamoose
Copy link
Member

This pull request is ready to merge

openssl-machine pushed a commit that referenced this pull request Feb 10, 2020
When setting up the hash function for DSA signature, the encoded
AlgorithmIdentifier for the DSA+hash combination is queried, but not
stored, which leads to problems when signing ASN.1 items in libcrypto.

Reviewed-by: Shane Lontis <[email protected]>
(Merged from #11037)
@levitte
Copy link
Member Author

levitte commented Feb 10, 2020

Merged.

b0593c0 PROV: Ensure the AlgorithmIdentifier registers in DSA signature impl

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: ready to merge The 24 hour grace period has passed, ready to merge branch: master Applies to master branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants