From: Felix Fietkau <nbd@openwrt.org>
Date: Wed, 12 Oct 2011 14:35:48 +0000 (+0200)
Subject: kill the proto task after teardown is complete, ignore proto task completion event... 
X-Git-Url: http://git.cdn.openwrt.org/?a=commitdiff_plain;h=545f0fad20b68957e245b3b05ace935cd1089726;p=project%2Fnetifd.git

kill the proto task after teardown is complete, ignore proto task completion event while teardown is pending or active
---

diff --git a/proto-shell.c b/proto-shell.c
index ee2366f..32a3016 100644
--- a/proto-shell.c
+++ b/proto-shell.c
@@ -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);