summaryrefslogtreecommitdiff
path: root/package/network/config/firewall
diff options
context:
space:
mode:
authorJohn Crispin <john@openwrt.org>2014-08-25 06:35:50 +0000
committerJohn Crispin <john@openwrt.org>2014-08-25 06:35:50 +0000
commit2ae05c57f8d4cb557a5ea733c2795b8462f00062 (patch)
tree8f54c6b5b6868ff35369f650de1a01f7eef4a296 /package/network/config/firewall
parentfdc41e2fd79dac12f3c18aadef077f0f140f464c (diff)
downloadmtk-20170518-2ae05c57f8d4cb557a5ea733c2795b8462f00062.zip
mtk-20170518-2ae05c57f8d4cb557a5ea733c2795b8462f00062.tar.gz
mtk-20170518-2ae05c57f8d4cb557a5ea733c2795b8462f00062.tar.bz2
package/*: remove useless explicit set of function returncode
somebody started to set a function returncode in the validation stuff and everybody copies it, e.g. myfunction() { fire_command return $? } a function automatically returns with the last returncode, so we can safely remove the command 'return $?'. reference: http://tldp.org/LDP/abs/html/exit-status.html "The last command executed in the function or script determines the exit status." Signed-off-by: Bastian Bittorf <bittorf@bluebottle.com> SVN-Revision: 42278
Diffstat (limited to 'package/network/config/firewall')
-rwxr-xr-xpackage/network/config/firewall/files/firewall.init4
1 files changed, 0 insertions, 4 deletions
diff --git a/package/network/config/firewall/files/firewall.init b/package/network/config/firewall/files/firewall.init
index 662dd80..ee3ed1a 100755
--- a/package/network/config/firewall/files/firewall.init
+++ b/package/network/config/firewall/files/firewall.init
@@ -15,8 +15,6 @@ validate_firewall_redirect()
'dest_ip:cidr' \
'dest_port:or(port, portrange)' \
'target:or("SNAT", "DNAT")'
-
- return $?
}
validate_firewall_rule()
@@ -28,8 +26,6 @@ validate_firewall_rule()
'src_port:or(port, portrange)' \
'dest_port:or(port, portrange)' \
'target:string'
-
- return $?
}
service_triggers() {