Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

secp256k1_fe_sqrt: Verify that the arguments don't alias. #551

Merged
merged 1 commit into from
Feb 21, 2019

Conversation

roconnor-blockstream
Copy link
Contributor

No description provided.

@roconnor-blockstream roconnor-blockstream changed the title secp256k1_fe_sqrt: Verify that the arguements don't alias. secp256k1_fe_sqrt: Verify that the arguments don't alias. Aug 14, 2018
@DesWurstes
Copy link
Contributor

Please change

static int secp256k1_fe_sqrt(secp256k1_fe *r, const secp256k1_fe *a)

to

static int secp256k1_fe_sqrt(secp256k1_fe *r, const secp256k1_fe * SECP256K1_RESTRICT a)

@roconnor-blockstream
Copy link
Contributor Author

I think your requested amendment is a good idea. However, I do not currently have the energy to shepherd this PR. Maintainers have permission to edit this branch, otherwise you can fork this PR and continue with a new PR.

@gmaxwell gmaxwell merged commit bf8b86c into bitcoin-core:master Feb 21, 2019
gmaxwell added a commit that referenced this pull request Feb 21, 2019
bf8b86c secp256k1_fe_sqrt: Verify that the arguments don't alias. (Russell O'Connor)

Pull request description:

Tree-SHA512: 4e9128625817be83a1259e7dbdb82c4be8ee0174af7d5e76eeac3245d23d6bf01378d9697dcb870eb61577afd7c0556cb3c6633e3b27504add14c10b18f7c607
gmaxwell added a commit that referenced this pull request Feb 24, 2019
… aliasing.

b76e45d Make bench_internal obey secp256k1_fe_sqrt's contract wrt aliasing. (Gregory Maxwell)

Pull request description:

  Bench_internal was previously incorrect but wasn't detected by -DVERIFY until PR #551.

Tree-SHA512: 5ecb9e3ba63ddf28cde4012154dbb1dda9b7178016d970dba58a4af20c31aa1ec3458ba2623541233fb16fceff7b3053054d6eee0229d0bfb6281d1a10554e9f
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.

3 participants