Typo in rc.d/netoptions?

Hello Current, 

	I was wondering if there is a typo in rc.d/netoptions that has inverted logic 
for tcp_extensions and missing the banner before changing the ipv4 to 6 
mapping? I think I have patched it, but would like a second opinion.

Peg

--- /usr/src/etc/rc.d/netoptions        2009-09-13 00:02:18.984185765 +0100
+++ /etc/rc.d/netoptions        2009-09-29 00:02:46.210831591 +0100
_at__at_ -32,7 +32,7 _at__at_
                ${SYSCTL_W} net.inet.udp.log_in_vain="${log_in_vain}" 
>/dev/null
        fi

-       if checkyesno tcp_extensions; then
+       if ! checkyesno tcp_extensions; then
                netoptions_init
                echo -n ' rfc1323 extensions=NO'
                ${SYSCTL_W} net.inet.tcp.rfc1323=0 >/dev/null
_at__at_ -69,6 +69,7 _at__at_
        if checkyesno ipv6_ipv4mapping; then
                ${SYSCTL_W} net.inet6.ip6.v6only=0 >/dev/null
        else
+               netoptions_init
                echo -n " no-ipv4-mapped-ipv6"
                ${SYSCTL_W} net.inet6.ip6.v6only=1 >/dev/null
        fi