[MPlayer-dev-eng] Recent regression in FreeMediaType

[Vladimir Voroshilov]

2007/4/9, Zuxy Meng <zuxy.meng at gmail.com>:
> Hi,
>
> 2007/3/14, Alan Nisota <alannisota at gmail.com>:
> > The changes to how AM_MEDIA_TYPE is handled about 2 weeks ago seems to
> > be a bit broken.
> > Specifically, in FreeMediaType, this code:
> >     if (pMediaType->pUnk)
> >     {
> >         pMediaType->pUnk->vt->Release(pMediaType->pUnk);
> >         pMediaType->pUnk = NULL;
> >     }
> > (well the 'Release') causes a segfault with CoreAVC (gdb reports
> > pUnk->vt as inaccessible memory).  I believe this data has already been
> > released (but pUnk not zeroed) by the time this gets called
> > (MemAllocator_Decommit) though I haven't dug into it much.
> > All I know so far is it didn't used to segfault on termination before
> > the patch was applied.
> >
> > I'll try to collect more info, and I realize that as CoreAVC isn't a
> > supported codec, I'm on my own, but I figured I'd report it nonetheless.
>
> Reproducible with wmv8 too. Disabling the Release statement avoids
> SIGSEV. Another blocker for RC2?

Well. In this case we can just comment out Release statement with
apropriate note about reason.

> --
> Zuxy
> Beauty is truth,
> While truth is beauty.
> PGP KeyID: E8555ED6
> _______________________________________________
> MPlayer-dev-eng mailing list
> MPlayer-dev-eng at mplayerhq.hu
> http://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
>


-- 
Regards,
Vladimir Voroshilov     mailto:voroshil at gmail.com
JID: voroshil at jabber.ru
ICQ: 95587719