free failed-connection context if redisAsyncConnect() returns one

sflow · Jun 1, 2023 · 2f795bb · 2f795bb
1 parent 1696a0f
commit 2f795bb
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/Linux/mod_sonic.c b/src/Linux/mod_sonic.c
@@ -724,7 +724,10 @@ extern "C" {
     }
     char *errm = ctx ? ctx->errstr : "ctx=NULL";
     myDebug(1, "sonic db_connectClient failed (fd=%d) err=%s", fd, errm);
-    // No more cleanup required here (calling redisAsyncFree(ctx) triggers SIGSEGV)
+    if(ctx) {
+      myDebug(1, "Connection failed but context still returned - calling redisAsyncFree(ctx)");
+      redisAsyncFree(ctx);
+    }
     return NO;
   }