Erreur de segmentation en manipulant une chaîne

Bonjour,
la compilation de mon code ne pose pas de problème et j'ai une erreur de segmentation à l’exécution.
Je voudrais simplement modifier un caractère dans une chaîne.
Voilà mon code :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21

#include <stdlib.h> #include <stdio.h> #include <string.h>   int clr(char**);   int main() { char* t="azerty"; printf("%s\n",t); printf("%d\n",clr(&t)); return 0; }   int clr(char** ch) { char* str; str=*ch; str[1] = 'C'; return 0; }

---

Quelqu'un pourrait m'éclairer ?
Merci.

Noe,

La question est : pourquoi passer un type char **, qui est un tableau de chaînes, alors qu'ailleurs tu utilises correctement le classique char* ?

:)

Le passage d'un pointeur sur pointeur sur char (légèrement différent d'un tableau de pointeur) n'a en effet pas d'intérêt, car tu modifies la zone pointée et non le pointeur lui-même.

Pour une erreur de segmentation ? Car t pointe vers une zone en mémoire constante et que tu ne peux pas y écrire. Remplacer ta déclaration par les deux lignes suivantes pour voir la différence :

Code:

---

1 2	char* t= malloc(100); strcpy(t, "azerty");

---

Citation:

---

Envoyé par Bktero

Remplacer ta déclaration par les deux lignes suivantes pour voir la différence :

Code:

---

1 2	char* t= malloc(100); strcpy(t, "azerty");

---

---

Ou plus simplement par:

Code:

---

char t[] = "azerty";

---

:)

EDIT: Sans ta fonction avec **

Citation:

---

Envoyé par Bktero

Remplacer ta déclaration par les deux lignes suivantes pour voir la différence :

Code:

---

1 2	char* t= malloc(100); strcpy(t, "azerty");

---

---

Voire :

Code:

---

char t[] ="azerty";

---

S‘il peut utiliser un tableau à la place de son pointeur.
S‘il a absolument besoin d‘un pointeur, il faut rester sur ta solution.

Au passage, le prototype du main n‘est pas standard (en C++ ok, en C non) :

Code:

---

1 2 3

toto.c:7:5: attention : function declaration isn’t a prototype [-Wstrict-prototypes] toto.c: In function ‘main’: toto.c:7:5: attention : old-style function definition [-Wold-style-definition]

---

Ça devrait être

Code:

---

int main(void)

---

pas

Code:

---

int main()

---

En C, une fonction qui prend void en paramètre c‘est TRÈS différent d‘une fonction avec une liste d’arguments vide…

Le tableau étant implicitement convertible en pointeur, ceci marcherait:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20

#include <stdlib.h> #include <stdio.h> #include <string.h>   int ChangeSecondCaractere(char*);   int main(void) { char t[] ="azerty"; printf("%s\n",t); printf("%d\n", ChangeSecondCaractere(t)); printf("%s\n",t); return 0; }   int ChangeSecondCaractere(char* str) { str[1] = 'C'; return 0; }

---

Si j'ai mis :

Code:

---

1 2	char* t= malloc(100); strcpy(t, "azerty");

---

et non :

Code:

---

char t[] ="azerty";

---

c'est qu'il y a une grande différence entre les deux. Le premier donne un pointeur vers une zone constante (ou se trouve la chaine littérale) ; le second crée un tableau (donc une zone modifiable) et l'initialiser avec la chaine littérale. Ici, on voulait bien montrer le problème sur un pointeur.

Euh, les deux exemples de code de ton message (#7) créent une zone mémoire modifiable initialisée avec "azerty"... La seule différence est que le premier code crée une zone de 100 octets alors que le second code en crée une juste à la taille de la chaîne...

(et bien sûr la différence pile/tas, qu'il faut faire un free() sur le premier pour éviter une fuite de mémoire)

Oui, bien sûr :)

Je trouvais juste que ça se rapprochait plus du cas original. Mais je préfère ton code !

Désolé, un problème matériel m'a empêché de me connecter jusqu'à ce matin.
Je viens de lire tous les messages, je comprends le problème (je dois encore approfondir la question de la gestion de la mémoire). Merci à tous.