Как освободить указатель в структуре?

Question

Это мой код:

typedef struct bobTheBuilder{ 
    char *name; 
    int fix; 
    int max; 
}; 

int main(void){ 

    struct bobTheBuilder bob; 
    initBob(&bob); 
    del(&bob); 

    system("PAUSE"); 
    return (0); 
} 

void initBob(struct bobTheBuilder *currBob) 
{ 
    char *n="bob"; 
    currBob->name = (char*)malloc(sizeof(char)*strlen(n)); 
    strcpy((*currBob).name, n); 
    (*currBob).fix = 0; 
    (*currBob).max = 3; 
} 

void del(struct bobTheBuilder *currBob) 
{ 
    free(currBob->name); 
} 

Визуальные перерывы студии на free предложения.

Что мне делать? Является ли проблема с free или malloc?

Answer 1

Линия

currBob->name = (char*)malloc(sizeof(char)*strlen(n)); 

неправильно, потому что

1. Вы не включили пространство для NUL-терминатор.
2. Вы should not cast the result of malloc(and family) в С.

Фикс проблемы с помощью

currBob->name = malloc(strlen(n) + 1); 

Если вы задаетесь вопросом, почему я удалил sizeof(char), это потому, что sizeof(char) является guarenteed быть 1. Таким образом, это не обязательно.

---

@EdHeal mentions in a comment, есть функция strdup(), которая делает malloc + strcpy. Это функция POSIX. Если он доступен, вы можете сократить

currBob->name = malloc(strlen(n) + 1); 
strcpy((*currBob).name, n); 

в

currBob->name = strdup(n); 

с помощью этой функции. Кроме того, обратите внимание, что

(*currBob).fix = 0; 
(*currBob).max = 3; 

эквивалентно

currBob -> fix = 0; 
currBob -> max = 3; 

в @Edhealmentions in another comment.