rc.d startup script problem

[Paul Schmehl]

--On Wednesday, May 14, 2008 06:56:26 -0500 Brooks Davis <brooks at freebsd.org> 
wrote:

> On Tue, May 13, 2008 at 02:35:54PM -0500, Paul Schmehl wrote:
>> This script will not start if you change the name or location of the conf
>> file in /etc/rc.conf.  For some reason it's not parsing /etc/rc.conf.
>> Anyone know why?
>>
>> # !/bin/sh
>>
>> # PROVIDE: sancp_agent
>> # REQUIRE: DAEMON
>> # KEYWORD: shutdown
>>
>> # Add the following line to /etc/sguil-sensor/rc.conf to enable sancp_agent:
>> # sancp_agent_enable (bool):    Set to YES to enable sancp_agent
>> #                               Default: NO
>> # sancp_agent_conf (str):       Sensor_agent configuration file
>> #                               Default:
>> /usr/local/etc/sguil-sensor/sancp_agent.conf
>> #
>>
>> . /etc/rc.subr
>>
>> name="sancp_agent"
>> rcvar=`set_rcvar`
>> command="/usr/local/bin/sguil-sensor/sancp_agent.tcl"
>> procname="/usr/local/bin/tclsh8.4"
>> pidfile="/var/run/${name}.pid"
>> check_pidfile="${pidfile} ${procname} /bin/sh"
>>
>> [ -z "$sancp_agent_enable" ]    && sancp_agent_enable="NO"
>> [ -z "$sancp_agent_conf" ]      &&
>> sancp_agent_conf="/usr/local/etc/sguil-sensor/sancp_agent.conf"
>> [ -z "$sancp_agent_flags" ]     && sancp_agent_flags="-D"
>>
>> [ -n "$sancp_agent_conf" ]      && sancp_agent_flags="$sancp_agent_flags -c
>> $sancp_agent_conf"
>
> This section needs to go below load_rc_conf so the variables are
> reliably defined.

Doh!  Thanks Brooks.  I knew it had to be something I'd overlooked.  That one 
is dumb.

> Also, command_args should generally be used instead of ${name}_flags.

Thanks.  I'll start using that.

-- 
Paul Schmehl (pauls at utdallas.edu)
Senior Information Security Analyst
The University of Texas at Dallas
http://www.utdallas.edu/ir/security/