Discussion :

[TheDrev]

Bonjour !

Je travail avec une chaine de caractères allouée dynamiquement, après un appel a strtok, lors de l'appel a free, toute la mémoire sera t'elle correctement libérée ?

Code :

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

1
2
3
4
5
 
char *str = malloc(sizeof(char) * BUFSIZ);
strcat(str, "une chaine de caracteres");
strtok(str, " ");
free(str);

es-ce valide ou dois-je modifier mon code pour que les chaines utilisée avec strtok soit des copies de str ou des tableaux de caractères ?

[schonai]

Bonjour,
le man de strtok nous explique que..

A sequence of calls to strtok() that operate on the same string
maintains a pointer that determines the point from which to start
searching for the next token. The first call to strtok() sets this
pointer to point to the first byte of the string.

Donc strtok utilise un simple pointeur vers ta chaine de caractère. L'appel de free libérera ta mémoire proprement.

des copies de str ou des tableaux de caractères ?

C'est la même chose

N'oublies pas de tester que ton allocation c'est bien passée en vérifiant que ton pointeur str n'est pas nul.

[TheDrev]

l'extrait du man dont tu parles expliques comment en interne la strtok gère les appelles successif.

Je n'avais pas pensé au man ! du coup j'ai quand même ma réponse !

Be cautious when using these functions. If you do use them, note that:

* These functions modify their first argument.

Comme le montre :

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
 
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
 
void main(){
  char *str = malloc(sizeof(char) * BUFSIZ);
  printf("%p\n", str);
  strcpy(str, "une chaine de caracteres");
  strtok(str, " ");
  str = strtok(NULL, " ");
  puts(str);
  printf("%p\n", str);
  free(str);
}

0x910d008
chaine
0x910d00c
Segmentation fault


Une solution possible

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
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
 
void main(){
  char *str = malloc(sizeof(char) * BUFSIZ);
  char *tok = str;
  printf("str %p tok %p\n", str, tok);
  strcpy(str, "une chaine de caracteres");
  strtok(tok, " ");
  tok = strtok(NULL, " ");
  puts(tok);
  printf("str %p tok %p\n", str, tok);
  free(str);
}

str 0x95e8008 tok 0x95e8008
chaine
str 0x95e8008 tok 0x95e800c


Merci tout de même d'avoir pris le temps de répondre !

[schonai]

l'extrait du man dont tu parles expliques comment en interne la strtok gère les appelles successif.

Je n'avais pas pensé au man ! du coup j'ai quand même ma réponse !


Comme le montre :

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
 
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
 
void main(){
  char *str = malloc(sizeof(char) * BUFSIZ);
  printf("%p\n", str);
  strcpy(str, "une chaine de caracteres");
  strtok(str, " ");
  str = strtok(NULL, " ");
  puts(str);
  printf("%p\n", str);
  free(str);
}

0x910d008
chaine
0x910d00c
Segmentation fault

Ce code ne montre rien du tout ^^. En faisant str = strtok(NULL, " "); tu changes la valeur du pointeur str qui contient l'adresse du début de ton tableau par l'adresse du dernier token analysé. ll est donc normal que str change. Quand tu appelles la fonction free tu fais alors référence à une allocation qui n'existe pas vu que tu as changé le pointeur d'où la seg fault.

Be cautious when using these functions. If you do use them, note that:

* These functions modify their first argument.

La modification a lieu à l'intérieur de la chaine de caractère : les délimiteurs (ici " ") sont remplacés par des '\0'.

[TheDrev]

La modification a lieu à l'intérieur de la chaine de caractère : les délimiteurs (ici " ") sont remplacés par des '\0'.

d'accord ! Je pensais que la chaine était (ré)allouée.

Évidement free ne tiens compte que de la valeur allouée dans malloc et pas des \0 !