conf/161710: SSHD starts on boot even if sshd_enable=" NO" in 8.2

[Dimitry Andric]

The following reply was made to PR conf/161710; it has been noted by GNATS.

From: Dimitry Andric <dim at FreeBSD.org>
To: bug-followup at FreeBSD.org, freebsd.org at logicalsolutns.com
Cc:  
Subject: Re: conf/161710: SSHD starts on boot even if  sshd_enable=&quot;NO&quot;
 in 8.2
Date: Sat, 29 Oct 2011 01:30:45 +0200

 This is a multi-part message in MIME format.
 --------------050401080107000606050906
 Content-Type: text/plain; charset=ISO-8859-1; format=flowed
 Content-Transfer-Encoding: 7bit
 
 Aha, now I see the cause of the problem.  A similar case to yours was
 actually reported in conf/156427:
 
    http://www.freebsd.org/cgi/query-pr.cgi?pr=156427
 
 There is an issue in /etc/rc.subr, which was introduced accidentally in
 r206686, fixed in head in r220760, and later merged to stable/8 in
 r220958:
 
    http://svnweb.freebsd.org/base?view=revision&revision=220958
 
 but this was after 8.2 released, unfortunately.
 
 You should be able to fix it by applying just the diff to rc.subr, which
 I have attached for your convenience.
 
 --------------050401080107000606050906
 Content-Type: text/x-diff;
  name="pr156427.diff"
 Content-Transfer-Encoding: 7bit
 Content-Disposition: attachment;
  filename="pr156427.diff"
 
 --- etc/rc.subr	2011/04/22 23:42:23	220957
 +++ etc/rc.subr	2011/04/22 23:44:25	220958
 @@ -651,7 +651,8 @@
  					#	checkyesno ${rcvar}
  					# and return if that failed
  					#
 -		if [ -n "${rcvar}" -a "$rc_arg" != "rcvar" -a -z "${rc_pid}" ]; then
 +		if [ -n "${rcvar}" -a "$rc_arg" != "rcvar" -a "$rc_arg" != "stop" ] ||
 +		    [ -n "${rcvar}" -a "$rc_arg" = "stop" -a -z "${rc_pid}" ]; then
  			if ! checkyesno ${rcvar}; then
  				if [ -n "${rc_quiet}" ]; then
  					return 0
 
 --------------050401080107000606050906--