X-Git-Url: https://git.distorted.org.uk/~mdw/tripe/blobdiff_plain/870ff51ad8e7fd068afb22c25701577d54828edb..89640f3f83c29967e524d754bf130bc452bdefe2:/server/admin.c

diff --git a/server/admin.c b/server/admin.c
index 72f42f29..44222eb8 100644
--- a/server/admin.c
+++ b/server/admin.c
@@ -272,14 +272,19 @@ void a_vformat(dstr *d, const char *fmt, va_list *ap)
     } else if (*fmt == '?') {
       if (strcmp(fmt, "?ADDR") == 0) {
 	const addr *a = va_arg(*ap, const addr *);
-	switch (a->sa.sa_family) {
-	  case AF_INET:
-	    u_quotify(d, "INET");
-	    u_quotify(d, inet_ntoa(a->sin.sin_addr));
-	    dstr_putf(d, " %u", (unsigned)ntohs(a->sin.sin_port));
-	    break;
-	  default:
-	    abort();
+	char name[NI_MAXHOST], serv[NI_MAXSERV];
+	int ix, err;
+	if ((err = getnameinfo(&a->sa, addrsz(a),
+			       name, sizeof(name), serv, sizeof(serv),
+			       (NI_NUMERICHOST | NI_NUMERICSERV |
+				NI_DGRAM)))) {
+	  dstr_putf(d, " E%d", err);
+	  u_quotify(d, gai_strerror(err));
+	} else {
+	  ix = afix(a->sa.sa_family); assert(ix >= 0);
+	  u_quotify(d, aftab[ix].name);
+	  u_quotify(d, name);
+	  u_quotify(d, serv);
 	}
       } else if (strcmp(fmt, "?B64") == 0) {
 	const octet *p = va_arg(*ap, const octet *);
@@ -1104,6 +1109,7 @@ static void a_resolve(admin *a, admin_resop *r, const char *tag,
   const char *fam = "ANY";
   char *p;
   int i = 0, j;
+  struct addrinfo *ai, *ailist, aihint = { 0 };
 
   /* --- Fill in the easy bits of address --- */
 
@@ -1168,11 +1174,28 @@ static void a_resolve(admin *a, admin_resop *r, const char *tag,
 
   /* --- If the name is numeric, do it the easy way --- */
 
-  if (inet_aton(av[i], &r->sa.sin.sin_addr)) {
-    T( trace(T_ADMIN, "admin: resop %s done the easy way", BGTAG(r)); )
-    r->sa.sin.sin_family = AF_INET;
-    setport(&r->sa, r->port);
-    func(r, ARES_OK);
+  aihint.ai_family = AF_INET;
+  aihint.ai_socktype = SOCK_DGRAM;
+  aihint.ai_protocol = IPPROTO_UDP;
+  aihint.ai_flags = AI_NUMERICHOST;
+  if (!getaddrinfo(av[i], 0, &aihint, &ailist)) {
+    for (ai = ailist; ai; ai = ai->ai_next) {
+      if ((j = afix(ai->ai_family)) >= 0 && udpsock[j].fd >= 0)
+	break;
+    }
+    if (!ai) {
+      T( trace(T_ADMIN, "admin: resop %s failed: "
+	       "no suitable addresses returned", BGTAG(r)); )
+      a_bgfail(&r->bg, "resolve-error", "%s" , r->addr, A_END);
+      func(r, ARES_FAIL);
+    } else {
+      T( trace(T_ADMIN, "admin: resop %s done the easy way", BGTAG(r)); )
+      assert(ai->ai_addrlen <= sizeof(r->sa));
+      memcpy(&r->sa, ai->ai_addr, ai->ai_addrlen);
+      setport(&r->sa, r->port);
+      func(r, ARES_OK);
+    }
+    freeaddrinfo(ailist);
     xfree(r->addr);
     a_bgrelease(&r->bg);
     return;