[MPlayer-dev-eng] RealMedia playing broken with demux_real.c 1.58

[Dominik 'Rathann' Mierzejewski]

On Friday, 30 April 2004 at 23:58, Roberto Togni wrote:
> On 2004.04.30 23:18, Dominik 'Rathann' Mierzejewski wrote:
> > Hi.
> > 
> > The CVS change below breaks plaing all RealMedia files I have.
> > Reversing
> > it fixes the problem. Interestingly, all these files have hdr_size
> > less
> > than 0x4e, so stream_skip is actually called with negative argument
> > (-8
> > for example), which results in sig11.
> > 
> > @@ -1190,6 +1193,10 @@
> >                    sh->wf->cbSize = 0;
> >                    sh->format = MKTAG(buf[0], buf[1], buf[2],
> > buf[3]);
> > 
> > +                   if ((version != 3) && (hdr_size != 0x4e)) {
> > +                       mp_msg(MSGT_DEMUX,MSGL_V,"skipping %d extra
> > header bytes\n", hdr_size-0x4e);
> > +                       stream_skip(demuxer->stream, hdr_size-0x4e);
> > +                   }
> > #if 0
> >                    switch (sh->format){
> > 			case MKTAG('d', 'n', 'e', 't'):
> > 
> > Shall I provide more info?

> Yes, thanks. I think i understood the problem, but i'd loke to test it
> before doing any change.
> Is some sample available somewhere (i'll test with all my samples
> anyway)?

For example http://www.scifi.com/b5rangers/trailer/
Try Trailer 1, RealPlayer 8 350k version.

And I've found some samples which still work. I'll upload both to mphq
in a moment as demux-real-sig11*.

> If not, can you post some info about codec used and mplyer
> output information about demuxer (file version and such)?
> If you change it to (hdr_size > 0x4e), does it fix your problem?

Recompiling now. I'll keep you posted.

Thanks for the quick reply.

-- 
MPlayer RPMs maintainer: http://greysector.rangers.eu.org/mplayer.html
"I am Grey. I stand between the candle and the star. We are Grey.
 We stand between the darkness ... and the light."
        -- Delenn in Grey Council in Babylon 5:"Babylon Squared"