[MPlayer-dev-eng] Re: Re: Re: Re: [PATCH] fix for faad2 infinite loop after decoding error

[Guillaume POIRIER]

Hi,

On 6/8/06, Bryan Alton <balton at eircom.net> wrote:
> Guillaume POIRIER wrote:
>
> > Hi,
> >
> > On 6/8/06, Bryan Alton <balton at eircom.net> wrote:
> >>  The errors you got before the crash was
> >>  FAAD: error: Unable to find ADTS syncword, trying to resync!
> >>  FAAD: Failed to decode frame: Unable to find ADTS syncword
> >>
> >> I have seen these when mplayer is trying to decode/resync but it has
> >> reached
> >> the end of file.  It is possible the error Guilaume is seeing is due to a
> >> resync near the end of file as the patch has already been run once in
> >> Guillaume test.   To that end I have uploaded a longer of the same
> >> stream. It starts from the same place just that it runs for about 1min30.
> >>
> >> The file is called faad_infinite_long.aacp
> >
> > Looks like you re-uploaded the shorter version again. I can only see
> > faad_infinite.aacp on the ftp server....
> >
> > Guillaume
> >
>
> OK it was late I probably made a mistake - I've tried again - with the
> "write only" I can't see if it was a success. The filesize is about 244k -
> the short one is about 122k.

Ok, I just tried with your new sample, and it works as expected.
I guess the other problem with FAAD will have to be dealt with in another patch.

Patch applied, thanks.

Guillaume
-- 
"Success consists of going from failure to failure without loss of enthusiasm."
 -- Winston Churchill