[sr-dev] git:5.3:f35c7fe0: db_redis: clean previous connection before trying to reconnect

[Victor Seva]

Module: kamailio
Branch: 5.3
Commit: f35c7fe0016fea44bbfa4d686f985cf5db4a3372
URL: https://github.com/kamailio/kamailio/commit/f35c7fe0016fea44bbfa4d686f985cf5db4a3372

Author: Victor Seva <vseva at sipwise.com>
Committer: Victor Seva <linuxmaniac at torreviejawireless.org>
Date: 2022-03-15T08:50:54+01:00

db_redis: clean previous connection before trying to reconnect

(cherry picked from commit 78c8d084e30373aacc456fd41062b68e6e82ea6f)
(cherry picked from commit 2e0537ba306219df0ce71605848e811027c69a57)
(cherry picked from commit 78506f7f2dfcdbf2bc9e21168929437516a368bd)

---

Modified: src/modules/db_redis/redis_connection.c

---

Diff:  https://github.com/kamailio/kamailio/commit/f35c7fe0016fea44bbfa4d686f985cf5db4a3372.diff
Patch: https://github.com/kamailio/kamailio/commit/f35c7fe0016fea44bbfa4d686f985cf5db4a3372.patch

---

diff --git a/src/modules/db_redis/redis_connection.c b/src/modules/db_redis/redis_connection.c
index 5a5887d658..bb7423fcbd 100644
--- a/src/modules/db_redis/redis_connection.c
+++ b/src/modules/db_redis/redis_connection.c
@@ -111,6 +111,11 @@ int db_redis_connect(km_redis_con_t *con) {
     db = atoi(con->id->database);
     reply = NULL;
 
+    if(con->con) {
+        LM_DBG("free old connection first\n");
+        redisFree(con->con);
+    }
+
     // TODO: introduce require_master mod-param and check if we're indeed master
     // TODO: on carrier, if we have db fail-over, the currently connected
     // redis server will become slave without dropping connections?