From b3bfc09852007fed60bf71451a6c71df8c28b3ed Mon Sep 17 00:00:00 2001 From: Michael Sevakis Date: Wed, 2 Mar 2011 03:47:42 +0000 Subject: r29348 changes can cause certain codecs to jump to code on early track change that assumes initializations have been done. Make sure that track change cleanups are only called if the state is sane to do so. Stops my vorbis data abort issue. Correctness for speex is a guess based on the library calls' source. It appears only speex/vorbis should have been bothered by said revision. git-svn-id: svn://svn.rockbox.org/rockbox/trunk@29489 a1c6a512-1295-4272-9138-f99709370657 --- apps/codecs/speex.c | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) (limited to 'apps/codecs/speex.c') diff --git a/apps/codecs/speex.c b/apps/codecs/speex.c index e38a04495c..7a1efa9753 100644 --- a/apps/codecs/speex.c +++ b/apps/codecs/speex.c @@ -393,6 +393,9 @@ enum codec_status codec_main(void) void *st = NULL; int j = 0; + memset(&bits, 0, sizeof(bits)); + memset(&oy, 0, sizeof(oy)); + /* Ogg handling still uses mallocs, so reset the malloc buffer per track */ next_track: error = CODEC_OK; @@ -401,16 +404,16 @@ next_track: error = CODEC_ERROR; goto exit; } + stereo = speex_stereo_state_init(); + spx_ogg_sync_init(&oy); + spx_ogg_alloc_buffer(&oy,2*CHUNKSIZE); if (codec_wait_taginfo() != 0) goto done; strtoffset = ci->id3->offset; - spx_ogg_sync_init(&oy); - spx_ogg_alloc_buffer(&oy,2*CHUNKSIZE); - samplerate = ci->id3->frequency; codec_set_replaygain(ci->id3); @@ -558,7 +561,8 @@ done: /* Clean things up for the next track */ - speex_decoder_destroy(st); + if (st) + speex_decoder_destroy(st); if (stream_init == 1) spx_ogg_stream_reset(&os); -- cgit v1.2.3