Segmentation fault autour de strcpy?

Je sais que vous vous rap-moi sur les doigts mais.

Pourquoi faire la même erreur de Segmentation

char* cmd;
strcpy(cmd, argv[0]);

quand ce n'est pas

char *cmd;
cmd = "plop";

Je ne pratique pas depuis un moment, et ne me souviens pas pourquoi.

ps: en fait, je sais que quelque chose comme ça, avant de le strcpy, il serait préférable de

char *cmd = (char*) malloc(strlen(argv[0]));

mais je me demande pourquoi cette erreur de segmentation.

Merci !

double possible de Quel est le problème avec ce C (strcpy) code?
stackoverflow.com/questions/9677654/...

OriginalL'auteur roro | 2011-06-22

  1. 9

    Lorsque vous faites:

    char * cmd;

    Vous êtes allouer un pointeur sur la pile. Ce pointeur n'est pas initialisé de toute valeur.

    Puis, quand vous faites cela:

    strcpy(cmd, argv[0]);

    Vous copie la chaîne de caractères contenue dans argv[0] à l'adresse pointée cmd, qui est... quelque chose dénuée de sens. Puisque vous avez de la chance, c'est tout simplement de segmentation.

    Quand vous faites cela:

    cmd = "plop";

    Que vous attribuez à cmd l'adresse d'une chaîne statique constante. Depuis ces chaînes sont en lecture seule, écriture sur eux, c'est un comportement indéfini.

    Alors, comment résoudre ce problème? Allouer de la mémoire pour l'exécution d'écrire. Il y a deux façons:

    La première consiste à répartir les données sur la pile, comme ceci:

    char cmd[100]; //for instance

    Il alloue un tableau de 100 chars sur la pile. Cependant, il n'est pas nécessairement robuste, parce que vous devez savoir à l'avance la quantité de mémoire dont vous aurez besoin. La pile est aussi plus petit que le tas. Ce qui nous amène à l'option numéro 2:

    char *cmd = malloc(whatever_you_need); //no need to cast, by the way, unless you're in C++

    Il alloue whatever_you_need chars sur le tas. Ne pas oublier de libérer la mémoire avec free une fois que vous avez terminé avec elle.

    OriginalL'auteur Etienne de Martel

  2. 5

    Vous obtenez un seg. faute, parce que cmd dans votre premier exemple n'est pas de pointage à rien (ou, plutôt, il pointe vers quelque chose d'indéfini - afin de tenter de lire les caractères à partir de ou écrire des caractères dans le pointeur probablement entraîner une violation d'accès).

    Dans le deuxième exemple, la configuration de cmd pour point de légitime d'une chaîne de caractères.

    OriginalL'auteur Will A

  3. 3

    Si vous voulez faire une copie de argv[0] facilement,

    char* cmd = strdup(argv[0]);

    Bien sûr, vous avez mieux à vérifier le résultat de strdup est null ou pas. 🙂

    merci, c'est utile !

    OriginalL'auteur mattn

  4. 1

    je me demande pourquoi cette erreur de segmentation.

    Parce que si cmd est une variable globale, sa valeur est NULL, ce qui n'est pas accessible en écriture, et si c'est une variable locale, alors sa valeur est indéterminée et vous ne devriez pas l'utiliser (mais il peut faire quelque chose si vous le faites, ce qui est pire que NUL, dans de nombreux cas).

    OriginalL'auteur Pascal Cuoq