-
Notifications
You must be signed in to change notification settings - Fork 19
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
Make libjade use DOIT instructions on secrets #108
Merged
tfaoliveira-sb
merged 16 commits into
formosa-crypto:feature/doit
from
J08nY:feature/doit
May 23, 2024
Merged
Make libjade use DOIT instructions on secrets #108
tfaoliveira-sb
merged 16 commits into
formosa-crypto:feature/doit
from
J08nY:feature/doit
May 23, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
vbgl
reviewed
Feb 27, 2024
Good points, thanks. Though this is only a draft, worth looking at after some more work. |
J08nY
force-pushed
the
feature/doit
branch
4 times, most recently
from
March 7, 2024 14:54
9d7bb27
to
d45a434
Compare
Alright, this is ready for some review now. I used something like: env JASMIN="jasminc -checkCT -checkCTafter=propagate -doit -infer -nowarning" make 2>&1 | grep -E "(Entering|line |constant)" to check that all implementations pass the CT checker as expected. |
J08nY
force-pushed
the
feature/doit
branch
2 times, most recently
from
March 15, 2024 16:46
141114f
to
d59b920
Compare
egrep is deprecated.
This spills some more registers to the stack in the reference chacha core. This comes at a significant performance cost. Further experimentation should be done with register packing instead.
But at what cost?!?
The rotate instruction just ignores these, but the ">>=" operators complain.
Fix xsalsa20poly1305 by swapping its xsalsa20 with the DOIT one. Adds declassifies into xsalsa20poly1305. This declassifies the result of tag verification, which leaks via a branch (decryption is not done if tag does not verify).
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This ties in to the effort in jasmin-lang/jasmin#736.
-lazy-regalloc
.publicseed
, necessary to make the CT checker pass even without DOIT.