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

Use other basepoints #6

Open
swasilyev opened this issue Nov 11, 2019 · 3 comments
Open

Use other basepoints #6

swasilyev opened this issue Nov 11, 2019 · 3 comments

Comments

@swasilyev
Copy link
Collaborator

For fixed basepoint multiplication and also for Pedersen hashes we use basepoints provided by Zcash. We should better use our basepoints to prevent any kind of collision.

@burdges
Copy link
Collaborator

burdges commented Apr 2, 2020

We've no problem for the Schnorr DLEQ proof here. And the SNARK should handle the same points too, no? Are you worried about Pederson hashes agreeing due to people reusing keys?

@swasilyev
Copy link
Collaborator Author

Smth like that. Anyway it's obviously not less safer to have custom Personalization. Maybe in Zexe version

@burdges
Copy link
Collaborator

burdges commented Apr 7, 2020

We'll see how ZEXE handles JubjubEngine::Params too, maybe my JubjubEngineWithParams became unnecessary, or maybe we want it for exactly this.

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

No branches or pull requests

2 participants