[SSSD] [PATCH] AD: Add a missing break statement to the GPO code

Jakub Hrozek jhrozek at redhat.com
Mon Sep 29 11:17:32 UTC 2014


Found by Coverity. Moreover, I think the DEBUG message shouldn't be so
loud.
-------------- next part --------------
>From fd56898b90db4f6c8149418df10309650f3188e2 Mon Sep 17 00:00:00 2001
From: Jakub Hrozek <jhrozek at redhat.com>
Date: Wed, 24 Sep 2014 16:44:15 +0200
Subject: [PATCH] AD: Add a missing break statement to the GPO code

---
 src/providers/ad/ad_gpo.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/providers/ad/ad_gpo.c b/src/providers/ad/ad_gpo.c
index de4d44166b85ccd85ed36bcb11f0596e0020af11..fad0101c66041fbebc5d41f20437c91cd15e26f1 100644
--- a/src/providers/ad/ad_gpo.c
+++ b/src/providers/ad/ad_gpo.c
@@ -1833,7 +1833,8 @@ ad_gpo_process_gpo_done(struct tevent_req *subreq)
     if (ret != EOK) {
         switch (ret) {
         case ENOENT:
-            DEBUG(SSSDBG_OP_FAILURE, "No GPOs available in cache\n");
+            DEBUG(SSSDBG_TRACE_FUNC, "No GPOs available in cache\n");
+            break;
         default:
             DEBUG(SSSDBG_FATAL_FAILURE,
                   "Could not delete stale GPOs from cache: [%s]\n",
-- 
1.9.3



More information about the sssd-devel mailing list