From 40c8c483273724bbc6033a0ebebefda00fed69bc Mon Sep 17 00:00:00 2001 From: Guillem Jover Date: Thu, 22 Aug 2019 22:17:13 +0200 Subject: [PATCH] TT#65100 Fix double free for redis context on closure We need to set the redis context to NULL after freeing it, otherwise other code will try to free it again, which will make the program abort when exiting. Change-Id: Id634075344351eb1c924c59739b72bbf57de3c89 (cherry picked from commit 781b275295d07cf9df4240e4c7d6d789a6b4a791) (cherry picked from commit 9d320ab400d6f3e36ced468487cacdd60a8b1087) --- daemon/redis.c | 1 + 1 file changed, 1 insertion(+) diff --git a/daemon/redis.c b/daemon/redis.c index ac606fe92..f94986a67 100644 --- a/daemon/redis.c +++ b/daemon/redis.c @@ -672,6 +672,7 @@ err: static void redis_close(struct redis *r) { if (r->ctx) redisFree(r->ctx); + r->ctx = NULL; mutex_destroy(&r->lock); g_slice_free1(sizeof(*r), r); }