cluster: RHEL6 - cman: fix data copy and memory leak when reloading config

Gitweb: http://git.fedorahosted.org/git/?p=cluster.git;a=commitdiff;h=fa9267535f7... Commit: fa9267535f7bbb49dcc766f87f35502dfa623847 Parent: bdbed1f76496e2f3cfc86535314cbeffd4d20257 Author: Fabio M. Di Nitto <fdinitto(a)redhat.com&gt; AuthorDate: Wed Jul 11 11:44:49 2012 +0200 Committer: Fabio M. Di Nitto <fdinitto(a)redhat.com&gt; CommitterDate: Tue Jul 24 11:00:04 2012 +0200 cman: fix data copy and memory leak when reloading config cman.cluster_id,nodename,two_node information were not copied from the old to the new config at reload time. This triggers a problem when <cman is set in cluster.conf and we effectively drop information from objdb (suboptimal). Also fix a possible memory leak when we have reload issues. Resolves: rhbz#839241 Signed-off-by: Fabio M. Di Nitto <fdinitto(a)redhat.com&gt; Reviewed-by: Christine Caulfield <ccaulfie(a)redhat.com&gt; --- cman/daemon/cman-preconfig.c | 31 +++++++++++++++++++++++++++++-- 1 files changed, 29 insertions(+), 2 deletions(-) diff --git a/cman/daemon/cman-preconfig.c b/cman/daemon/cman-preconfig.c index c42052e..321d78f 100644 --- a/cman/daemon/cman-preconfig.c +++ b/cman/daemon/cman-preconfig.c @@ -1483,6 +1483,7 @@ static int cmanpre_reloadconfig(struct objdb_iface_ver0 *objdb, int flush, const hdb_handle_t cluster_parent_handle_new; unsigned int config_version = 0, config_version_new = 0; char *config_value = NULL; + char str[255]; /* don't reload if we've been told to run configless */ if (getenv("CMAN_NOCONFIG")) { @@ -1494,16 +1495,16 @@ static int cmanpre_reloadconfig(struct objdb_iface_ver0 *objdb, int flush, const /* find both /cluster entries */ objdb->object_find_create(OBJECT_PARENT_HANDLE, "cluster", strlen("cluster"), &find_handle); objdb->object_find_next(find_handle, &cluster_parent_handle); + objdb->object_find_next(find_handle, &cluster_parent_handle_new); + objdb->object_find_destroy(find_handle); if (!cluster_parent_handle) { sprintf (error_reason, "%s", "Cannot find old /cluster/ key in configuration\n"); goto err; } - objdb->object_find_next(find_handle, &cluster_parent_handle_new); if (!cluster_parent_handle_new) { sprintf (error_reason, "%s", "Cannot find new /cluster/ key in configuration\n"); goto err; } - objdb->object_find_destroy(find_handle); if (!objdb->object_key_get(cluster_parent_handle, "config_version", strlen("config_version"), (void *)&config_value, NULL)) { if (config_value) { @@ -1536,6 +1537,32 @@ static int cmanpre_reloadconfig(struct objdb_iface_ver0 *objdb, int flush, const /* destroy the old one */ objdb->object_destroy(cluster_parent_handle); + /* + * create cluster.cman in the new config if it doesn't exists + */ + objdb->object_find_create(cluster_parent_handle_new, "cman", strlen("cman"), &find_handle); + if (objdb->object_find_next(find_handle, &object_handle)) { + objdb->object_create(cluster_parent_handle_new, &object_handle, + "cman", strlen("cman")); + } + objdb->object_find_destroy(find_handle); + + /* + * readd cluster_id/two_node/nodename + */ + snprintf(str, sizeof(str) - 1, "%d", cluster_id); + objdb->object_key_create_typed(object_handle, "cluster_id", + str, strlen(str) + 1, OBJDB_VALUETYPE_STRING); + + if (two_node) { + snprintf(str, sizeof(str) - 1, "%d", 1); + objdb->object_key_create_typed(object_handle, "two_node", + str, strlen(str) + 1, OBJDB_VALUETYPE_STRING); + } + + objdb->object_key_create_typed(object_handle, "nodename", + nodename, strlen(nodename)+1, OBJDB_VALUETYPE_STRING); + /* update the reference to the new config */ cluster_parent_handle = cluster_parent_handle_new;