Skip to content

Comments

Document EC_POINT_get_affine_coordinates_*.#5532

Closed
davidben wants to merge 1 commit intoopenssl:masterfrom
davidben:doc-get-affine
Closed

Document EC_POINT_get_affine_coordinates_*.#5532
davidben wants to merge 1 commit intoopenssl:masterfrom
davidben:doc-get-affine

Conversation

@davidben
Copy link
Contributor

@davidben davidben commented Mar 6, 2018

In particular, x and y may be NULL, as used in ecdsa_ossl.c. Make use of
this in ecdh_ossl.c as well, to save an otherwise unnecessary temporary.

Checklist
  • documentation is added or updated

In particular, x and y may be NULL, as used in ecdsa_ossl.c. Make use of
this in ecdh_ossl.c as well, to save an otherwise unnecessary temporary.
@mattcaswell mattcaswell added this to the 1.1.1 milestone Mar 20, 2018
@mattcaswell mattcaswell added approval: done This pull request has the required number of approvals branch: master Applies to master branch 1.1.0 branch: 1.1.1 Applies to OpenSSL_1_1_1-stable branch (EOL) labels Mar 21, 2018
@mattcaswell
Copy link
Member

My approval counts for 1.1.0 too if you want to backport it there.

@davidben
Copy link
Contributor Author

Merged as ddc1caa. Hopefully I got it right this time. :-)

@davidben davidben closed this Mar 21, 2018
levitte pushed a commit that referenced this pull request Mar 21, 2018
In particular, x and y may be NULL, as used in ecdsa_ossl.c. Make use of
this in ecdh_ossl.c as well, to save an otherwise unnecessary temporary.

Reviewed-by: Paul Dale <[email protected]>
Reviewed-by: Matt Caswell <[email protected]>
(Merged from #5532)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approval: done This pull request has the required number of approvals branch: master Applies to master branch branch: 1.1.1 Applies to OpenSSL_1_1_1-stable branch (EOL)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants