Do not launch a shell for each check

Gerhard Lausser Gerhard.Lausser at consol.de
Wed Sep 29 15:16:20 CEST 2010


I had some problems with command/service-definitions which use quotes. 
example: check_command check_something!"--param1 --param2"
Please don't forget to add single and double quotes to the list of special
characters.

if(!strpbrk(processed_command,"!$^&*()~[]|{};<>?`'\"")){

Gerhard 

> -----Ursprüngliche Nachricht-----
> Von: Matthieu Kermagoret [mailto:mkermagoret at merethis.com] 
> Gesendet: Mittwoch, 29. September 2010 13:37
> An: Nagios Developers List
> Betreff: Re: [Nagios-devel] Do not launch a shell for each check
> 
> On Tue, Jun 1, 2010 at 3:49 PM, Matthieu Kermagoret 
> <mkermagoret at merethis.com> wrote:
> > The patch I propose, handle simple commands with shell 
> quoting (simple 
> > and double quote). Every command containing any of these characters 
> > (escaped or not) will be handled by the shell --> !$^&*()~[]|{};<>?`
> > <--
> >
> > So any feedback on this new proposal ?
> >
> 
> Could it somehow make it into HEAD ? It would resolve issue #86.
> 
> Best regards,
> 
> --
> Matthieu KERMAGORET | Développeur
> 
> mkermagoret at merethis.com
> 
> MERETHIS est éditeur du logiciel Centreon.
> 
> --------------------------------------------------------------
> ----------------
> Start uncovering the many advantages of virtual appliances 
> and start using them to simplify application deployment and 
> accelerate your shift to cloud computing.
> http://p.sf.net/sfu/novell-sfdev2dev
> _______________________________________________
> Nagios-devel mailing list
> Nagios-devel at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/nagios-devel
> 


------------------------------------------------------------------------------
Start uncovering the many advantages of virtual appliances
and start using them to simplify application deployment and
accelerate your shift to cloud computing.
http://p.sf.net/sfu/novell-sfdev2dev




More information about the Developers mailing list