[sr-dev] git:master:f82a2563: ndb_redis: add result check when reconnecting

[Victor Seva]

Module: kamailio
Branch: master
Commit: f82a2563000361b6b1b12b4bb72b9a8ee4c62292
URL: https://github.com/kamailio/kamailio/commit/f82a2563000361b6b1b12b4bb72b9a8ee4c62292

Author: Victor Seva <linuxmaniac at torreviejawireless.org>
Committer: Victor Seva <linuxmaniac at torreviejawireless.org>
Date: 2021-03-02T15:20:44+01:00

ndb_redis: add result check when reconnecting

Reported and provided solution by @kritarthh

follow-up #2461

---

Modified: src/modules/ndb_redis/redis_client.c

---

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

---

diff --git a/src/modules/ndb_redis/redis_client.c b/src/modules/ndb_redis/redis_client.c
index 42880310f2..befb69ab79 100644
--- a/src/modules/ndb_redis/redis_client.c
+++ b/src/modules/ndb_redis/redis_client.c
@@ -1059,6 +1059,11 @@ int redisc_exec(str *srv, str *res, str *cmd, ...)
 			if(redisc_reconnect_server(rsrv)==0)
 			{
 				rpl->rplRedis = redisvCommand(rsrv->ctxRedis, cmd->s, ap4);
+				if(rpl->rplRedis == NULL)
+				{
+					redis_count_err_and_disable(rsrv);
+					goto error_exec;
+				}
 			} else {
 				LM_ERR("unable to reconnect to redis server: %.*s\n",
 						srv->len, srv->s);