Discussion :

[Mr_Chut]

bonsoir,

Je me suis fait un code qui permet de concatener des chaine de caracteres.

Voici le code:

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
22
23
24
25
26
27
28
29
30
31
32
char *str_concat(char *cs, ...) {
	va_list va;
	const char *ct;
	char *s;
	size_t size;
 
	size = 1;
	s = NULL;
 
	va_start(va, cs);
	while ((ct = va_arg(va, char *)) != NULL) 
		size += strlen(ct);
 
	s = (char *) calloc(size * sizeof(char), sizeof(char));
 
	if (s == NULL)
	{
		printf("in str_concat : malloc() failed.\n");
		fflush(stdout);
	}
 
	else
	{
		va_start(va, cs); 	
		strcpy(s, cs);
 
		while ((ct = va_arg(va, char *)) != NULL) 
			strcat(s, ct);
	}
 
	return s;
}

lorsque je l utilise je fais par exemple :

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
22
 
void init_dir(char *rep_dir_path, list_t l) 
{
	int path_len;
	char *path;
 
	path_len = strlen(rep_dir_path);
 
	mkdir(rep_dir_path, S_IRWXU);
 
	path = str_concat(rep_dir_path, "/Divers", NULL);
	mkdir(path, S_IRWXU);
 
	while (fdt_list != NULL) 
	{
		path = str_concat(rep_dir_path, "/", l->name, NULL);
		mkdir(path, S_IRWXU);
		l = l->next;
		free(path);
	}
 
}

list_t est une liste de chaine de caracteres.

Ca me fait

Code :

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

1
2
*** glibc detected *** free(): invalid next size (fast): 0x08053298 ***
Aborted

Je ne comprend pas pourquoi .... Meme sans le free cela me fait une erreure qui est

Code :

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

1
2
*** glibc detected *** malloc(): memory corruption: 0x08052cc8 ***
Aborted

Merci

[Emmanuel Delahaye]

Je me suis fait un code qui permet de concatener des chaine de caracteres.

Ca me fait

Code :

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

1
2
*** glibc detected *** free(): invalid next size (fast): 0x08053298 ***
Aborted

<...>

Ceci fonctionne :

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
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
 
#include <stdio.h>
#include <stdlib.h>
#include <stdarg.h>
#include <string.h>
 
static char *str_concat (char *cs,...)
{
   char *s = NULL;
   size_t size = 1;
   {
      const char *ct;
      va_list va;
      va_start (va, cs);
      while ((ct = va_arg (va, char *)) != NULL)
      {
         size += strlen (ct);
      }
      va_end (va);
   }
   printf ("size = %lu\n", (unsigned long) size);
   s = malloc (size);
 
   if (s == NULL)
   {
      printf ("in str_concat : malloc() failed.\n");
   }
   else
   {
      const char *ct;
      va_list va;
      va_start (va, cs);
      strcpy (s, cs);
 
      while ((ct = va_arg (va, char *)) != NULL)
      {
         strcat (s, ct);
      }
      va_end (va);
   }
   return s;
}
 
int main ()
{
   char *s = str_concat ("a", "bc", "def", NULL);
 
   if (s != NULL)
   {
      printf ("'%s'\n", s);
      free (s), s = NULL;
   }
 
   return 0;
}

[Mr_Chut]

merci je vais essayer.

Tu pourrais me dire l utilisation de static pour une fonction, car je l utilise jamais, et je ai que trouver une explication pour son utilisation avec une variable.

Merci.

[Mr_Chut]

ok static sert juste a etre vu par les autres fonctions present dans le fichier.
Donc je dois l enlever pour qu il soit utiliser autre part.


Dis moi il ne faudrait pas mettre le charactere '\0' a la fin ??

[Mr_Chut]

j ai toujours l erreur :

Code :

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

1
2
*** glibc detected *** free(): invalid next size (fast): 0x08053298 ***
Aborted

Je ne sais vraiment pas a quoi cela est du

[Emmanuel Delahaye]

j ai toujours l erreur :

Code :

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

1
2
*** glibc detected *** free(): invalid next size (fast): 0x08053298 ***
Aborted

Je ne sais vraiment pas a quoi cela est du

Un problème ailleurs. (Débordement de tableau, par exemple...)

Essaye d'isoler le code qui provoque ce problème à coup de

Code :

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

1
2
3
 
#if 0
#endif

c'est la méthode la plus rapide pour trouver ce genre de bug...

[DaZumba]

Ceci fonctionne :

Juste pour faire joli, je rajouterais un premier test pour assurer que cs n'est pas NULL, histoire de ne pas avoir de coredump sur un appel du genre:

Code :

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

1
2
 
s = str_concat(NULL);

Il suffit que str_concat() renvoie NULL dans ce cas.

[Médinoc]

Code C à éviter :

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

s = (char *) realloc(s, sizeof(char) * (size + 1));

Il ne faut pas faire ça, car si realloc() retourne NULL, s est toujours alloué mais perdu (fuite mémoire). Tu dois utiliser un temporaire.

Code C non-testé :

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
/* Retourne 0 si OK et -1 si erreur
   Nécessite un cast si le pointeur de départ n'est pas void*
   (mais bon, on peut toujours faire des wrappers pour les types courants) */
int MonRealloc(void ** ppData, size_t taille)
{
	void *ptr = realloc(*ppData, taille);
	if(ptr != NULL)
		*ppData = ptr;
	return (ptr==NULL ? -1 : 0);
}
 
/* Et comme je suis de bonne humeur, un petit wrapper pour char */
static inline /*si supporté*/ int MonReallocChar(char ** ppData, size_t taille)
{
	return MonRealloc((void **)ppData, taille);
}

Code C suspect :

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

size += strlen(ct) + strlen(s);

un size += strlen(ct); (ou bien size = strlen(ct) + strlen(s) + 1;) ne suffirait-il pas ?

[Mr_Chut]

Code C à éviter :

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

s = (char *) realloc(s, sizeof(char) * (size + 1));

Il ne faut pas faire ça, car si realloc() retourne NULL, s est toujours alloué mais perdu (fuite mémoire). Tu dois utiliser un temporaire...

Je ne l utilise plus, j utilise une autre maniere, je parcours une fois la liste des parametres et je calcul la taille total d 'un seul coup.
Le probleme viens apparement du malloc qui me fais un memory cooruption

[Mr_Chut]

J ai trouvé ceci :

Memory Corruption: Memory when altered without an explicit assignment due to the inadvertent and unexpected altering of data held in memory or the altering of a pointer to a specific place in memory.

Je ne vois pas en quoi le malloc altere la memoire, vu que s dans la fonction est NULL et est de plus une variable locale.