From: Daniel Stenberg <daniel@haxx.se>
Date: Thu, 27 May 2004 06:42:48 +0000 (+0000)
Subject: James Bursa's fix to prevent free(NULL) to occur
X-Git-Tag: upstream/1.10.0~1425
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=d870be584d57130accf54602383c8325123fce4d;p=platform%2Fupstream%2Fc-ares.git

James Bursa's fix to prevent free(NULL) to occur
---

diff --git a/ares__get_hostent.c b/ares__get_hostent.c
index ecba75c..f05d712 100644
--- a/ares__get_hostent.c
+++ b/ares__get_hostent.c
@@ -140,7 +140,8 @@ int ares__get_hostent(FILE *fp, struct hostent **host)
       memcpy(hostent->h_addr_list[0], &addr, sizeof(struct in_addr));
       hostent->h_addr_list[1] = NULL;
       *host = hostent;
-      free(line);
+      if(line)
+        free(line);
       return ARES_SUCCESS;
     }
   free(line);
@@ -150,18 +151,21 @@ int ares__get_hostent(FILE *fp, struct hostent **host)
       /* Memory allocation failure; clean up. */
       if (hostent)
 	{
-	  free((char *) hostent->h_name);
+          if(hostent->h_name)
+            free((char *) hostent->h_name);
 	  if (hostent->h_aliases)
 	    {
 	      for (alias = hostent->h_aliases; *alias; alias++)
 		free(*alias);
 	    }
-	  free(hostent->h_aliases);
+          if(hostent->h_aliases)
+            free(hostent->h_aliases);
 	  if (hostent->h_addr_list)
 	    free(hostent->h_addr_list[0]);
-	  free(hostent->h_addr_list);
+          if(hostent->h_addr_list)
+            free(hostent->h_addr_list);
+          free(hostent);
 	}
-      free(hostent);
       return ARES_ENOMEM;
     }