C設置實現使用鏈接列表內存泄漏

Question

Valgrind堅持認爲這個函數有內存泄漏，但我無法找到它。這是c中使用鏈接列表的集合實現的一部分。

int set_add(set * s,int e[2]){ 
     if(set_empty(*s)) { 
       element * new=malloc(sizeof (element)); 
       new->coord[0]=e[0]; 
       new->coord[1]=e[1]; 
       new->next =NULL; 
       s->head=new; 
       return 1; 
     } 
     element * current=s->head; 
     while(current != NULL) { 
       if(coord_equal(current->coord,e)) { 
         return 0; 
       } 
       if(current->next ==NULL){ 
        break; 
       } 
       current=current->next; 
     } 
     element * new=malloc(sizeof (element)); 
     new->coord[0]=e[0]; 
     new->coord[1]=e[1]; 
     new->next = NULL; 
     current->next=new; 
     return 1; 
} 

Answer 1

我認爲做正確的事是每個malloc的驗證後，如果確實被分配，如果沒有，你應該釋放該內存區和退出功能。

事情是這樣的：

value = malloc(); 
if (value){ 
//value was allocated correctly 
//do the things you want with it 
free(value); 
} 
else{ 
return 0; //exit your function 
} 

希望這有助於。