Fix selection of Tunnel* attributes by tag

The initial implementation was doing useless work.

Signed-off-by: Tom Barthe <jeltz+hostap@auro.re>

src/radius/radius.c

@@ -1470,6 +1470,11 @@ int radius_msg_get_vlanid(struct radius_msg *msg, int *untagged, int numtagged,
 		tagged[j] = 0;
 	*untagged = 0;
 
+	if (tag_is_valid(tag))
+		tun = &tunnel[tag];
+	else
+		tun = &tunnel[0];
+
 	for (i = 0; i < msg->attr_used; i++) {
 		attr = radius_get_attr_hdr(msg, i);
 		if (attr->length < sizeof(*attr))
@@ -1478,10 +1483,8 @@ int radius_msg_get_vlanid(struct radius_msg *msg, int *untagged, int numtagged,
 		dlen = attr->length - sizeof(*attr);
 		if (attr->length < 3)
 			continue;
-		if (data[0] >= RADIUS_TUNNEL_TAGS)
+		if (tag_is_valid(tag) && data[0] != tag)
-			tun = &tunnel[0];
+			continue;
-		else
-			tun = &tunnel[data[0]];
 
 		switch (attr->type) {
 		case RADIUS_ATTR_TUNNEL_TYPE:
@@ -1526,19 +1529,11 @@ int radius_msg_get_vlanid(struct radius_msg *msg, int *untagged, int numtagged,
 		}
 	}
 
-	/* Use tunnel with the lowest tag for untagged VLAN id */
+	if (tun->tag_used &&
-	for (i = 0; i < RADIUS_TUNNEL_TAGS; i++) {
+	    tun->type == RADIUS_TUNNEL_TYPE_VLAN &&
-		tun = &tunnel[i];
+	    tun->medium_type == RADIUS_TUNNEL_MEDIUM_TYPE_802 &&
-		if (tag_is_valid(tag) && i != tag)
+	    tun->vlanid > 0)
-			continue;
+		*untagged = tun->vlanid;
-		if (tun->tag_used &&
-		    tun->type == RADIUS_TUNNEL_TYPE_VLAN &&
-		    tun->medium_type == RADIUS_TUNNEL_MEDIUM_TYPE_802 &&
-		    tun->vlanid > 0) {
-			*untagged = tun->vlanid;
-			break;
-		}
-	}
 
 	if (taggedidx)
 		qsort(tagged, taggedidx, sizeof(int), cmp_int);