Skip to content
This repository has been archived by the owner on Jan 13, 2025. It is now read-only.

[zk-token-sdk] Fix ElGamal key derivation #28792

Merged
merged 2 commits into from
Nov 11, 2022
Merged

[zk-token-sdk] Fix ElGamal key derivation #28792

merged 2 commits into from
Nov 11, 2022

Conversation

samkim-crypto
Copy link
Contributor

Problem

The inversion method in the ElGamal pubkey derivation was removed when I was adding additional comments and cleaning up the code.

Summary of Changes

  • Add the inversion method again
  • Augment the tests so that these type of misses can be caught in the future

@samkim-crypto samkim-crypto merged commit c828031 into solana-labs:master Nov 11, 2022
xiangzhu70 pushed a commit to xiangzhu70/solana that referenced this pull request Nov 16, 2022
@samkim-crypto samkim-crypto deleted the zk-token-sdk/fix-pubkey-derivation branch December 1, 2022 00:34
mergify bot pushed a commit that referenced this pull request Dec 2, 2022
* fix ElGamal key derivation

* cargo fmt

(cherry picked from commit c828031)
mergify bot added a commit that referenced this pull request Dec 2, 2022
[zk-token-sdk] Fix ElGamal key derivation (#28792)

* fix ElGamal key derivation

* cargo fmt

(cherry picked from commit c828031)

Co-authored-by: samkim-crypto <skim13@cs.stanford.edu>
gnapoli23 pushed a commit to gnapoli23/solana that referenced this pull request Dec 16, 2022
nickfrosty pushed a commit to nickfrosty/solana that referenced this pull request Jan 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant