-
-
Notifications
You must be signed in to change notification settings - Fork 158
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
Add more defensive exceptions #16
Labels
enhancement
Enhancements or general improvements
Comments
Put in a good amount of this. |
Do we have many more to add or should we go ahead and close? (unable to run the testing tool for a couple of days) |
I have a few more, then it's mostly fixing cert and sign parsing issues for all the data samples. |
Attestation code is fairly well defended now, with the exception of certificate parsing. |
I think we should go ahead and close this now. Please reopen if you deem necessary. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
We pass a lot of the "Expect failure" tests in the conformance tool, e.g:
But it's often because of null reference etc. Might be better for a library consumer if we threw more precise exceptions.
The text was updated successfully, but these errors were encountered: