Discussion :

[tiresias54]

Bonjour à tous,

J'ai une segmentation fault sur le sprintf lorsque j'execute le code suivant:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
#include <stdio.h>
 
int main () {
 
    char *pt="Pb0";
    sprintf(pt,"Pb0%1d",1);
 
    return 0;
}

quelqu'un voit pourquoi?

De plus la ligne

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

char *pt="Pb0";

me surprend: ne serait-il pas plus correct d'écrire:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
char *pt=NULL;
*pt="Pb0";

?

Merci d'avance ^^

[deletme]

Salut,

J'ai une segmentation fault sur le sprintf lorsque j'execute le code suivant:

Tu manipules une chaine de caractère à l'aide d'un pointeur, il faut donc faire attention à l'endroit où tu vas écrire et la quantité que tu vas écrire. Ici le dernier point est respecté... mais pas le premier

Tu donnes a ta fonction sprintf l'adresse du pointeur (qui est à la discrétion de ton compilateur) et non l'adresse de ta chaine :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
#include <stdio.h>
 
int main () {
 
    char *pt="Pb0";
    sprintf(&pt,"Pb0%1d",1);
 
    return 0;
}

L'adresse sur laquelle pointe pt est caractérisée par &pt.

[Bktero]

Bonjour,

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
char *pt="Pb0";
sprintf(pt,"Pb0%1d",1)

La string litteral "Pb0" est placée en mémoire constante. Le pointeur vise donc une zone mémoire qui n'est pas modifiable. Le sprintf() essaye et se fait bouler : segmentation fault. Voir la : http://c.developpez.com/faq/?page=st...RINGS_pointeur

EDIT : d'ailleurs le compilateur te donne une indication sur la ligne de création de la variable :

D:\main.c|5|warning: initialization discards 'const' qualifier from pointer target type [enabled by default]|

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
char *pt=NULL;
*pt="Pb0";

Un compilateur bien réglé émet un warning sur la seconde ligne car tu essayes de mettre une adresse (celle de la chaine littérale) dans un char:

D:\main.c|9|warning: assignment makes integer from pointer without a cast [enabled by default]|

Et l'exécution plantera. Ici, tu tentes de déréfencer un pointeur qui ne pointe vers rien (NULL). Tu ne peux pas écrire à cet endroit : crash.

Dans les deux cas, il faut un pointeur vers une zone modifiable : malloc() ou un char[].


EDIT : deletme, je pense que tu as tord. Ton code donne l'adresse du pointeur, pas celui du PO. Ce qui me fait dire ça ? Ton code compile avec des erreurs :

D:\main.c||In function 'main': |
D:main.c|5|warning: initialization discards 'const' qualifier from pointer target type [enabled by default]|
D:\main.c|6|warning: passing argument 1 of 'sprintf' from incompatible pointer type [enabled by default]|
c:\program files (x86)\codeblocks\mingw\bin\..\lib\gcc\mingw32\4.7.1\..\..\..\..\include\stdio.h|262|note: expected 'char *' but argument is of type 'char **'|
||=== Build finished: 0 errors, 2 warnings (0 minutes, 0 seconds) ===|

[deletme]

Oui en effet, en reconfigurant correctement mon compilateur (nouvelle installation ) ça warning ^^

Pour rester en statique, passe par un tableau (de taille suffisante) sinon tu peux faire de l'allocation dynamique.

EDIT :

Bon, en prêtant un peu plus attention à mon code généré, voici deux alternatives :

En dynamique :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
#include <stdio.h>
#include <stdlib.h>
 
int main () {
 
    char *pt;
 
    pt = malloc(sizeof(char) * 10); // allocation dynamique de l'espace mémoire
 
    if(pt == NULL)
        return EXIT_FAILURE; // l'allocation a échoué
    else
    {
        sprintf(pt,"test");
        printf("%s\r\n", pt);
    }
 
    free(pt); // libération de la mémoire allouée dynamiquement
 
    return EXIT_SUCCESS;
}

En statique :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
#include <stdio.h>
 
int main () {
 
    char pt[10];
 
    sprintf(pt,"test");
    printf("%s\r\n", pt);
 
    return EXIT_SUCCESS;
}

[Bktero]

Je préfère ça

Deux petites remarques : je pense que tu n'as pas besoin d''\r', '\n' est converti comme il faut par le C ; puts(pt) c'est plus court.

Il faut toutefois faire attention avec ce genre de code à ne pas écrire trop avec sprintf() car si on dépasse de la zone réservée, on risque (encore) une erreur de segmentation. Soit on fait attention, soit on utilise snprintf().

"Un petit vote négatif m'apprendra à répondre trop vite ^^" --> Si tu insistes...

[tiresias54]

merci pour vos explications, c'est beaucoup plus clair maintenant

[Bktero]

Si le problème est résolu, merci de cliquer sur le bouton en bas de l'écran

[Mipwiq]

Bon, en prêtant un peu plus attention à mon code généré, voici deux alternatives :

En dynamique :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
#include <stdio.h>
#include <stdlib.h>
 
int main () {
 
    char *pt = malloc(sizeof(char) * 10);
 
    sprintf(pt,"test");
    printf("%s\r\n", pt);
 
    return 0;
}

Fait attention quand tu utilises malloc n'oublie pas de n'allouer que le nécessaire et surtout de free à la fin de l'utilisation du pointeur.

EDIT: Aussi ne jamais oublier les valeurs de retour.

[deletme]

Fait attention quand tu utilises malloc n'oublie pas de n'allouer que le nécessaire et surtout de free à la fin de l'utilisation du pointeur.

oui tu as raison, les modifications ont été apportées au code.