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

ABY innerproduct does not match expected results. #19

Open
marsella opened this issue Nov 8, 2019 · 2 comments
Open

ABY innerproduct does not match expected results. #19

marsella opened this issue Nov 8, 2019 · 2 comments
Labels
bug Something isn't working

Comments

@marsella
Copy link
Collaborator

marsella commented Nov 8, 2019

I noticed some mismatch when running the innerprod example for ABY. Occasionally the expected value returned after running python ~/ABY/src/examples/geninput.py -e innerprod doesn't match the circuit result from running ./ABY/build/bin/innerprod_test -r 0 & ./ABY/build/bin/innerprod_test -r 1.

Reported by PFW, verified by me.

Possible fix: update to the latest ABY version.

@marsella marsella added the bug Something isn't working label May 25, 2020
@marsella
Copy link
Collaborator Author

marsella commented Aug 9, 2020

Got another report of this problem from MB and RI.

It seems we are already using the most up-to-date version of ABY.

The error happens in both my modified inner product and their inner product examples. This implies it is not an issue with our method of input generation (Also, in retrospect, it was an awful idea to name the two versions "innerprod" and "innerproduct').

The bug probably lies in the BuildInnerProductCircuit function. Since this is a straightforward implementation, there is probably a bug in the library somewhere.

@locomotive-crypto
Copy link

I have the same problem.

@gutjuri gutjuri mentioned this issue May 12, 2023
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants