vsf_quorum: fix potential memory leak

Signed-off-by: Fabio M. Di Nitto <fdinitto@redhat.com>
Reviewed-by: Angus Salkeld <asalked@redhat.com>

diff --git a/exec/vsf_quorum.c b/exec/vsf_quorum.c
index c258c145c4037940c35b9f0c29dd6c67f2ae2f2b..e0831e138613025a237fb1c99f3e48c067af41e5 100644 (file)
--- a/exec/vsf_quorum.c
+++ b/exec/vsf_quorum.c
@@ -252,7 +252,7 @@ static struct quorum_callin_functions callins = {
 
 static int quorum_exec_init_fn (struct corosync_api_v1 *api)
 {
-       char *quorum_module;
+       char *quorum_module = NULL;
 
 #ifdef COROSYNC_SOLARIS
        logsys_subsys_init();
@@ -280,6 +280,11 @@ static int quorum_exec_init_fn (struct corosync_api_v1 *api)
                }
        }
 
+       if (quorum_module) {
+               free(quorum_module);
+               quorum_module = NULL;
+       }
+
        /*
         * setting quorum_type and primary_designated in the right order is important
         * always try to lookup/init a quorum module, then revert back to be quorate