From: Phil Mayers <p.mayers@imperial.ac.uk>
Date: Wed, 14 Nov 2012 17:30:39 +0000 (+0000)
Subject: don't finish the query, our caller already does that
X-Git-Tag: release_2_2_1~217^2
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=refs%2Fpull%2F130%2Fhead;p=thirdparty%2Ffreeradius-server.git

don't finish the query, our caller already does that
---

diff --git a/src/modules/rlm_redis/rlm_redis.c b/src/modules/rlm_redis/rlm_redis.c
index 2eb778d5e0..bddb5eab3a 100644
--- a/src/modules/rlm_redis/rlm_redis.c
+++ b/src/modules/rlm_redis/rlm_redis.c
@@ -413,10 +413,6 @@ int rlm_redis_query(REDISSOCK *dissocket, REDIS_INST *inst, char *query)
 	if (dissocket->reply->type == REDIS_REPLY_ERROR) {
 		radlog(L_ERR, "rlm_redis (%s): query failed, %s",
 		       inst->xlat_name, query);
-
-		/* Free the reply just in case */
-		rlm_redis_finish_query(dissocket);
-
 		return -1;
 	}