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

Michael Niedermayer michaelni at gmx.at
Sat Aug 31 03:57:30 CEST 2013


On Sat, Aug 31, 2013 at 03:54:31AM +0200, Michael Niedermayer wrote:
> 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

but make sure that the error cases work without leaks and double
frees

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Asymptotically faster algorithms should always be preferred if you have
asymptotical amounts of data
-------------- 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/de14c920/attachment.asc>


More information about the ffmpeg-devel mailing list