This is the mail archive of the
cygwin-patches
mailing list for the Cygwin project.
[Patch] gethostby_helper
- From: "Pierre A. Humblet" <phumblet at phumblet dot no-ip dot org>
- To: cygwin-patches at cygwin dot com
- Date: Tue, 16 Aug 2011 19:38:26 -0400
- Subject: [Patch] gethostby_helper
This patch has already been already applied.
Diff in attachment and also below.
Pierre
2011-08-16 Pierre Humblet <Pierre.Humblet@ieee.org>
* net.cc (gethostby_helper): Remove DEBUGGING code from and
streamline the second pass.
Index: net.cc
===================================================================
RCS file: /cvs/src/src/winsup/cygwin/net.cc,v
retrieving revision 1.289
diff -u -p -r1.289 net.cc
--- net.cc 4 Aug 2011 08:22:11 -0000 1.289
+++ net.cc 16 Aug 2011 23:31:44 -0000
@@ -1198,57 +1198,34 @@ gethostby_helper (const char *name, cons
string_ptr = (char *) (ret->h_addr_list + address_count + 1);
/* Rescan the answers */
- ancount = alias_count + address_count; /* Valid records */
alias_count = address_count = 0;
+ prevptr->set_next (prevptr + 1);
- for (i = 0, curptr = anptr; i < ancount; i++, curptr = curptr->next ())
+ for (curptr = anptr; curptr <= prevptr; curptr = curptr->next ())
{
antype = curptr->type;
if (antype == ns_t_cname)
{
- complen = dn_expand (msg, eomsg, curptr->name (),
string_ptr, string_size);
-#ifdef DEBUGGING
- if (complen != curptr->complen)
- goto debugging;
-#endif
+ dn_expand (msg, eomsg, curptr->name (), string_ptr,
curptr->namelen1);
ret->h_aliases[alias_count++] = string_ptr;
- namelen1 = curptr->namelen1;
- string_ptr += namelen1;
- string_size -= namelen1;
- continue;
+ string_ptr += curptr->namelen1;
}
- if (antype == type)
+ else
+ {
+ if (address_count == 0)
{
- if (address_count == 0)
- {
- complen = dn_expand (msg, eomsg, curptr->name(),
string_ptr, string_size);
-#ifdef DEBUGGING
- if (complen != curptr->complen)
- goto debugging;
-#endif
- ret->h_name = string_ptr;
- namelen1 = curptr->namelen1;
- string_ptr += namelen1;
- string_size -= namelen1;
- }
- ret->h_addr_list[address_count++] = string_ptr;
- if (addrsize_in != addrsize_out)
- memcpy4to6 (string_ptr, curptr->data);
- else
- memcpy (string_ptr, curptr->data, addrsize_in);
- string_ptr += addrsize_out;
- string_size -= addrsize_out;
- continue;
- }
-#ifdef DEBUGGING
- /* Should not get here */
- goto debugging;
-#endif
+ dn_expand (msg, eomsg, curptr->name (), string_ptr,
curptr->namelen1);
+ ret->h_name = string_ptr;
+ string_ptr += curptr->namelen1;
+ }
+ ret->h_addr_list[address_count++] = string_ptr;
+ if (addrsize_in != addrsize_out)
+ memcpy4to6 (string_ptr, curptr->data);
+ else
+ memcpy (string_ptr, curptr->data, addrsize_in);
+ string_ptr += addrsize_out;
+ }
}
-#ifdef DEBUGGING
- if (string_size < 0)
- goto debugging;
-#endif
free (msg);
@@ -1263,16 +1240,6 @@ gethostby_helper (const char *name, cons
Should it be NO_RECOVERY ? */
h_errno = TRY_AGAIN;
return NULL;
-
-
-#ifdef DEBUGGING
- debugging:
- system_printf ("Please debug.");
- free (msg);
- free (ret);
- h_errno = NO_RECOVERY;
- return NULL;
-#endif
}
/* gethostbyname2: standards? */Attachment:
net.cc.diff
Description: Binary data