Discussion :

[Kimly]

Bonjour,

Voilà mon erreur :

Code :

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

Program received signal SIGSEGV, Segmentation fault. [Switching to Thread 0x8f88340 (LWP 3494)] 0xb7e7470a in ?? () from /lib/i386-linux-gnu/libc.so.6

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
 
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
 
typedef struct
{
	char * key;
	char * value;
} stringPair;
 
int main(const int argc, const char * argv[])
{
char* list1;
char *key1;
stringPair * pPair1;
 
key1="";
key1="test";
list1="";
list1="bonjour";
pPair1=malloc(sizeof(stringPair));/*malloc(strlen(key1)+strlen(list1));*/
strcpy (pPair1->key,key1);
	strcpy (pPair1->value,list1);
fprintf(stdout, "%s and %s\n",pPair1->key, pPair1->value);
free(pPair1);
}

Donc je ne comprends pas pourquoi.

Merci.

[Bktero]

Code :

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

1
2
3
pPair1=malloc(sizeof(stringPair));/*malloc(strlen(key1)+strlen(list1));*/
strcpy (pPair1->key,key1);
strcpy (pPair1->value,list1);

En faisant ça, tu alloues de la place pour la structure, mais tu n'alloues pas de place pour les zones pointées par les champs de la structure.

Code :

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

1
2
3
4
5
6
7
pPair1=malloc(sizeof(stringPair));/*malloc(strlen(key1)+strlen(list1));*/
 
pPair1->key = malloc(200);
pPair1->value = malloc(200);
 
strcpy (pPair1->key,key1);
strcpy (pPair1->value,list1);

marche beaucoup mieux.

[srvremi]

Pourquoi 200 ?

@+
Rémi

[Bktero]

C'était une taille tout à fait arbitraire, pour simplement illustrer le fait qu'il faut allouer de la mémoire et ainsi avoir un espace réservé pour y copier les chaines. J'aurais dû le préciser .
Dans la réalité, il faudrait donner une taille justifiée et libérer la mémoire après utilisation.


Je relis le code et je me rend compte de ça :

Code :

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

1
2
3
4
key1="";
key1="test";
list1="";
list1="bonjour";

Les affectations à "" ne servent à rien, puisqu'elles sont immédiatement écrasées.

[gangsoleil]

Je relis le code et je me rend compte de ça :

Code :

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

1
2
3
4
key1="";
key1="test";
list1="";
list1="bonjour";

Les affectations à "" ne servent à rien, puisqu'elles sont immédiatement écrasées.

Je pense que tu es fatigue : les chaines de caracteres ne se copient pas comme ca

[Bktero]

Euh.... Je sais que ça ne se copie pas comme ça !

char *ptr = "developpez.com" crée un pointeur sur une chaine et comme la chaine est constante, cette chaine est dans la zone mémoire réservée aux constantes. Si a la suite, je fais ptr= "developpez.net", ptr pointe maintenant vers une nouvelle chaine constante, qui est à un autre endroit de la mémoire statique (à voir même si le compilateur n'est pas suffisamment malin pour ne garder que la seconde affectation dans le code cité à mon message précédent).

Le code suivant est valide, non ? (en tout cas, ni MinGW ni Dr Memory ne me jettent) :

Code :

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

1
2
3
4
5
6
7
8
int main(void)
{
    char *ptr = "developpez.com";
    printf("%p : %s\n", ptr, ptr);
 
    ptr = "developpez.net";
    printf("%p : %s\n", ptr, ptr);
}

[gangsoleil]

Oula, alors y'a pas mal de problemes. Mes commentaires dans le code. Pose des questions si tu ne comprends pas.

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
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
 
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
 
typedef struct
{
	char * key;
	char * value;
} stringPair;
 
int main(int argc, char * argv[]) /* pas de const dans le prototype de main */
{
char* list1;
char *key1;
stringPair * pPair1;
 
/*********************
key1="";
key1="test";
list1="";
list1="bonjour";
**********************/
/* allocation de l'espace memoire pour key1 et list1 */
list1 = key1 = pPair1 = NULL; /* initialisation a NULL pour la liberation en cas d'erreur */
 
key1 = malloc (200); /* taille a definir */
list1 = malloc (200); /* taille a definir */
pPair1=malloc(sizeof(stringPair));
 
/* verification */
if (key1 == NULL  ||  list1 == NULL  ||  pPair1 == NULL)
{
  fprintf (stderr, "Erreur d'allocation memoire\n");
  free (key1), key1 = NULL;
  free (list1), list1 = NULL;
  free (pPair1), pPair1 = NULL; 
 return EXIT_FAILURE;
}
 
/* la memoire est allouee. On peut ecrire dedans */
sprintf (key1, "test");
sprintf (list1, "bonjour");
 
/* on alloue le contenu de la structure */
pPair1->key = pPair1->value = NULL;
 
pPair1->key = malloc (300);
ppair1->value = malloc (300);
if (pPair1->key == NULL  || pPair1->value == NULL)
{
  fprintf (stderr, "Erreur d'allocation memoire\n");
  free (key1), key1 = NULL;
  free (list1), list1 = NULL;
  free (pPair1->key), pPair1->key = NULL; 
  free (pPair1->value), pPair1->value = NULL; 
  free (pPair1), pPair1 = NULL; 
  return EXIT_FAILURE;
}
 
/* on peut copier les donnees. Utiliser strncpy serait mieux */
strcpy (pPair1->key, key1);
strcpy (pPair1->value,list1);
 
/* affichage */
fprintf(stdout, "%s and %s\n",pPair1->key, pPair1->value);
 
/* liberation memoire */
  free (key1), key1 = NULL;
  free (list1), list1 = NULL;
  free (pPair1->key), pPair1->key = NULL; 
  free (pPair1->value), pPair1->value = NULL; 
  free (pPair1), pPair1 = NULL; 
 
/* fin */
  return EXIT_SUCCESS;
}

A changer dans ce code : utiliser snprintf, strncpy.

Pourquoi utilises-tu de la memoire dynamique ? Pour l'exercice, ou bien il n'y a aucune raison, et tu pourrais utiliser de la memoire statique, ce qui serait beaucoup plus simple pour toi ?