Skip to content

[TEST] Update CBMC to #8573 #688

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

Closed
wants to merge 4 commits into from
Closed

[TEST] Update CBMC to #8573 #688

wants to merge 4 commits into from

Conversation

hanno-becker
Copy link
Contributor

@hanno-becker hanno-becker commented Jan 23, 2025

This commit tests the mlkem-native CBMC proofs with diffblue/cbmc#8573. This includes diffblue/cbmc#8562 solving a soundness issue we previously reported, but was noted to sometimes cause performance regressions.

Opening this PR to check whether proofs remain functional and practical in mlkem-native.

@hanno-becker hanno-becker changed the title [TEST] Update CBMC to d4757e2231b236ddd1d1933b4002a5aa4ca36db9 [TEST] Update CBMC to d4757e Jan 23, 2025
@hanno-becker
Copy link
Contributor Author

Fix from diffblue/cbmc#8574 works for MLKEM-512 and MLKEM-768. MLKEM-1024 still times out -- investigating.

This commit tests the mlkem-native CBMC proofs with the current
upstream d4757e2231b236ddd1d1933b4002a5aa4ca36db9 of CBMC. This
merges diffblue/cbmc#8562 solving a
soundness issue we previously reported, but was noted to sometimes
cause performance regressions.

Signed-off-by: Hanno Becker <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant