Discussion :

[omedo]

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

Code C :

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
 
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

[Celelibi]

Outch, ça pique.

Tiens, regarde l'erreur

Code :

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

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 :

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

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 C :

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

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

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

Code C :

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

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.

[Sve@r]

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

Code C :

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

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 C :

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

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

[Celelibi]

Pourquoi tout ça pour ça ???

Code C :

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

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.

[omedo]

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