[MPlayer-dev-eng] [PATCH] latm faad handling

Dan Oscarsson Dan.Oscarsson at tieto.com
Mon Jun 21 18:16:40 CEST 2010


On 2010-06-17 at 22:05 +0200 Reimar Döffinger wrote:
> On Wed, Jun 16, 2010 at 09:14:05PM +0200, Dan Oscarsson wrote:
> > +    if (faac_init < 0 && sh->format == mmioFOURCC('M', 'P', '4', 'L')) {
> > +        // working LATM not found at first try, look further on in stream
> > +        for (int i = 0; i < 5; i++) {
> 
> Move the "int" out, this will not work with some older compilers.
> 
> > +            pos = sh->a_in_buffer_len-3;
> 
> This can overflow (or rather underfflow).
> 
> > +            memmove(sh->a_in_buffer, &(sh->a_in_buffer[pos]), 3);
> 
> Pointless ().
> 

Fixed in the attached updated patch.

   Dan

-------------- next part --------------
A non-text attachment was scrubbed...
Name: faad-latm.diff
Type: text/x-patch
Size: 6276 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20100621/0cf42614/attachment.bin>


More information about the MPlayer-dev-eng mailing list