Don't follow the NULL pointer, not even in debug printouts.

Bug reported by Leonhard Knauff. Closes RADSECPROXY-68.
mariux64 · Oct 17, 2016 · 6a45e69 · 6a45e69
1 parent e901dd8
commit 6a45e69
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/AUTHORS b/AUTHORS
@@ -13,6 +13,7 @@ Fabian Mauchle
 Faidon Liambotis
 Hans Zandbelt
 Kolbjørn Barmen
+Leonhard Knauff
 Linus Nordberg
 Luke Benes
 Maja Wolniewicz

diff --git a/hostport.c b/hostport.c
@@ -163,7 +163,10 @@ int resolvehostport(struct hostportres *hp, int af, int socktype, uint8_t passiv
 	    }
 	}
     }
-    debug(DBG_DBG, "%s: %s -> %s", __func__, hp->host, addr2string(hp->addrinfo->ai_addr));
+    debug(DBG_DBG, "%s: %s -> %s", __func__,
+          (hp->host ? hp->host : "(src info not available)"),
+          ((hp->addrinfo && hp->addrinfo->ai_addr) ?
+           addr2string(hp->addrinfo->ai_addr) : "(dst info not available)"));
     return 1;
 
 errexit: