Problem init.d script

Morten Guldager morten.guldager at gmail.com
Wed May 7 14:57:44 CEST 2008


On Wed, May 7, 2008 at 2:32 PM, Max <perldork at webwizarddesign.com> wrote:

> On Wed, May 7, 2008 at 7:17 AM, Morten Guldager
> <morten.guldager at gmail.com> wrote:
> > 'Aloha!
> >
> > I have found what looks like a bug in the file nagios-3.0.1/
> daemon-init.in
> > Find the function killproc_nagios
> >
> > Looks like something is missing.... At least mine looks like this
> straight
> > form the tar-ball:
> >
> > killproc_nagios ()
> > {
> >
> >         kill $2 $NagiosPID
> >
> > }
> >
> > What do you think?
>
> it is strange code :) .. $1 looks like it is always nagios and it is
> always ignored, $2 is the signal to send and that is used properly ..
>
> example from script ..
>
>  if status_nagios > /dev/null; then
>                    printf "Reloading nagios configuration..."
>                    killproc_nagios nagios -HUP
>                    echo "done"
>                else
>
> the script could use some syntax clean up for sure :) but that
> function looks like it works correctly even though it is strange to
> have kill_proc always passed an argument that is ignored.


Agreed! (after reading the code a few times more...)


/Morten %-)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://www.monitoring-lists.org/archive/developers/attachments/20080507/1e5be36d/attachment.html>
-------------- next part --------------
-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
-------------- next part --------------
_______________________________________________
Nagios-devel mailing list
Nagios-devel at lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nagios-devel


More information about the Developers mailing list