On Mon, Jan 01, 2018 at 11:28:37AM -0500, Derek Buitenhuis wrote: > This fixes a double free in he error case. > > Signed-off-by: Derek Buitenhuis <derek.buitenh...@gmail.com> > --- > This does fix the double free, but I am unsure if it is the correct free > to removed to fix it. Comments welcome. > --- > libavfilter/vf_paletteuse.c | 1 - > 1 file changed, 1 deletion(-) > > diff --git a/libavfilter/vf_paletteuse.c b/libavfilter/vf_paletteuse.c > index ede2e2e..c2d0c6b 100644 > --- a/libavfilter/vf_paletteuse.c > +++ b/libavfilter/vf_paletteuse.c > @@ -1037,7 +1037,6 @@ static int load_apply_palette(FFFrameSync *fs) > > error: > av_frame_free(&master); > - av_frame_free(&second); > return ret; > }
That's some weird ownership semantic for the error-path, but Nicolas knows better this API so I'll trust him on this one. -- Clément B.
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
