Introduce an _rc_check_name() function to check the input script name so that

we don't end up with cryptic error messages.
Regex help from rpe@
Issue reported by Anthony Coulter in rcctl(8), but better fix the root cause.

Also clarify the mage page that rc.d script name must follow ksh(1) variable
naming.

ok robert@ sthen@

1 files changed, 7 insertions, 3 deletions

diff --git a/share/man/man8/rc.subr.8 b/share/man/man8/rc.subr.8
index d72b3e4ea73..5b9c0bf1741 100644
--- a/share/man/man8/rc.subr.8
+++ b/share/man/man8/rc.subr.8
@@ -1,4 +1,4 @@
-.\" 	$OpenBSD: rc.subr.8,v 1.31 2016/04/26 18:30:22 jmc Exp $
+.\" 	$OpenBSD: rc.subr.8,v 1.32 2016/09/07 13:12:42 ajacoutot Exp $
 .\"
 .\" Copyright (c) 2011 Robert Nagy, Antoine Jacoutot, Ingo Schwarze
 .\" All rights reserved.
@@ -24,7 +24,7 @@
 .\" (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF
 .\" THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 .\"
-.Dd $Mdocdate: April 26 2016 $
+.Dd $Mdocdate: September 7 2016 $
 .Dt RC.SUBR 8
 .Os
 .Sh NAME
@@ -42,10 +42,14 @@ naming policy:
 .Pp
 .Bl -enum -compact
 .It
-Use the same name as the
+When possible, use the same name as the
 .Nm daemon
 it is referring to.
 .It
+It must follow
+.Xr ksh 1
+variable naming: begin with an alphabetic or underscore character, followed by
+one or more alphanumeric or underscore characters.
 Dashes
 .Pq Sq -
 have to be converted to