Two NRPE bugs (and patches)

[Andrew Boyce-Lewis (Contractor)]

Sorry for the delay in writing back, things have ben quite busy.

As I said before, I made some minor changes to the NRPE source to  
address two bugs that I found.

1) If  openssl installed but in a non-standard location, configuring  
and making nrpe would fail generating DH parameters because the  
openssl binary could not be found. I added the configure time option  
--with-ssl for setting the install path of openssl. Also, as far as I  
can tell with-ssl-lib is never used, so I removed it. Finally, I  
beefed up the logic that it used to find both the include files and  
the openssl binary so that they are more likely to succeed if you  
have a non-standard installation.

2) Fixed a bug that caused NRPE to stop parsing the config file after  
a non-fatal error. This happened when an invalid config_dir was  
specified after sufficient configuration options were parsed. For  
example, I had all of the core options (port, user, group.... etc) at  
the top of my config, then an include_dir that specified a directory  
that did not exist and then all of the command definitions. The  
daemon would launch, report only that the config dir was not found  
yet it would have no checks available and report no other errors  
(because the checks were defined after the invalid include_dir).

I enabled some additional error output and allowed read_config_file()  
to keep going (with warnings) even if a non-existent include_dir or  
include_file are specified.

Feedback welcome.

  -Andrew

-------------- next part --------------
A non-text attachment was scrubbed...
Name: abl.patch
Type: application/octet-stream
Size: 2260 bytes
Desc: not available
URL: <https://www.monitoring-lists.org/archive/developers/attachments/20070202/9d1ff543/attachment.obj>
-------------- next part --------------

On Jan 29, 2007, at 1:57 PM, Ethan Galstad wrote:

> Andreas Ericsson wrote:
>> Andrew Boyce-Lewis (Contractor) wrote:
>>> Is this the correct list to send patches for NRPE? I've made some  
>>> minor
>>> usability enhancements to the configure process that were needed  
>>> for my
>>> site and they seem like good candidates to be released into the  
>>> wild.
>>>
>>
>> NRPE doesn't have a list of its own (or at least an active one),  
>> so here's
>> as good as anywhere, I guess.
>>
>> Fire away. :)
>>
>
> Yep, this is indeed the place to post patches for both NRPE and  
> NSCA.  I
> suppose I should update the docs in each addon to make a note of  
> this...
>
>
> Ethan Galstad,
> Nagios Developer
> ---
> Email: nagios at nagios.org
> Website: http://www.nagios.org
>
> ---------------------------------------------------------------------- 
> ---
> Take Surveys. Earn Cash. Influence the Future of IT
> Join SourceForge.net's Techsay panel and you'll get the chance to  
> share your
> opinions on IT & business topics through brief surveys - and earn cash
> http://www.techsay.com/default.php? 
> page=join.php&p=sourceforge&CID=DEVDEV
> _______________________________________________
> Nagios-devel mailing list
> Nagios-devel at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/nagios-devel

-------------- next part --------------
-------------------------------------------------------------------------
Using Tomcat but need to do more? Need to support web services, security?
Get stuff done quickly with pre-integrated technology to make your job easier.
Download IBM WebSphere Application Server v.1.0.1 based on Apache Geronimo
http://sel.as-us.falkag.net/sel?cmd=lnk&kid=120709&bid=263057&dat=121642
-------------- next part --------------
_______________________________________________
Nagios-devel mailing list
Nagios-devel at lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nagios-devel