-
Notifications
You must be signed in to change notification settings - Fork 4
CBMC: Add proof and spec for poly_uniform_4x
and poly_uniform_eta_4x
#254
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
Conversation
c8d2f43
to
d717297
Compare
d717297
to
2df7c40
Compare
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.
Looks good to me. Please merge.
2df7c40
to
e50362d
Compare
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.
Harnesses must pass uninitialized pointers, not arrays. Otherwise they inadvertently add to the memory preconditions of the function under proof.
5cb87d5
to
ffee934
Compare
Resolves #248 Inspired by the corresponding proof in mlkem-native: https://github.com/pq-code-package/mlkem-native/blob/e4d143181422f08bcad3a15fe5b0e0fe64228e2b/mlkem/sampling.c#L149 Signed-off-by: Matthias J. Kannwischer <[email protected]>
Resolves #249 Signed-off-by: Matthias J. Kannwischer <[email protected]>
ffee934
to
22b5e0c
Compare
polyvecl_uniform_gamma1
,poly_uniform_gamma1_4x
,mld_shake256x4_absorb_once
,mld_shake256x4_squeezeblocks
#253poly_uniform_4x
#248poly_uniform_eta_4x
#249