Caught by RHEL Coverity scans.

Luckily all the bug would have caused is a spurious DEBUG message.
>From ab11c0a6d32ae3981af6bc703d5a7460d7d42718 Mon Sep 17 00:00:00 2001
From: Jakub Hrozek <jhro...@redhat.com>
Date: Thu, 30 May 2013 13:28:59 +0200
Subject: [PATCH] monitor: Add forgotten break

---
 src/monitor/monitor.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/src/monitor/monitor.c b/src/monitor/monitor.c
index 
8098a583068f3d2b483972b0c6119e1508c862d3..cf8479c4b4e0b9c52ae6862793d6daef4edf353d
 100644
--- a/src/monitor/monitor.c
+++ b/src/monitor/monitor.c
@@ -516,6 +516,7 @@ static void tasks_check_handler(struct tevent_context *ev,
 
     case ENXIO:
         DEBUG(1,("Child (%s) not responding! (yet)\n", svc->name));
+        break;
 
     default:
         /* TODO: should we tear it down ? */
-- 
1.8.2.1

_______________________________________________
sssd-devel mailing list
sssd-devel@lists.fedorahosted.org
https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Reply via email to