From 8f6f496c7e64fa50f72777d9ec290da85e77e722 Mon Sep 17 00:00:00 2001
From: John Sully <john@csquare.ca>
Date: Sun, 16 Feb 2020 17:08:00 -0500
Subject: [PATCH] Memory leak fix on config, and redisDb dtor

Former-commit-id: b92bbf4de8ffc3edc965e2f9da4dd82ed7071559
---
 src/config.cpp | 2 +-
 src/db.cpp     | 8 ++++++++
 src/server.h   | 7 +++++--
 3 files changed, 14 insertions(+), 3 deletions(-)

diff --git a/src/config.cpp b/src/config.cpp
index 2aaad825e..2c9a9d518 100644
--- a/src/config.cpp
+++ b/src/config.cpp
@@ -405,7 +405,7 @@ void loadServerConfigFromString(char *config) {
         } else if ((!strcasecmp(argv[0],"slaveof") ||
                     !strcasecmp(argv[0],"replicaof")) && argc == 3) {
             slaveof_linenum = linenum;
-            replicationAddMaster(sdsnew(argv[1]), atoi(argv[2]));
+            replicationAddMaster(argv[1], atoi(argv[2]));
         } else if ((!strcasecmp(argv[0],"repl-ping-slave-period") ||
                     !strcasecmp(argv[0],"repl-ping-replica-period")) &&
                     argc == 2)
diff --git a/src/db.cpp b/src/db.cpp
index 9190d22f1..e94e0cdb1 100644
--- a/src/db.cpp
+++ b/src/db.cpp
@@ -1305,6 +1305,14 @@ void setExpire(client *c, redisDb *db, robj *key, robj *subkey, long long when)
         rememberSlaveKeyWithExpire(db,key);
 }
 
+redisDb::~redisDb()
+{
+    dictRelease(watched_keys);
+    dictRelease(ready_keys);
+    dictRelease(blocking_keys);
+    listRelease(defrag_later);
+}
+
 void setExpire(client *c, redisDb *db, robj *key, expireEntry &&e)
 {
     dictEntry *kde;
diff --git a/src/server.h b/src/server.h
index 3ec2e8948..135a216ae 100644
--- a/src/server.h
+++ b/src/server.h
@@ -1127,10 +1127,13 @@ typedef struct clientReplyBlock {
 /* Redis database representation. There are multiple databases identified
  * by integers from 0 (the default database) up to the max configured
  * database. The database number is the 'id' field in the structure. */
-typedef struct redisDb {
+struct redisDb {
     redisDb() 
         : expireitr(nullptr)
     {};
+    
+    ~redisDb();
+
     dict *pdict;                 /* The keyspace for this DB */
     expireset *setexpire;
     expireset::setiter expireitr;
@@ -1142,7 +1145,7 @@ typedef struct redisDb {
     long long last_expire_set;  /* when the last expire was set */
     double avg_ttl;             /* Average TTL, just for stats */
     list *defrag_later;         /* List of key names to attempt to defrag one by one, gradually. */
-} redisDb;
+};
 
 /* Client MULTI/EXEC state */
 typedef struct multiCmd {