gfapi: Closed the logfile fd and initialize to NULL in glfs_fini

Currently if logfile is closed and other threads call gf_log
after glfs_fini() is executed, it may lead to memory corruption.
Adding gf_log_fini() which closes the logfile and initializes
the logfile to NULL, thus any further logging happens to stderr.
Also added gf_log_globals_fini() which should be filled in the
future to release all the logging resources.

Change-Id: I879163e1a3636e65300d166f782517ee773cab65
BUG: 1030228
Signed-off-by: Poornima G <pgurusid@redhat.com>
Reviewed-on: http://review.gluster.org/6552
Reviewed-by: Pranith Kumar Karampuri <pkarampu@redhat.com>
Reviewed-by: Shyamsundar Ranganathan <srangana@redhat.com>
Tested-by: Gluster Build System <jenkins@build.gluster.com>
Reviewed-by: Vijay Bellur <vbellur@redhat.com>

1 files changed, 34 insertions, 1 deletions

diff --git a/libglusterfs/src/logging.c b/libglusterfs/src/logging.c
index e3a4a9fde09..0058233a7cf 100644
--- a/libglusterfs/src/logging.c
+++ b/libglusterfs/src/logging.c
@@ -108,11 +108,44 @@ gf_log_set_xl_loglevel (void *this, gf_loglevel_t level)
 }
 
 void
-gf_log_fini (void)
+gf_log_globals_fini (void)
 {
         pthread_mutex_destroy (&THIS->ctx->log.logfile_mutex);
 }
 
+/** gf_log_fini - function to perform the cleanup of the log information
+ * @data - glusterfs context
+ * @return: success: 0
+ *          failure: -1
+ */
+int
+gf_log_fini (void *data)
+{
+        glusterfs_ctx_t *ctx = data;
+        int ret = 0;
+
+        if (ctx == NULL) {
+	        ret = -1;
+                goto out;
+        }
+
+        pthread_mutex_lock (&ctx->log.logfile_mutex);
+        {
+                if (ctx->log.logfile) {
+                        if (fclose (ctx->log.logfile) != 0)
+                                ret = -1;
+                        /* Logfile needs to be set to NULL, so that any
+                           call to gf_log after calling gf_log_fini, will
+                           log the message to stderr.
+                        */
+                        ctx->log.logfile = NULL;
+                }
+        }
+        pthread_mutex_unlock (&ctx->log.logfile_mutex);
+
+ out:
+        return ret;
+}
 
 #ifdef GF_USE_SYSLOG
 /**