Re: [FFmpeg-devel] [PATCH] ffmpeg: check fclose return values

2016-01-08 Thread Ganesh Ajjanagadde
On Jan 8, 2016 1:59 AM, "wm4" wrote: > > On Thu, 7 Jan 2016 16:01:14 -0800 > Ganesh Ajjanagadde wrote: > > > On Thu, Jan 7, 2016 at 3:57 PM, Ganesh Ajjanagadde wrote: > > > On Thu, Jan 7, 2016 at 2:18 PM, Michael Niedermayer > > > wrote: > > >> On Thu, Jan 07, 2016 at 11:16:27PM +0100, Michael

Re: [FFmpeg-devel] [PATCH] ffmpeg: check fclose return values

2016-01-08 Thread wm4
On Thu, 7 Jan 2016 16:01:14 -0800 Ganesh Ajjanagadde wrote: > On Thu, Jan 7, 2016 at 3:57 PM, Ganesh Ajjanagadde wrote: > > On Thu, Jan 7, 2016 at 2:18 PM, Michael Niedermayer > > wrote: > >> On Thu, Jan 07, 2016 at 11:16:27PM +0100, Michael Niedermayer wrote: > >>> On Thu, Jan 07, 2016 at

Re: [FFmpeg-devel] [PATCH] ffmpeg: check fclose return values

2016-01-07 Thread Ganesh Ajjanagadde
On Thu, Jan 7, 2016 at 4:34 PM, Michael Niedermayer wrote: > On Thu, Jan 07, 2016 at 04:01:14PM -0800, Ganesh Ajjanagadde wrote: >> On Thu, Jan 7, 2016 at 3:57 PM, Ganesh Ajjanagadde wrote: >> > On Thu, Jan 7, 2016 at 2:18 PM, Michael Niedermayer >> > wrote: >> >> On Thu, Jan 07, 2016 at 11:16:2

Re: [FFmpeg-devel] [PATCH] ffmpeg: check fclose return values

2016-01-07 Thread Michael Niedermayer
On Thu, Jan 07, 2016 at 04:01:14PM -0800, Ganesh Ajjanagadde wrote: > On Thu, Jan 7, 2016 at 3:57 PM, Ganesh Ajjanagadde wrote: > > On Thu, Jan 7, 2016 at 2:18 PM, Michael Niedermayer > > wrote: > >> On Thu, Jan 07, 2016 at 11:16:27PM +0100, Michael Niedermayer wrote: > >>> On Thu, Jan 07, 2016 a

Re: [FFmpeg-devel] [PATCH] ffmpeg: check fclose return values

2016-01-07 Thread Ganesh Ajjanagadde
On Thu, Jan 7, 2016 at 3:57 PM, Ganesh Ajjanagadde wrote: > On Thu, Jan 7, 2016 at 2:18 PM, Michael Niedermayer > wrote: >> On Thu, Jan 07, 2016 at 11:16:27PM +0100, Michael Niedermayer wrote: >>> On Thu, Jan 07, 2016 at 10:00:47AM -0800, Ganesh Ajjanagadde wrote: >>> > On Thu, Jan 7, 2016 at 9:2

Re: [FFmpeg-devel] [PATCH] ffmpeg: check fclose return values

2016-01-07 Thread Ganesh Ajjanagadde
On Thu, Jan 7, 2016 at 2:18 PM, Michael Niedermayer wrote: > On Thu, Jan 07, 2016 at 11:16:27PM +0100, Michael Niedermayer wrote: >> On Thu, Jan 07, 2016 at 10:00:47AM -0800, Ganesh Ajjanagadde wrote: >> > On Thu, Jan 7, 2016 at 9:27 AM, Michael Niedermayer >> > wrote: >> > > On Wed, Jan 06, 2016

Re: [FFmpeg-devel] [PATCH] ffmpeg: check fclose return values

2016-01-07 Thread Michael Niedermayer
On Thu, Jan 07, 2016 at 11:16:27PM +0100, Michael Niedermayer wrote: > On Thu, Jan 07, 2016 at 10:00:47AM -0800, Ganesh Ajjanagadde wrote: > > On Thu, Jan 7, 2016 at 9:27 AM, Michael Niedermayer > > wrote: > > > On Wed, Jan 06, 2016 at 09:00:46PM -0800, Ganesh Ajjanagadde wrote: > > >> In the spir

Re: [FFmpeg-devel] [PATCH] ffmpeg: check fclose return values

2016-01-07 Thread Michael Niedermayer
On Thu, Jan 07, 2016 at 10:00:47AM -0800, Ganesh Ajjanagadde wrote: > On Thu, Jan 7, 2016 at 9:27 AM, Michael Niedermayer > wrote: > > On Wed, Jan 06, 2016 at 09:00:46PM -0800, Ganesh Ajjanagadde wrote: > >> In the spirit of commit a956840cbc. Simple method to reproduce: > >> pass -vstats_file /de

Re: [FFmpeg-devel] [PATCH] ffmpeg: check fclose return values

2016-01-07 Thread Ganesh Ajjanagadde
On Thu, Jan 7, 2016 at 9:27 AM, Michael Niedermayer wrote: > On Wed, Jan 06, 2016 at 09:00:46PM -0800, Ganesh Ajjanagadde wrote: >> In the spirit of commit a956840cbc. Simple method to reproduce: >> pass -vstats_file /dev/full to ffmpeg. >> >> All raw fclose usages in ffmpeg.c taken care of here.

Re: [FFmpeg-devel] [PATCH] ffmpeg: check fclose return values

2016-01-06 Thread Ganesh Ajjanagadde
On Wed, Jan 6, 2016 at 9:00 PM, Ganesh Ajjanagadde wrote: > In the spirit of commit a956840cbc. Simple method to reproduce: > pass -vstats_file /dev/full to ffmpeg. > > All raw fclose usages in ffmpeg.c taken care of here. > > Signed-off-by: Ganesh Ajjanagadde > --- > ffmpeg.c | 13 ++---

[FFmpeg-devel] [PATCH] ffmpeg: check fclose return values

2016-01-06 Thread Ganesh Ajjanagadde
In the spirit of commit a956840cbc. Simple method to reproduce: pass -vstats_file /dev/full to ffmpeg. All raw fclose usages in ffmpeg.c taken care of here. Signed-off-by: Ganesh Ajjanagadde --- ffmpeg.c | 13 ++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/ffmpeg.c