Скопируйте одну строку в другую

Question

У меня проблема с передачей текста члену структуры. Вот мой код

#include <stdio.h> 
#include <string.h> 


typedef struct { 
    char *name; 
    int rank; 
    int weapons; 

}player; 

player create_player(char name[], int rank, int weapons); 
void display_player(player data); 

int main (int argc, const char * argv[]) 
{ 
    player tmp = create_player("First", 3, 3); 
    display_player(tmp); 
} 

player create_player(char name[], int rank, int weapons) 
{ 
    player newobj; 
    char *tmp = malloc(strlen(name) + 1); 

    strcpy(newobj.name, tmp); 
    newobj.rank = rank; 
    newobj.weapons = weapons; 
    free(tmp); 
    return newobj; 
} 

void display_player(player data) 
{ 
    printf("Player name %s\n", data.name); 
    printf("Player rang %d\n", data.rank); 
    printf("Player weapons %d", data.weapons); 
} 

А вот выход

Player name 
Player rang 3 
Player weapons 3 

Как вы можете видеть результат «имя игрока» пусто. Также вы можете дать мне более элегантный способ присвоить текст newobj.name? Что я пропущу?

Answer 1

strcpy в вашем коде неверен (он копирует неинициализированные данные в нераспределенную память). Вы, вероятно, хотите:

newobj.name = malloc(strlen(name) + 1); 
strcpy(newobj.name, name); 

Или еще проще, если у вас есть strdup:

newobj.name = strdup(name);