changeset 2625:3e7623dd431a

(canon_host): Use malloc and memcpy to copy an address, not strdup. Include <stdlib.h> and don't declare free().
author Jim Meyering <jim@meyering.net>
date Wed, 21 Jun 2000 13:00:49 +0000
parents 8d0959af71c4
children 1710f067f0d1
files lib/canon-host.c
diffstat 1 files changed, 5 insertions(+), 2 deletions(-) [+]
line wrap: on
line diff
--- a/lib/canon-host.c
+++ b/lib/canon-host.c
@@ -26,6 +26,9 @@
 #ifdef HAVE_UNISTD_H
 # include <unistd.h>
 #endif
+#ifdef HAVE_STDLIB_H
+# include <stdlib.h>
+#endif
 #ifdef HAVE_STRING_H
 # include <string.h>
 #endif
@@ -46,7 +49,6 @@
 #ifndef strdup
 char *strdup ();
 #endif
-void free ();
 
 /* Returns the canonical hostname associated with HOST (allocated in a static
    buffer), or 0 if it can't be determined.  */
@@ -80,11 +82,12 @@
 	     directly to gethostbyaddr because on some systems he->h_addr
 	     is located in a static library buffer that is reused in the
 	     gethostbyaddr call.  Make a copy and use that instead.  */
-	  char *h_addr_copy = strdup (he->h_addr);
+	  char *h_addr_copy = (char *) malloc (he->h_length);
 	  if (h_addr_copy == NULL)
 	    he = NULL;
 	  else
 	    {
+	      memcpy (h_addr_copy, he->h_addr, he->h_length);
 	      he = gethostbyaddr (h_addr_copy, he->h_length, he->h_addrtype);
 	      free (h_addr_copy);
 	    }