[FFmpeg-devel] [PATCH 1/2] lavfi/video: ensure that filter_frame is called.

Michael Niedermayer michaelni at gmx.at
Fri Dec 28 14:36:38 CET 2012


On Fri, Dec 28, 2012 at 02:24:57PM +0100, Michael Niedermayer wrote:
> On Fri, Dec 28, 2012 at 10:52:27AM +0100, Nicolas George wrote:
> > Le septidi 7 nivôse, an CCXXI, Michael Niedermayer a écrit :
> > > after pull --rebase and both patches:
> > > make fate-lavfi-tile
> > > ==12655== Invalid read of size 8
> > <snip>
> > > Assertion link->cur_buf_copy->buf->refcount > 0 failed at libavfilter/video.c:381
> > 
> > I am sorry, but I am still unable to reproduce the issue. I am working with
> > the following tree, without local changes:
> > 
> > ae69e57 lavfi/sink_buffer: switch to filter_frame.
> > 909e799 lavfi/video: ensure that filter_frame is called.
> > 9e04e11 libpostproc: alloc a few bytes more in the temporary buffer
> > f1185fb libpostproc: Fix off by 1 error in the C dering code
> > 69db85c mjpegdec: Support 16bit DQT
> > 
> > 9e04e11 is currently the head of the official repository, and AFAIK the
> > (short) hash uniquely identifies the whole tree anyway (except for unlikely
> > collisions. I tried both this and with only 909e799, with or without
> > valgrind on x86_64, and x86_32, and fate-lavfi-tile (and the rest of FATE)
> > succeeds.
> 
> you might need to build with --assert-level=1 or 2
> and i did test on x86-64
> 
> ill retry with current HEAD and report back in a moment

I cant reproduce with current HEAD (0d4572a54b04ac983e6fb00bcdab6d9516726dbb)

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

There will always be a question for which you do not know the correct awnser.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20121228/d8e3e9f7/attachment.asc>


More information about the ffmpeg-devel mailing list