better check during redisFree() in the rlm_redis
authorJorge Pereira <jpereiran@gmail.com>
Thu, 11 Jun 2015 19:18:58 +0000 (16:18 -0300)
committerArran Cudbard-Bell <a.cudbardb@freeradius.org>
Thu, 11 Jun 2015 19:54:41 +0000 (15:54 -0400)
src/modules/rlm_redis/rlm_redis.c

index c0dd08d..f8250c3 100644 (file)
@@ -42,7 +42,10 @@ static const CONF_PARSER module_config[] = {
 
 static int _mod_conn_free(REDISSOCK *dissocket)
 {
-       redisFree(dissocket->conn);
+       if (dissocket->conn) {
+               redisFree(dissocket->conn);
+               dissocket->conn = NULL;
+       }
 
        if (dissocket->reply) {
                freeReplyObject(dissocket->reply);
@@ -108,7 +111,6 @@ static void *mod_conn_create(TALLOC_CTX *ctx, void *instance)
                        goto do_close;
                }
 
-
                switch (reply->type) {
                case REDIS_REPLY_STATUS:
                        if (strcmp(reply->str, "OK") != 0) {