Revert "glusterfsd: Memleak in glusterfsd process while brick mux is on"

Updates: bz#1582286
This reverts commit 7c3cc485054e4ede1efb358552135b432fb7047a.
Change-Id: I831d646112bcfa13d0c2153482ad00ff1b23aa6c
Signed-off-by: Mohit Agrawal <moagrawa@redhat.com>

1 files changed, 10 insertions, 5 deletions

diff --git a/xlators/features/changetimerecorder/src/changetimerecorder.c b/xlators/features/changetimerecorder/src/changetimerecorder.c
index 40bfcf4a32a..e97e3ad6cfa 100644
--- a/xlators/features/changetimerecorder/src/changetimerecorder.c
+++ b/xlators/features/changetimerecorder/src/changetimerecorder.c
@@ -19,6 +19,7 @@
 #include "tier-ctr-interface.h"
 
 /*******************************inode forget***********************************/
+
 int
 ctr_forget (xlator_t *this, inode_t *inode)
 {
@@ -2305,6 +2306,15 @@ notify (xlator_t *this, int event, void *data, ...)
        if (!priv)
                goto out;
 
+       if (event == GF_EVENT_CLEANUP) {
+               if (fini_db (priv->_db_conn)) {
+                       gf_msg (this->name, GF_LOG_WARNING, 0,
+                               CTR_MSG_CLOSE_DB_CONN_FAILED, "Failed closing "
+                               "db connection");
+               }
+               if (priv->_db_conn)
+                        priv->_db_conn = NULL;
+       }
        ret = default_notify (this, event, data);
 
 out:
@@ -2345,10 +2355,6 @@ fini (xlator_t *this)
                                 CTR_MSG_CLOSE_DB_CONN_FAILED, "Failed closing "
                                 "db connection");
                 }
-
-                if (priv->_db_conn)
-                        priv->_db_conn = NULL;
-
                 GF_FREE (priv->ctr_db_path);
                 if (pthread_mutex_destroy (&priv->compact_lock)) {
                         gf_msg (this->name, GF_LOG_WARNING, 0,
@@ -2358,7 +2364,6 @@ fini (xlator_t *this)
         }
         GF_FREE (priv);
         mem_pool_destroy (this->local_pool);
-        this->local_pool = NULL;
 
         return;
 }