cluster: RHEL510 - rgmanager: Fix improper use of cman_get_node()

Ryan McCabe rmccabe at fedoraproject.org
Thu Jun 13 18:58:49 UTC 2013


Gitweb:        http://git.fedorahosted.org/git/?p=cluster.git;a=commitdiff;h=be593c8577d5c405facb5a454f5d934c3f9fb23b
Commit:        be593c8577d5c405facb5a454f5d934c3f9fb23b
Parent:        75f10d2cb456040124adf819adbe077d9954b320
Author:        Ryan McCabe <rmccabe at redhat.com>
AuthorDate:    Thu Jun 13 14:54:30 2013 -0400
Committer:     Ryan McCabe <rmccabe at redhat.com>
CommitterDate: Thu Jun 13 14:58:03 2013 -0400

rgmanager: Fix improper use of cman_get_node()

Fix an improper use of cman_get_node() where the function is used to
get a node by id without first zeroing the cman_node_t struct.

Resolves: rhbz#889098

Signed-off-by: Ryan McCabe <rmccabe at redhat.com>
---
 rgmanager/src/daemons/rg_event.c |    1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/rgmanager/src/daemons/rg_event.c b/rgmanager/src/daemons/rg_event.c
index 606d41b..fe532d7 100644
--- a/rgmanager/src/daemons/rg_event.c
+++ b/rgmanager/src/daemons/rg_event.c
@@ -227,6 +227,7 @@ node_fenced(int nodeid)
 	 * if CMAN has admitted a node back in to the cluster, 
 	 * then we can stop waiting for fencing (#527777)
 	 */
+	memset(&node, 0, sizeof(node));
 	if (cman_get_node(ch, nodeid, &node) < 0)
 		goto out;
 


More information about the cluster-commits mailing list