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

Problems with equality operation #453

Open
1 of 4 tasks
amiller opened this issue Jul 10, 2020 · 0 comments
Open
1 of 4 tasks

Problems with equality operation #453

amiller opened this issue Jul 10, 2020 · 0 comments
Assignees

Comments

@amiller
Copy link
Contributor

amiller commented Jul 10, 2020

There are several problems with the euqality implementation.

  • 1. The intended functionality is that equal([x], [y]) should return [1] if x==y, and [0] otherwise. However, the implementation appears to be incorrect, and it returns [0] if x!=y. If x==y, it returns an arbitrary nonzero number, not necessarily [1].
  • 2. There is no documentation for the equality function indicating what the intended behavior should be.
  • 3. The test case is not precise enough to serve as documentation either. The test case opens the returned value, but does not check that it is either [0] or [1], instead only checks the python "truth" value, hence the test passes even if the return value is any non-zero value.
    https://github.com/initc3/HoneyBadgerMPC/blob/dev/tests/progs/mixins/test_share_comparison.py#L65
  • 4. The source code is missing references, both to the paper, and to the implementation in Viff that this was built from.
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