[FFmpeg-cvslog] ffmpeg: Don't trigger url_interrupt_cb on the first signal

Martin Storsjö git at videolan.org
Mon May 23 04:40:14 CEST 2011


ffmpeg | branch: master | Martin Storsjö <martin at martin.st> | Sun May 22 14:40:33 2011 +0300| [a121754852a69b4879a39ba78863404c13c54f61] | committer: Martin Storsjö

ffmpeg: Don't trigger url_interrupt_cb on the first signal

Currently, the url_interrupt_cb callback will abort all IO
after the first received signal. This makes the output files
from e.g. the mov muxer to be unreadable if the transcode is
aborted with ctrl+c.

After this patch, the first signal cleanly breaks out of
the transcoding loop, but won't forcibly abort all IO.
After the second signal is received, the url_interrupt_cb
callback will abort all IO.

Signed-off-by: Martin Storsjö <martin at martin.st>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=a121754852a69b4879a39ba78863404c13c54f61
---

 ffmpeg.c |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/ffmpeg.c b/ffmpeg.c
index 0c95451..8673253 100644
--- a/ffmpeg.c
+++ b/ffmpeg.c
@@ -426,11 +426,13 @@ static void term_exit(void)
 }
 
 static volatile int received_sigterm = 0;
+static volatile int received_nb_signals = 0;
 
 static void
 sigterm_handler(int sig)
 {
     received_sigterm = sig;
+    received_nb_signals++;
     term_exit();
 }
 
@@ -445,7 +447,7 @@ static void term_init(void)
 
 static int decode_interrupt_cb(void)
 {
-    return received_sigterm;
+    return received_nb_signals > 1;
 }
 
 static int ffmpeg_exit(int ret)



More information about the ffmpeg-cvslog mailing list