[SSSD] [PATCH] Fix warning unused variable ap_fallback

Lukas Slebodnik lslebodn at redhat.com
Fri Feb 21 10:44:38 UTC 2014


ehlo,

Yesterday, I spent a lot of time with building sssd in mock and did not notice
warning introduced by myself. I apologize for that.

The variable ap_fallback is used only if sssd is build with journald, but
this variable was declared outside of "#ifdef WITH_JOURNALD"

Patch is attached.

LS
-------------- next part --------------
>From f5d966b43288e53266bd2f8f100858c75f1ca5c0 Mon Sep 17 00:00:00 2001
From: Lukas Slebodnik <lslebodn at redhat.com>
Date: Fri, 21 Feb 2014 11:38:10 +0100
Subject: [PATCH] Fix warning unused variable ap_fallback

The variable ap_fallback is used only if sssd is build with journald, but
this variable was declared outside of "#ifdef WITH_JOURNALD"
---
 src/util/debug.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/util/debug.c b/src/util/debug.c
index 1db3693a11bef9d6dced6b085e320d99c4fe3384..b66de07b4e3a4549548f627f07a45f97068fa21b 100644
--- a/src/util/debug.c
+++ b/src/util/debug.c
@@ -203,7 +203,6 @@ void debug_fn(const char *file,
               const char *format, ...)
 {
     va_list ap;
-    va_list ap_fallback;
     struct timeval tv;
     struct tm *tm;
     char datetime[20];
@@ -211,6 +210,7 @@ void debug_fn(const char *file,
 
 #ifdef WITH_JOURNALD
     errno_t ret;
+    va_list ap_fallback;
 
     if (!debug_file) {
         /* If we are not outputting logs to files, we should be sending them
-- 
1.8.5.3



More information about the sssd-devel mailing list