preview release: 2.6.0pre1

Paul Slootman paul at debian.org
Sat Dec 20 09:14:56 EST 2003


On Fri 19 Dec 2003, Wayne Davison wrote:

> Please test this and let me know if we have any remaining issues to work
> through before the final release.  Thanks,

One tiny little nit, when compiling on alpha, I get the following
warning:

alpha-linux-gcc -I. -I. -Wall -O2  -c sender.c -o sender.o
sender.c: In function `read_sum_head':
sender.c:51: warning: int format, different type arg (arg 3)

It's a size_t being printed as an int. Here's a patch:


--- sender.c.orig	2003-12-19 23:08:13.000000000 +0100
+++ sender.c	2003-12-19 23:08:21.000000000 +0100
@@ -47,8 +47,8 @@
 	} else {
 		sum->s2length = read_int(f);
 		if (sum->s2length > MD4_SUM_LENGTH) {
-			rprintf(FERROR, "Invalid checksum length %d\n",
-			    sum->s2length);
+			rprintf(FERROR, "Invalid checksum length %ld\n",
+			    (long)sum->s2length);
 			exit_cleanup(RERR_PROTOCOL);
 		}
 	}

Alternatively, it could have been cast to a normal int and the format
left as %d. However, I thought that this was the safest, as although I
can't really see how sum->s2length could contain more than 32 bits, I
guess it was declared as size_t for some reason...



Paul Slootman



More information about the rsync mailing list