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

X coordinates collision #1

Open
gching opened this issue Mar 21, 2018 · 2 comments
Open

X coordinates collision #1

gching opened this issue Mar 21, 2018 · 2 comments

Comments

@gching
Copy link

gching commented Mar 21, 2018

value = @util.random()

I know this codebase is pretty old, but it seems like there is a possibility of x coordinates colliding upon generating the points / parts at random.

As we do not insert a selected x coordinate for a point into numbers, another point's x coordinate can possibly collide.

@cipherboy
Copy link
Member

Fair point, I do believe that's an oversight on our part. I probably won't fix it though -- the probability of it occurring is extraordinarily low considering the prime in question (115792089237316195423570985008687907853269984665640564039457584007913129639747) and a reasonable quantity of shares (< 2^16).

If you want to file a PR, fine, but neither I or @Mab879 are currently using this. I'd also suggest the protocol should be changed before others use this, so I'm not sure its worth your time.

@gching
Copy link
Author

gching commented Mar 26, 2018

Makes sense 👍 was just researching up a lot of sss algorithms. Great clean code btw!

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