diff options
author | Jelmer Vernooij <jelmer@jelmer.uk> | 2016-12-28 13:50:14 +0000 |
---|---|---|
committer | dx <dx@dxzone.com.ar> | 2016-12-31 17:40:09 -0300 |
commit | 2e8523b30bafa86685655001457a2615936b646a (patch) | |
tree | 4e0c9c6509d1b3aeca395bc17e258f4a38b48325 /lib/ftutil.c | |
parent | 7801298f6a855cda0c62433d45ec717d2773ef73 (diff) |
Use NI_MAXHOST rather than HOST_NAME_MAX for host lengths.
This constant is always available and meant to be used with
getnameinfo().
This fixes the build on Debian GNU/kFreeBSD.
Diffstat (limited to 'lib/ftutil.c')
-rw-r--r-- | lib/ftutil.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/lib/ftutil.c b/lib/ftutil.c index 3c79ba1f..8da88de9 100644 --- a/lib/ftutil.c +++ b/lib/ftutil.c @@ -62,13 +62,13 @@ int ft_listen(struct sockaddr_storage *saddr_ptr, char *host, char *port, int co if (scolon) { if (for_bitlbee_client) { *scolon = '\0'; - strncpy(host, ftlisten, HOST_NAME_MAX); + strncpy(host, ftlisten, NI_MAXHOST); *scolon = ';'; } else { - strncpy(host, scolon + 1, HOST_NAME_MAX); + strncpy(host, scolon + 1, NI_MAXHOST); } } else { - strncpy(host, ftlisten, HOST_NAME_MAX); + strncpy(host, ftlisten, NI_MAXHOST); } if ((colon = strchr(host, ':'))) { @@ -77,13 +77,13 @@ int ft_listen(struct sockaddr_storage *saddr_ptr, char *host, char *port, int co } } else if (copy_fd >= 0 && getsockname(copy_fd, (struct sockaddr*) &saddrs, &ssize) == 0 && (saddrs.ss_family == AF_INET || saddrs.ss_family == AF_INET6) && - getnameinfo((struct sockaddr*) &saddrs, ssize, host, HOST_NAME_MAX, + getnameinfo((struct sockaddr*) &saddrs, ssize, host, NI_MAXHOST, NULL, 0, NI_NUMERICHOST) == 0) { /* We just took our local address on copy_fd, which is likely to be a sensible address from which we can do a file transfer now - the most sensible we can get easily. */ } else { - ASSERTSOCKOP(gethostname(host, HOST_NAME_MAX + 1), "gethostname()"); + ASSERTSOCKOP(gethostname(host, NI_MAXHOST), "gethostname()"); } memset(&hints, 0, sizeof(struct addrinfo)); @@ -108,7 +108,7 @@ int ft_listen(struct sockaddr_storage *saddr_ptr, char *host, char *port, int co if (!inet_ntop(saddr->ss_family, saddr->ss_family == AF_INET ? ( void * ) &(( struct sockaddr_in * ) saddr)->sin_addr.s_addr : ( void * ) &(( struct sockaddr_in6 * ) saddr)->sin6_addr.s6_addr, - host, HOST_NAME_MAX)) { + host, NI_MAXHOST)) { strcpy(errmsg, "inet_ntop failed on listening socket"); return -1; } @@ -127,7 +127,7 @@ int ft_listen(struct sockaddr_storage *saddr_ptr, char *host, char *port, int co } /* I hate static-length strings.. */ - host[HOST_NAME_MAX - 1] = '\0'; + host[NI_MAXHOST - 1] = '\0'; port[5] = '\0'; return fd; |