On Thu, Jul 19, 2012 at 7:05 AM, Michael Tokarev <[email protected]> wrote:
> Why udhcpd does not write leases file upon
> receipt of termination signal?  Here's the
> code in question (networking/udhcp/udhcpd.c):
>
>                 switch (udhcp_sp_read(&rfds)) {
>                 case SIGUSR1:
>                         bb_info_msg("Received SIGUSR1");
>                         write_leases();
>                         /* why not just reset the timeout, eh */
>                         timeout_end = monotonic_sec() + 
> server_config.auto_time;
>                         continue;
>                 case SIGTERM:
>                         bb_info_msg("Received SIGTERM");
>                         goto ret0;
>                 case 0: /* no signal: read a packet */
>                         break;
>                 default: /* signal or error (probably EINTR): back to select 
> */
>                         continue;
>                 }
>
> I think there should be a call to write_leases()
> right after "Received SIGTERM" message, just like
> it is done in SIGUSR1 case?

Indeed. I added it.

-- 
vda
_______________________________________________
busybox mailing list
[email protected]
http://lists.busybox.net/mailman/listinfo/busybox

Reply via email to