Browse Source

Merge pull request #1444 from grumvalski/redis_server_len_fix

ndb_redis: fix check on server name len when adding a new server to t…
Federico Cabiddu 7 năm trước cách đây
mục cha
commit
62b00fc883

+ 1 - 1
src/modules/ndb_redis/redis_client.c

@@ -707,7 +707,7 @@ int check_cluster_reply(redisReply *reply, redisc_server_t **rsrv)
 						"name=%.*s;addr=%.*s;port=%i", name.len, name.s,
 						"name=%.*s;addr=%.*s;port=%i", name.len, name.s,
 						addr.len, addr.s, port);
 						addr.len, addr.s, port);
 
 
-				if(server_len>0 || server_len>sizeof(spec_new) - 1) {
+				if(server_len<0 || server_len>sizeof(spec_new) - 1) {
 					LM_ERR("failed to print server spec string\n");
 					LM_ERR("failed to print server spec string\n");
 					return 0;
 					return 0;
 				}
 				}