[FFmpeg-devel] [PATCH 2/3] avpacket: Removing a memset that could cause double free

[Michael Niedermayer]

On Fri, Aug 30, 2013 at 03:28:45PM -0700, Vignesh Venkatasubramanian wrote:
> On Fri, Aug 30, 2013 at 12:49 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> >
> > On Fri, Aug 30, 2013 at 11:26:52AM -0700, Vignesh Venkatasubramanian wrote:
> > > Removing a memset from copy_side_data which could potentially cause
> > > double free.
> >
> > this is a misunderstanding, this patch causes the problems, it does
> > not fix them
> > you can just force a error path and remove the memset to see this
> > in make fate
> > (needs latest git head as there where other bugs when the error paths
> >  where used that i just fixed)
> 
> ok, any idea as to how i can fix this to get it working to do what i want
> with the side data copying?

if i undestand the issue correctly puting the memset under
if(src != dst) might work

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

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- 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/20130831/cfe3481b/attachment.asc>