[Cluster-devel] cluster/fence/agents/xvm virt.c

[Cluster-devel] cluster/fence/agents/xvm virt.c

[lhh]

CVSROOT:	/cvs/cluster
Module name:	cluster
Branch: 	RHEL5
Changes by:	lhh at sourceware.org	2007-07-30 16:14:18

Modified files:
	fence/agents/xvm: virt.c 

Log message:
	Fix memory leak in virt.c

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/cluster/fence/agents/xvm/virt.c.diff?cvsroot=cluster&only_with_tag=RHEL5&r1=1.3&r2=1.3.2.1

--- cluster/fence/agents/xvm/virt.c	2006/11/03 15:58:56	1.3
+++ cluster/fence/agents/xvm/virt.c	2007/07/30 16:14:18	1.3.2.1
@@ -108,6 +108,7 @@
 
 	/* We have all the locally running domains & states now */
 	/* Sort */
+	free(d_ids);
 	qsort(&vl->vm_states[0], vl->vm_count, sizeof(vl->vm_states[0]),
 	      _compare_virt);
 	return vl;	

[Cluster-devel] cluster/fence/agents/xvm virt.c

[lhh]

CVSROOT:	/cvs/cluster
Module name:	cluster
Branch: 	RHEL5
Changes by:	lhh at sourceware.org	2007-07-30 20:37:08

Modified files:
	fence/agents/xvm: virt.c 

Log message:
	Fix memory leak.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/cluster/fence/agents/xvm/virt.c.diff?cvsroot=cluster&only_with_tag=RHEL5&r1=1.3.2.1&r2=1.3.2.2

--- cluster/fence/agents/xvm/virt.c	2007/07/30 16:14:18	1.3.2.1
+++ cluster/fence/agents/xvm/virt.c	2007/07/30 20:37:08	1.3.2.2
@@ -104,6 +104,8 @@
 		strncpy(vl->vm_states[x].v_uuid, d_uuid, MAX_DOMAINNAME_LENGTH);
 		vl->vm_states[x].v_state.s_state = d_info.state;
 		vl->vm_states[x].v_state.s_owner = my_id;
+
+		virDomainFree(dom);
 	}
 
 	/* We have all the locally running domains & states now */

[Cluster-devel] cluster/fence/agents/xvm virt.c

[lhh]

CVSROOT:	/cvs/cluster
Module name:	cluster
Changes by:	lhh at sourceware.org	2007-07-30 20:41:44

Modified files:
	fence/agents/xvm: virt.c 

Log message:
	Fix #250152

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/cluster/fence/agents/xvm/virt.c.diff?cvsroot=cluster&r1=1.3&r2=1.4

--- cluster/fence/agents/xvm/virt.c	2006/11/03 15:58:56	1.3
+++ cluster/fence/agents/xvm/virt.c	2007/07/30 20:41:44	1.4
@@ -104,10 +104,13 @@
 		strncpy(vl->vm_states[x].v_uuid, d_uuid, MAX_DOMAINNAME_LENGTH);
 		vl->vm_states[x].v_state.s_state = d_info.state;
 		vl->vm_states[x].v_state.s_owner = my_id;
+
+		virDomainFree(dom);
 	}
 
 	/* We have all the locally running domains & states now */
 	/* Sort */
+	free(d_ids);
 	qsort(&vl->vm_states[0], vl->vm_count, sizeof(vl->vm_states[0]),
 	      _compare_virt);
 	return vl;	

[Cluster-devel] cluster/fence/agents/xvm virt.c

[lhh]

CVSROOT:	/cvs/cluster
Module name:	cluster
Branch: 	RHEL51
Changes by:	lhh at sourceware.org	2007-07-30 21:15:22

Modified files:
	fence/agents/xvm: virt.c 

Log message:
	Resolves: #250152  (at least, the fence_xvmd side)

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/cluster/fence/agents/xvm/virt.c.diff?cvsroot=cluster&only_with_tag=RHEL51&r1=1.3&r2=1.3.6.1

--- cluster/fence/agents/xvm/virt.c	2006/11/03 15:58:56	1.3
+++ cluster/fence/agents/xvm/virt.c	2007/07/30 21:15:22	1.3.6.1
@@ -104,10 +104,13 @@
 		strncpy(vl->vm_states[x].v_uuid, d_uuid, MAX_DOMAINNAME_LENGTH);
 		vl->vm_states[x].v_state.s_state = d_info.state;
 		vl->vm_states[x].v_state.s_owner = my_id;
+
+		virDomainFree(dom);
 	}
 
 	/* We have all the locally running domains & states now */
 	/* Sort */
+	free(d_ids);
 	qsort(&vl->vm_states[0], vl->vm_count, sizeof(vl->vm_states[0]),
 	      _compare_virt);
 	return vl;