diff options
| author |   tobhe <tobhe@openbsd.org> | 2019-12-16 13:58:53 +0000 |
|---|---|---|
| committer | tobhe <tobhe@openbsd.org> | 2019-12-16 13:58:53 +0000 |
| commit | 43e343f8aa17502e68dbb74fa3dd463280c74fe5 (patch) | |
| tree | 328404629833eaa99ce5c7c5814e25329366aad3 /usr.bin/ssh/auth.c | |
| parent | When rsync exits non zero because of network issues or because the (diff) | |
| download | wireguard-openbsd-43e343f8aa17502e68dbb74fa3dd463280c74fe5.tar.xz wireguard-openbsd-43e343f8aa17502e68dbb74fa3dd463280c74fe5.zip | |
strdup may return NULL if memory allocation fails. Use the safer xstrdup
which fatals on allocation failures.
ok markus@
Diffstat (limited to 'usr.bin/ssh/auth.c')
| -rw-r--r-- | usr.bin/ssh/auth.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/usr.bin/ssh/auth.c b/usr.bin/ssh/auth.c index c696b8759d1..f0340a81b7e 100644 --- a/usr.bin/ssh/auth.c +++ b/usr.bin/ssh/auth.c @@ -1,4 +1,4 @@ -/* $OpenBSD: auth.c,v 1.143 2019/11/25 00:54:23 djm Exp $ */ +/* $OpenBSD: auth.c,v 1.144 2019/12/16 13:58:53 tobhe Exp $ */ /* * Copyright (c) 2000 Markus Friedl. All rights reserved. * @@ -631,7 +631,7 @@ remote_hostname(struct ssh *ssh) if (getpeername(ssh_packet_get_connection_in(ssh), (struct sockaddr *)&from, &fromlen) == -1) { debug("getpeername failed: %.100s", strerror(errno)); - return strdup(ntop); + return xstrdup(ntop); } debug3("Trying to reverse map address %.100s.", ntop); @@ -639,7 +639,7 @@ remote_hostname(struct ssh *ssh) if (getnameinfo((struct sockaddr *)&from, fromlen, name, sizeof(name), NULL, 0, NI_NAMEREQD) != 0) { /* Host name not found. Use ip address. */ - return strdup(ntop); + return xstrdup(ntop); } /* @@ -654,7 +654,7 @@ remote_hostname(struct ssh *ssh) logit("Nasty PTR record \"%s\" is set up for %s, ignoring", name, ntop); freeaddrinfo(ai); - return strdup(ntop); + return xstrdup(ntop); } /* Names are stored in lowercase. */ @@ -675,7 +675,7 @@ remote_hostname(struct ssh *ssh) if (getaddrinfo(name, NULL, &hints, &aitop) != 0) { logit("reverse mapping checking getaddrinfo for %.700s " "[%s] failed.", name, ntop); - return strdup(ntop); + return xstrdup(ntop); } /* Look for the address from the list of addresses. */ for (ai = aitop; ai; ai = ai->ai_next) { @@ -690,9 +690,9 @@ remote_hostname(struct ssh *ssh) /* Address not found for the host name. */ logit("Address %.100s maps to %.600s, but this does not " "map back to the address.", ntop, name); - return strdup(ntop); + return xstrdup(ntop); } - return strdup(name); + return xstrdup(name); } /* |