Report info_slave errors to stderr and do not use them in place of data to MUSH
authorAri Johnson <ari@theari.com>
Mon, 10 Aug 2015 14:39:31 +0000 (10:39 -0400)
committerAri Johnson <ari@theari.com>
Mon, 10 Aug 2015 14:39:31 +0000 (10:39 -0400)
src/info_slave.c

index 4e1ed54eabdea5bc28ef692b59259feb548b730d..f0845b68a3c3599885a89fa62c6050f1cba6b42d 100644 (file)
@@ -193,16 +193,11 @@ main(int argc, char *argv[])
     bp = buf;
     if ((gairesult = getnameinfo(&remote.addr, rlen, host, sizeof host, NULL,
                                 0, NI_NUMERICHOST | NI_NUMERICSERV)) != 0) {
-      safe_str("Error: getnameinfo(remote): ", buf, &bp);
-      if (gairesult == EAI_SYSTEM) {
-        safe_integer(errno, buf, &bp);
-        safe_chr('/', buf, &bp);
-        safe_str(strerror(errno), buf, &bp);
-      } else {
-        safe_integer(gairesult, buf, &bp);
-        safe_chr('/', buf, &bp);
-        safe_str(gai_strerror(gairesult), buf, &bp);
-      }
+      if (gairesult == EAI_SYSTEM)
+        perror("info_slave getnameinfo");
+      else
+        fprintf(stderr, "info_slave getnameinfo: %s\n",
+               gai_strerror(gairesult));
     } else {
       safe_str(host, buf, &bp);
     }
@@ -210,16 +205,11 @@ main(int argc, char *argv[])
     if ((gairesult = getnameinfo(&local.addr, llen, NULL, 0, lport,
                                 sizeof lport,
                                 NI_NUMERICHOST | NI_NUMERICSERV)) != 0) {
-      safe_str("Error: getnameinfo(local): ", buf, &bp);
-      if (gairesult == EAI_SYSTEM) {
-        safe_integer(errno, buf, &bp);
-        safe_chr('/', buf, &bp);
-        safe_str(strerror(errno), buf, &bp);
-      } else {
-        safe_integer(gairesult, buf, &bp);
-        safe_chr('/', buf, &bp);
-        safe_str(gai_strerror(gairesult), buf, &bp);
-      }
+      if (gairesult == EAI_SYSTEM)
+        perror("info_slave getnameinfo");
+      else
+        fprintf(stderr, "info_slave getnameinfo: %s\n",
+               gai_strerror(gairesult));
     } else {
       safe_str(lport, buf, &bp);
     }
@@ -239,18 +229,13 @@ main(int argc, char *argv[])
     if (use_dns) {
       if ((gairesult = getnameinfo(&remote.addr, rlen, host, sizeof host, NULL,
                                   0, NI_NUMERICSERV) != 0)) {
-        safe_str("Error: getnameinfo(local/dns): ", buf, &bp);
-        if (gairesult == EAI_SYSTEM) {
-          safe_integer(errno, buf, &bp);
-          safe_chr('/', buf, &bp);
-          safe_str(strerror(errno), buf, &bp);
-        } else {
-          safe_integer(gairesult, buf, &bp);
-          safe_chr('/', buf, &bp);
-          safe_str(gai_strerror(gairesult), buf, &bp);
-        }
+        if (gairesult == EAI_SYSTEM)
+          perror("info_slave getnameinfo");
+        else
+          fprintf(stderr, "info_slave getnameinfo: %s\n",
+                 gai_strerror(gairesult));
       } else {
-       safe_str(host, buf, &bp);
+        safe_str(host, buf, &bp);
       }
     } else
       safe_str(host, buf, &bp);