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

[Roberto Togni]

On 2004.05.01 01:18, Dominik 'Rathann' Mierzejewski wrote:
> 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)? 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?
> 
> Yes, it does.
good.
I can't get your clips. They have permission rw------- , so i can't  
read them (not from web nor ftp).

Anyway, i'll committed a patch that removes that nonsense header seek.

> 
> Thanks,
> Dominik
Ciao,
 Roberto