Re: [FFmpeg-devel] [PATCH] libavcodec/rv30: fix mem leak in case of init failure

2014-11-23 Thread Lukasz Marek
On 23.11.2014 00:52, Michael Niedermayer wrote: On Sat, Nov 22, 2014 at 11:12:53PM +0100, Lukasz Marek wrote: Also replaced return -1 with return AVERROR(EINVAL) Signed-off-by: Lukasz Marek --- libavcodec/rv30.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) LGTM pushed

Re: [FFmpeg-devel] [PATCH] libavcodec/rv30: fix mem leak in case of init failure

2014-11-22 Thread Michael Niedermayer
On Sat, Nov 22, 2014 at 11:12:53PM +0100, Lukasz Marek wrote: > Also replaced return -1 with return AVERROR(EINVAL) > > Signed-off-by: Lukasz Marek > --- > libavcodec/rv30.c | 8 > 1 file changed, 4 insertions(+), 4 deletions(-) LGTM thx [...] -- Michael GnuPG fingerprint: 9FF21

[FFmpeg-devel] [PATCH] libavcodec/rv30: fix mem leak in case of init failure

2014-11-22 Thread Lukasz Marek
Also replaced return -1 with return AVERROR(EINVAL) Signed-off-by: Lukasz Marek --- libavcodec/rv30.c | 8 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/libavcodec/rv30.c b/libavcodec/rv30.c index fd8fd4f..1483107 100644 --- a/libavcodec/rv30.c +++ b/libavcodec/rv30.c @@