Segfault memcpy thread

Bonjour,
ma fonction qui gere mes thread lance cette fonction pour rajouter un prefixe a ma var id

Code:

---

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

void RecupRes(char* id){ char* prefixe; int i; char* nomTable; prefixe="prefixe"; i=strlen(id)+1; char* test; char* concat=(char*)malloc(sizeof(char)*(strlen(prefixe)+i)); char* concat2=(char*)malloc(sizeof(char)*(strlen(prefixe)+i)); concat=prefixe; concat2=memcpy(concat,id,i); printf("Nom table :[%s]\n",concat2); if (concat!=NULL){ free(concat); concat=NULL; } if (concat2!=NULL){ free(concat2); concat2=NULL; }

---

Le problème est que j'ai une erreur de segmentation au niveau de memcopy valgrind me dit plus precisement
Bad permission for mapped region at address ....
Mais le error summary me dit 0 errors from 0 contexts
Pourriez vous m'aider a resoudre ce probleme car je ne vois pas quoi faire pour mapped correctement

En vous remerciant

Omedo

Outch, ça pique.

Tiens, regarde l'erreur

Code:

---

concat=prefixe;

---

Tu copies le pointeur. Du coup concat ne pointe plus vers la mémoire que tu as mallocé mais vers la chaîne constante "prefixe". Ensuite tu essayes d'écrire dedans, forcément ça segfault.

De plus tu essayes d'écraser les premiers octets de "prefixe" au lieux de concaténer.

Toujours très étrange :

Code:

---

concat2=memcpy(concat,id,i);

---

memcpy renvoie la valeur de son premier argument. Soit le pointeur concat. Du coup tu as concat2 = concat. Et donc à quoi sert le deuxième malloc ?

Si tu veux juste te contenter d'afficher le nom de la table, tu peux faire :

Code:

---

printf("Nom table :[%s%s]\n", prefix, id);

---

Moi je te propose ce code qui me semble un peu moins le bazar.

Code:

---

1 2 3 4 5 6 7 8 9 10 11

void recup_res(const char *id) { const char *pre = "prefix"; char *full_name = malloc(sizeof(char) * (strlen(pre) + strlen(id) + 1));   assert(full_name != NULL);   strcpy(full_name, pre); strcat(full_name, id); printf("Nom de la table : [%s]\n", full_name); free(full_name); }

---

Remarque les const, le test du malloc juste à son retour, l'utilisation de strcat à la place de strlen+memcpy. Et strcpy+strcat pour concaténer le préfixe et l'id.

Citation:

---

Envoyé par Celelibi

Moi je te propose ce code qui me semble un peu moins le bazar.

Code:

---

1 2 3 4 5 6 7 8 9 10 11

void recup_res(const char *id) { const char *pre = "prefix"; char *full_name = malloc(sizeof(char) * (strlen(pre) + strlen(id) + 1));   assert(full_name != NULL);   strcpy(full_name, pre); strcat(full_name, id); printf("Nom de la table : [%s]\n", full_name); free(full_name); }

---

Remarque les const, le test du malloc juste à son retour, l'utilisation de strcat à la place de strlen+memcpy. Et strcpy+strcat pour concaténer le préfixe et l'id.

---

Pourquoi tout ça pour ça ???

Code:

---

printf("Nom de la table : [prefix%s]\n", id);

---

Citation:

---

Envoyé par Sve@r

Pourquoi tout ça pour ça ???

Code:

---

printf("Nom de la table : [prefix%s]\n", id);

---

---

J'espère que son code était une version simplifiée et qu'il fait quelque chose d'autre de la chaîne préfixée.

oui effectivement je faisais autre chose que l'afficher à l’écran :ccool:
j'avais trouver une autre solution mais la tienne est effectivement très intéressante