malloc() 을 이용한 메모리 할당 관련 - malloc()호출이 기존의 연결 리스트 영역에 영향을 미칠 수 있나요? ( 테스트한 내용 추가)

addr_list_entry *insert_addr(char *retrieved_url,addr_list_entry *insert_position)
{
addr_list_entry *new_entry=NULL;
char *for_url;

new_entry=(addr_list_entry *)malloc(sizeof(addr_list_entry));

// for_url을 MAX_URL_SIZE 만큼 할당 받을 필요가 있으신가요?
// 항상 MAX_URL_SIZE 만큼 할당을 받으실것이라면,
// addr_list_entry 의 struct에서 url을 char[MAX_URL_SIZE]로 하시는게 더 나을듯 한데...
// 보통은 malloc( strlen(retrieved_url) + 1 ) 만큼 할당받는게 malloc을 쓰는 의미가 있을듯 합니다.
// 아래에서 strcpy를 사용하는데, 만일 strlen(retrieved_url) > MAX_URL_SIZE 라면, overflow가 발생할 수 있습니다.
for_url=(char *)malloc(MAX_URL_SIZE);

if(new_entry==NULL || for_url==NULL)
{
inform("Memory allocation failed in addr_list.c:insert_addr()");
return insert_position;
}

//Clear memory space before use
memset(new_entry,'\0',sizeof(addr_list_entry));

// 위에서 만일 malloc( strlen(retrieved_url) + 1 )로 하셨다면,
// 이 부분의 MAX_URL_SIZE도 strlen(retrieved_url) + 1 로 변경하시길...
memset(for_url,'\0',MAX_URL_SIZE);

//------------------------------
//Set url value of structure
//------------------------------
strcpy(for_url,retrieved_url);

new_entry->url=for_url;

// insert_position->next 가 항상 NULL이라는것을 보장하는지 모르겠습니다.
// 저라면.. new_entry->next = NULL; 이라고 쓸듯...
new_entry->next=insert_position->next; //Attach address to the end of address list
insert_position->next=new_entry;

return new_entry;
}

int clear_addr_list(addr_list_entry *head)
{

// 아래의 pointer들은 malloc을 해줄 필요가 없습니다.
// head로 부터 pointing만 하기에...
// addr_list_entry *for_free,*for_check; 만 해주세요.
addr_list_entry *for_free=(addr_list_entry *)malloc(sizeof(addr_list_entry));
addr_list_entry *for_check=(addr_list_entry *)malloc(sizeof(addr_list_entry));

// 아래의 if 부분도 필요 없습니다. malloc을 할 필요가 없기에.
if((for_free==NULL) || (for_check==NULL))
{
fatal("Failure of memory allocation in clear_addr_list()");
}

// 만일, malloc을 하셨다면, 여기 부분에서 leak이 발생해 버립니다.
// 위에서 malloc을 제거하고 아래의 부분은 남겨두세요.
for_check=head->next;

// 아래의 while 구문을
// while( for_check != NULL ) 로 바꾸세요.
// 처음 들어왔을때, for_check->next는 head->next->next가 되어 버려서,
// 제일 앞에 있는 list는 무시가 될것입니다.
while(for_check->next != NULL)
{
for_free=for_check;
for_check=for_check->next;

free(for_free->url);
free(for_free);

// free를 해주었는데, memset을 하면 어떻게 동작할지 알 수 없습니다.
// memset을 할 필요가 없으니, 삭제하시구요.
memset(for_free->url,'\0',MAX_URL_SIZE);
memset(for_free,'\0',sizeof(addr_list_entry));
}

// 위에서 free해준 것을 또 free 하기에, 원하는대로 동작하는것을 보장 할 수 없습니다.
// 메모리가 꼬이게 되는 것이지요..
// 아래의 두줄은 삭제하세요.
free(for_free);
free(for_check);

// 아래의 이것만 남기세요.
free(head);

inform("clearing address list completed");

return 0;
}

원래의 위의 함수를 사용한다면, 메모리는 꼬여서 바보가 되어 있을 것입니다.

int print_list(addr_list_entry *list_head)
{
#if 0
addr_list_entry *entry=(addr_list_entry *)malloc(sizeof(addr_list_entry));
entry=list_head->next;
#endif

addr_list_entry *entry=list_head->next;

printf("\n");

// 아래의 while 구문을
// while( entry != NULL ) 로 바꾸세요.
// entry = entry->next; 로 해 놓고,
// while에서 entry->next != NULL 로 비교를 하면,
// 중간것을 빼먹게 될 것입니다.
while(entry->next != NULL)
{
printf("%s\n",entry->url);

entry=entry->next;
}

// free(entry);

return 0;
}

우선 올려주신 코드에서 제 나름(?)대로 문제가 될만한걸 정리해 드리긴 했는데요....

다른 코드들에서도 비슷한 상황(?)이 있을 듯 합니다.