device/bridge: support passing extra vlans in the device_set_state call
authorFelix Fietkau <nbd@nbd.name>
Wed, 1 Nov 2023 05:25:16 +0000 (06:25 +0100)
committerFelix Fietkau <nbd@nbd.name>
Wed, 1 Nov 2023 05:27:46 +0000 (06:27 +0100)
This can be used by RADIUS based 802.1x auth to dynamically define the member
vlans for the port based on its authentication status.

Signed-off-by: Felix Fietkau <nbd@nbd.name>
bridge.c
device.c
device.h
ubus.c

index dbeea469160c039393b45863c1067dc215925876..60134a21203effafc33b9945ac640dab59fe65b1 100644 (file)
--- a/bridge.c
+++ b/bridge.c
@@ -401,6 +401,155 @@ bridge_disable_interface(struct bridge_state *bst)
        bst->active = false;
 }
 
+static struct bridge_vlan *
+bridge_hotplug_get_vlan(struct bridge_state *bst, uint16_t vid, bool create)
+{
+       struct bridge_vlan *vlan;
+
+       vlan = vlist_find(&bst->dev.vlans, &vid, vlan, node);
+       if (vlan || !create)
+               return vlan;
+
+       vlan = calloc(1, sizeof(*vlan));
+       vlan->vid = vid;
+       vlan->local = true;
+       INIT_LIST_HEAD(&vlan->hotplug_ports);
+       vlist_add(&bst->dev.vlans, &vlan->node, &vlan->vid);
+       vlan->node.version = -1;
+
+       return vlan;
+}
+
+static struct bridge_vlan_hotplug_port *
+bridge_hotplug_get_vlan_port(struct bridge_vlan *vlan, const char *ifname)
+{
+       struct bridge_vlan_hotplug_port *port;
+
+       list_for_each_entry(port, &vlan->hotplug_ports, list)
+               if (!strcmp(port->port.ifname, ifname))
+                       return port;
+
+       return NULL;
+}
+
+static void
+bridge_hotplug_set_member_vlans(struct bridge_state *bst, struct blob_attr *vlans,
+                               struct bridge_member *bm, bool add, bool untracked)
+{
+       const char *ifname = bm->name;
+       struct device_vlan_range *r;
+       struct bridge_vlan *vlan;
+       struct blob_attr *cur;
+       int n_vlans;
+       size_t rem;
+
+       if (!vlans)
+               return;
+
+       if (add) {
+               bm->n_extra_vlan = 0;
+               n_vlans = blobmsg_check_array(vlans, BLOBMSG_TYPE_STRING);
+               if (n_vlans < 1)
+                       return;
+
+               bm->extra_vlan = realloc(bm->extra_vlan, n_vlans * sizeof(*bm->extra_vlan));
+       }
+
+       blobmsg_for_each_attr(cur, vlans, rem) {
+               struct bridge_vlan_hotplug_port *port;
+               unsigned int vid, vid_end;
+               uint16_t flags = 0;
+               char *name_buf;
+               char *end;
+
+               if (blobmsg_type(cur) != BLOBMSG_TYPE_STRING)
+                       continue;
+
+               vid = strtoul(blobmsg_get_string(cur), &end, 0);
+               vid_end = vid;
+               if (!vid || vid > 4095)
+                       continue;
+
+               if (*end == '-') {
+                       vid_end = strtoul(end + 1, &end, 0);
+                       if (vid_end < vid)
+                               continue;
+               }
+
+               if (end && *end) {
+                       if (*end != ':')
+                               continue;
+
+                       for (end++; *end; end++) {
+                               switch (*end) {
+                               case 'u':
+                                       flags |= BRVLAN_F_UNTAGGED;
+                                       fallthrough;
+                               case '*':
+                                       flags |= BRVLAN_F_PVID;
+                                       break;
+                               }
+                       }
+               }
+
+               vlan = bridge_hotplug_get_vlan(bst, vid, !!flags);
+               if (!vlan || vid_end > vid || untracked) {
+                       if (add) {
+                               if (!untracked) {
+                                       r = &bm->extra_vlan[bm->n_extra_vlan++];
+                                       r->start = vid;
+                                       r->end = vid_end;
+                               }
+                               if (bm->active)
+                                       system_bridge_vlan(ifname, vid, vid_end, true, flags);
+                       } else if (bm->active) {
+                               system_bridge_vlan(ifname, vid, vid_end, false, 0);
+                       }
+                       continue;
+               }
+
+               if (vlan->pending) {
+                       vlan->pending = false;
+                       bridge_set_vlan_state(bst, vlan, true);
+               }
+
+               port = bridge_hotplug_get_vlan_port(vlan, ifname);
+               if (!add) {
+                       if (!port)
+                               continue;
+
+                       __bridge_set_member_vlan(bm, vlan, &port->port, false);
+                       list_del(&port->list);
+                       free(port);
+                       continue;
+               }
+
+               if (port) {
+                       if (port->port.flags == flags)
+                               continue;
+
+                       __bridge_set_member_vlan(bm, vlan, &port->port, false);
+                       port->port.flags = flags;
+                       __bridge_set_member_vlan(bm, vlan, &port->port, true);
+                       continue;
+               }
+
+               port = calloc_a(sizeof(*port), &name_buf, strlen(ifname) + 1);
+               if (!port)
+                       continue;
+
+               port->port.flags = flags;
+               port->port.ifname = strcpy(name_buf, ifname);
+               list_add_tail(&port->list, &vlan->hotplug_ports);
+
+               if (!bm)
+                       continue;
+
+               __bridge_set_member_vlan(bm, vlan, &port->port, true);
+       }
+}
+
+
 static void
 bridge_member_add_extra_vlans(struct bridge_member *bm)
 {
@@ -461,6 +610,8 @@ bridge_enable_member(struct bridge_member *bm)
                bridge_member_add_extra_vlans(bm);
                vlist_for_each_element(&bst->dev.vlans, vlan, node)
                        bridge_set_member_vlan(bm, vlan, true);
+               if (dev->settings.auth && dev->auth_vlans)
+                       bridge_hotplug_set_member_vlans(bst, dev->auth_vlans, bm, true, true);
        }
 
        device_set_present(&bst->dev, true);
@@ -785,151 +936,6 @@ bridge_add_member(struct bridge_state *bst, const char *name)
        bridge_create_member(bst, name, dev, false);
 }
 
-static struct bridge_vlan *
-bridge_hotplug_get_vlan(struct bridge_state *bst, uint16_t vid, bool create)
-{
-       struct bridge_vlan *vlan;
-
-       vlan = vlist_find(&bst->dev.vlans, &vid, vlan, node);
-       if (vlan || !create)
-               return vlan;
-
-       vlan = calloc(1, sizeof(*vlan));
-       vlan->vid = vid;
-       vlan->local = true;
-       INIT_LIST_HEAD(&vlan->hotplug_ports);
-       vlist_add(&bst->dev.vlans, &vlan->node, &vlan->vid);
-       vlan->node.version = -1;
-
-       return vlan;
-}
-
-static struct bridge_vlan_hotplug_port *
-bridge_hotplug_get_vlan_port(struct bridge_vlan *vlan, const char *ifname)
-{
-       struct bridge_vlan_hotplug_port *port;
-
-       list_for_each_entry(port, &vlan->hotplug_ports, list)
-               if (!strcmp(port->port.ifname, ifname))
-                       return port;
-
-       return NULL;
-}
-
-static void
-bridge_hotplug_set_member_vlans(struct bridge_state *bst, struct blob_attr *vlans,
-                               const char *ifname, struct bridge_member *bm, bool add)
-{
-       struct device_vlan_range *r;
-       struct bridge_vlan *vlan;
-       struct blob_attr *cur;
-       int n_vlans;
-       size_t rem;
-
-       if (!vlans)
-               return;
-
-       if (add) {
-               bm->n_extra_vlan = 0;
-               n_vlans = blobmsg_check_array(vlans, BLOBMSG_TYPE_STRING);
-               if (n_vlans < 1)
-                       return;
-
-               bm->extra_vlan = realloc(bm->extra_vlan, n_vlans * sizeof(*bm->extra_vlan));
-       }
-
-       blobmsg_for_each_attr(cur, vlans, rem) {
-               struct bridge_vlan_hotplug_port *port;
-               unsigned int vid, vid_end;
-               uint16_t flags = 0;
-               char *name_buf;
-               char *end;
-
-               if (blobmsg_type(cur) != BLOBMSG_TYPE_STRING)
-                       continue;
-
-               vid = strtoul(blobmsg_get_string(cur), &end, 0);
-               vid_end = vid;
-               if (!vid || vid > 4095)
-                       continue;
-
-               if (*end == '-') {
-                       vid_end = strtoul(end + 1, &end, 0);
-                       if (vid_end < vid)
-                               continue;
-               }
-
-               if (end && *end) {
-                       if (*end != ':')
-                               continue;
-
-                       for (end++; *end; end++) {
-                               switch (*end) {
-                               case 'u':
-                                       flags |= BRVLAN_F_UNTAGGED;
-                                       break;
-                               case '*':
-                                       flags |= BRVLAN_F_PVID;
-                                       break;
-                               }
-                       }
-               }
-
-               vlan = bridge_hotplug_get_vlan(bst, vid, !!flags);
-               if (!vlan || vid_end > vid) {
-                       if (add) {
-                               r = &bm->extra_vlan[bm->n_extra_vlan++];
-                               r->start = vid;
-                               r->end = vid_end;
-                               if (bm->active)
-                                       system_bridge_vlan(ifname, vid, vid_end, true, 0);
-                       } else if (bm->active) {
-                               system_bridge_vlan(ifname, vid, vid_end, false, 0);
-                       }
-                       continue;
-               }
-
-               if (vlan->pending) {
-                       vlan->pending = false;
-                       bridge_set_vlan_state(bst, vlan, true);
-               }
-
-               port = bridge_hotplug_get_vlan_port(vlan, ifname);
-               if (!add) {
-                       if (!port)
-                               continue;
-
-                       __bridge_set_member_vlan(bm, vlan, &port->port, false);
-                       list_del(&port->list);
-                       free(port);
-                       continue;
-               }
-
-               if (port) {
-                       if (port->port.flags == flags)
-                               continue;
-
-                       __bridge_set_member_vlan(bm, vlan, &port->port, false);
-                       port->port.flags = flags;
-                       __bridge_set_member_vlan(bm, vlan, &port->port, true);
-                       continue;
-               }
-
-               port = calloc_a(sizeof(*port), &name_buf, strlen(ifname) + 1);
-               if (!port)
-                       continue;
-
-               port->port.flags = flags;
-               port->port.ifname = strcpy(name_buf, ifname);
-               list_add_tail(&port->list, &vlan->hotplug_ports);
-
-               if (!bm)
-                       continue;
-
-               __bridge_set_member_vlan(bm, vlan, &port->port, true);
-       }
-}
-
 static int
 bridge_hotplug_add(struct device *dev, struct device *member, struct blob_attr *vlan)
 {
@@ -942,7 +948,7 @@ bridge_hotplug_add(struct device *dev, struct device *member, struct blob_attr *
            new_entry = true;
            bm = bridge_alloc_member(bst, member->ifname, member, true);
        }
-       bridge_hotplug_set_member_vlans(bst, vlan, member->ifname, bm, true);
+       bridge_hotplug_set_member_vlans(bst, vlan, bm, true, false);
        if (new_entry)
                bridge_insert_member(bm, member->ifname);
 
@@ -959,7 +965,7 @@ bridge_hotplug_del(struct device *dev, struct device *member, struct blob_attr *
        if (!bm)
                return UBUS_STATUS_NOT_FOUND;
 
-       bridge_hotplug_set_member_vlans(bst, vlan, member->ifname, bm, false);
+       bridge_hotplug_set_member_vlans(bst, vlan, bm, false, false);
        if (!bm->dev.hotplug)
                return 0;
 
index 24175564f8558df03aa54470ac72aaea23fc697c..42a7c387c9f3b3e0338f2e6aecec9af551419a97 100644 (file)
--- a/device.c
+++ b/device.c
@@ -830,8 +830,16 @@ device_refresh_present(struct device *dev)
 }
 
 void
-device_set_auth_status(struct device *dev, bool value)
+device_set_auth_status(struct device *dev, bool value, struct blob_attr *vlans)
 {
+       if (!value)
+               vlans = NULL;
+       else if (!blob_attr_equal(vlans, dev->auth_vlans))
+               device_set_auth_status(dev, false, NULL);
+
+       free(dev->auth_vlans);
+       dev->auth_vlans = vlans ? blob_memdup(vlans) : NULL;
+
        if (dev->auth_status == value)
                return;
 
@@ -966,6 +974,7 @@ static void
 device_free(struct device *dev)
 {
        __devlock++;
+       free(dev->auth_vlans);
        free(dev->config);
        device_cleanup(dev);
        free(dev->extra_vlan);
index 14d7486b7317799e2a431e01c826b9c0fdb0728a..804efb38a048fa919daa0b61247c43e7ba955eb4 100644 (file)
--- a/device.h
+++ b/device.h
@@ -238,6 +238,7 @@ struct device {
 
        struct vlist_tree vlans;
        struct kvlist vlan_aliases;
+       struct blob_attr *auth_vlans;
 
        char ifname[IFNAMSIZ];
        int ifindex;
@@ -371,7 +372,7 @@ struct device *get_vlan_device_chain(const char *ifname, int create);
 void alias_notify_device(const char *name, struct device *dev);
 struct device *device_alias_get(const char *name);
 
-void device_set_auth_status(struct device *dev, bool value);
+void device_set_auth_status(struct device *dev, bool value, struct blob_attr *vlans);
 
 static inline void
 device_set_deferred(struct device *dev, bool value)
diff --git a/ubus.c b/ubus.c
index 3504a4976aacb71d024f7a060ec69a1268d0c1f9..e2aa3f80b7e1493ffe7eafda6808604fdb5c243c 100644 (file)
--- a/ubus.c
+++ b/ubus.c
@@ -305,6 +305,7 @@ enum {
        DEV_STATE_NAME,
        DEV_STATE_DEFER,
        DEV_STATE_AUTH_STATUS,
+       DEV_STATE_AUTH_VLANS,
        __DEV_STATE_MAX,
 };
 
@@ -312,6 +313,7 @@ static const struct blobmsg_policy dev_state_policy[__DEV_STATE_MAX] = {
        [DEV_STATE_NAME] = { .name = "name", .type = BLOBMSG_TYPE_STRING },
        [DEV_STATE_DEFER] = { .name = "defer", .type = BLOBMSG_TYPE_BOOL },
        [DEV_STATE_AUTH_STATUS] = { .name = "auth_status", .type = BLOBMSG_TYPE_BOOL },
+       [DEV_STATE_AUTH_VLANS] = { .name = "auth_vlans", BLOBMSG_TYPE_ARRAY },
 };
 
 static int
@@ -322,6 +324,7 @@ netifd_handle_set_state(struct ubus_context *ctx, struct ubus_object *obj,
        struct device *dev = NULL;
        struct blob_attr *tb[__DEV_STATE_MAX];
        struct blob_attr *cur;
+       bool auth_status;
 
        blobmsg_parse(dev_state_policy, __DEV_STATE_MAX, tb, blob_data(msg), blob_len(msg));
 
@@ -337,9 +340,12 @@ netifd_handle_set_state(struct ubus_context *ctx, struct ubus_object *obj,
        if (cur)
                device_set_deferred(dev, !!blobmsg_get_u8(cur));
 
-       cur = tb[DEV_STATE_AUTH_STATUS];
-       if (cur)
-               device_set_auth_status(dev, !!blobmsg_get_u8(cur));
+       if ((cur = tb[DEV_STATE_AUTH_STATUS]) != NULL)
+               auth_status = blobmsg_get_bool(cur);
+       else
+               auth_status = dev->auth_status;
+       if (tb[DEV_STATE_AUTH_STATUS] || tb[DEV_STATE_AUTH_VLANS])
+               device_set_auth_status(dev, auth_status, tb[DEV_STATE_AUTH_VLANS]);
 
        return 0;
 }