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

Fix out of bound crash #8

Merged
merged 7 commits into from
Sep 12, 2015
Merged

Conversation

pmezard
Copy link
Contributor

@pmezard pmezard commented Sep 11, 2015

Hello,

I am interested in the go-fuzz tool and run it on random libraries. Here is your turn. It triggered one out of bounds panic in vorbis comment decoding and various hard coded panics in other parts of the code. I believe panicking on user inputs in third party libraries is not really good style so made some others return an error instead.

I have not written additional tests because I do not have too much time to spend on this, but I can provide binary inputs triggering the issues if you want to,

@eaburns
Copy link
Owner

eaburns commented Sep 12, 2015

Looks good. Thanks a lot!

eaburns added a commit that referenced this pull request Sep 12, 2015
@eaburns eaburns merged commit c03c247 into eaburns:master Sep 12, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants