Page 1 of 1

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

Posted: Fri Jan 07, 2011 10:59 am
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]