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.

Sign up for GitHub

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

Jump to bottom

crashers found by go-fuzz #279

Closed
dgryski opened this issue Jul 27, 2015 · 3 comments
Closed

crashers found by go-fuzz #279

dgryski opened this issue Jul 27, 2015 · 3 comments

Comments

@dgryski
Copy link

dgryski commented Jul 27, 2015

cfssl includes a mostly vendored copy of Azure/go-pkcs12. I just filed Azure/go-pkcs12#25 that might affect cfssl too.
@kisom
Copy link
Contributor

kisom commented Jul 27, 2015

@dgryski Thanks for the heads up. We'll keep an eye out for patches upstream and investigate patching our copy.

@dgryski
Copy link
Author

dgryski commented Oct 5, 2015

Azure/go-pkcs12 has now been merged into /x/crypto: golang/crypto@c8b9e63 , including fixes for these crashers. Consider moving to that code base instead?

@andrewbuss andrewbuss mentioned this issue Oct 9, 2015
Merged
@dgryski
Copy link
Author

dgryski commented Dec 1, 2015

Closing since cfssl now uses /x/crypto/pkc12 which has fixes for these crashes.

@dgryski dgryski closed this as completed Dec 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kisom @dgryski