[Cluster-devel] cluster/cman/tests libtest.c

[Cluster-devel] cluster/cman/tests libtest.c

[pcaulfield]

CVSROOT:	/cvs/cluster
Module name:	cluster
Changes by:	pcaulfield at sourceware.org	2006-08-11 10:53:48

Modified files:
	cman/tests     : libtest.c 

Log message:
	Set a good example by checking return values.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/cluster/cman/tests/libtest.c.diff?cvsroot=cluster&r1=1.3&r2=1.4

--- cluster/cman/tests/libtest.c	2006/04/06 12:11:50	1.3
+++ cluster/cman/tests/libtest.c	2006/08/11 10:53:48	1.4
@@ -87,7 +87,10 @@
 		perror("get_node failed");
 	}
 
-	cman_start_notification(h, cman_callback);
+	if (!cman_start_notification(h, cman_callback))
+	{
+		perror("start_notification");
+	}
 	while (1) {
 	  int ret = cman_dispatch(h, CMAN_DISPATCH_BLOCKING | CMAN_DISPATCH_ALL);
 	  if (ret == -1) {

[Cluster-devel] cluster/cman/tests libtest.c

[pcaulfield]

CVSROOT:	/cvs/cluster
Module name:	cluster
Changes by:	pcaulfield at sourceware.org	2006-08-17 14:30:12

Modified files:
	cman/tests     : libtest.c 

Log message:
	Fix a bug in the demo prog.
	no point in setting a bad example (again)

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/cluster/cman/tests/libtest.c.diff?cvsroot=cluster&r1=1.5&r2=1.6

--- cluster/cman/tests/libtest.c	2006/08/17 13:22:39	1.5
+++ cluster/cman/tests/libtest.c	2006/08/17 14:30:11	1.6
@@ -95,6 +95,9 @@
 		perror("get_nodes failed");
 	}
 
+	// Need to clear this.
+	// Who wrote this rubbish? oh, I did.
+	nodes[0].cn_name[0] = '\0';
 	if (!cman_get_node(h, CMAN_NODEID_US, &nodes[0]))
 	{
 		printf("Getting our info:\n");