about summary refs log tree commit diff
diff options
context:
space:
mode:
authorFlorian Weimer <fweimer@redhat.com>2019-02-18 13:42:00 +0100
committerFlorian Weimer <fweimer@redhat.com>2019-02-18 16:39:15 +0100
commit308e022fc46a944700ad5a41530f36aaa6cc7dae (patch)
tree0ae0d37dd8bbcbd4d8e5899cf2a717cec8ee0c46
parenta5406364ac949f91867352cb2ae867629f083c5b (diff)
downloadglibc-308e022fc46a944700ad5a41530f36aaa6cc7dae.tar.gz
glibc-308e022fc46a944700ad5a41530f36aaa6cc7dae.tar.xz
glibc-308e022fc46a944700ad5a41530f36aaa6cc7dae.zip
resolv: Remove debugging code from compat-gethnamaddr.c
-rw-r--r--ChangeLog7
-rw-r--r--resolv/compat-gethnamaddr.c23
2 files changed, 7 insertions, 23 deletions
diff --git a/ChangeLog b/ChangeLog
index a359b0631b..312ef3bd8f 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,5 +1,12 @@
 2019-02-18  Florian Weimer  <fweimer@redhat.com>
 
+	* resolv/compat-gethnamaddr.c (Dprintf): Remove definition.
+	(getanswer): Do not call Dprintf.
+	(res_gethostbyname2_context): Likewise.
+	(res_gethostbyaddr_context): Likewise.
+
+2019-02-18  Florian Weimer  <fweimer@redhat.com>
+
 	* libio/libio.h (_IO_stdin, _IO_stdout, _IO_stderr): Remove
 	declaration.
 	* libio/stdio.c (AL, AL2, _IO_stdin, _IO_stdout, _IO_stderr):
diff --git a/resolv/compat-gethnamaddr.c b/resolv/compat-gethnamaddr.c
index 259378b2be..0e24dcbbcc 100644
--- a/resolv/compat-gethnamaddr.c
+++ b/resolv/compat-gethnamaddr.c
@@ -119,21 +119,6 @@ typedef union {
 extern int h_errno;
 # endif
 
-# ifdef DEBUG
-static void
-Dprintf (char *msg, int num)
-{
-	if (_res.options & RES_DEBUG) {
-		int save = errno;
-
-		printf(msg, num);
-		__set_errno (save);
-	}
-}
-# else
-#  define Dprintf(msg, num) /*nada*/
-# endif
-
 # define BOUNDED_INCR(x) \
 	do { \
 		cp += x; \
@@ -162,7 +147,6 @@ getanswer (const querybuf *answer, int anslen, const char *qname, int qtype)
 	char *bp, **ap, **hap;
 	int type, class, buflen, ancount, qdcount;
 	int haveanswer, had_error;
-	int toobig = 0;
 	char tbuf[MAXDNAME];
 	const char *tname;
 	int (*name_ok) (const char *);
@@ -370,15 +354,10 @@ getanswer (const querybuf *answer, int anslen, const char *qname, int qtype)
 			bp += sizeof(align) - ((u_long)bp % sizeof(align));
 
 			if (bp + n >= &hostbuf[sizeof hostbuf]) {
-				Dprintf("size (%d) too big\n", n);
 				had_error++;
 				continue;
 			}
 			if (hap >= &h_addr_ptrs[MAXADDRS-1]) {
-				if (!toobig++) {
-					Dprintf("Too many addresses (%d)\n",
-						MAXADDRS);
-				}
 				cp += n;
 				continue;
 			}
@@ -572,7 +551,6 @@ res_gethostbyname2_context (struct resolv_context *ctx,
 	      &buf.ptr, NULL, NULL, NULL, NULL)) < 0) {
 		if (buf.buf != origbuf)
 			free (buf.buf);
-		Dprintf("res_nsearch failed (%d)\n", n);
 		if (errno == ECONNREFUSED)
 			return (_gethtbyname2(name, af));
 		return (NULL);
@@ -671,7 +649,6 @@ res_gethostbyaddr_context (struct resolv_context *ctx,
 	if (n < 0) {
 		if (buf.buf != orig_buf)
 			free (buf.buf);
-		Dprintf("res_nquery failed (%d)\n", n);
 		if (errno == ECONNREFUSED)
 			return (_gethtbyaddr(addr, len, af));
 		return (NULL);