hostapd: make the snooping interface (for proxyarp) configurable
Use the VLAN interface instead of the bridge, to ensure that hostapd receives untagged DHCP packets Signed-off-by: Felix Fietkau <nbd@nbd.name>
This commit is contained in:
		| @@ -754,6 +754,7 @@ hostapd_set_bss_options() { | |||||||
|  |  | ||||||
| 	append bss_conf "ssid=$ssid" "$N" | 	append bss_conf "ssid=$ssid" "$N" | ||||||
| 	[ -n "$network_bridge" ] && append bss_conf "bridge=$network_bridge" "$N" | 	[ -n "$network_bridge" ] && append bss_conf "bridge=$network_bridge" "$N" | ||||||
|  | 	[ -n "$network_ifname" ] && append bss_conf "snoop_iface=$network_ifname" "$N" | ||||||
| 	[ -n "$iapp_interface" ] && { | 	[ -n "$iapp_interface" ] && { | ||||||
| 		local ifname | 		local ifname | ||||||
| 		network_get_device ifname "$iapp_interface" || ifname="$iapp_interface" | 		network_get_device ifname "$iapp_interface" || ifname="$iapp_interface" | ||||||
|   | |||||||
| @@ -0,0 +1,34 @@ | |||||||
|  | --- a/src/ap/ap_config.h | ||||||
|  | +++ b/src/ap/ap_config.h | ||||||
|  | @@ -278,6 +278,7 @@ struct hostapd_bss_config { | ||||||
|  |  	char iface[IFNAMSIZ + 1]; | ||||||
|  |  	char bridge[IFNAMSIZ + 1]; | ||||||
|  |  	char ft_iface[IFNAMSIZ + 1]; | ||||||
|  | +	char snoop_iface[IFNAMSIZ + 1]; | ||||||
|  |  	char vlan_bridge[IFNAMSIZ + 1]; | ||||||
|  |  	char wds_bridge[IFNAMSIZ + 1]; | ||||||
|  |   | ||||||
|  | --- a/src/ap/x_snoop.c | ||||||
|  | +++ b/src/ap/x_snoop.c | ||||||
|  | @@ -71,6 +71,10 @@ x_snoop_get_l2_packet(struct hostapd_dat | ||||||
|  |  { | ||||||
|  |  	struct hostapd_bss_config *conf = hapd->conf; | ||||||
|  |  	struct l2_packet_data *l2; | ||||||
|  | +	const char *ifname = conf->bridge; | ||||||
|  | + | ||||||
|  | +	if (conf->snoop_iface[0]) | ||||||
|  | +		ifname = conf->snoop_iface; | ||||||
|  |   | ||||||
|  |  	l2 = l2_packet_init(conf->bridge, NULL, ETH_P_ALL, handler, hapd, 1); | ||||||
|  |  	if (l2 == NULL) { | ||||||
|  | --- a/hostapd/config_file.c | ||||||
|  | +++ b/hostapd/config_file.c | ||||||
|  | @@ -2357,6 +2357,8 @@ static int hostapd_config_fill(struct ho | ||||||
|  |  			   sizeof(conf->bss[0]->iface)); | ||||||
|  |  	} else if (os_strcmp(buf, "bridge") == 0) { | ||||||
|  |  		os_strlcpy(bss->bridge, pos, sizeof(bss->bridge)); | ||||||
|  | +	} else if (os_strcmp(buf, "snoop_iface") == 0) { | ||||||
|  | +		os_strlcpy(bss->snoop_iface, pos, sizeof(bss->snoop_iface)); | ||||||
|  |  	} else if (os_strcmp(buf, "vlan_bridge") == 0) { | ||||||
|  |  		os_strlcpy(bss->vlan_bridge, pos, sizeof(bss->vlan_bridge)); | ||||||
|  |  	} else if (os_strcmp(buf, "wds_bridge") == 0) { | ||||||
		Reference in New Issue
	
	Block a user
	 Felix Fietkau
					Felix Fietkau