We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

Bug 820

Summary: Crash playing OGG files with SDL 1.3
Product: SDL_mixer Reporter: Sam Lantinga <slouken>
Component: miscAssignee: Ryan C. Gordon <icculus>
Status: RESOLVED DUPLICATE QA Contact: Sam Lantinga <slouken>
Severity: blocker    
Priority: P2    
Version: unspecified   
Hardware: All   
OS: All   
Attachments: Crashing test case

Description Sam Lantinga 2009-10-02 21:34:40 UTC 
The SDL 1.3 audio conversion functions look at len_cvt instead of len in the SDL_AudioCVT structure.

The SDL_mixer Ogg Vorbis playback code correctly just fills in len when converting sample rates, and crashes.

Here's a transcript of IRC for ya:
Ryan:  Arguably, SDL should be calculating len_cvt, the caller should be supplying len.
So 1.2 is correct here
Ryan:  If we do it the other way, we're just inviting apps to screw up the math.
me:  The bug shows up in music_ogg.c:171
Ryan:  yeah, I see
Comment 1 Sam Lantinga 2009-10-02 23:38:29 UTC 
Created attachment 381 [details]
Crashing test case

It looks like cvt_len might be a red herring.  Here's the crashing Ogg Vorbis file:
drumloop051505.ogg: Ogg data, Vorbis audio, stereo, 44100 Hz, ~112001 bps, created by: Xiph.Org libVorbis I (1.0.1)
Comment 2 Sam Lantinga 2009-10-03 01:57:23 UTC 
This was caused by the memory corruption in bug 819, helpfully fixed by Ozkan

*** This bug has been marked as a duplicate of bug 819 ***