From: serassio <>
Date: Sun, 23 Jan 2005 19:39:45 +0000 (+0000)
Subject: Bug #1136: Internal DNS sometimes truncates host names in reverse (PTR)
X-Git-Tag: SQUID_3_0_PRE4~897
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=cb4e6a9036117351da2a52717f06df2f7499dc67;p=thirdparty%2Fsquid.git

Bug #1136: Internal DNS sometimes truncates host names in reverse (PTR)
lookups

Compressed hostnames was not dealt with proper, only saving the data of
the initial string + 2 bytes.
---

diff --git a/lib/rfc1035.c b/lib/rfc1035.c
index bad96e9c4c..c07ddf654f 100644
--- a/lib/rfc1035.c
+++ b/lib/rfc1035.c
@@ -1,6 +1,6 @@
 
 /*
- * $Id: rfc1035.c,v 1.31 2004/04/11 09:15:11 hno Exp $
+ * $Id: rfc1035.c,v 1.32 2005/01/23 12:39:45 serassio Exp $
  *
  * Low level DNS protocol routines
  * AUTHOR: Duane Wessels
@@ -554,11 +554,10 @@ rfc1035AnswersUnpack(const char *buf,
 	    return -rfc1035_unpack_error;
 	}
     }
-    i = (int) hdr.ancount;
-    if (i == 0)
+    if (hdr.ancount == 0)
 	return 0;
-    recs = calloc(i, sizeof(*recs));
-    while (i--) {
+    recs = calloc((int)hdr.ancount, sizeof(*recs));
+    for (i = 0; i < (int)hdr.ancount; i++) {
 	if (off >= sz) {	/* corrupt packet */
 	    RFC1035_UNPACK_DEBUG;
 	    break;