diff options
author | Matthias Schiffer <mschiffer@universe-factory.net> | 2016-06-21 18:19:11 +0300 |
---|---|---|
committer | Felix Fietkau <nbd@nbd.name> | 2016-06-26 13:55:31 +0300 |
commit | 1ad3d93eb888ae7db3dd97e43e611979352b1aac (patch) | |
tree | 999111caeec71c4e5af728b384d02a91b8d76b3d /uloop.c | |
parent | 1f019ceea1ed39286e6bccfb3ff936c22fe0f7c0 (diff) |
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 <mschiffer@universe-factory.net>
Diffstat (limited to 'uloop.c')
-rw-r--r-- | uloop.c | 9 |
1 files changed, 7 insertions, 2 deletions
@@ -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) |