# Fix implicit declaration of function 'bsd_signal'; did you mean 'ssignal'? # # On sufficiently new glibc, signal with defined _DEFAULT_SOURCE is equivalent # to bsd_signal. --- a/config.h 2022-03-26 01:45:23.602395248 +0100 +++ b/config.h 2022-03-26 01:45:01.076491553 +0100 @@ -294,9 +294,6 @@ void *xrealloc(void *, size_t); char *xstrdup(const char *); -#ifndef HAVE_BSD_SIGNAL -void (*bsd_signal(int, void (*)(int))) (int); -#endif #ifndef HAVE_DUP2 int dup2(int, int); #endif --- a/configure.in 2022-03-26 01:56:06.656577548 +0100 +++ b/configure.in 2022-03-26 01:56:04.739068636 +0100 @@ -160,7 +160,6 @@ PA_SEARCH_LIBS_AND_ADD(xmalloc, iberty) PA_SEARCH_LIBS_AND_ADD(xrealloc, iberty) PA_SEARCH_LIBS_AND_ADD(xstrdup, iberty) -PA_SEARCH_LIBS_AND_ADD(bsd_signal, bsd, bsdsignal) PA_SEARCH_LIBS_AND_ADD(getopt_long, getopt, getopt_long) PA_SEARCH_LIBS_AND_ADD(getaddrinfo, [nsl resolv]) if $pa_add_getaddrinfo diff --git a/tftp/main.c b/tftp/main.c index b2f9059..d658230 100644 --- a/tftp/main.c +++ b/tftp/main.c @@ -305,7 +305,7 @@ int main(int argc, char *argv[]) sp->s_proto = (char *)"udp"; } - bsd_signal(SIGINT, intr); + signal(SIGINT, intr); if (peerargc) { /* Set peer */ @@ -768,7 +768,7 @@ void intr(int sig) { (void)sig; /* Quiet unused warning */ - bsd_signal(SIGALRM, SIG_IGN); + signal(SIGALRM, SIG_IGN); alarm(0); siglongjmp(toplevel, -1); } diff --git a/tftp/tftp.c b/tftp/tftp.c index d15da22..52f5be0 100644 --- a/tftp/tftp.c +++ b/tftp/tftp.c @@ -85,7 +85,7 @@ void tftp_sendfile(int fd, const char *name, const char *mode) is_request = 1; /* First packet is the actual WRQ */ amount = 0; - bsd_signal(SIGALRM, timer); + signal(SIGALRM, timer); do { if (is_request) { size = makerequest(WRQ, name, dp, mode) - 4; @@ -191,7 +191,7 @@ void tftp_recvfile(int fd, const char *name, const char *mode) firsttrip = 1; amount = 0; - bsd_signal(SIGALRM, timer); + signal(SIGALRM, timer); do { if (firsttrip) { size = makerequest(RRQ, name, ap, mode); # Fix leaked_handle: Handle variable "fd" going out of scope leaks the handle. diff --git a/tftpd/tftpd.c b/tftpd/tftpd.c index 364e7d2..cbd6093 100644 --- a/tftpd/tftpd.c +++ b/tftpd/tftpd.c @@ -1505,6 +1505,7 @@ static int validate_access(char *filename, int mode, if (mode == RRQ) { if (!unixperms && (stbuf.st_mode & (S_IREAD >> 6)) == 0) { + close(fd); *errmsg = "File must have global read permissions"; return (EACCESS); } @@ -1514,6 +1515,7 @@ static int validate_access(char *filename, int mode, } else { if (!unixperms) { if ((stbuf.st_mode & (S_IWRITE >> 6)) == 0) { + close(fd); *errmsg = "File must have global write permissions"; return (EACCESS); } @@ -1522,6 +1524,7 @@ static int validate_access(char *filename, int mode, #ifdef HAVE_FTRUNCATE /* We didn't get to truncate the file at open() time */ if (ftruncate(fd, (off_t) 0)) { + close(fd); *errmsg = "Cannot reset file size"; return (EACCESS); } # Fix warnings about useless inline in int usage(int) From 8ddf0d87d7463c21e28dd2bea6f3f42d4c92cb1d Mon Sep 17 00:00:00 2001 From: "H. Peter Anvin" Date: Sat, 7 Jun 2014 13:00:46 -0700 Subject: [PATCH] tftp: drop "inline" from definition of usage() It is pointless and newer gcc say it is a lose. Signed-off-by: H. Peter Anvin --- tftp/main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tftp/main.c b/tftp/main.c index 1b8a881..b2f9059 100644 --- a/tftp/main.c +++ b/tftp/main.c @@ -188,7 +188,7 @@ char *xstrdup(const char *); const char *program; -static inline void usage(int errcode) +static void usage(int errcode) { fprintf(stderr, #ifdef HAVE_IPV6 -- 2.35.1 # Fixes negative_returns: "fd" is passed to a parameter of pmtu_discovery_off # that cannot be negative From 0b44159b3a2f51d350f309d3f6d14a17e74e8231 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Luk=C3=A1=C5=A1=20Zaoral?= Date: Wed, 6 Apr 2022 09:33:33 +0200 Subject: [PATCH 1/2] tftpd: Correctly disable path MTU discovery in standalone mode --- tftpd/tftpd.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tftpd/tftpd.c b/tftpd/tftpd.c index 364e7d2..00fa1cf 100644 --- a/tftpd/tftpd.c +++ b/tftpd/tftpd.c @@ -769,7 +769,7 @@ int main(int argc, char **argv) } /* Disable path MTU discovery */ - pmtu_discovery_off(fd); + pmtu_discovery_off(fdmax); /* This means we don't want to wait() for children */ #ifdef SA_NOCLDWAIT -- 2.35.1 From 5f60355c4bd10b866847a0d58a9582bda7db72aa Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Luk=C3=A1=C5=A1=20Zaoral?= Date: Wed, 6 Apr 2022 09:34:46 +0200 Subject: [PATCH 2/2] tftpd: Fix a possible usage of -1 file descriptor in standalone mode --- tftpd/tftpd.c | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/tftpd/tftpd.c b/tftpd/tftpd.c index 00fa1cf..afd595d 100644 --- a/tftpd/tftpd.c +++ b/tftpd/tftpd.c @@ -622,6 +622,13 @@ int main(int argc, char **argv) exit(EX_USAGE); } ai_fam = AF_INET6; + + if (fd6 < 0) { + syslog(LOG_ERR, + "IPv6 was disabled but address %s is in address " + "family AF_INET6", address); + exit(EX_USAGE); + } } break; #endif -- 2.35.1