[Debian-ha-commits] [pacemaker] 61/69: Fix: fencing: free only after last usage makes coverity happy

Christoph Berg myon at debian.org
Tue Jan 26 09:14:30 UTC 2016


This is an automated email from the git hooks/post-receive script.

myon pushed a commit to branch debian/master
in repository pacemaker.

commit 93b9f2c30dd4b03265b34c13175f142bf270202a
Author: Klaus Wenninger <klaus.wenninger at aon.at>
Date:   Thu Jan 14 17:18:29 2016 +0100

    Fix: fencing: free only after last usage
      makes coverity happy
---
 fencing/commands.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fencing/commands.c b/fencing/commands.c
index e933404..5eb6eff 100644
--- a/fencing/commands.c
+++ b/fencing/commands.c
@@ -1224,8 +1224,8 @@ stonith_level_register(xmlNode *msg, char **desc)
 
     /* Sanity-check arguments */
     if (mode >= 3 || (id <= 0) || (id >= ST_LEVEL_MAX)) {
-        free(target);
         crm_trace("Could not add %s[%d] (%d) to the topology (%d active entries)", target, id, mode, g_hash_table_size(topology));
+        free(target);
         crm_log_xml_err(level, "Bad topology");
         return -EINVAL;
     }

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/debian-ha/pacemaker.git



More information about the Debian-HA-Commits mailing list