Re: [FFmpeg-devel] [PATCH] h261dec, ituh263dec: ensure CLOSE_READER is done on error paths, too.

2014-09-01 Thread Reimar Döffinger
On Sun, Aug 31, 2014 at 10:50:16PM +0200, Reimar Döffinger wrote: > On Sun, Aug 31, 2014 at 09:56:03PM +0200, Michael Niedermayer wrote: > > On Sun, Aug 31, 2014 at 09:40:05PM +0200, Reimar Döffinger wrote: > > > Signed-off-by: Reimar Döffinger > > > --- > > > libavcodec/h261dec.c| 2 ++ > > >

Re: [FFmpeg-devel] [PATCH] h261dec, ituh263dec: ensure CLOSE_READER is done on error paths, too.

2014-08-31 Thread Reimar Döffinger
On Sun, Aug 31, 2014 at 09:56:03PM +0200, Michael Niedermayer wrote: > On Sun, Aug 31, 2014 at 09:40:05PM +0200, Reimar Döffinger wrote: > > Signed-off-by: Reimar Döffinger > > --- > > libavcodec/h261dec.c| 2 ++ > > libavcodec/ituh263dec.c | 4 ++-- > > 2 files changed, 4 insertions(+), 2 de

Re: [FFmpeg-devel] [PATCH] h261dec, ituh263dec: ensure CLOSE_READER is done on error paths, too.

2014-08-31 Thread Michael Niedermayer
On Sun, Aug 31, 2014 at 09:40:05PM +0200, Reimar Döffinger wrote: > Signed-off-by: Reimar Döffinger > --- > libavcodec/h261dec.c| 2 ++ > libavcodec/ituh263dec.c | 4 ++-- > 2 files changed, 4 insertions(+), 2 deletions(-) LGTM though in theory its probably also ok not to write back on error

[FFmpeg-devel] [PATCH] h261dec, ituh263dec: ensure CLOSE_READER is done on error paths, too.

2014-08-31 Thread Reimar Döffinger
Signed-off-by: Reimar Döffinger --- libavcodec/h261dec.c| 2 ++ libavcodec/ituh263dec.c | 4 ++-- 2 files changed, 4 insertions(+), 2 deletions(-) diff --git a/libavcodec/h261dec.c b/libavcodec/h261dec.c index c9470b1..9b95775 100644 --- a/libavcodec/h261dec.c +++ b/libavcodec/h261dec.c @@ -