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

Another panic on malformed input #33

Closed
Shnatsel opened this issue Sep 7, 2018 · 4 comments
Closed

Another panic on malformed input #33

Shnatsel opened this issue Sep 7, 2018 · 4 comments
Labels

Comments

@Shnatsel
Copy link
Contributor

Shnatsel commented Sep 7, 2018

When given a malformed input lewton panics with the following error:

thread 'main' panicked at 'index out of bounds: the len is 256 but the index is 256', src/audio.rs:558:13

Testcase: minimized_panic2_testcase.gz (gzipped so that github would accept the upload)

Command to reproduce the crash:
RUSTFLAGS='--cfg=fuzzing' cargo run --release --example perf /path/to/testcase

Tested on git commit ddd2408. Found using afl.rs

@est31 est31 added the bug label Sep 8, 2018
@est31
Copy link
Member

est31 commented Sep 8, 2018

Repacked version: 33_minimized_panic_testcase.ogg.zip

@Shnatsel btw, which are the license terms of the samples that were, will and are being submitted by you in the bug reports? Is cc0 okay?

@Shnatsel
Copy link
Contributor Author

Shnatsel commented Sep 8, 2018

Yeah, they're autogenerated by a tool I'm using, so I'm the copyright holder. I hereby release them under Creative Commons Zero 1.0.

@est31
Copy link
Member

est31 commented Sep 10, 2018

@Shnatsel thanks!

@est31
Copy link
Member

est31 commented Sep 13, 2018

Hmmm turns out the file was actually valid, not malformed. But the panic is a bug nontheless.

@est31 est31 closed this as completed in 75e4c1c Sep 13, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants