summaryrefslogtreecommitdiff
path: root/ssl.c
diff options
context:
space:
mode:
authorNathan Scott <nathans@redhat.com>2019-11-25 15:21:05 +1100
committerNathan Scott <nathans@redhat.com>2019-11-25 15:21:05 +1100
commit42697bd45a226d0841189c5ee6d4987d6cd7f481 (patch)
treec9fc32068dcdfc4f5cb58210a608f57ca9ce5269 /ssl.c
parent5d0568d9ae49363ad21b512c52025e45bbf8d02e (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).
Diffstat (limited to 'ssl.c')
-rw-r--r--ssl.c4
1 files changed, 2 insertions, 2 deletions
diff --git a/ssl.c b/ssl.c
index 78ab9e4..b97fc1d 100644
--- a/ssl.c
+++ b/ssl.c
@@ -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) {