[FFmpeg-devel] [PATCH] RDT/Realmedia patches #2

Michael Niedermayer michaelni
Sat Nov 15 20:21:28 CET 2008


On Sat, Nov 15, 2008 at 11:25:59AM -0500, Ronald S. Bultje wrote:
> Hi,
> 
> On Sat, Nov 15, 2008 at 4:36 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
> > On Fri, Nov 14, 2008 at 10:05:36PM -0500, Ronald S. Bultje wrote:
> >> -    if (len > 0 && (buf[0] < 0x40 || buf[0] > 0x42)) {
> >> -        buf += 9;
> >> -        len -= 9;
> >> -        consumed += 9;
> >> +    /* skip status packets */
> >> +    while (len >= 5 && buf[1] == 0xFF /* status packet */) {
> >> +        int pkt_len;
> >> +
> >> +        if (!(buf[0] & 0x80))
> >> +            return -1; /* frame contains no data packet */
> >> +        pkt_len  = AV_RB16(buf+3);
> >> +        buf += pkt_len;
> >> +        len -= pkt_len;
> >> +        consumed += pkt_len;
> >>      }
> 
> Here's this part of the patch alone...

this looks ok (without crosschecking against the bitstream)

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Dictatorship naturally arises out of democracy, and the most aggravated
form of tyranny and slavery out of the most extreme liberty. -- Plato
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081115/90d0f983/attachment.pgp>



More information about the ffmpeg-devel mailing list