aboutsummaryrefslogtreecommitdiffstats
path: root/package/firewall
diff options
context:
space:
mode:
authorJohn Crispin <john@openwrt.org>2008-08-15 20:18:13 +0000
committerJohn Crispin <john@openwrt.org>2008-08-15 20:18:13 +0000
commit7681547293dda1e0d93c203353b3eaab860dfa63 (patch)
treef34919f3ec689190b8758ef73f77b50c72bd1acf /package/firewall
parente7c9479ae4856a3b8b6b6e62a22489a039f26c4f (diff)
downloadupstream-7681547293dda1e0d93c203353b3eaab860dfa63.tar.gz
upstream-7681547293dda1e0d93c203353b3eaab860dfa63.tar.bz2
upstream-7681547293dda1e0d93c203353b3eaab860dfa63.zip
trigger error if dport is used when no proto is defined
SVN-Revision: 12317
Diffstat (limited to 'package/firewall')
-rwxr-xr-xpackage/firewall/files/uci_firewall.sh4
1 files changed, 4 insertions, 0 deletions
diff --git a/package/firewall/files/uci_firewall.sh b/package/firewall/files/uci_firewall.sh
index e1683e9cf0..27e9dd71d6 100755
--- a/package/firewall/files/uci_firewall.sh
+++ b/package/firewall/files/uci_firewall.sh
@@ -178,6 +178,8 @@ fw_rule() {
[ -z "$target" ] && target=DROP
[ -n "$src" ] && ZONE=zone_$src || ZONE=INPUT
[ -n "$dest" ] && TARGET=zone_${dest}_$target || TARGET=$target
+ [ -n "$dest_port" -a -z "$proto" ] && { \
+ echo "dport may only be used it proto is defined"; return; }
$IPTABLES -I $ZONE 1 \
${proto:+-p $proto} \
${src_ip:+-s $src_ip} \
@@ -220,6 +222,8 @@ fw_redirect() {
config_get protocol $1 protocol
[ -z "$src" -o -z "$dest_ip" ] && { \
echo "redirect needs src and dest_ip"; return ; }
+ [ -n "$dest_port" -a -z "$proto" ] && { \
+ echo "dport may only be used it proto is defined"; return; }
$IPTABLES -A zone_${src}_prerouting -t nat \
${protocol:+-p $protocol} \
${src_ip:+-s $srcdip} \