patch for unscheduling triggered downtime

Ethan Galstad nagios at nagios.org
Tue Aug 1 22:36:29 CEST 2006


Good catch Graham!  This is indeed a bug.  The patch will be applied to 
CVS shortly.  Thanks.

Graham Collinson wrote:
> There appears to be an error in common/downtime.c for unscheduling 
> downtimes triggered by the downtime currently being unscheduled:
> 
> diff -r -U 3 nagios-tofix/common/downtime.c nagios-fixed/common/downtime.c
> --- nagios-tofix/common/downtime.c    2006-07-30 19:56: 22.000000000 +0100
> +++ nagios-fixed/common/downtime.c    2006-07-30 19:57:32.000000000 +0100
> @@ -214,7 +214,7 @@
>      /* unschedule all downtime entries that were triggered by this one */
>      
> for(temp_downtime=scheduled_downtime_list;temp_downtime!=NULL;temp_downtime=temp_downtime->next){ 
> 
>          if(temp_downtime->triggered_by==downtime_id)
> -            unschedule_downtime(ANY_DOWNTIME,downtime_id);
> +            unschedule_downtime(ANY_DOWNTIME,temp_downtime->downtime_id);
>              }
>  
>      return OK;
> 
> 
> or perhaps I misunderstood?
> 
> 
> patch is against latest cvs source
> 
> cheers,
> Graham


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




More information about the Developers mailing list