Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Prevent memory leak in case of invalid metadata reads.

2014-10-21 Thread Michael Niedermayer
On Tue, Oct 21, 2014 at 11:04:28AM +0100, Derek Buitenhuis wrote: > On 10/21/2014 9:12 AM, Thilo Borgmann wrote: > > Subject: [PATCH] lavf/mov.c: Prevent memory leak in case of invalid metadata > > reads. > > > > --- > > libavformat/mov.c | 14 -- > > 1 file changed, 8 insertions(+),

Re: [FFmpeg-devel] [PATCH] lavf/mov.c: Prevent memory leak in case of invalid metadata reads.

2014-10-21 Thread Derek Buitenhuis
On 10/21/2014 9:12 AM, Thilo Borgmann wrote: > Subject: [PATCH] lavf/mov.c: Prevent memory leak in case of invalid metadata > reads. > > --- > libavformat/mov.c | 14 -- > 1 file changed, 8 insertions(+), 6 deletions(-) LGTM. - Derek ___

[FFmpeg-devel] [PATCH] lavf/mov.c: Prevent memory leak in case of invalid metadata reads.

2014-10-21 Thread Thilo Borgmann
Hi, the last commit to lavf/mov.c might leave the metadata buffer allocated in certain error conditions. This patch fixes it. -Thilo From a2fabe107796cf0af5cd18310fad178c5650b168 Mon Sep 17 00:00:00 2001 From: Thilo Borgmann Date: Tue, 21 Oct 2014 10:10:55 +0200 Subject: [PATCH] lavf/mov.c: Prev