strdupを使用してCでリンクリストをコピー

Question

私は元のリンクをコピーして保存するリンク先のデータ構造を持っています。だから私はコピーされたリンクリストに影響を与えずに元のリンクリストを編集することができます。私は次の方法を試して、私はセグメンテーションフォールトを持っています。

struct link { 
     char * name; 
     struct link *next; 
    }; 


struct list{ 
     struct link *first; 
     struct link *last; 
    }; 



struct list *list_new(){ 
    struct list *n = calloc(1, sizeof(struct list)); 
    return n; 
}; 




struct list* copyList(struct list*list){ 
    struct list*new = list_new(); 
    struct link *current = list -> first; 
    struct link *newCurrent = new -> first; 
    struct link *p; 
     if(current == NULL) 
     return NULL; 


     newCurrent = malloc(sizeof(struct link)); 
     newCurrent = p; 

     while (current != NULL) { 
     p ->name = strdup((char*)current -> name); 
     p->next = malloc(sizeof(struct link)); 
     p = p->next; 
     current = current->next; 
     } 

     return new; 
    } 

Answer 1

は、私はあなたのような何かをしたいと思う：

struct list *copyList(struct list *list) 
{ 
    struct list *new = list_new(); 
    struct link *current = list->first; 
    struct link *newCurrent = malloc(sizeof(struct link)); 

    if((current == NULL) || (newCurrent == NULL) || (new == NULL)) 
    { 
     if(newCurrent != NULL) 
     { 
      free(newCurrent); 
     } 
     if(new != NULL) 
     { 
      free(new); 
     } 

     return NULL; 
    } 
    else 
    { 
     new->first = newCurrent; 
    } 


    while(current != NULL) 
    { 
     newCurrent->name = strdup((char*)current -> name); 
     current = current->next;   
     if(current != NULL) 
     { 
      newCurrent->next = malloc(sizeof(struct link)); 
      newCurrent = newCurrent->next; 
     } 
     else 
     { 
      newCurrent->next = NULL; 
      new->last = newCurrent; 
     } 
    } 

    return new; 
}