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

Should the proof of soundness (Schnorr) in page 12 use fixed C_0 (as well as the corresponding x_0) to extract (x_1, x_2, ..., x_m) by running m+1 times? #9

Open
GoldSaintEagle opened this issue Jan 3, 2019 · 2 comments

Comments

@GoldSaintEagle
Copy link

Should the proof of soundness (Schnorr) in page 12 use fixed C_0 (as well as the corresponding x_0) to extract (x_1, x_2, ..., x_m) by running m+1 times?

I think C_0 should be fixed to get the Vandermonde matrix, which should be as follows:
(C_0, e_1, (z_1, s_1))
(C_0, e_2, (z_2, s_2))
...
(C_0, e_m, (z_m, s_m))
Otherwise we cannot get the Vandermonde matrix since x_0 changes every time.

Anyway, thanks for your perfect introduction of ZK and bullet proof.

@baro77
Copy link

baro77 commented Jul 29, 2022

A lot of time since you opening this issue, anyway....

nope, Vandermonde matrix is a math entity existing regardless of prover rewinding (resulting in constant C_0) or not during the m+1 runs. What could be improved in the text is run index starting from 0 e not from 1 , so the first run transcript should be (C_0,0, e_0, (z_0, s_0)) - see PR #17

Extractor proof doesn't rely on initial commitment being constant on multiple rounds, which seems a relevant point in commitments case compared to Schnorr identity case: it seems to suggest rewinding is not necessary in commitments case.. which I fear it could cause problems to ZK, but that's another matter..

@baro77
Copy link

baro77 commented Aug 3, 2022

Ok, I have studied better the proof: I was wrong, it needs constant C_0, because it uses C_i with i in [0...m] range

However I underline again it's not because otherwise you cannot define Vandermonde matrix, in fact it's a consequence of equalities on page 14

Im going to update my PR #17

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