Re: [FFmpeg-devel] [PATCH 2/2] lavc/utils: free private options on avcodec_open2 fail

2014-11-25 Thread Lukasz Marek
On 24.11.2014 21:46, Michael Niedermayer wrote: On Sun, Nov 23, 2014 at 12:58:07AM +0100, Lukasz Marek wrote: It protects leaking string/binary/dict options from priv context. Signed-off-by: Lukasz Marek --- libavcodec/utils.c | 2 ++ 1 file changed, 2 insertions(+) LGTM pushed both __

Re: [FFmpeg-devel] [PATCH 2/2] lavc/utils: free private options on avcodec_open2 fail

2014-11-24 Thread Michael Niedermayer
On Sun, Nov 23, 2014 at 12:58:07AM +0100, Lukasz Marek wrote: > It protects leaking string/binary/dict options from priv context. > > Signed-off-by: Lukasz Marek > --- > libavcodec/utils.c | 2 ++ > 1 file changed, 2 insertions(+) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF2128B147E

[FFmpeg-devel] [PATCH 2/2] lavc/utils: free private options on avcodec_open2 fail

2014-11-22 Thread Lukasz Marek
It protects leaking string/binary/dict options from priv context. Signed-off-by: Lukasz Marek --- libavcodec/utils.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/libavcodec/utils.c b/libavcodec/utils.c index bf2a5b9..7e37b82 100644 --- a/libavcodec/utils.c +++ b/libavcodec/utils.c @@ -1