GitList - GitList

Browse code

ndb_redis: fix SIGSEGV in redisc_check_auth

- Added the check for NULL pointer. When TCP session to redis reset,
redisCommand function can return NULL pointer.

(cherry picked from commit c32a0be89cc92a0649277774d1e7aebf349b8b49)

Dennis Yurasov authored on 24/08/2021 14:15:24 • Daniel-Constantin Mierla committed on 09/09/2021 08:30:30
Showing 1 changed files

src/modules/ndb_redis/redis_client.c index befb69ab79..bb6adc20b1 100644

src/modules/ndb_redis/redis_client.c

History View file @ 5dbf99eb03

@@ -1269,6 +1269,10 @@ int redisc_check_auth(redisc_server_t *rsrv, char *pass)
                      	int retval = 0;
                      	reply = redisCommand(rsrv->ctxRedis, "AUTH %s", pass);
                     +	if(!reply) {
                     +		LM_ERR("Redis authentication error\n");
                     +		return -1;
                     +	}
                      	if (reply->type == REDIS_REPLY_ERROR) {
                      		LM_ERR("Redis authentication error\n");
                      		retval = -1;