Re: [Nagios-devel] [PATCH] NDO - Check for configuration

Support forum for Nagios Core, Nagios Plugins, NCPA, NRPE, NSCA, NDOUtils and more. Engage with the community of users including those using the open source solutions.
Locked
Guest

Re: [Nagios-devel] [PATCH] NDO - Check for configuration

Post by Guest »

On 01/07/2011 10:08 AM, Michael Friedrich wrote:
> Ricardo Maraschini wrote:
>> Hello,
>>
>> I don't know if this is the right list for NDO, but i'll learn it on the hard way, though.
>> I installed NDO into a CentOS, and when i started Nagios i received the following message at nagios.log:
>>
>> ndomod: Could not open data sink! I'll keep trying, but some output may get lost...
>>
>> I figured out that nagios' user couldn't access NDO configuration file due to a permission issue. It took me some time to discover the source of the problem, so i thought that maybe an accurate error message can help on this.
>>
>> I've applied the patch below against ndoutils-1.4b9.
>>
>>
>> --- ndomod.c 2009-10-27 01:08:29.000000000 -0700
>> +++ ndomod.c-modified 2011-01-05 03:40:08.000000000 -0800
>> @@ -330,10 +330,15 @@
>> ndo_mmapfile *thefile=NULL;
>> char *buf=NULL;
>> int result=NDO_OK;
>> + char temp_buffer[NDOMOD_MAX_BUFLEN];
>>
>> /* open the file */
>> - if((thefile=ndo_mmap_fopen(filename))==NULL)
>> + if((thefile=ndo_mmap_fopen(filename))==NULL) {
>> + snprintf(temp_buffer,sizeof(temp_buffer)-1,"ndomod: Unable to open configuration file, please check permissions.\n");
>
> it would make sense addig the filename over here too. just as
>
> snprintf(temp_buffer,sizeof(temp_buffer)-1,"idomod: Unable to open
> configuration file, please check permissions on %s\n", filename);
>

It would make even more sense to use the value of errno instead of just
randomly guessing why it failed.

--
Andreas Ericsson [email protected]
OP5 AB www.op5.se
Tel: +46 8-230225 Fax: +46 8-230231

Considering the successes of the wars on alcohol, poverty, drugs and
terror, I think we should give some serious thought to declaring war
on peace.





This post was automatically imported from historical nagios-devel mailing list archives
Original poster: [email protected]
Locked