Динамически распределенный список динамически выделяемых структур переписывается

Question

В следующем коде я не могу понять, насколько я должен использовать тот же результат (то есть то же имя) при запуске метода dump_data() после добавления 2 (или более) User_struct (с разными именами) с помощью метода addUser().

struct Friends_struct { 
    struct User_struct *this; 
    struct Friends_struct *next; 
}; 

typedef struct Friends_struct *Friends;  

typedef struct User_struct { 
    const char *name; 
    Friends amigos; 
} User; 

static User **userList;    // global table of users 
static int numUsers;    // global number of users 
static int tableSize;    // global size of table 


void create_amigonet() {  
    tableSize = INIT_TABLE_SIZE; 
    numUsers = 0; 

    userList = malloc(tableSize * sizeof(User *)); 
} 

void addUser(const char *name) { 
    userList[numUsers] = malloc(sizeof(User)); 
    userList[numUsers]->name = name; 
    userList[numUsers]->amigos = 0; 

    numUsers++; 

    if (numUsers == tableSize) { 
     tableSize += INIT_TABLE_SIZE; 

     User **moreUsers; 
     moreUsers = realloc(userList, tableSize * sizeof(User *)); 
     userList = moreUsers; 
    } 
} 

void dump_data() { 
    for (int i=0; i<numUsers; i++) { 
     printf("%s; friends:", userList[i]->name); 
    } 

    printf("\n"); 
} 

Различные .c файл:

void do_add(const char *name) { 
    addUser(name); 
} 

int main(void) { 
    create_amigonet(); 
    do_add("Dan"); 
    do_add("Jim"); 
    dump_data(); 
} 

Если это выполняется, то выход я ожидал бы что-то вроде

Dan; friends: 
Jim; friends: 

пока я получаю

Jim; friends: 
Jim; friends: 

Как/где я переписываю свои данные?

Answer 1

Проблема в том, что вы храните только указатель char в User_struct, а указатели, которые вы передаете в звонках addUser, являются временными. Вы должны изменить тип name на массив достаточной длины и использовать strcpy для копирования имени в пределах addUser.

Я не могу полностью объяснить, почему пример действительно показывает ошибку, потому что я думал, что обе строки имен должны быть действительными в течение main(), но кажется, что сгенерированный код повторно использует место, где хранился «Дэн», и поэтому вы получаете «Джим» дважды после второго звонка addUser.