Make WARNS=6 clean.

MFC after:	1 week

6 files changed, 37 insertions, 44 deletions

diff --git a/libexec/tftpd/Makefile b/libexec/tftpd/Makefile
index 48c5e0d..8445829 100644
--- a/libexec/tftpd/Makefile
+++ b/libexec/tftpd/Makefile
@@ -5,7 +5,6 @@ PROG=	tftpd
 MAN=	tftpd.8
 SRCS=	tftp-file.c tftp-io.c tftp-options.c tftp-transfer.c tftp-utils.c
 SRCS+=	tftpd.c
-WARNS=	3
 WFORMAT=0
 LDFLAGS= -lwrap
 
diff --git a/libexec/tftpd/tftp-file.c b/libexec/tftpd/tftp-file.c
index e0f8e78..1ef8820 100644
--- a/libexec/tftpd/tftp-file.c
+++ b/libexec/tftpd/tftp-file.c
@@ -103,13 +103,13 @@ static size_t
 convert_to_net(char *buffer, size_t count, int init)
 {
 	size_t i;
-	static size_t n = 0, read = 0;
+	static size_t n = 0, in = 0;
 	static int newline = 0;
 
 	if (init) {
 		newline = 0;
 		n = 0;
-		read = 0;
+		in = 0;
 		return 0 ;
 	}
 
@@ -124,13 +124,13 @@ convert_to_net(char *buffer, size_t count, int init)
 	}
 
 	while (i < count) {
-		if (n == read) {
+		if (n == in) {
 			/* When done we're done */
 			if (feof(file)) break;
 
 			/* Otherwise read another bunch */
-			read = fread(convbuffer, 1, count, file);
-			if (read == 0) break;
+			in = fread(convbuffer, 1, count, file);
+			if (in == 0) break;
 			n = 0;
 		}
 
@@ -250,7 +250,7 @@ read_close(void)
 
 
 int
-synchnet(int peer)
+synchnet(int peer __unused)
 {
 
 	return 0;
diff --git a/libexec/tftpd/tftp-options.c b/libexec/tftpd/tftp-options.c
index 0b97aaf..d5b6386 100644
--- a/libexec/tftpd/tftp-options.c
+++ b/libexec/tftpd/tftp-options.c
@@ -79,7 +79,8 @@ int options_extra_enabled = 1;
  */
 
 int
-option_tsize(int peer, struct tftphdr *tp, int mode, struct stat *stbuf)
+option_tsize(int peer __unused, struct tftphdr *tp __unused, int mode,
+    struct stat *stbuf)
 {
 
 	if (options[OPT_TSIZE].o_request == NULL)
@@ -159,21 +160,19 @@ option_rollover(int peer)
 int
 option_blksize(int peer)
 {
-	int *maxdgram;
-	char maxbuffer[100];
+	u_long maxdgram;
 	size_t len;
 
 	if (options[OPT_BLKSIZE].o_request == NULL)
 		return (0);
 
 	/* maximum size of an UDP packet according to the system */
-	len = sizeof(maxbuffer);
+	len = sizeof(maxdgram);
 	if (sysctlbyname("net.inet.udp.maxdgram",
-	    maxbuffer, &len, NULL, 0) < 0) {
+	    &maxdgram, &len, NULL, 0) < 0) {
 		tftp_log(LOG_ERR, "sysctl: net.inet.udp.maxdgram");
 		return (acting_as_client ? 1 : 0);
 	}
-	maxdgram = (int *)maxbuffer;
 
 	int size = atoi(options[OPT_BLKSIZE].o_request);
 	if (size < BLKSIZE_MIN || size > BLKSIZE_MAX) {
@@ -191,20 +190,20 @@ option_blksize(int peer)
 		}
 	}
 
-	if (size > *maxdgram) {
+	if (size > (int)maxdgram) {
 		if (acting_as_client) {
 			tftp_log(LOG_ERR,
 			    "Invalid blocksize (%d bytes), "
 			    "net.inet.udp.maxdgram sysctl limits it to "
-			    "%d bytes.\n", size, *maxdgram);
+			    "%d bytes.\n", size, maxdgram);
 			send_error(peer, EBADOP);
 			return (1);
 		} else {
 			tftp_log(LOG_WARNING,
 			    "Invalid blocksize (%d bytes), "
 			    "net.inet.udp.maxdgram sysctl limits it to "
-			    "%d bytes.\n", size, *maxdgram);
-			size = *maxdgram;
+			    "%d bytes.\n", size, maxdgram);
+			size = maxdgram;
 			/* No reason to return */
 		}
 	}
@@ -220,10 +219,9 @@ option_blksize(int peer)
 }
 
 int
-option_blksize2(int peer)
+option_blksize2(int peer __unused)
 {
-	int	*maxdgram;
-	char	maxbuffer[100];
+	u_long	maxdgram;
 	int	size, i;
 	size_t	len;
 
@@ -236,13 +234,12 @@ option_blksize2(int peer)
 		return (0);
 
 	/* maximum size of an UDP packet according to the system */
-	len = sizeof(maxbuffer);
+	len = sizeof(maxdgram);
 	if (sysctlbyname("net.inet.udp.maxdgram",
-	    maxbuffer, &len, NULL, 0) < 0) {
+	    &maxdgram, &len, NULL, 0) < 0) {
 		tftp_log(LOG_ERR, "sysctl: net.inet.udp.maxdgram");
 		return (acting_as_client ? 1 : 0);
 	}
-	maxdgram = (int *)maxbuffer;
 
 	size = atoi(options[OPT_BLKSIZE2].o_request);
 	for (i = 0; sizes[i] != 0; i++) {
@@ -254,13 +251,13 @@ option_blksize2(int peer)
 		return (acting_as_client ? 1 : 0);
 	}
 
-	if (size > *maxdgram) {
+	if (size > (int)maxdgram) {
 		for (i = 0; sizes[i+1] != 0; i++) {
-			if (*maxdgram < sizes[i+1]) break;
+			if ((int)maxdgram < sizes[i+1]) break;
 		}
 		tftp_log(LOG_INFO,
 		    "Invalid blocksize2 (%d bytes), net.inet.udp.maxdgram "
-		    "sysctl limits it to %d bytes.\n", size, *maxdgram);
+		    "sysctl limits it to %d bytes.\n", size, maxdgram);
 		size = sizes[i];
 		/* No need to return */
 	}
@@ -279,7 +276,7 @@ option_blksize2(int peer)
  * Append the available options to the header
  */
 uint16_t
-make_options(int peer, char *buffer, uint16_t size) {
+make_options(int peer __unused, char *buffer, uint16_t size) {
 	int	i;
 	char	*value;
 	const char *option;
diff --git a/libexec/tftpd/tftp-utils.c b/libexec/tftpd/tftp-utils.c
index da58064..d6d0558 100644
--- a/libexec/tftpd/tftp-utils.c
+++ b/libexec/tftpd/tftp-utils.c
@@ -59,7 +59,7 @@ int	acting_as_client;
  * first timeout) to 'timeoutnetwork' (i.e. the last timeout)
  */
 int
-settimeouts(int _timeoutpacket, int _timeoutnetwork, int _maxtimeouts)
+settimeouts(int _timeoutpacket, int _timeoutnetwork, int _maxtimeouts __unused)
 {
 	int i;
 	
@@ -91,7 +91,7 @@ unmappedaddr(struct sockaddr_in6 *sin6)
 	    !IN6_IS_ADDR_V4MAPPED(&sin6->sin6_addr))
 		return;
 	sin4 = (struct sockaddr_in *)sin6;
-	addr = *(u_int32_t *)&sin6->sin6_addr.s6_addr[12];
+	memcpy(&addr, &sin6->sin6_addr.s6_addr[12], sizeof(addr));
 	port = sin6->sin6_port;
 	memset(sin4, 0, sizeof(struct sockaddr_in));
 	sin4->sin_addr.s_addr = addr;
@@ -170,7 +170,7 @@ struct packettypes packettypes[] = {
 	{ 0,		NULL	},
 };
 
-char *
+const char *
 packettype(int type)
 {
 	static char failed[100];
@@ -231,7 +231,7 @@ debug_finds(char *s)
 	return (i);
 }
 
-char *
+const char *
 debug_show(int d)
 {
 	static char s[100];
@@ -317,4 +317,3 @@ printstats(const char *direction, int verbose, struct tftp_stats *ts)
 		printf(" [%.0f bits/sec]", (ts->amount*8.)/delta);
 	putchar('\n');
 }
-
diff --git a/libexec/tftpd/tftp-utils.h b/libexec/tftpd/tftp-utils.h
index d072479..a789252 100644
--- a/libexec/tftpd/tftp-utils.h
+++ b/libexec/tftpd/tftp-utils.h
@@ -64,17 +64,17 @@ ssize_t	get_field(int peer, char *buffer, ssize_t size);
  */
 struct packettypes {
 	int	value;
-	char	*name;
+	const char *const name;
 };
 extern struct packettypes packettypes[];
-char *packettype(int);
+const char *packettype(int);
 
 /*
  * RP_
  */
 struct rp_errors {
 	int	error;
-	char	*desc;
+	const char *const desc;
 };
 extern struct rp_errors rp_errors[];
 char	*rp_strerror(int error);
@@ -89,15 +89,15 @@ char	*rp_strerror(int error);
 #define DEBUG_ACCESS	0x0008
 struct debugs {
 	int	value;
-	char	*name;
-	char	*desc;
+	const char *const name;
+	const char *const desc;
 };
 extern int	debug;
 extern struct debugs debugs[];
 extern int	packetdroppercentage;
 int	debug_find(char *s);
 int	debug_finds(char *s);
-char	*debug_show(int d);
+const char *debug_show(int d);
 
 /*
  * Log routines
diff --git a/libexec/tftpd/tftpd.c b/libexec/tftpd/tftpd.c
index 93943f5..33911ec 100644
--- a/libexec/tftpd/tftpd.c
+++ b/libexec/tftpd/tftpd.c
@@ -97,7 +97,7 @@ static int	suppress_naks;
 static int	logging;
 static int	ipchroot;
 static int	create_new = 0;
-static char	*newfile_format = "%Y%m%d";
+static const char *newfile_format = "%Y%m%d";
 static int	increase_name = 0;
 static mode_t	mask = S_IWGRP | S_IWOTH;
 
@@ -785,7 +785,6 @@ static void
 tftp_xmitfile(int peer, const char *mode)
 {
 	uint16_t block;
-	uint32_t amount;
 	time_t now;
 	struct tftp_stats ts;
 
@@ -799,13 +798,12 @@ tftp_xmitfile(int peer, const char *mode)
 	read_close();
 	if (debug&DEBUG_SIMPLE)
 		tftp_log(LOG_INFO, "Sent %d bytes in %d seconds",
-		    amount, time(NULL) - now);
+		    ts.amount, time(NULL) - now);
 }
 
 static void
 tftp_recvfile(int peer, const char *mode)
 {
-	uint32_t filesize; 
 	uint16_t block;
 	struct timeval now1, now2;
 	struct tftp_stats ts;
@@ -820,6 +818,7 @@ tftp_recvfile(int peer, const char *mode)
 	tftp_receive(peer, &block, &ts, NULL, 0);
 
 	write_close();
+	gettimeofday(&now2, NULL);
 
 	if (debug&DEBUG_SIMPLE) {
 		double f;
@@ -832,9 +831,8 @@ tftp_recvfile(int peer, const char *mode)
 		    (now2.tv_usec - now1.tv_usec) / 100000.0;
 		tftp_log(LOG_INFO,
 		    "Download of %d bytes in %d blocks completed after %0.1f seconds\n",
-		    filesize, block, f);
+		    ts.amount, block, f);
 	}
 
 	return;
 }
-