-
Notifications
You must be signed in to change notification settings - Fork 464
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
Patch Kyber to fix ASAN error on ARM64 #1922
Conversation
Signed-off-by: Pravek Sharma <[email protected]>
Signed-off-by: Pravek Sharma <[email protected]>
Co-Authored-By: Kyle Nekritz <[email protected]> Co-Authored-By: Mingtao Yang <[email protected]> Signed-off-by: Pravek Sharma <[email protected]>
@praveksharma Thanks for pulling this over from PQClean. To make reviewing easier would you mind pointing to the actual PQClean commit you pulled over? This PR doesn't seem to be the identical logic as in #1914. |
Sorry for the inadequate documentation @baentsch. The changes were first introduced this commit. The logic is slightly different where instead of defining a function to round up integers this is done manually -- |
Patch Kyber to fix address sanitiser issue on ARM64 as described in #1914. Patch copied from PQClean, in line with fix submitted in #1914.