aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAndreas Schneider <asn@samba.org>2017-07-18 10:12:39 +0200
committerAndreas Schneider <asn@samba.org>2018-08-28 11:46:10 +0200
commitc7777b96c52a4d87543a6ffdf941bae1a44b12b5 (patch)
treef1fa47f2e35f2bf28b7bb45a5c57d1af01fc81c9
parentc87f179b8ec238820c913ca0cf8d852603e8f0b7 (diff)
downloadresolv_wrapper-c7777b96c52a4d87543a6ffdf941bae1a44b12b5.tar.gz
resolv_wrapper-c7777b96c52a4d87543a6ffdf941bae1a44b12b5.tar.xz
resolv_wrapper-c7777b96c52a4d87543a6ffdf941bae1a44b12b5.zip
rwrap: Do an early return if log level doesn't match
Signed-off-by: Andreas Schneider <asn@samba.org> Reviewed-by: Ralph Boehme <slow@samba.org>
-rw-r--r--src/resolv_wrapper.c48
1 files changed, 25 insertions, 23 deletions
diff --git a/src/resolv_wrapper.c b/src/resolv_wrapper.c
index 1c71447..0b05ea5 100644
--- a/src/resolv_wrapper.c
+++ b/src/resolv_wrapper.c
@@ -100,33 +100,35 @@ static void rwrap_log(enum rwrap_dbglvl_e dbglvl,
lvl = atoi(d);
}
+ if (lvl < dbglvl) {
+ return;
+ }
+
va_start(va, format);
vsnprintf(buffer, sizeof(buffer), format, va);
va_end(va);
- if (lvl >= dbglvl) {
- switch (dbglvl) {
- case RWRAP_LOG_ERROR:
- fprintf(stderr,
- "RWRAP_ERROR(%d) - %s: %s\n",
- pid, func, buffer);
- break;
- case RWRAP_LOG_WARN:
- fprintf(stderr,
- "RWRAP_WARN(%d) - %s: %s\n",
- pid, func, buffer);
- break;
- case RWRAP_LOG_DEBUG:
- fprintf(stderr,
- "RWRAP_DEBUG(%d) - %s: %s\n",
- pid, func, buffer);
- break;
- case RWRAP_LOG_TRACE:
- fprintf(stderr,
- "RWRAP_TRACE(%d) - %s: %s\n",
- pid, func, buffer);
- break;
- }
+ switch (dbglvl) {
+ case RWRAP_LOG_ERROR:
+ fprintf(stderr,
+ "RWRAP_ERROR(%d) - %s: %s\n",
+ pid, func, buffer);
+ break;
+ case RWRAP_LOG_WARN:
+ fprintf(stderr,
+ "RWRAP_WARN(%d) - %s: %s\n",
+ pid, func, buffer);
+ break;
+ case RWRAP_LOG_DEBUG:
+ fprintf(stderr,
+ "RWRAP_DEBUG(%d) - %s: %s\n",
+ pid, func, buffer);
+ break;
+ case RWRAP_LOG_TRACE:
+ fprintf(stderr,
+ "RWRAP_TRACE(%d) - %s: %s\n",
+ pid, func, buffer);
+ break;
}
}