[FFmpeg-devel] [PATCH] lavf/tee: fix side data double free.

Michael Niedermayer michaelni at gmx.at
Sun Dec 13 10:03:29 CET 2015


On Sat, Oct 10, 2015 at 07:13:04PM +0200, Nicolas George wrote:
> Le nonidi 19 vendémiaire, an CCXXIV, wm4 a écrit :
> > This looks suspicious. Like some code above this does unclean tricks
> > with keeping side-data somehow referenced, instead of using proper
> > methods like creating a new AVPacket reference.
> 
> It is the same code than in ffmpeg.c, but I must admit I did not take time
> to really understand how side data ownership is handled.
> 
> I suspect the best course of action would be to wait for Roger's patch to be
> applied, it should fix this very issue at the same time, and applying this
> patch or Bela's latest version would only annoy him with rebase conflicts.

2 months have passed it hasnt happened, thus applied this
also this affects a release and rogers patch wont help the release

and i think serious bugs should not be left open when rather simple
fixes are known.

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

What does censorship reveal? It reveals fear. -- Julian Assange
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20151213/7a29dd4d/attachment.sig>


More information about the ffmpeg-devel mailing list