Merge pull request #3463 from vladpaiu/fix_redis_tls_connection_bug

Fix fetching TLS domain for a redis connection
OpenSIPS · Sep 11, 2024 · 377d72b · 377d72b
2 parents 09faaca + b3e81a6
commit 377d72b
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/modules/cachedb_redis/cachedb_redis_dbase.c b/modules/cachedb_redis/cachedb_redis_dbase.c
@@ -97,7 +97,7 @@ static int redis_init_ssl(char *url_extra_opts, redisContext *ctx,
 	SSL *ssl;
 	struct tls_domain *d;
 
-	if (tls_dom == NULL) {
+	if (tls_dom == NULL || *tls_dom == NULL) {
 		if (strncmp(url_extra_opts, CACHEDB_TLS_DOM_PARAM,
 				CACHEDB_TLS_DOM_PARAM_LEN)) {
 			LM_ERR("Invalid Redis URL parameter: %s\n", url_extra_opts);
@@ -258,6 +258,8 @@ int redis_connect(redis_con *con)
 				freeReplyObject(rpl);
 			goto error;
 		}
+
+		memset(con->nodes,0,sizeof(cluster_node) + len + 1);
 		con->nodes->ip = (char *)(con->nodes + 1);
 
 		strcpy(con->nodes->ip,con->host);