Cluster Project branch, STABLE2, updated. cluster-2.03.04-9-g62b847a

Cluster Project branch, STABLE2, updated. cluster-2.03.04-9-g62b847a

[rpeterso]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Cluster Project".

http://sources.redhat.com/git/gitweb.cgi?p=cluster.git;a=commitdiff;h=62b847a636ea65b080f3a8b203384a2c5a18de25

The branch, STABLE2 has been updated
       via  62b847a636ea65b080f3a8b203384a2c5a18de25 (commit)
      from  d6e657e2882ac2fa89567c87189c3bbc49902f06 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 62b847a636ea65b080f3a8b203384a2c5a18de25
Author: Bob Peterson <rpeterso@redhat.com>
Date:   Thu Jun 12 13:42:27 2008 -0500

    Fix gfs_tool build warnings

-----------------------------------------------------------------------

Summary of changes:
 gfs/gfs_tool/counters.c |    2 +-
 gfs/gfs_tool/main.c     |    2 +-
 gfs/gfs_tool/misc.c     |    6 ++----
 gfs/gfs_tool/sb.c       |   11 +++++------
 4 files changed, 9 insertions(+), 12 deletions(-)

diff --git a/gfs/gfs_tool/counters.c b/gfs/gfs_tool/counters.c
index f0da2df..641428a 100644
--- a/gfs/gfs_tool/counters.c
+++ b/gfs/gfs_tool/counters.c
@@ -186,7 +186,7 @@ print_counters(int argc, char **argv)
 			die("can't get counters: %s\n", strerror(errno));
 
 		if (debug)
-			write(STDOUT_FILENO, data, error);
+			error = write(STDOUT_FILENO, data, error);
 
 		parse_lines(data, error);
 
diff --git a/gfs/gfs_tool/main.c b/gfs/gfs_tool/main.c
index d71373f..3de8580 100644
--- a/gfs/gfs_tool/main.c
+++ b/gfs/gfs_tool/main.c
@@ -110,7 +110,7 @@ print_usage(void)
 	int x;
 
 	for (x = 0; usage[x][0]; x++)
-		printf(usage[x]);
+		printf("%s", usage[x]);
 }
 
 /**
diff --git a/gfs/gfs_tool/misc.c b/gfs/gfs_tool/misc.c
index 2ec3ca6..bd699f2 100644
--- a/gfs/gfs_tool/misc.c
+++ b/gfs/gfs_tool/misc.c
@@ -158,7 +158,7 @@ print_lockdump(int argc, char **argv)
 	close(fd);
 
 
-	write(STDOUT_FILENO, data, count);
+	x = write(STDOUT_FILENO, data, count);
 
 	free(data);
 }
@@ -640,9 +640,7 @@ reclaim_metadata(int argc, char **argv)
 	if (!override) {
 		printf("Don't do this if this file system is being exported by NFS (on any machine).\n");
 		printf("\nAre you sure you want to proceed? [y/n] ");
-		fgets(buf, 255, stdin);
-
-		if (buf[0] != 'y')
+		if (fgets(buf, 255, stdin) == NULL || buf[0] != 'y')
 			die("aborted\n");
 
 		printf("\n");
diff --git a/gfs/gfs_tool/sb.c b/gfs/gfs_tool/sb.c
index 9a999f4..130e039 100644
--- a/gfs/gfs_tool/sb.c
+++ b/gfs/gfs_tool/sb.c
@@ -48,7 +48,8 @@ do_sb(int argc, char **argv)
 {
 	char *device, *field, *newval = NULL;
 	int fd;
-	unsigned char buf[GFS_BASIC_BLOCK], input[256];
+	unsigned char buf[GFS_BASIC_BLOCK];
+	char input[256];
 	struct gfs_sb sb;
 
 	if (optind == argc)
@@ -75,9 +76,7 @@ do_sb(int argc, char **argv)
 	if (newval && !override) {
 		printf("You shouldn't change any of these values if the filesystem is mounted.\n");
 		printf("\nAre you sure? [y/n] ");
-		fgets(input, 255, stdin);
-
-		if (input[0] != 'y')
+		if (fgets(input, 255, stdin) == NULL || input[0] != 'y')
 			die("aborted\n");
 
 		printf("\n");
@@ -86,7 +85,7 @@ do_sb(int argc, char **argv)
 	do_lseek(fd, GFS_SB_ADDR * GFS_BASIC_BLOCK);
 	do_read(fd, buf, GFS_BASIC_BLOCK);
 
-	gfs_sb_in(&sb, buf);
+	gfs_sb_in(&sb, (char *)buf);
 
 	if (sb.sb_header.mh_magic != GFS_MAGIC ||
 	    sb.sb_header.mh_type != GFS_METATYPE_SB)
@@ -139,7 +138,7 @@ do_sb(int argc, char **argv)
 		die("unknown field %s\n", field);
 
 	if (newval) {
-		gfs_sb_out(&sb, buf);
+		gfs_sb_out(&sb, (char *)buf);
 
 		do_lseek(fd, GFS_SB_ADDR * GFS_BASIC_BLOCK);
 		do_write(fd, buf, GFS_BASIC_BLOCK);


hooks/post-receive
--
Cluster Project