ok, tried it out... passes test/test_seeking.sh and my
"xmms twitch" test, checked in to CVS.  thanks!

Josh


--- Miroslav Lichvar <[EMAIL PROTECTED]> wrote:

> On Fri, Nov 03, 2006 at 10:01:42AM +0100, Miroslav Lichvar wrote:
> > Thanks. Sending latest version of the patch. Now it can seek in
> files
> > that have large id3 tag (or any random data) at the end and it
> won't loop on
> > streams with shuffled frames.
> 
> One small patch on top of that and it's bug free. ;)
> 
> -- 
> Miroslav Lichvar
> > --- flac/src/libFLAC/stream_decoder.c.orig  2006-11-03
> 18:52:38.104291323 +0100
> +++ flac/src/libFLAC/stream_decoder.c 2006-11-03 18:53:41.350727144
> +0100
> @@ -2995,7 +2995,7 @@
>  
>       while(1) {
>               /* check if the bounds are still ok */
> -             if (lower_bound_sample + FLAC__MIN_BLOCK_SIZE > 
> upper_bound_sample
> || lower_bound > upper_bound) {
> +             if (lower_bound_sample >= upper_bound_sample || lower_bound >
> upper_bound) {
>                       decoder->protected_->state = 
> FLAC__STREAM_DECODER_SEEK_ERROR;
>                       return false;
>               }
> 




 
____________________________________________________________________________________
Do you Yahoo!?
Everyone is raving about the all-new Yahoo! Mail.
http://new.mail.yahoo.com
_______________________________________________
Flac-dev mailing list
Flac-dev@xiph.org
http://lists.xiph.org/mailman/listinfo/flac-dev

Reply via email to