erreur de realloc

Version imprimable

erreur de realloc

Bonjour j essaie de creer une fonction concat qui me concat des chaines de caracteres.
Voila comment je fais :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 char *concat(int nb, ...) { va_list vl; int i; char *res = ""; char *tmp; va_start(vl, nb); for (i = 0; i < nb; i++) { tmp = va_arg(vl, char *); res = (char *) realloc(res, sizeof(tmp) + sizeof(res)); sprintf(res, "%s%s", res, tmp); } return res; }
cela me dit

Code:

*** glibc detected *** realloc(): invalid pointer: 0x0804860c ***

d ou vient le problem ?

Merci

06/07/2006, 14h30
Mr_Chut

c est peut etre pas tres propre, mais j ai creer une variable intermediaire pour ensuite faire un malloc. :)

Pour le pointeur invalide, je dois avouer que je ne vois pas trop. Mais pour le reste, ce n'est pas étonnant que ça plante:
Tu utilises des sizeof sur des pointeurs, il te faut utiliser strlen() pour cela.
Quant au sprintf(), c'est vraiment déconseillé de l'utiliser comme ça. La fonction strcat() est encore préférable...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
/* On ne doit pas affecter directement 
   le retour de realloc() au pointeur original :
   Sinon, en cas d'échec, on le perd.*/
/* Retourne 0 si OK, -1 si échec
   (comme les fonctions standard/POSIX) */
int MyReallocChar(char **ppC, int nTaille) {
	char *pTmpC = realloc(*ppC, nTaille);
	if(pTmpC != NULL) {
		*ppC = pTmpC;
	}
	return ((pTmpC != NULL) ? 0 : -1);
}
 
char *concat(int nb, ...) {
	va_list vl;
	int i;
	/* ou NULL, mais toujours un pointeur du même type:
	   ici, un pointeur libérable avec free(). */
	char *pRes = strdup(""); 
	char const *pcTmp = NULL;
 
	va_start(vl, nb);
 
	for (i = 0; i < nb; i++) {
		pcTmp = va_arg(vl, char const*);
		if(MyRealloc(&pRes, strlen(pcTmp) + strlen(pRes) + 1)) {
			return pRes; /* N'a pas pu agrandir : stop (ou erreur) */
		}
		strcat(pRes, pcTmp);
	}
	va_end(vl);
	return pRes;
}

Je n'ai pas testé ce code, mais ça devrait marcher...

06/07/2006, 16h48
Mr_Chut

le strlen c est une erreur de frappe (j ai recopier la fonction qui etait sur un ordinateur non relie au net :p).

Merci pour le conseil du sprintf.

pour le realloc tu rajoute "+ 1" pour la fin de la chaine ?
si oui, le realloc initialise implicitement le nouvel espace memoire a '\0' ?
07/07/2006, 09h19
Médinoc

Oui je rajoute +1 pour le \0, et pour le realloc(), je ne sais plus, je ne sais même plus si c'est spécifié dans la norme.

Mais ce n'est pas grave, puisque le \0 de fin de la chaîne précédente est gardé.
09/07/2006, 01h42
fearyourself

Citation:

Envoyé par Mr_Chut

si oui, le realloc initialise implicitement le nouvel espace memoire a '\0' ?

Non, voir le man de realloc : man

Le contenu de la zone de mémoire nouvellement allouée n'est pas initialisé

Jc