[PATCH] Fall-through bugs in base/commands.c

Alexios Chouchoulas alexios at arkcomputing.com
Sat Nov 15 15:59:48 CET 2008


Folks,

  I've just discovered two minor (but annoying) bugs in commands.c: there are
two break statements missing, making CMD_ENABLE_ALL_NOTIFICATIONS_BEYOND_HOST
fail. Here's a pathetic little patch for the problem (against the latest code
in CVS):


diff -U3 -r nagios-orig/base/commands.c nagios/base/commands.c
--- nagios-orig/base/commands.c 2008-11-02 20:51:29.000000000 +0200
+++ nagios/base/commands.c      2008-11-15 16:24:57.000000000 +0200
@@ -1186,9 +1186,11 @@

        case CMD_ENABLE_ALL_NOTIFICATIONS_BEYOND_HOST:
                
enable_and_propagate_notifications(temp_host,0,FALSE,TRUE,TRUE);
+               break;

        case CMD_DISABLE_ALL_NOTIFICATIONS_BEYOND_HOST:
                
disable_and_propagate_notifications(temp_host,0,FALSE,TRUE,TRUE);
+               break;

        case CMD_ENABLE_HOST_SVC_NOTIFICATIONS:
        case CMD_DISABLE_HOST_SVC_NOTIFICATIONS:



Many thanks,
Alexios.


-- 
Alexios Chouchoulas
Ark Computing (www.arkcomputing.com)


-------------------------------------------------------------------------
This SF.Net email is sponsored by the Moblin Your Move Developer's challenge
Build the coolest Linux based applications with Moblin SDK & win great prizes
Grand prize is a trip for two to an Open Source event anywhere in the world
http://moblin-contest.org/redirect.php?banner_id=100&url=/




More information about the Developers mailing list