summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorbenno <benno@openbsd.org>2020-12-30 18:56:56 +0000
committerbenno <benno@openbsd.org>2020-12-30 18:56:56 +0000
commita3bb2ea1af2817e43380b769d4b0ab0919156bc8 (patch)
treedd0a91ba402b50d3a745054a163741ec01af9c67
parentgetifaddrs() can return entries where ifa_addr is NULL. Check for this (diff)
downloadwireguard-openbsd-a3bb2ea1af2817e43380b769d4b0ab0919156bc8.tar.xz
wireguard-openbsd-a3bb2ea1af2817e43380b769d4b0ab0919156bc8.zip
getifaddrs() can return entries where ifa_addr is NULL. Check for this
before accessing anything in ifa_addr. ok claudio@
-rw-r--r--regress/sys/netinet/in_pcbbind/runtest.c5
1 files changed, 3 insertions, 2 deletions
diff --git a/regress/sys/netinet/in_pcbbind/runtest.c b/regress/sys/netinet/in_pcbbind/runtest.c
index e26d2832dce..2842a2c7b7d 100644
--- a/regress/sys/netinet/in_pcbbind/runtest.c
+++ b/regress/sys/netinet/in_pcbbind/runtest.c
@@ -1,4 +1,4 @@
-/* $OpenBSD: runtest.c,v 1.5 2017/07/06 13:20:54 bluhm Exp $ */
+/* $OpenBSD: runtest.c,v 1.6 2020/12/30 18:56:56 benno Exp $ */
/*
* Copyright (c) 2015 Vincent Gross <vincent.gross@kilob.yt>
*
@@ -415,7 +415,8 @@ main(int argc, char *argv[])
err(2, "getifaddrs()");
curifa = ifap;
while (curifa) {
- if (memcmp(curifa->ifa_addr,
+ if (curifa->ifa_addr != NULL &&
+ memcmp(curifa->ifa_addr,
mifa->ai_addr,
mifa->ai_addrlen) == 0)
break;