Liberation memoire chaine de caractere

Version imprimable

Bonsoir a vous,

Comment libérer l'espace mémoire occupée par la chaîne de caractère res?
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 char *CopieLigne (const char *s) { int k = 0; char *res = malloc(sizeof(char)*strlen (s)); while (s[k]!='\n') { res[k] = s[k]; k++; } return res; } int main (void) { char *origine = "Bonjour\nComment allez vous?\n"; char *res = CopieLigne (origine); printf("%s\n",res); return 0; }
De plus valgrind me donne se message

==4725== Conditional jump or move depends on uninitialised value(s)
==4725== at 0x401E215: strlen (mc_replace_strmem.c:246)
==4725== by 0x4088134: puts (in /lib/tls/i686/cmov/libc-2.3.6.so)
==4725== by 0x8048423: main

Merci pour votre aide

Citation:

Envoyé par BatuBou

Comment libérer l'espace mémoire occupée par la chaîne de caractère res?
<...>
De plus valgrind me donne se message

Code:

1
2
3
4
5
 
==4725== Conditional jump or move depends on uninitialised value(s)
==4725==    at 0x401E215: strlen (mc_replace_strmem.c:246)
==4725==    by 0x4088134: puts (in /lib/tls/i686/cmov/libc-2.3.6.so)
==4725==    by 0x8048423: main

Merci pour votre aide

Déjà, tu peux corriger ça :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c: In function `CopieLigne':
main.c:4: warning: implicit declaration of function `malloc'
main.c:4: warning: implicit declaration of function `strlen'
main.c: In function `main':
main.c:15: warning: initialization discards qualifiers from pointer target type
main.c:17: warning: implicit declaration of function `printf'
main.c:19:2: warning: no newline at end of file
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 0 seconds)
0 errors, 5 warnings

http://emmanuel-delahaye.developpez....tm#cfg_compilo

ensuite un simple free() de res dans le main suffit, sauf complications que je n'aurais pas vues...

Quelques corrections :

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
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
char *CopieLigne (const char *s)
{
   int k = 0;
   /* -ed- de la place pour le 0 final */
   char *const res = malloc (strlen (s) +1);
   while (s[k] != '\n')
   {
      res[k] = s[k];
      k++;
   }
 
    /* -ed- le 0 final... */
      res[k] = 0;
   return res;
}
 
int main (void)
{
   char const *origine = "Bonjour\n" 
                          "Comment allez vous?\n";
   char *res = CopieLigne (origine);
   if (res != NULL)
   {
      printf ("'%s'\n", res);
      free (res), res = NULL;
   }
   return 0;
}

Pose des questions si tu ne comprends pas.

02/01/2008, 21h36
BatuBou

Merci beaucoup pour votre reponse.

J'ai vraiment besoin de rajouter les options de compilation pr mon gcc :oops:

Bonne soirée a vous
02/01/2008, 21h38
Pacorabanix

Citation:

Envoyé par BatuBou

Merci beaucoup pour votre reponse.

J'ai vraiment besoin de rajouter les options de compilation pr mon gcc :oops:

Bonne soirée a vous

-Wall (c-à-d afficher simplement tous les warnings) comme option est deja fortement conseillé
02/01/2008, 21h55
BatuBou

oui je compile tout le temps avec -Wall -ansi... Mais il m'a pas affiché tous ces warning lol.
C'est pour ca que je vais rajouter les autres options

Citation:
Envoyé par Emmanuel Delahaye
ensuite un simple free() de res dans le main suffit, sauf complications que je n'aurais pas vues...

Quelques corrections :
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 #include <stdio.h> #include <stdlib.h> #include <string.h> char *CopieLigne (const char *s) { int k = 0; /* -ed- de la place pour le 0 final */ char *const res = malloc (strlen (s) +1); while (s[k] != '\n') { res[k] = s[k]; k++; } /* -ed- le 0 final... */ res[k] = 0; return res; } int main (void) { char const *origine = "Bonjour\n" "Comment allez vous?\n"; char *res = CopieLigne (origine); if (res != NULL) { printf ("'%s'\n", res); free (res), res = NULL; } return 0; }
Pose des questions si tu ne comprends pas.
Personellement, j'aurais aussi rajoute un test sur la valeur de res (et donc sur le resultat de malloc) avant de l'utiliser.
En outre, arreter la boucle lorsque la fin de chaine est atteinte me semble preferable pour gerer le cas ou il n'y a pas de '\n' dans la chaine initiale et eviter un debordement de buffer.

03/01/2008, 13h22
Emmanuel Delahaye

Citation:

Envoyé par gl

Personellement, j'aurais aussi rajoute un test sur la valeur de res (et donc sur le resultat de malloc) avant de l'utiliser.
En outre, arreter la boucle lorsque la fin de chaine est atteinte me semble preferable pour gerer le cas ou il n'y a pas de '\n' dans la chaine initiale et eviter un debordement de buffer.

Oui, j'ai juste corrigé les bugs de bases. La fonction est améliorable, c'est sûr.

Citation:
Envoyé par Emmanuel Delahaye
Déjà, tu peux corriger ça :
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 #include <stdio.h> #include <stdlib.h> #include <string.h> char *CopieLigne (const char *s) { int k = 0; /* -ed- de la place pour le 0 final */ char *const res = malloc (strlen (s) +1); while (s[k] != '\n') { res[k] = s[k]; k++; } /* -ed- le 0 final... */ res[k] = 0; return res; } int main (void) { char const *origine = "Bonjour\n" "Comment allez vous?\n"; char *res = CopieLigne (origine); if (res != NULL) { printf ("'%s'\n", res); free (res), res = NULL; } return 0; }
A quoi ça sert de mettre res à NULL ??
C'est en prévision du cas où on ne prendrait pas soin d'initialiser un futur malloc qui retomberait au même endroit ??

Il y a pas des situations où cela est superflu de nettoyer les variables.

Les programmeurs professionnels le font systématiquement ???

Ca sert à indiquer que le pointeur ne pointe plus sur une région valide. Cela permet d'écrire ultérieurement par exemple :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 if (res == NULL) { /* res est libre, on va lui donner du boulot */ ... res = <...>; ... } else { /* res est encore occupe pour le moment, vaut mieux ne pas le deranger */ ... }

11/01/2008, 08h46
Black-Hawk

Ok bon c'est utile selon les codes qu'on implémente.

Mais bon je vais commençer quand même à prendre ça comme une habitude systématique .

Ce qui est sûr , c'est que ça fera pas de mal à mon code :mouarf:

PS : je crois que tu t'es trompé dans ton if , tu as inversé les conditions .
11/01/2008, 09h06
Melem

Merci.

Citation:

Envoyé par Black-Hawk

Mais bon je vais commençer quand même à prendre ça comme une habitude systématique .

Ce qui est sûr , c'est que ça fera pas de mal à mon code

Oui ça fait partie des bonnes habitudes de programmation, d'ailleurs c'est très souvent utile.
11/01/2008, 10h27
Emmanuel Delahaye

Citation:

Envoyé par Black-Hawk

A quoi ça sert de mettre res à NULL ??
C'est en prévision du cas où on ne prendrait pas soin d'initialiser un futur malloc qui retomberait au même endroit ??

Parce que c'est une Bonne Pratique

http://emmanuel-delahaye.developpez.....htm#pointeurs

Citation:

Il y a pas des situations où cela est superflu de nettoyer les variables.

Certainement.

Citation:

Les programmeurs professionnels le font systématiquement ???

Moi, je le fais.
12/01/2008, 04h52
Black-Hawk

Très intéressant ton site qui doit sûrement regorger de pleins d'autres bonnes pratiques.:D

Je le mets dans mes favoris.

Merci de partager tout ça !!:bravo::bravo: