Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # 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
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants