X-Git-Url: https://git.archive.openwrt.org/?a=blobdiff_plain;f=proto-static.c;h=bbe304b1a67ccec9758e81abb1f718a1771a50f8;hb=e70cc01c34a4a3faded4bbdd50b9b716bd2ca94c;hp=a4938f41a23e8ffcb6248658f18d7960298ba98d;hpb=818abd9f2f42c36a0f91ff6d29a9a635398216e0;p=project%2Fnetifd.git diff --git a/proto-static.c b/proto-static.c index a4938f4..bbe304b 100644 --- a/proto-static.c +++ b/proto-static.c @@ -43,7 +43,6 @@ struct static_proto_state { struct interface_proto_state proto; struct blob_attr *config; - struct interface *iface; }; static bool @@ -83,26 +82,25 @@ parse_ip_and_netmask(int af, const char *str, void *addr, unsigned int *netmask) } static bool -parse_addr(struct static_proto_state *state, const char *str, bool v6, int mask) +parse_addr(struct interface_proto_state *state, const char *str, bool v6, int mask) { struct device_addr *addr; int af = v6 ? AF_INET6 : AF_INET; addr = calloc(1, sizeof(*addr)); addr->flags = v6 ? DEVADDR_INET6 : DEVADDR_INET4; - addr->ctx = state; addr->mask = mask; if (!parse_ip_and_netmask(af, str, &addr->addr, &addr->mask)) { interface_add_error(state->iface, "proto-static", "INVALID_ADDRESS", &str, 1); free(addr); return false; } - interface_add_address(state->iface, addr); + vlist_add(&state->iface->proto_addr, &addr->node); return true; } static int -parse_address_option(struct static_proto_state *state, struct blob_attr *attr, bool v6, int netmask) +parse_address_option(struct interface_proto_state *state, struct blob_attr *attr, bool v6, int netmask) { struct blob_attr *cur; int n_addr = 0; @@ -118,7 +116,7 @@ parse_address_option(struct static_proto_state *state, struct blob_attr *attr, b } static bool -parse_gateway_option(struct static_proto_state *state, struct blob_attr *attr, bool v6) +parse_gateway_option(struct interface_proto_state *state, struct blob_attr *attr, bool v6) { struct device_route *route; const char *str = blobmsg_data(attr); @@ -133,13 +131,13 @@ parse_gateway_option(struct static_proto_state *state, struct blob_attr *attr, b } route->mask = 0; route->flags = DEVADDR_DEVICE | (v6 ? DEVADDR_INET6 : DEVADDR_INET4); - interface_add_route(state->iface, route); + vlist_add(&state->iface->proto_route, &route->node); return true; } -static bool -static_proto_setup(struct static_proto_state *state) +int +proto_apply_static_settings(struct interface_proto_state *state, struct blob_attr *attr) { struct blob_attr *tb[__OPT_MAX]; struct in_addr ina; @@ -147,8 +145,7 @@ static_proto_setup(struct static_proto_state *state) int netmask = 32; int n_v4 = 0, n_v6 = 0; - blobmsg_parse(static_attrs, __OPT_MAX, tb, blob_data(state->config), - blob_len(state->config)); + blobmsg_parse(static_attrs, __OPT_MAX, tb, blob_data(attr), blob_len(attr)); if (tb[OPT_NETMASK]) { if (!inet_aton(blobmsg_data(tb[OPT_NETMASK]), &ina)) { @@ -183,12 +180,18 @@ static_proto_setup(struct static_proto_state *state) goto out; } - return true; + return 0; error: interface_add_error(state->iface, "proto-static", error, NULL, 0); out: - return false; + return -1; +} + +static bool +static_proto_setup(struct static_proto_state *state) +{ + return proto_apply_static_settings(&state->proto, state->config) == 0; } static int @@ -202,12 +205,11 @@ static_handler(struct interface_proto_state *proto, switch (cmd) { case PROTO_CMD_SETUP: - if (static_proto_setup(state)) - break; + if (!static_proto_setup(state)) + return -1; - /* fall through */ + break; case PROTO_CMD_TEARDOWN: - interface_del_ctx_addr(state->iface, state); break; } return ret; @@ -233,15 +235,13 @@ static_attach(const struct proto_handler *h, struct interface *iface, if (!state) return NULL; - state->iface = iface; state->config = malloc(blob_pad_len(attr)); if (!state->config) goto error; memcpy(state->config, attr, blob_pad_len(attr)); state->proto.free = static_free; - state->proto.handler = static_handler; - state->proto.flags = PROTO_FLAG_IMMEDIATE; + state->proto.cb = static_handler; return &state->proto; @@ -252,6 +252,7 @@ error: static struct proto_handler static_proto = { .name = "static", + .flags = PROTO_FLAG_IMMEDIATE, .config_params = &static_attr_list, .attach = static_attach, };