projects
/
project
/
netifd.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
netifd: GRE Tos support
[project/netifd.git]
/
proto-shell.c
diff --git
a/proto-shell.c
b/proto-shell.c
index
77c9b87
..
7c23caa
100644
(file)
--- a/
proto-shell.c
+++ b/
proto-shell.c
@@
-440,11
+440,14
@@
proto_shell_update_link(struct proto_shell_state *state, struct blob_attr *data,
bool keep = false;
bool up;
bool keep = false;
bool up;
+ if (state->sm == S_TEARDOWN || state->sm == S_SETUP_ABORT)
+ return UBUS_STATUS_PERMISSION_DENIED;
+
if (!tb[NOTIFY_LINK_UP])
return UBUS_STATUS_INVALID_ARGUMENT;
up = blobmsg_get_bool(tb[NOTIFY_LINK_UP]);
if (!tb[NOTIFY_LINK_UP])
return UBUS_STATUS_INVALID_ARGUMENT;
up = blobmsg_get_bool(tb[NOTIFY_LINK_UP]);
- if (!up
|| state->sm == S_TEARDOWN
) {
+ if (!up) {
state->proto.proto_event(&state->proto, IFPEV_LINK_LOST);
return 0;
}
state->proto.proto_event(&state->proto, IFPEV_LINK_LOST);
return 0;
}
@@
-543,6
+546,9
@@
proto_shell_run_command(struct proto_shell_state *state, struct blob_attr **tb)
static char *argv[64];
static char *env[32];
static char *argv[64];
static char *env[32];
+ if (state->sm == S_TEARDOWN || state->sm == S_SETUP_ABORT)
+ return UBUS_STATUS_PERMISSION_DENIED;
+
if (!tb[NOTIFY_COMMAND])
goto error;
if (!tb[NOTIFY_COMMAND])
goto error;
@@
-572,7
+578,8
@@
proto_shell_kill_command(struct proto_shell_state *state, struct blob_attr **tb)
signal = SIGTERM;
if (state->proto_task.uloop.pending) {
signal = SIGTERM;
if (state->proto_task.uloop.pending) {
- state->proto_task_killed = true;
+ if (signal == SIGTERM || signal == SIGKILL)
+ state->proto_task_killed = true;
kill(state->proto_task.uloop.pid, signal);
}
kill(state->proto_task.uloop.pid, signal);
}
@@
-641,6
+648,9
@@
proto_shell_add_host_dependency(struct proto_shell_state *state, struct blob_att
const char *ifname_str = ifname_a ? blobmsg_data(ifname_a) : "";
char *ifname;
const char *ifname_str = ifname_a ? blobmsg_data(ifname_a) : "";
char *ifname;
+ if (state->sm == S_TEARDOWN || state->sm == S_SETUP_ABORT)
+ return UBUS_STATUS_PERMISSION_DENIED;
+
if (!host)
return UBUS_STATUS_INVALID_ARGUMENT;
if (!host)
return UBUS_STATUS_INVALID_ARGUMENT;
@@
-670,6
+680,8
@@
proto_shell_add_host_dependency(struct proto_shell_state *state, struct blob_att
static int
proto_shell_setup_failed(struct proto_shell_state *state)
{
static int
proto_shell_setup_failed(struct proto_shell_state *state)
{
+ int ret = 0;
+
switch (state->sm) {
case S_IDLE:
state->proto.proto_event(&state->proto, IFPEV_LINK_LOST);
switch (state->sm) {
case S_IDLE:
state->proto.proto_event(&state->proto, IFPEV_LINK_LOST);
@@
-677,10
+689,13
@@
proto_shell_setup_failed(struct proto_shell_state *state)
case S_SETUP:
proto_shell_handler(&state->proto, PROTO_CMD_TEARDOWN, false);
break;
case S_SETUP:
proto_shell_handler(&state->proto, PROTO_CMD_TEARDOWN, false);
break;
+ case S_SETUP_ABORT:
+ case S_TEARDOWN:
default:
default:
+ ret = UBUS_STATUS_PERMISSION_DENIED;
break;
}
break;
}
- return
0
;
+ return
ret
;
}
static int
}
static int