'strcpy' avec 'malloc'?

Question

Est-il sécuritaire de faire quelque chose comme ce qui suit?

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

int main(void) 
{ 
    char* msg; 

    strcpy(msg, "Hello World!!!"); //<--------- 

    printf("%s\n", msg); 

    return 0; 
} 

Ou faut-il utiliser ce qui suit?

char* msg = (char*)malloc(sizeof(char) * 15); 

Answer 1

Votre code original n'assigne pas de message. Tenter de le strcpy serait mauvais. Vous devez allouer de l'espace avant de vous y connecter. Vous pouvez utiliser malloc comme vous le suggérez ou allouer de l'espace sur la pile comme ceci:

char msg[15]; 

Si vous malloc la mémoire, vous devez vous rappeler de le libérer à un moment donné. Si vous allouez sur la pile, la mémoire sera automatiquement renvoyée à la pile lorsqu'elle sortira de la portée (par exemple, la sortie de la fonction). Dans les deux cas, vous devez faire attention à allouer suffisamment pour pouvoir copier la chaîne la plus longue. Vous pourriez vouloir jeter un oeil à strncpy pour éviter de déborder le tableau.

Answer 2

La première version n'est pas sûre. Et, msg doit pointer vers l'emplacement de mémoire valide pour "Hello World !!!" pour être copié.

char* msg = (char*)malloc(sizeof(char) * 15); 
strcpy(msg, "Hello World!!!"); 

Answer 3

Vous devez allouer de l'espace. Utilisez malloc avant le strcpy.

Answer 4

strdup fait le malloc et strcpy pour vous

char *msg = strdup("hello world"); 

Answer 5

char* msg; 
strcpy(msg, "Hello World!!!"); //<---------Ewwwww 
printf("%s\n", msg); 

C'est UB. Pas de seconde pensée. msg est un pointeur sauvage et essayer de le déréférencer peut provoquer une erreur de segmentation sur votre implémentation.

msg à pointer vers un emplacement de mémoire valide assez grand pour contenir "Hello World".

Essayez

char* msg = malloc(15); 
strcpy(msg, "Hello World!!!"); 

ou

char msg[20]; 
strcpy(msg, "Hello World!!!"); 

Answer 6

Utilisation:

#define MYSTRDUP(str,lit) strcpy(str = malloc(strlen(lit)+1), lit) 

Et maintenant c'est facile et standard conforme:

char *s; 
MYSTRDUP(s, "foo bar");