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
Uninitialized Read (and Divide by Zero) in ParseWave64HeaderConfig() #68
Comments
dbry
added a commit
that referenced
this issue
Mar 6, 2019
|
This is fixed now, thanks! |
|
Thanks for the reports and retest! |
Aw, it's interesting and weird that when I wget the zip and unzip it and run I am not sure why it gives me On the contrary: |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
uninit-divzero-waveheader.wav.zip - contains fuzzed input
The parsing of the attached file uninit-divzero-waveheader.wav leads to a read of an uninitialized location in memory. The uninitialized read sometimes further leads to a divide-by-zero error. The uninitialized read can be uncovered using a tool such as Valgrind or MemorySanitizer. For example:
It appears that this is an uninitialized read of the field
WaveHeader.NumChannelson this line. This sometimes leads to a subsequent divide by zero on this line -- I guess the division instruction may or may not be executed based on what value is read forWaveHeader.NumChannels.The text was updated successfully, but these errors were encountered: