[FFmpeg-devel] [WIP] Fix unnecessary memcpy in ffmpeg when libavfilter is enabled

Michael Niedermayer michaelni
Tue Nov 2 21:52:49 CET 2010


On Tue, Nov 02, 2010 at 09:37:54PM +0100, Stefano Sabatini wrote:
> On date Tuesday 2010-11-02 02:28:53 +0100, Michael Niedermayer encoded:
> > On Tue, Nov 02, 2010 at 02:23:45AM +0100, Stefano Sabatini wrote:
> > > Hi all,
> > > 
> > > check this one. Again another big patch stack to climb before to reach
> > > the top, and I suspect that there are problems with direct rendering
> > > for some codecs, the given patches compile but then they fail
> > > regressions (FFV1 and snow in particular).
> > > 
> > 
> > > Also ffplay always crash when playing a FFV1 file, maybe someone can
> > > have look at it.
> > 
> > its because the buffer is released before its returned
> 
> Do you have any evidence on this?

yeah if i remove the release_buffer() line (and let it leak) it doesnt crash


> Also are you saying the problem is
> in ffplay or?

the bug is in ffv1.c it should release the previous frame at the begin not the
current frame at the end

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

In fact, the RIAA has been known to suggest that students drop out
of college or go to community college in order to be able to afford
settlements. -- The RIAA
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20101102/2bdb7ee2/attachment.pgp>



More information about the ffmpeg-devel mailing list