Skip to content
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

very few specs include properties #32

Open
kiniry opened this issue Nov 13, 2020 · 2 comments
Open

very few specs include properties #32

kiniry opened this issue Nov 13, 2020 · 2 comments
Labels
enhancement New feature or request

Comments

@kiniry
Copy link
Member

kiniry commented Nov 13, 2020

We have tons of non-trivial properties specified for various algorithms. Sharing specs with properties permits us to easily demonstrate the power and utility of stating and having assurance about theorems in order to ensure that a specification is validated, as well as to help drive an assurance case of an implementation of the algorithm. We should endeavor to include these properties in our public specs.

@kiniry kiniry added the enhancement New feature or request label Nov 13, 2020
@weaversa
Copy link
Contributor

weaversa commented Nov 13, 2020 via email

@robdockins
Copy link
Contributor

Indeed; test vectors should be an absolute minimum. For AES, SHA-2 and ECDSA, I spent a fair amount of time munging some of the public NIST test vectors into Cryptol syntax to validate the new primitive implementations. They could pretty easily be reused here, I think.

https://github.com/GaloisInc/cryptol/tree/master/tests/suiteb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants