Set the sysctl(8) value in the same shell, not a subshell. This was

causing calls to netoptions_init() to not properly set a global variable,
which ended up being in the parent shell.

etc/rc.d/netoptions

@@ -36,12 +36,12 @@ case ${log_in_vain} in
 	;;
 esac
 
-[ "${log_in_vain}" -ne 0 ] && (
+if [ "${log_in_vain}" -ne 0 ]; then
 	netoptions_init
 	echo -n " log_in_vain=${log_in_vain}"
 	sysctl net.inet.tcp.log_in_vain="${log_in_vain}" >/dev/null
 	sysctl net.inet.udp.log_in_vain="${log_in_vain}" >/dev/null
-)
+fi
 
 case ${tcp_extensions} in
 [Yy][Ee][Ss] | '')