[FFmpeg-devel] [PATCH 3/3] ffmpeg: properly close down muxers on av_interleaved_write_frame() failure

Nicolas George george at nsup.org
Tue Jan 14 20:29:05 CET 2014


Le quintidi 25 nivôse, an CCXXII, Michael Niedermayer a écrit :
> Signed-off-by: Michael Niedermayer <michaelni at gmx.at>
> ---
>  ffmpeg.c |   17 ++++++++++++++---
>  1 file changed, 14 insertions(+), 3 deletions(-)
> 
> diff --git a/ffmpeg.c b/ffmpeg.c
> index 5831a78..228849f 100644
> --- a/ffmpeg.c
> +++ b/ffmpeg.c
> @@ -313,6 +313,7 @@ void term_exit(void)
>  
>  static volatile int received_sigterm = 0;
>  static volatile int received_nb_signals = 0;

> +static int main_return_code = 0;

I do not like that use of a global variable, but it makes the patch simpler.

The logic of the three patches seems right to me.

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140114/1a4b1d66/attachment.asc>


More information about the ffmpeg-devel mailing list