diff options
author | Nathan Scott <nathans@redhat.com> | 2019-11-25 15:21:05 +1100 |
---|---|---|
committer | Nathan Scott <nathans@redhat.com> | 2019-11-25 15:21:05 +1100 |
commit | 42697bd45a226d0841189c5ee6d4987d6cd7f481 (patch) | |
tree | c9fc32068dcdfc4f5cb58210a608f57ca9ce5269 | |
parent | 5d0568d9ae49363ad21b512c52025e45bbf8d02e (diff) |
Fix dead code in sslLogCallback relating to should_log variable.
Coverity scans found that the should_log logic in sslLogCallback
is not working as expected because the variable is not correctly
initialised (the conditional code before logging always sets the
value to 1, which it already is).
-rw-r--r-- | ssl.c | 4 |
1 files changed, 2 insertions, 2 deletions
@@ -82,8 +82,8 @@ redisContextFuncs redisContextSSLFuncs; * Callback used for debugging */ static void sslLogCallback(const SSL *ssl, int where, int ret) { - const char *retstr = ""; - int should_log = 1; + const char *retstr; + int should_log = 0; /* Ignore low-level SSL stuff */ if (where & SSL_CB_ALERT) { |