On Monday 10 November 2008 15:24, Vladimir Dronnikov wrote:
> Denys, do we mean "break" and not "continue" right before
> return (SIGHUP == bb_got_signal) ? 111 : EXIT_SUCCESS;
> at the end of runsvdir.c?

Yes:

                switch (bb_got_signal) {
...
                default: /* SIGTERM (or SIGUSRn if we are init) */
                        /* Exit unless we are init */
                        if (getpid() == 1)
                                break;
                        return (SIGHUP == bb_got_signal) ? 111 : EXIT_SUCCESS;
                }
                bb_got_signal = 0;
        } /* for (;;) */


"break" breaks from switch(), not from for().
We are still inside for(), we reset bb_got_signal
and go back to watching and restarting services.
--
vda
_______________________________________________
busybox mailing list
[email protected]
http://busybox.net/cgi-bin/mailman/listinfo/busybox

Reply via email to