mxb <[email protected]> wrote:

>Then running dhcpd with pf-support (-A <tbl> -C <tbl>).
>dhcpd spawns child process which is not handled by rc-script then
>stop/restart.
>
>Here is a diff to fix it.
>
>Yes, I know, normally one might want to flush PF-tables as well and
>this is not handled by the diff.
>But at least I don't have to kill child process manually.
>
>--- dhcpd       Wed Nov 28 10:16:33 2012
>+++ dhcpd.my    Wed Nov 28 09:47:28 2012
>@@ -12,4 +12,17 @@
>        touch /var/db/dhcpd.leases
> }
> 
>-rc_cmd $1
>+case $1 in
>+        restart)
>+                rc_stop
>+                pkill dhcpd 
>+                rc_start
>+                ;;
>+       stop)
>+               rc_stop
>+               pkill dhcpd
>+               ;;
>+        *)
>+                rc_cmd $1
>+                ;;
>+esac

If the parent process is killed but not the child, it's a bug in dhcpd. If not, 
I'd say its a bug in rc.subr or the pexp.

The proposed diff looks like the wrong approach.

/Alexander

Reply via email to