Bug in Dependencies

[Ethan Galstad]

Gerhard Lausser wrote:
> Hi Ethan,
> 
> i tested the current cvs code and it reported me (unjustified) circular
> dependencies. I think this patch will help:
> 
> Greetings from Munich,
> Gerhard 
> 
> --- nagios-cvs/common/objects.c 2006-09-18 19:03:50.000000000 +0200
> +++ nagios-consol/common/objects.c      2006-10-13 19:17:03.064383605 +0200
> @@ -4119,7 +4119,7 @@
>  
> for(temp_sd=servicedependency_list;temp_sd!=NULL;temp_sd=temp_sd->next){
>  
>                 /* only check parent dependencies */
> -               if(dep->master_service_ptr==temp_sd->dependent_service_ptr)
> +               if(dep->master_service_ptr!=temp_sd->dependent_service_ptr)
>                         continue;
>  
>  
> if(check_for_circular_servicedependency_path(root_dep,temp_sd,dependency_typ
> e)==TRUE)
>  

Thanks Gerhard!  I'll get this patch (and the corresponding one for host 
dependencies) in CVS shortly.


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