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
[seraphis] seraphis_crypto: add crypto utils #9196
base: master
Are you sure you want to change the base?
Conversation
{ | ||
return MINUS_ONE; | ||
} | ||
//------------------------------------------------------------------------------------------------------------------- |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where did you get the invert function from?
What are the requirements on the scalar so this operation works properly?
What would be the effect of inverting on clamped scalars?
Any comments on this phrase here?
In order to prevent attacks using small subgroups, the scalarmult functions above clear lower bits of the scalar. This may be indesirable to build protocols that requires n to be invertible.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Where did you get the invert function from?
As I said in the PR comment "Some functions were taken from code written by Sarang:" "invert(): bulletproofs2.cpp"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Quick comment about decompose()
Co-authored-by: Jeffro <jeffreyryan@tutanota.com>
This is a PR in my 'upstreaming seraphis_lib project', the changes here are not used anywhere yet.
Adds a set of crypto utility functions. Some functions were taken from code written by Sarang:
invert()
: bulletproofs2.cppdecompose()
: triptych experimental branchkronecker_delta()
: triptych experimental branchconvolve()
: triptych experimental branchThese are security-critical functions.