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

speexenc hangs on corrupted input file #9

Closed
tianxiaogu opened this issue Sep 11, 2018 · 4 comments
Closed

speexenc hangs on corrupted input file #9

tianxiaogu opened this issue Sep 11, 2018 · 4 comments
Assignees
@tmatth

Comments

@tianxiaogu
Copy link

Our fuzzing tool has found an input that makes speexenc hang on it. Attached is the corrupted input.

32.zip

speexenc  32 -
@tmatth tmatth self-assigned this Sep 11, 2018
tmatth added a commit that referenced this issue Sep 11, 2018
@tmatth
wav_io: check for EOF when seeking in wav
f405273 
Fixes hang discovered by fuzzing: #9
@tmatth tmatth mentioned this issue Sep 11, 2018
Closed
@tmatth
Copy link
Member

tmatth commented Sep 11, 2018

@tianxiaogu nice catch, do you want to try #10 ?

tmatth added a commit that referenced this issue Sep 11, 2018
@tmatth
wav_io: check for EOF when seeking in wav
0aaf169 
Fixes hang discovered by fuzzing: #9
tmatth added a commit that referenced this issue Sep 11, 2018
@tmatth
wav_io: check for EOF when seeking in wav
bc93757 
Fixes hang discovered by fuzzing: #9
@tianxiaogu tianxiaogu reopened this Sep 11, 2018
@tianxiaogu
Copy link
Author

@tmatth , Thanks very much for your reply. We have run the fuzzer for hours. And this is the only issue we have found. speex is really reliable. We will improve our fuzzer and keep fuzzing on speex.

ghost pushed a commit that referenced this issue Sep 12, 2018
@tmatth
wav_io: check for EOF when seeking in wav
bdc3922 
Fixes hang discovered by fuzzing: #9
@tmatth
Copy link
Member

tmatth commented Sep 12, 2018

@tianxiaogu great, let me know if you find others.

@tmatth
Copy link
Member

tmatth commented Sep 12, 2018

Fixed in bdc3922

@tmatth tmatth closed this as completed Sep 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tmatth tmatth

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tmatth @tianxiaogu