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

This commit is contained in:
Daniel Stenberg 2004-05-27 06:42:48 +00:00
parent 46cdc83b81
commit ff0429a802
1 changed files with 9 additions and 5 deletions

View File

@ -140,7 +140,8 @@ int ares__get_hostent(FILE *fp, struct hostent **host)
memcpy(hostent->h_addr_list[0], &addr, sizeof(struct in_addr)); memcpy(hostent->h_addr_list[0], &addr, sizeof(struct in_addr));
hostent->h_addr_list[1] = NULL; hostent->h_addr_list[1] = NULL;
*host = hostent; *host = hostent;
free(line); if(line)
free(line);
return ARES_SUCCESS; return ARES_SUCCESS;
} }
free(line); free(line);
@ -150,18 +151,21 @@ int ares__get_hostent(FILE *fp, struct hostent **host)
/* Memory allocation failure; clean up. */ /* Memory allocation failure; clean up. */
if (hostent) if (hostent)
{ {
free((char *) hostent->h_name); if(hostent->h_name)
free((char *) hostent->h_name);
if (hostent->h_aliases) if (hostent->h_aliases)
{ {
for (alias = hostent->h_aliases; *alias; alias++) for (alias = hostent->h_aliases; *alias; alias++)
free(*alias); free(*alias);
} }
free(hostent->h_aliases); if(hostent->h_aliases)
free(hostent->h_aliases);
if (hostent->h_addr_list) if (hostent->h_addr_list)
free(hostent->h_addr_list[0]); 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; return ARES_ENOMEM;
} }