From mboxrd@z Thu Jan  1 00:00:00 1970
From: pcaulfield@sourceware.org <pcaulfield@sourceware.org>
Date: 12 Sep 2006 12:45:53 -0000
Subject: [Cluster-devel] cluster/cman/daemon commands.c
Message-ID: <20060912124553.21061.qmail@sourceware.org>
List-Id: <cluster-devel.redhat.com>
To: cluster-devel.redhat.com
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit

CVSROOT:	/cvs/cluster
Module name:	cluster
Changes by:	pcaulfield at sourceware.org	2006-09-12 12:45:52

Modified files:
	cman/daemon    : commands.c 

Log message:
	Fix strdup braindamage that probably caused segfaults when nodes
	left the cluster.
	
	This is likely to be the cause of bz#206083
	(thanks to Steven Dake for most of the diagnostics on this).

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/cluster/cman/daemon/commands.c.diff?cvsroot=cluster&r1=1.48&r2=1.49

--- cluster/cman/daemon/commands.c	2006/08/17 13:22:39	1.48
+++ cluster/cman/daemon/commands.c	2006/09/12 12:45:52	1.49
@@ -1644,7 +1644,7 @@
 	if (node->fence_agent && msg->fence_agent[0] && strcmp(node->fence_agent, msg->fence_agent))
 	{
 		free(node->fence_agent);
-		strdup(node->fence_agent);
+		node->fence_agent = strdup(msg->fence_agent);
 		node->fence_time = msg->fence_time;
 	}