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

snark-verifier test failed #24

Open
Velaciela opened this issue Dec 30, 2022 · 4 comments
Open

snark-verifier test failed #24

Velaciela opened this issue Dec 30, 2022 · 4 comments

Comments

@Velaciela
Copy link

Velaciela commented Dec 30, 2022

recent dev branch (scroll-dev-1220) does not pass the tests in snark-verifier

by adding commit: [revert vanishing.commit to PSE's version] to branch: [pse_snark_verifier_test]
the proofs can be verified by two examples in the snark-verifier and pass default tests

@jonathanpwang @kunxian-xia @lispc

@Velaciela
Copy link
Author

Velaciela commented Dec 30, 2022

Here is another test from @kunxian-xia
If we change C::identity to C::generator, which writes point (1,2)
it can also pass evm-verifier
6067D673-F33F-48E2-80B6-FDA324B20282

        // Sample a random polynomial of degree n - 1
        let mut random_poly = domain.empty_coeff();
        random_poly[0] = C::Scalar::one();
        // Sample a random blinding factor
        let random_blind = Blind(C::Scalar::zero());
        let c = C::generator();

@kunxian-xia
Copy link

Is it possible that the evm-verifier cannot handle the point at infinity properly?

@jonathanpwang
Copy link

jonathanpwang commented Dec 30, 2022

Good find!

BTW for true no-zk option, we should try to merge privacy-scaling-explorations#76
snark-verifier should be able to support this once it's merged (most of the code is already there), due to @han0110.

@jonathanpwang
Copy link

Is it possible that the evm-verifier cannot handle the point at infinity properly?

@han0110 told me the evm-verifier does sanity checks for safety and specifically rejects point at infinity: https://github.com/privacy-scaling-explorations/snark-verifier/blob/main/snark-verifier/src/loader/evm/code.rs#L41-L62

@29988122 29988122 reopened this Mar 29, 2023
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

4 participants