[package] firewall: fix fw__uci_state_del() procedure (#11132)
[openwrt.git] / package / firewall / files / lib / core_interface.sh
index 56ff1a1..3d67184 100644 (file)
@@ -1,4 +1,35 @@
-# Copyright (C) 2009-2010 OpenWrt.org
+# Copyright (C) 2009-2012 OpenWrt.org
+
+fw__uci_state_add() {
+       local var="$1"
+       local item="$2"
+
+       local val="$(uci_get_state firewall core $var)"
+       local e1; for e1 in $item; do
+               local e2; for e2 in $val; do
+                       [ "$e1" = "$e2" ] && e1=""
+               done
+               val="${val:+$val${e1:+ }}$e1"
+       done
+
+       uci_toggle_state firewall core $var "$val"
+}
+
+fw__uci_state_del() {
+       local var="$1"
+       local item="$2"
+
+       local rest=""
+       local val="$(uci_get_state firewall core $var)"
+       local e1; for e1 in $val; do
+               local e2; for e2 in $item; do
+                       [ "$e1" = "$e2" ] && e1=""
+               done
+               rest="${rest:+$rest${e1:+ }}$e1"
+       done
+
+       uci_toggle_state firewall core $var "$rest"
+}
 
 fw_configure_interface() {
        local iface=$1
@@ -11,7 +42,12 @@ fw_configure_interface() {
                [ "$status" == 1 ] || [ -n "$aliasnet" ] || return 0
        }
 
-       [ -n "$ifname" ] || ifname=$(uci_get_state network "$iface" ifname "$iface")
+       [ -n "$ifname" ] || {
+               ifname=$(uci_get_state network "$iface" ifname)
+               ifname="${ifname%%:*}"
+               [ -z "$ifname" ] && return 0
+       }
+
        [ "$ifname" == "lo" ] && return 0
 
        fw_callback pre interface
@@ -23,21 +59,21 @@ fw_configure_interface() {
                local ifname=$3
                local subnet=$4
 
-               local inet onet
-               local mode=$(fw_get_family_mode x $zone i)
+               local inet onet mode
+               fw_get_family_mode mode x $zone i
 
                case "$mode/$subnet" in
                        # Zone supports v6 only or dual, need v6
-                       6/*:*|i/*:*)
-                               inet="{ -s $subnet -d ::/0 }"
-                               onet="{ -s ::/0 -d $subnet }"
+                       G6/*:*|i/*:*)
+                               inet="-s $subnet -d ::/0"
+                               onet="-s ::/0 -d $subnet"
                                mode=6
                        ;;
 
                        # Zone supports v4 only or dual, need v4
-                       4/*.*.*.*|i/*.*.*.*)
-                               inet="{ -s $subnet -d 0.0.0.0/0 }"
-                               onet="{ -s 0.0.0.0/0 -d $subnet }"
+                       G4/*.*.*.*|i/*.*.*.*)
+                               inet="-s $subnet -d 0.0.0.0/0"
+                               onet="-s 0.0.0.0/0 -d $subnet"
                                mode=4
                        ;;
 
@@ -46,36 +82,51 @@ fw_configure_interface() {
 
                        # Need v4 while zone is v6
                        */*.*) fw_log info "zone $zone does not support IPv4 address family, skipping"; return ;;
+
+                       # Strip prefix
+                       *) mode="${mode#G}" ;;
                esac
 
-               fw $action $mode f ${chain}_ACCEPT ACCEPT ^ $onet { -o "$ifname" }
-               fw $action $mode f ${chain}_ACCEPT ACCEPT ^ $inet { -i "$ifname" }
-               fw $action $mode f ${chain}_DROP   DROP   ^ $onet { -o "$ifname" }
-               fw $action $mode f ${chain}_DROP   DROP   ^ $inet { -i "$ifname" }
-               fw $action $mode f ${chain}_REJECT reject ^ $onet { -o "$ifname" }
-               fw $action $mode f ${chain}_REJECT reject ^ $inet { -i "$ifname" }
+               lock /var/run/firewall-interface.lock
+
+               fw $action $mode f ${chain}_ACCEPT ACCEPT $ { -o "$ifname" $onet }
+               fw $action $mode f ${chain}_ACCEPT ACCEPT $ { -i "$ifname" $inet }
+               fw $action $mode f ${chain}_DROP   DROP   $ { -o "$ifname" $onet }
+               fw $action $mode f ${chain}_DROP   DROP   $ { -i "$ifname" $inet }
+               fw $action $mode f ${chain}_REJECT reject $ { -o "$ifname" $onet }
+               fw $action $mode f ${chain}_REJECT reject $ { -i "$ifname" $inet }
+
+               [ "$(uci_get_state firewall core "${zone}_tcpmss")" == 1 ] && \
+                       fw $action $mode m ${chain}_MSSFIX TCPMSS $ \
+                               { -o "$ifname" -p tcp --tcp-flags SYN,RST SYN --clamp-mss-to-pmtu $onet }
 
-               fw $action $mode n ${chain}_nat MASQUERADE ^ $onet { -o "$ifname" }
-               fw $action $mode f ${chain}_MSSFIX TCPMSS  ^ $onet { -o "$ifname" -p tcp --tcp-flags SYN,RST SYN --clamp-mss-to-pmtu }
+               fw $action $mode f input   ${chain}         $ { -i "$ifname" $inet }
+               fw $action $mode f forward ${chain}_forward $ { -i "$ifname" $inet }
+               fw $action $mode n PREROUTING ${chain}_prerouting $ { -i "$ifname" $inet }
+               fw $action $mode r PREROUTING ${chain}_notrack    $ { -i "$ifname" $inet }
+               fw $action $mode n POSTROUTING ${chain}_nat       $ { -o "$ifname" $onet }
 
-               fw $action $mode f input   ${chain}         $ $inet { -i "$ifname" }
-               fw $action $mode f forward ${chain}_forward $ $inet { -i "$ifname" }
-               fw $action $mode n PREROUTING ${chain}_prerouting ^ $inet { -i "$ifname" }
-               fw $action $mode r PREROUTING ${chain}_notrack    ^ $inet { -i "$ifname" }
+               lock -u /var/run/firewall-interface.lock
        }
 
-       local old_zones old_ifname old_subnet
+       local old_zones old_ifname old_subnets
        config_get old_zones core "${iface}_zone"
        [ -n "$old_zones" ] && {
                config_get old_ifname core "${iface}_ifname"
-               config_get old_subnet core "${iface}_subnet"
+               config_get old_subnets core "${iface}_subnets"
 
                local z
                for z in $old_zones; do
-                       fw_log info "removing $iface ($old_ifname${old_subnet:+ alias $old_subnet}) from zone $z"
-                       fw__do_rules del $z $old_ifname $old_subnet
-
-                       [ -n "$old_subnet" ] || ACTION=remove ZONE="$z" INTERFACE="$iface" DEVICE="$ifname" /sbin/hotplug-call firewall
+                       local n
+                       for n in ${old_subnets:-""}; do
+                               fw_log info "removing $iface ($old_ifname${n:+ alias $n}) from zone $z"
+                               fw__do_rules del $z $old_ifname $n
+                       done
+
+                       [ -n "$old_subnets" ] || {
+                               fw__uci_state_del "${z}_networks" "$iface"
+                               env -i ACTION=remove ZONE="$z" INTERFACE="$iface" DEVICE="$ifname" /sbin/hotplug-call firewall
+                       }
                done
 
                local old_aliases
@@ -88,25 +139,12 @@ fw_configure_interface() {
 
                uci_revert_state firewall core "${iface}_zone"
                uci_revert_state firewall core "${iface}_ifname"
-               uci_revert_state firewall core "${iface}_subnet"
+               uci_revert_state firewall core "${iface}_subnets"
                uci_revert_state firewall core "${iface}_aliases"
        }
 
        [ "$action" == del ] && return
 
-       local new_zones=
-       load_zone() {
-               fw_config_get_zone "$1"
-               list_contains zone_network "$iface" || return
-
-               fw_log info "adding $iface ($ifname${aliasnet:+ alias $aliasnet}) to zone $zone_name"
-               fw__do_rules add ${zone_name} "$ifname" $aliasnet
-               append new_zones $zone_name
-
-               [ -n "$aliasnet" ] || ACTION=add ZONE="$zone_name" INTERFACE="$iface" DEVICE="$ifname" /sbin/hotplug-call firewall
-       }
-       config_foreach load_zone zone
-
        [ -z "$aliasnet" ] && {
                local aliases
                config_get aliases "$iface" aliases
@@ -118,19 +156,41 @@ fw_configure_interface() {
                        config_get netmask "$a" netmask
                        config_get ip6addr "$a" ip6addr
 
-                       [ -n "$ipaddr" ] && fw_configure_interface "$a" add "$ifname" "$ipaddr${netmask:+/$netmask}"
-                       [ -n "$ip6addr" ] && fw_configure_interface "$a" add "$ifname" "$ip6addr"
+                       [ -n "$ipaddr" ] && fw_configure_interface "$a" add "" "$ipaddr${netmask:+/$netmask}"
+                       [ -n "$ip6addr" ] && fw_configure_interface "$a" add "" "$ip6addr"
                done
 
                fw_sysctl_interface $ifname
                fw_callback post interface
 
-               uci_set_state firewall core "${iface}_aliases" "$aliases"
+               uci_toggle_state firewall core "${iface}_aliases" "$aliases"
+       } || {
+               local subnets=
+               config_get subnets core "${iface}_subnets"
+               append subnets "$aliasnet"
+
+               config_set core "${iface}_subnets" "$subnets"
+               uci_toggle_state firewall core "${iface}_subnets" "$subnets"
        }
 
-       uci_set_state firewall core "${iface}_zone" "$new_zones"
-       uci_set_state firewall core "${iface}_ifname" "$ifname"
-       uci_set_state firewall core "${iface}_subnet" "$aliasnet"
+       local new_zones=
+       load_zone() {
+               fw_config_get_zone "$1"
+               list_contains zone_network "$iface" || return
+
+               fw_log info "adding $iface ($ifname${aliasnet:+ alias $aliasnet}) to zone $zone_name"
+               fw__do_rules add ${zone_name} "$ifname" "$aliasnet"
+               append new_zones $zone_name
+
+               [ -n "$aliasnet" ] || {
+                       fw__uci_state_add "${zone_name}_networks" "${zone_network}"
+                       env -i ACTION=add ZONE="$zone_name" INTERFACE="$iface" DEVICE="$ifname" /sbin/hotplug-call firewall
+               }
+       }
+       config_foreach load_zone zone
+
+       uci_toggle_state firewall core "${iface}_zone" "$new_zones"
+       uci_toggle_state firewall core "${iface}_ifname" "$ifname"
 }
 
 fw_sysctl_interface() {
This page took 0.038096 seconds and 4 git commands to generate.