Discussion :

[xc78370]

Bonjour à tous,

J'ai un petit souci lors d'allocation mémoire dans une de mes programmes.
J'alloue dynamiquement de la mémoire pour pouvoir conserver des pointeurs su des caracteres (le but étant de marquer certains caracteres d'une chaine et d'en conserver une trace).
Je dois donc allouer plusieurs fois une taille égale à n*sizeof(char *) ou n*4 dan smon cas.
Cependant lorsque j'effectue mon realloc, celui-ci m'écrase une partide ma mémoire allouée précédemment, comme si il ne pointait plus au bon endroit.

Voila mon code simplifé :

Code :

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

1
2
3
4
5
6
7
8
 
  strcpy(chaine, "Bonjour/a/tous");
  lS_tok = (char**) malloc (4);
  *(lS_tok) = chaine;
  lS_tok = (char**) realloc (lS_tok, 8);
  *(lS_tok+4) = (chaine+8);
  lS_tok = (char**) realloc (lS_tok, 12);
  *(lS_tok+8) = (chaine+10);

Quelqu'un aurait une idée ? A moins que je n'aie pas tout compris !!!

[diogene]

- Si lS_tok est un char**, l'incrémentation n'est pas correcte :

Code :

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

1
2
*(lS_tok+1) = (chaine+8); // ou lS_tok[1] = chaine+8;
*(lS_tok+2) = (chaine+10); // ou lS_tok[2] = chaine+10;

- Ne pas se préoccuper de la taille d'un pointeur, le compilateur la connait :

Code :

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

1
2
3
  lS_tok = malloc (sizeof *lS_tok);
  lS_tok = realloc (lS_tok, 2*sizeof *lS_tok);
  lS_tok =  realloc (lS_tok, 3*sizeof *lS_tok);

- Attention, le realloc peut échouer et dans ce cas il renvoie NULL. Dans ce cas,et dans ton code, l'adresse précédente est perdue et la mémoire qui reste allouée devient inaccessible.
En général on procède de la façon suivante :

Code :

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

1
2
3
4
5
6
  p= realloc (lS_tok, 2*sizeof *lS_tok);
  if(p!= NULL) lS_tok = p;
  else 
  {
  // erreur de la réallocation. Les données sont toujours pointées par lS_tok
  }

[xc78370]

Merci diogene...

Pour :

- Si lS_tok est un char**, l'incrémentation n'est pas correcte :

Effectivement, a force de tourner en rond, je me suis embrouillé moi-meme !!

Pour :

- Ne pas se préoccuper de la taille d'un pointeur, le compilateur la connait :

Je suppose que tu parles de (char**) devant le realloc ? On m'a deja dit ca, mais mon compilateur (VC++6) me sort une erreur . Est-ce parceque j'ai indiqué un projet C++ ?

Pour :

- Attention, le realloc peut échouer et dans ce cas il renvoie NULL

Je le savais et ne l'ai pas mis en œuvre. Je pense que c'est le probleme majeur dans mon code...

Merci bien pour ton aide !

[dancingmad]

Est-ce parceque j'ai indiqué un projet C++ ?

Oui, probablement, car en c++, on est obligé de caster tous les (void*) explicitement.

Sinon une dernière remarque, il est préférable d'écrire :

Code :

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

malloc( 4 * sizeof( char ) )

meme si sizeof( char ) vaut toujours 1. En effet, c'est mieux de dire "je réserve de la mémoire pour 4 char" plutot que "je réserve 4 octets en mémoire".

EDIT : oups, j'avait pas complètement lu la réponse de diogene qui avait déjà traité la question.