From a8f9cec16a79d17f72c30a95773ba572bc3eabf5 Mon Sep 17 00:00:00 2001
From: Jan-Erik Rediger <badboy@archlinux.us>
Date: Fri, 18 Jan 2013 10:13:10 +0100
Subject: [PATCH] Always exit if connection fails.

This avoids unnecessary core dumps. Fixes antirez/redis#894
---
 src/redis-cli.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/redis-cli.c b/src/redis-cli.c
index 0bc61bb2..a789ab13 100644
--- a/src/redis-cli.c
+++ b/src/redis-cli.c
@@ -1288,19 +1288,19 @@ int main(int argc, char **argv) {
 
     /* Latency mode */
     if (config.latency_mode) {
-        cliConnect(0);
+        if (cliConnect(0) == REDIS_ERR) exit(1);
         latencyMode();
     }
 
     /* Slave mode */
     if (config.slave_mode) {
-        cliConnect(0);
+        if (cliConnect(0) == REDIS_ERR) exit(1);
         slaveMode();
     }
 
     /* Get RDB mode. */
     if (config.getrdb_mode) {
-        cliConnect(0);
+        if (cliConnect(0) == REDIS_ERR) exit(1);
         getRDB();
     }
 
@@ -1312,7 +1312,7 @@ int main(int argc, char **argv) {
 
     /* Find big keys */
     if (config.bigkeys) {
-        cliConnect(0);
+        if (cliConnect(0) == REDIS_ERR) exit(1);
         findBigKeys();
     }
 
-- 
GitLab