[Cluster-devel] cluster/group/gfs_controld main.c

[teigland@sourceware.org <teigland@sourceware.org>]

CVSROOT:	/cvs/cluster
Module name:	cluster
Branch: 	RHEL50
Changes by:	teigland at sourceware.org	2006-11-28 20:52:46

Modified files:
	group/gfs_controld: main.c 

Log message:
	the fix yesterday to prevent a segfault when mount failed mistakenly
	also changed the exit point from the function causing the error to not
	be written back to mount.gfs

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/cluster/group/gfs_controld/main.c.diff?cvsroot=cluster&only_with_tag=RHEL50&r1=1.18.4.4&r2=1.18.4.5

--- cluster/group/gfs_controld/main.c	2006/11/27 22:43:17	1.18.4.4
+++ cluster/group/gfs_controld/main.c	2006/11/28 20:52:46	1.18.4.5
@@ -293,13 +293,13 @@
 		/* ci, dir, type, proto, table, extra */
 		rv = do_mount(ci, argv[1], argv[2], argv[3], argv[4], argv[5],
 			      &mg);
+		fd = client[ci].fd;
+		fcntl(fd, F_SETFL, fcntl(fd, F_GETFL, 0) | O_NONBLOCK);
 		if (!rv) {
 			client[ci].mg = mg;
-			fd = client[ci].fd;
-			fcntl(fd, F_SETFL, fcntl(fd, F_GETFL, 0) | O_NONBLOCK);
 			mg->mount_client_fd = fd;
-			goto reply;
 		}
+		goto reply;
 	} else if (!strcmp(cmd, "mount_result")) {
 		got_mount_result(client[ci].mg, atoi(argv[3]));