Nagios Support Forum

--=-kh6P2l01oFz8qUbkzGwL
Content-Type: text/plain
Content-Transfer-Encoding: 7bit

Hi,

the attached diff removes two break statements that seem odd to me, is
this valid? would the break statement in the first function not cause it
to return 1 even though the number of immediate parent hosts could be
more?

--=-kh6P2l01oFz8qUbkzGwL
Content-Disposition: attachment; filename="objects.diff"
Content-Type: text/x-patch; name="objects.diff"; charset="UTF-8"
Content-Transfer-Encoding: 7bit

--- nagios/nagios/common/objects.c 2008-10-14 15:28:08.000000000 +0200
+++ /tmp/objects.c 2008-11-08 14:26:45.000000000 +0200
@@ -3082,7 +3082,6 @@
for(temp_host=host_list;temp_host!=NULL;temp_host=temp_host->next){
if(is_host_immediate_parent_of_host(hst,temp_host)==TRUE){
parents++;
- break;
}
}

@@ -3099,7 +3098,6 @@
for(temp_host=host_list;temp_host!=NULL;temp_host=temp_host->next){
if(is_host_immediate_parent_of_host(hst,temp_host)==TRUE){
parents+=number_of_total_parent_hosts(temp_host)+1;
- break;
}
}


--=-kh6P2l01oFz8qUbkzGwL--






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