kill the proto task after teardown is complete, ignore proto task completion event...
authorFelix Fietkau <nbd@openwrt.org>
Wed, 12 Oct 2011 14:35:48 +0000 (16:35 +0200)
committerFelix Fietkau <nbd@openwrt.org>
Wed, 12 Oct 2011 14:35:48 +0000 (16:35 +0200)
proto-shell.c

index ee2366f5ab8ad5b33e63cb59282db1f2716c9f9c..32a30161f7ff5d9473398af475f6eee0e17d6ee3 100644 (file)
@@ -43,15 +43,22 @@ struct proto_shell_state {
        struct uloop_process proto_task;
 };
 
+static void
+kill_process(struct uloop_process *proc)
+{
+       if (!proc->pending)
+               return;
+
+       kill(proc->pid, SIGTERM);
+       uloop_process_delete(proc);
+}
+
 static int
 start_process(const char **argv, struct uloop_process *proc)
 {
        int pid;
 
-       if (proc->pending) {
-               kill(proc->pid, SIGTERM);
-               uloop_process_delete(proc);
-       }
+       kill_process(proc);
 
        if ((pid = fork()) < 0)
                return -1;
@@ -148,6 +155,8 @@ proto_shell_teardown_cb(struct uloop_process *p, int ret)
 
        state = container_of(p, struct proto_shell_state, teardown_task);
 
+       kill_process(&state->proto_task);
+
        if (state->l3_dev.dev)
                device_remove_user(&state->l3_dev);
 
@@ -160,6 +169,8 @@ proto_shell_task_cb(struct uloop_process *p, int ret)
        struct proto_shell_state *state;
 
        state = container_of(p, struct proto_shell_state, proto_task);
+       if (state->teardown_pending || state->teardown_task.pending)
+               return;
 
        state->proto.proto_event(&state->proto, IFPEV_LINK_LOST);
        proto_shell_handler(&state->proto, PROTO_CMD_TEARDOWN, false);