Make the 'restart' command work. Otherwise, it would successfully

stop ipfw, but not enable it again.

Aesthetic changes
	o Use positve logic (instead of negative)
	o create a 'stop' function, rather than putting the
	  commands in the stop_cmd variable.

Submitted by:	des
Approved by:	markm (mentor) (implicit)
This commit is contained in:
Mike Makonnen 2003-03-30 15:52:18 +00:00
parent d2a0822e9d
commit 1d89dde13e
Notes: svn2git 2020-12-20 02:59:44 +00:00
svn path=/head/; revision=112849

View File

@ -14,7 +14,7 @@ name="ipfw"
rcvar="firewall_enable"
start_cmd="ipfw_start"
start_precmd="ipfw_precmd"
stop_cmd="${SYSCTL_W} net.inet.ip.fw.enable=0"
stop_cmd="ipfw_stop"
ipfw_precmd()
{
@ -60,10 +60,21 @@ ipfw_start()
# Firewall logging
#
! checkyesno firewall_logging && return 0
if checkyesno firewall_logging; then
echo 'Firewall logging enabled'
sysctl net.inet.ip.fw.verbose=1 >/dev/null
fi
echo 'Firewall logging=YES'
sysctl net.inet.ip.fw.verbose=1 >/dev/null
# Enable the firewall
#
${SYSCTL_W} net.inet.ip.fw.enable=1
}
ipfw_stop()
{
# Disable the firewall
#
${SYSCTL_W} net.inet.ip.fw.enable=0
}
load_rc_config $name