diff options
author | Andreas Schneider <asn@samba.org> | 2017-07-18 10:12:39 +0200 |
---|---|---|
committer | Andreas Schneider <asn@samba.org> | 2018-08-28 11:46:10 +0200 |
commit | c7777b96c52a4d87543a6ffdf941bae1a44b12b5 (patch) | |
tree | f1fa47f2e35f2bf28b7bb45a5c57d1af01fc81c9 | |
parent | c87f179b8ec238820c913ca0cf8d852603e8f0b7 (diff) | |
download | resolv_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.c | 48 |
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; } } |