From 1ad3d93eb888ae7db3dd97e43e611979352b1aac Mon Sep 17 00:00:00 2001 From: Matthias Schiffer Date: Tue, 21 Jun 2016 17:19:11 +0200 Subject: loop: make uloop_run() return the cancelling signal When a process quits in response to a signal it handles, it should to so be re-sending the signal to itself. This especially important for SIGINT, as is explained in [1]. uloop currently hides the reason for quitting uloop_run(). Fix this by returning the signal that caused the loop to quit (or 0 when uloop_end() was used), so a program using loop an comply with [1]. [1] https://www.cons.org/cracauer/sigint.html Signed-off-by: Matthias Schiffer --- uloop.c | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) (limited to 'uloop.c') diff --git a/uloop.c b/uloop.c index e60fb09..0b75d4b 100644 --- a/uloop.c +++ b/uloop.c @@ -58,6 +58,7 @@ static struct list_head processes = LIST_HEAD_INIT(processes); static int poll_fd = -1; bool uloop_cancelled = false; +static int uloop_status = 0; static bool do_sigchld = false; static struct uloop_fd_event cur_fds[ULOOP_MAX_EVENTS]; @@ -391,6 +392,7 @@ static void uloop_signal_wake(void) static void uloop_handle_sigint(int signo) { + uloop_status = signo; uloop_cancelled = true; uloop_signal_wake(); } @@ -506,7 +508,7 @@ static void uloop_clear_processes(void) uloop_process_delete(p); } -void uloop_run(void) +int uloop_run(void) { static int recursive_calls = 0; struct timeval tv; @@ -518,8 +520,9 @@ void uloop_run(void) if (!recursive_calls++) uloop_setup_signals(true); + uloop_status = 0; uloop_cancelled = false; - while(!uloop_cancelled) + while (!uloop_cancelled) { uloop_gettime(&tv); uloop_process_timeouts(&tv); @@ -536,6 +539,8 @@ void uloop_run(void) if (!--recursive_calls) uloop_setup_signals(false); + + return uloop_status; } void uloop_done(void) -- cgit v1.2.3