[MPlayer-dev-eng] libmpeg2 V0.40b plays jerky/jumpy

[Shachar Raindel]

On Thu, 26 Aug 2004 12:19:29 +0000, Sascha Sommer
<saschasommer at freenet.de> wrote:
> 
> 
> On Thursday 26 August 2004 10:08, Shachar Raindel wrote:
> > On Tue, 24 Aug 2004 19:07:03 +0200, Jindrich Makovicka
> >
> > <makovick at kmlinux.fjfi.cvut.cz> wrote:
> > > Dominik 'Rathann' Mierzejewski wrote:
> > > > On Monday, 23 August 2004 at 14:35, Attila Kinali wrote:
> > > >>On Mon, Aug 23, 2004 at 05:35:12AM +0300, Shachar Raindel wrote:
> > > >>>So, will you please revert it??????
> > > >>
> > > >>No, i leave that to iive, he knows libmpeg2 much better than i do.
> > > >>I just pull the threads from behind :)
> > > >
> > > > You meant, of course, "I just pull *strings* from behind". :P
> > > >
> > > > R.
> > >
> > > please test the current CVS, it should already play correctly.
> >
> > It crashes on some slightly corrupted MPEG 2 files, which played fine
> > with the old version. See the file
> > "broken_file_crashes_libmpeg2_040b_worked_fine_in_03.m2v" in the
> > incoming folder of MPlayer's FTP. I have uploaded a complete bug
> > report. Maybe I will have a look on this bug later, as it seems to be
> > some kind of double free or something similar. If it is so, it might
> > be an exploitable bug, which can be activated remotely (think about
> > streaming mpeg2....)
> >
> 
> Please try attached patch (still downloading).
> 
Will test it in few minutes.... I also done some diging into the code,
and here is a patch that works for me, and seems not to break anything
else in libmpeg2, when used by mplayer. It simply disable the internal
allocation/free of convert_id in libmpeg2. The proper fix might be to
change the call signature of draw_slice in vd_libmpeg2.c

   Cheers,
   Shachar
-------------- next part --------------
A non-text attachment was scrubbed...
Name: libmpeg2-fix.patch
Type: text/x-patch
Size: 965 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20040826/1d316f75/attachment.bin>