>Number:         19002
>Category:       pkg
>Synopsis:       cyrus rc script has bad behaviors
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    pkg-manager
>State:          open
>Class:          sw-bug
>Submitter-Id:   net
>Arrival-Date:   Sun Nov 10 02:53:00 PST 2002
>Closed-Date:
>Last-Modified:
>Originator:     Frank Cusack
>Release:        NetBSD 1.6
>Organization:
>Environment:
System: NetBSD moltar.savecore.net 1.6 NetBSD 1.6 (moltar) #14: Sat Nov 9 04:25:50 PST 2002 root@moltar.savecore.net:/usr/src/sys/arch/sparc64/compile/moltar sparc64
Architecture: sparc64
Machine: sparc64
>Description:
	1) stdin/stdout/stderr should be redirected to /dev/null.  I
	   won't go into why this should be, I'm sure it's well understood.
	2) The 'sleep 2' bit causes master to receive SIGHUP after 2s.
	   It doesn't seem to have any real function, but master
	   receiving an unnecessary SIGHUP *is* bad, so removing it seems
	   to be the easiest fix.
>How-To-Repeat:
	/etc/rc.d/cyrus restart
>Fix:
--- cyrus.orig	Sun Nov 10 01:57:23 2002
+++ cyrus	Sun Nov 10 02:24:14 2002
@@ -17,7 +17,7 @@
 name="cyrus"
 rcvar=$name
 command="/usr/pkg/cyrus/bin/master"
-command_args="& sleep 2"
+command_args="</dev/null >/dev/null 2>&1 &"
 required_files="/usr/pkg/etc/cyrus.conf /usr/pkg/etc/imapd.conf"
 extra_commands="mkimap reload"
 
>Release-Note:
>Audit-Trail:
>Unformatted: