summaryrefslogtreecommitdiffstats
path: root/libexec
diff options
context:
space:
mode:
authormarius <marius@FreeBSD.org>2010-09-24 10:40:17 +0000
committermarius <marius@FreeBSD.org>2010-09-24 10:40:17 +0000
commitb33b39bf8fca1665dc3b1f2d01092704fa9387d2 (patch)
tree2f8808cab176e2913d55c56aa2ac6b5eda69d0de /libexec
parentb9eeaa21c2c08a8a5837ece770038dd9704067f3 (diff)
downloadFreeBSD-src-b33b39bf8fca1665dc3b1f2d01092704fa9387d2.zip
FreeBSD-src-b33b39bf8fca1665dc3b1f2d01092704fa9387d2.tar.gz
Make WARNS=6 clean.
MFC after: 1 week
Diffstat (limited to 'libexec')
-rw-r--r--libexec/tftpd/Makefile1
-rw-r--r--libexec/tftpd/tftp-file.c12
-rw-r--r--libexec/tftpd/tftp-options.c37
-rw-r--r--libexec/tftpd/tftp-utils.c9
-rw-r--r--libexec/tftpd/tftp-utils.h12
-rw-r--r--libexec/tftpd/tftpd.c10
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;
}
-
OpenPOWER on IntegriCloud