[package] firewall:

	- fix possible endless loop when the family option is used for forwardings
	- only generate forwarding rules in SNAT redirect sections if src_dip is specified

SVN-Revision: 22938

diff --git a/package/firewall/files/lib/core_redirect.sh b/package/firewall/files/lib/core_redirect.sh
index 913f963562d4b6885506030393531d92bbf5d506..2f0e38f393c41d9a65d27f8456c68a3f74507a56 100644 (file)
--- a/package/firewall/files/lib/core_redirect.sh
+++ b/package/firewall/files/lib/core_redirect.sh
@@ -31,13 +31,15 @@ fw_load_redirect() {
                fw_die "redirect ${redirect_name}: needs src and dest_ip or dest_port"
        }
 
-       local chain destopt
+       local chain destopt destaddr
        if [ "$redirect_target" == "DNAT" ]; then
                chain="zone_${redirect_src}_prerouting"
                destopt="--to-destination"
+               destaddr="$redirect_dest_ip"
        elif [ "$redirect_target" == "SNAT" ]; then
                chain="zone_${redirect_src}_nat"
                destopt="--to-source"
+               destaddr="$redirect_src_dip"
        else
                fw_die "redirect ${redirect_name}: target must be either DNAT or SNAT"
        fi
@@ -65,9 +67,9 @@ fw_load_redirect() {
                        $destopt ${redirect_dest_ip}${redirect_dest_port:+:$nat_dest_port} \
                }
 
-               [ -n "$redirect_dest_ip" ] && \
+               [ -n "$destaddr" ] && \
                fw add $mode f zone_${redirect_src}_forward ACCEPT ^ { $redirect_src_ip $redirect_dest_ip } { \
-                       -d $redirect_dest_ip \
+                       -d $destaddr \
                        ${redirect_proto:+-p $redirect_proto} \
                        ${redirect_src_ip:+-s $redirect_src_ip/$redirect_src_ip_prefixlen} \
                        ${redirect_src_port:+--sport $redirect_src_port} \

diff --git a/package/firewall/files/lib/fw.sh b/package/firewall/files/lib/fw.sh
index 819aa48eae87f8efdc4100fc2aebe8714554f2eb..aaf3d14ef02f52769f5231737b5cb1798a16f28c 100644 (file)
--- a/package/firewall/files/lib/fw.sh
+++ b/package/firewall/files/lib/fw.sh
@@ -149,7 +149,7 @@ fw__exec() { # <action> <family> <table> <chain> <target> <position> { <rules> }
        fi
 
        case "$fam" in
-               G*) shift; while [ "$1" != "{" ]; do shift; done ;;
+               G*) shift; while [ $# -gt 0 ] && [ "$1" != "{" ]; do shift; done ;;
        esac
 
        if [ $# -gt 0 ]; then