retourner un pointeur

Citation:

---

Envoyé par psgman113

J'ai pourtant alloué de la memoire dynamique:

Code:

---

1 2	char *coded_msg; coded_msg= (char *)malloc (n * sizeof (char)) ;

---

---

est horriblement et inutilement complexe :

Code:

---

1 2	char *coded_msg = malloc (n) ;

---

http://emmanuel-delahaye.developpez....tes.htm#malloc

Citation:

---

Code:

---

1 2	//coded_msg a bien été construit!

---

---

Prouve le !

alors là , c'est la meilleure!

J'ai juste mis le code là où j'affiche le contenu de la chaine dans ma fonction

donc après la ligne char msg_decoe[1000]="\0"; , je perds le contenu du msg_code !!
Alors que cette declaration de tableau de caracteres n'a rien a voir avec le msg_code!

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

void decodage_msg(char *msg_code) { // declaration de variables que j'utilise mais je n'ai pas affiché ici char buf_un[9]="\0" ; char buf_deux[9]="\0" ; int T_new[9],T_chai[9],T_chaine_tab[9],T_chaine[9]; printf("a%s\n",msg_code); // m'affiche quelque chose char msg_decoe[1000]="\0"; printf("a%s\n",msg_code); // m'affiche plus rien :-( }

---

C'est quoi ce code 8O

Citation:

---

Envoyé par psgman113

alors là , c'est la meilleure!

J'ai juste mis le code là où j'affiche le contenu de la chaine dans ma fonction
<...>

---

Ceci fonctionne :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

#include <stdio.h> #include <stdlib.h>   void decodage_msg (char *msg_code) { printf ("a%s\n", msg_code); // m'affiche quelque chose char msg_decoe[1000] = "\0"; printf ("a%s\n", msg_code); // m'affiche plus rien :-( }   int main (void) { char coded_msg[]="hello world";   printf (" %s\n", coded_msg); // m'affiche une chaine de caracteres decodage_msg (coded_msg);   return 0; }

---

Code:

---

1 2 3 4 5 6

hello world ahello world ahello world   Press ENTER to continue.

---

Si tu postais ton code véritable, ça irait beaucoup plus vite que de jouer aux devinettes...

Mon code est assez enorme .. Ca ferait mal aux yeux
En fait , j'ai remarqué que le contenu de ma chaine de caractères est légèrmeent modifié entre le moment où je l'affiche avant l'appel d'une fonction et quand je l'affiche dans la fonction (pointeur passé en parametre), il y a 3 caractères qui manque dans ma chaine.

Citation:

---

Envoyé par psgman113

Mon code est assez enorme .. Ca ferait mal aux yeux<...>

---

Alors réduit ton code à ce qui ne va pas et poste le. Ca doit être compilable.

On est pas devins...

Voici mon code qui est long,
Le truc que je comprends pas c'est quand j'affiche le contenu de ma chaine de caractère avant et après l'appel de la fonction decodage_msg , ele differe e 3 caractères . J'ai écrit Avant après , vous pouvez le voir lors de l'exécution

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 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 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92 93 94 95 96 97 98 99 100 101 102 103 104 105 106 107 108 109 110 111 112 113 114 115 116 117 118 119 120 121 122 123 124 125 126 127 128 129 130 131 132 133 134 135 136 137 138 139 140 141 142 143 144 145 146 147 148 149 150 151 152 153 154 155 156 157 158 159 160 161 162 163 164 165 166 167 168 169 170 171 172 173 174 175 176 177 178 179 180 181 182 183 184 185 186 187 188 189 190 191 192 193 194 195 196 197 198 199 200 201 202 203 204 205 206 207 208 209 210 211 212 213 214 215 216 217 218 219 220 221 222 223 224 225 226 227 228 229 230 231 232 233 234 235 236 237 238 239 240 241 242 243 244 245 246 247 248 249 250 251 252 253 254 255 256 257 258 259 260 261 262 263 264 265 266 267 268 269 270 271 272 273 274 275 276 277 278 279 280 281 282 283 284 285 286 287 288 289 290 291 292 293 294 295 296 297 298 299 300 301 302 303 304 305 306 307 308 309 310 311 312 313 314 315 316 317 318 319 320 321 322 323 324 325 326 327 328 329 330 331 332 333 334 335 336 337 338 339 340 341 342 343 344 345 346 347 348 349 350 351 352 353 354 355 356 357 358 359 360 361 362 363 364 365 366 367 368 369 370 371 372 373 374 375 376 377 378 379 380 381 382 383 384 385 386 387 388 389 390 391 392 393 394 395 396 397 398 399 400 401 402 403 404 405 406 407 408 409 410 411 412 413 414 415 416 417 418 419 420 421 422 423 424 425 426 427 428 429 430 431 432 433 434 435 436 437 438 439 440 441 442 443 444 445 446 447 448 449 450 451 452 453 454 455 456 457 458 459 460 461 462 463 464 465 466 467 468 469 470 471 472 473 474 475 476 477 478 479 480 481 482 483 484 485 486 487 488 489 490 491 492 493 494 495 496 497 498 499 500 501 502 503 504 505 506 507 508 509 510 511 512 513 514 515 516 517 518 519 520 521 522 523 524 525 526 527 528 529 530 531 532 533 534 535 536 537 538 539 540 541 542 543 544 545 546 547 548 549 550 551 552 553 554 555 556 557 558 559 560 561 562 563 564 565 566 567 568 569 570 571 572 573 574 575 576 577 578 579 580

#include <stdlib.h> #include <stdio.h> #define P 256   void decodage_msg ( char * msg_deco); void conversion_wep_iv_un (char *cle_wep); char *codage_msg(char *message); char conversion_deco (char *chaine_decod, size_t size); int conversion_letre_decimal ( int decimal,char carac); char *saisie_wep (); char * conversion ( int decimal); int size; char * saisie_msg(); int T_deux[20],T_ini[20], T[20];   int size_wep; int n;   main(void) {   char *coded_msg = malloc (n) ; char *wep= malloc (n) ; char *msg= malloc (n) ;   wep= saisie_wep(); if (wep!= NULL) {   conversion_wep_iv_un(wep); msg = saisie_msg(); if (msg != NULL) {   coded_msg = codage_msg(msg);   if (coded_msg != NULL) {   printf("avnt %s\n",coded_msg); decodage_msg(coded_msg); free(coded_msg); } else { printf("message non code"); } free(msg); }   } else { printf("wep incorrect"); } system("pause");   }     void decodage_msg(char *msg_code) {   int i,z,j,n,k,y,t,err;   char buf_un[9]; char buf_deux[9]; int T_new[9],T_chai[9],T_chaine_tab[9],T_chaine[9]; char letter; printf("apre %s\n",msg_code); // m'affiche plus rien :-(   char msg_decoe[1000];     i=1; t=0; for(z=0;z<size;z++) // 6 c pr lexemple { j = 0;   j= (T_deux[i]+j)%15;   k= T_deux[i]; T_deux[i]=T_deux[j]; T_deux[j]=k; n = (T_deux[i] + T_deux[j]) % 15 ;     char *chai= conversion (T_deux[n]);   if (chai==NULL)   printf("erreur de retour");   for (y=0;y<8;y++) {   char s3[2] = {msg_code[t]}; // convertir un caractère en chaine puis en nombre T_chaine_tab[y]=atoi(s3); char s4[2] = {chai[y]}; T_chaine[y]=atoi(s4); T_new[y]=T_chaine_tab[y]^T_chaine[y];   sprintf(buf_un, "%d", T_new[y]); // on a un caractere     strcat(buf_deux,buf_un);// concatener printf("%s\n",buf_deux);   t++;   } letter= conversion_deco(buf_deux,sizeof buf_deux); // on a la lettre   buf_deux[0]='\0'; printf("la lettre decodee est %c\n",letter);   char s6[2] = {letter};       strcat(msg_decoe,s6);// concatener     printf("le msg decode est %s\n",msg_decoe); printf("\n");   i++; } }     char * codage_msg(char *msg) { int i,j,k,y,o,n,decimal,retour; // char chaine[9],chaine_tab[9];   int T_chaine[9],T_chaine_tab[9],T_new[9]; char code_un[9]="\0" ; char code[1000]="\0";   char *msg_code= malloc(n); char *chaine= malloc(n);   if (msg_code == NULL | chaine==NULL ) { /* l'allocation a echoué: */ /* message d'erreur et arrêt du programme */ printf ("l'allocation a echoue\n") ; exit (1) ; }     j = 0; i=1; for (y=0;y<size;y++) {   printf("lettre du message: %c\n",msg[y]);   decimal= conversion_letre_decimal(decimal, msg[y]); chaine= conversion ( decimal);     if (chaine==NULL) printf("erreur de retour"); //ici on a la chaine binaire en var globale corespondant à un caractère du msg   j= (T[i]+j)%15; k= T[i]; T[i]=T[j]; T[j]=k; n = (T[i] + T[j]) % 15 ;     char *chaine_tab = conversion ( T[n]); if (chaine_tab!=NULL) { printf("la chaine binaire du tableau est: %s\n",chaine_tab); // chaine binaire de l'entier du tablo printf("la chaine binaire du caractere est: %s\n",chaine); // dans chaine , on a la chaine binaire du caractère du texte } else printf("erreur de retour");   for (o=0;o<8;o++) {   char s1[2] = {chaine_tab[o]}; // convertir un caractère en chaine puis en nombre T_chaine_tab[o]=atoi(s1); char s2[2] = {chaine[o]}; T_chaine[o]=atoi(s2);   // printf("ici %d\n",T_chaine_tab[o]);   T_new[o]=T_chaine_tab[o]^T_chaine[o]; // printf("ici %d\n",T_new[o]); sprintf(code_un, "%d", T_new[o]); // convertit entier en caractère     strcat(code,code_un); // printf("ici %s\n",code); }       printf("construction du message code: \n"); printf("%s\n", code); printf("\n");     i++; }   printf("\n"); msg_code=code; printf("voici le msg code enn binaire %s\n",msg_code); // msg codé   printf("decodons le maintenant\n "); printf("\n");     return msg_code; }   /// saisie de la clé wep   char * saisie_wep () { char *cle_wep; cle_wep= (char *)malloc (n * sizeof (char)) ;   if (cle_wep== NULL ) { /* l'allocation a echoué: */ /* message d'erreur et arrêt du programme */ printf ("l'allocation a echoue\n") ; exit (1) ; }   cle_wep="12CA234567ABC"; size_wep=strlen(cle_wep); return cle_wep; }   ///////////////////////////////////////   // saisie du msg au clavier char * saisie_msg() {   char * message = malloc (n) ;   if (message== NULL ) { /* l'allocation a echoué: */ /* message d'erreur et arrêt du programme */ printf ("l'allocation a echoue\n") ; exit (1) ; }   message="crist"; size=0; size=strlen (message);   return message; }   /////////////////////////////// /*fonction de conversion de la clé wep ( char -> entier décimal ds un tablo) fonction de génération de la clé iv fonction de sauvegarde du tableau (clé + clé iv) clé wep en variable globale */ void conversion_wep_iv_un (char *cle_wep) { int i,n; char *elem, *end;   for (i=0;i<size_wep;i++) { if (cle_wep[i] == 'A') elem="10"; else if (cle_wep[i] == 'B') elem="11"; else if (cle_wep[i]== 'C') elem="12"; else if (cle_wep[i] == 'D') elem="13"; else if (cle_wep[i]== 'E') elem="14"; else { char s[2] = {cle_wep[i]}; // on crée une chaine de caractères avec un seul caractère elem=s; }   n=strtol(elem, &end, 10); // convertir la chaine elem en un entier décimal   if(*end == '\0') { T[i]=n; /* La chaine a enteirement pu etre convertie */ } else{ printf("chaine de caractere incorrect\n"); /* La chaine contenait au moins un caractere ne pouvant pas etre converti en nombre */ } }     for (i = size_wep; i<size_wep+6; i++) // on génére un tableau de nombre aléatoires (entre 0 et N ) de 26 octets {   T[i] = rand() % P;   }   for (i = 0; i<size_wep+6; i++) // on génére un tableau de nombre aléatoires (entre 0 et N ) de 26 octets {   T_ini[i]=T[i]; T_deux[i]=T[i];   }   }   ////////////////////////////////////////////////   //fonction de conversion lettre en binaire     int conversion_letre_decimal ( int decimal,char carac) {     if (carac == 'a') decimal=0;   if (carac == 'b') decimal=1;     if (carac == 'c') decimal=2;     if (carac == 'd') decimal=3;     if (carac == 'e') decimal=4;   if (carac == 'f') decimal=5;   if (carac == 'g') decimal=6;   if (carac == 'h') decimal=7;     if (carac == 'i') decimal=8;     if (carac == 'j') decimal=9;     if (carac == 'k') decimal=10;     if (carac =='l') decimal=11;   if (carac == 'm') decimal=12;   if (carac == 'n') decimal=13;     if (carac == 'o') decimal=14;     if (carac == 'p') decimal=15;     if (carac == 'q') decimal=16;     if (carac == 'r') decimal=17;   if (carac == 's') decimal=18;   if (carac == 't') decimal=19;       if (carac == 'u') decimal=20;     if (carac == 'v') decimal=21;     if (carac == 'w') decimal=22;   if (carac == 'x') decimal=23;   if (carac == 'y') decimal=24;     if (carac == 'z') decimal=25;       return decimal; }       char * conversion ( int decimal) { char *octet; int i,j,l; int reste; char *tempo; tempo = (char *)malloc (n * sizeof (char)) ; octet= (char *)malloc (n * sizeof (char)) ; for (i=0;i<8;i++) { octet[i]='0'; }   octet[8]='\0';     l=128;// l définit le premier seuil 128         if (decimal != 0) {     reste=decimal%2;   if (reste ==1) {   octet[7]='1';   decimal=decimal-1; }   else octet[0]='0';   i=0;     while ( decimal!=0) { if (decimal<l) { octet[i]='0'; l=l/2; i++; }   if (decimal>l) { octet[i]='1'; decimal=decimal -l; l=l/2; i++; }   if (decimal==l) { octet[i]='1'; i++; for (j=i;j<7;j++) { octet[j]='0'; } octet[8]='\0';   decimal=0; } }       } else { for (j=0;j<8;j++) { octet[j]='0'; }   }   return octet; } //////////////////////////////////////////////////////   char conversion_deco (char *chaine_decod, size_t size) // transforme une chaine en un caractere { char lettre; if (!strcmp(chaine_decod,"00000000")) lettre='a';   else if (!strcmp(chaine_decod,"00000001")) lettre='b';     else if (!strcmp(chaine_decod,"00000010")) { lettre='c';   }   else if (!strcmp(chaine_decod,"00000011")) lettre='d'; else if (!strcmp(chaine_decod,"00000100")) lettre='e'; else if (!strcmp(chaine_decod,"00000101")) lettre='f'; else if (!strcmp(chaine_decod,"00000110")) lettre='g'; else if (!strcmp(chaine_decod,"00000111")) lettre='h'; else if (!strcmp(chaine_decod,"00001000")) lettre='i'; else if (!strcmp(chaine_decod,"00001001")) lettre='j'; else if (!strcmp(chaine_decod,"00001010")) lettre='k'; else if (!strcmp(chaine_decod,"00001011")) lettre='l'; else if (!strcmp(chaine_decod,"00001100")) lettre='m'; else if (!strcmp(chaine_decod,"00001101")) lettre='n'; else if (!strcmp(chaine_decod,"00001110")) lettre='o'; else if (!strcmp(chaine_decod,"00001111")) lettre='p'; else if (!strcmp(chaine_decod,"00010000")) lettre='q'; else if (!strcmp(chaine_decod,"00010001")) lettre='r'; else if (!strcmp(chaine_decod,"00010010")) lettre='s'; else if (!strcmp(chaine_decod,"00010011")) lettre='t'; else if (!strcmp(chaine_decod,"00010100")) lettre='u'; else if (!strcmp(chaine_decod,"00010101")) lettre='v'; else if (!strcmp(chaine_decod,"00010110")) lettre='w'; else if (!strcmp(chaine_decod,"00010111")) lettre='x'; else if (!strcmp(chaine_decod,"00011000")) lettre='y'; else if (!strcmp(chaine_decod,"00011001")) lettre='z';   else lettre='<';   return lettre; }

---

Citation:

---

Envoyé par psgman113

Voici mon code qui est long,

---

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 34 35 36 37 38 39 40 41 42 43 44

Project : Forums Compiler : GNU GCC Compiler (called directly) Directory : C:\dev\forums2\ -------------------------------------------------------------------------------- Switching to target: default Compiling: main.c main.c:10: warning: function declaration isn't a prototype main.c:13: warning: function declaration isn't a prototype main.c:20: warning: return type defaults to `int' main.c: In function `decodage_msg': main.c:101: warning: implicit declaration of function `strcat' main.c:58: warning: unused variable `err' main.c:62: warning: unused variable `T_chai' main.c: In function `codage_msg': main.c:139: warning: suggest parentheses around comparison in operand of | main.c:129: warning: unused variable `retour' main.c: At top level: main.c:218: warning: function declaration isn't a prototype main.c: In function `saisie_wep': main.c:229: warning: assignment discards qualifiers from pointer target type main.c:230: warning: implicit declaration of function `strlen' main.c: At top level: main.c:238: warning: function declaration isn't a prototype main.c: In function `saisie_msg': main.c:249: warning: assignment discards qualifiers from pointer target type main.c: In function `conversion_wep_iv_un': main.c:270: warning: assignment discards qualifiers from pointer target type main.c:272: warning: assignment discards qualifiers from pointer target type main.c:274: warning: assignment discards qualifiers from pointer target type main.c:276: warning: assignment discards qualifiers from pointer target type main.c:278: warning: assignment discards qualifiers from pointer target type main.c: In function `conversion_deco': main.c:514: warning: implicit declaration of function `strcmp' main.c: At top level: main.c:511: warning: unused parameter 'size' main.c: In function `codage_msg': main.c:129: warning: 'n' might be used uninitialized in this function main.c:129: warning: 'decimal' might be used uninitialized in this function main.c: In function `main': main.c:52: warning: control reaches end of non-void function Linking console executable: console.exe Process terminated with status 0 (0 minutes, 5 seconds) 0 errors, 23 warnings

---

Ca fout la trouille...

Si ton compilateur ne dit rien, c'est peut être qu'il est mal configuré :

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

Corrige tout ça, déjà, on y verra plus clair...

Bonjour,

Aïe !!

des globales en veux-tu, en voilà

code mal indenté

mauvaise forme pour main

des allocations par malloc faites avec une variable non initialisée, PUIS écrasées par des retours de fonction qui ont fait leur propre allocation

...

j'en passe

pas envie d'aller plus loin

Je programme sous dev c++ sous windows ,
j'ai pas eu ces warnings,
mes variables globales? y en a pas tant que ca ,
je déclare juste pas mal de fonctions

Citation:

---

Envoyé par psgman113

Je programme sous dev c++ sous windows ,
j'ai pas eu ces warnings,

---

Alors ça y'est, tu as modifié ta config, tu as recompilé, corrigé ?

Je ne suis pas bien sûr que tu maitrises suffisamment le C pour mener à bien ton projet ... Tu ne veux pas commencer par quelque chose de plus simple ?

Au fait tu as fait les études préalables ? spécification, conception ?

http://emmanuel-delahaye.developpez.com/dev_proj.htm

c'est une application que je désire realiser . EN fait, je l'ai fait dans une premier temps en utilisant des variables globales et en mettant tout le code dans le main et ca marchait!

Je veux maintenant le structurer

J'ai compilé sous fedora core et j'ai obtenu ceci en warnings

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

essai.c: In function ‘decodage_msg’: essai.c:119: warning: incompatible implicit declaration of built-in function ‘st rcat’ essai.c:134: warning: incompatible implicit declaration of built-in function ‘st rcat’   essai.c: In function ‘codage_msg’: essai.c:228: warning: incompatible implicit declaration of built-in function ‘st rcat’   essai.c: In function ‘saisie_wep’: essai.c:268: warning: incompatible implicit declaration of built-in function ‘st rlen’   essai.c: In function ‘saisie_msg’: essai.c:291: warning: incompatible implicit declaration of built-in function ‘st rlen’

---

Ne vous fiez pas aux lignes car c'est une autre version du programme.


J'en ais déduis qu'il manquait un include quelque chose pour utiliser strlen et strcat .

Sinon , quand je fais ceci , il me dit que l'allocation a échoué.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

char *msg_code= malloc(n); char *chaine= malloc(n);   if (msg_code == NULL | chaine==NULL ) { /* l'allocation a echoué: */ /* message d'erreur et arrêt du programme */ printf ("l'allocation a echoue\n") ; exit (1) ; }

---

Citation:

---

Envoyé par psgman113

c'est une application que je désire realiser . EN fait, je l'ai fait dans une premier temps en utilisant des variables globales et en mettant tout le code dans le main et ca marchait!

---

Tu sais, c'est encore plus inutile d'utiliser des variables globales si tout le code est dans la même fonction...

j'utilisais quand meme une ou deux fonctions

Sinon dans ma fonction codage_msg , je veux créer une hcaine de caractères et la retourner (je ne connais pas sa longueur avant) donc je fais ceci :
mais il me dit bien que l'allocation a échouée
pourquoi?

Code:

---

1 2 3 4 5 6 7 8 9

char *msg_code= malloc(n);   if (msg_code == NULL ) { /* l'allocation a echoué: */ /* message d'erreur et arrêt du programme */ printf ("l'allocation a echoue\n") ; exit (1) ; }

---

J'ai simplifié mon probleme,

Je construis une chaine de caractères dans une fonction , je retourne cette chaine à l'aide d'un pointeur , je veux la réafficher dans mon main et là hic:

L'allocation mémoire doit se faire dans ma fonction , dans mon main ou dans les 2 ?

Merci

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 34

#include <stdio.h> #include <stdlib.h> #include <string.h>   char * decodage_msg () { int n;   char *a=malloc(n); if (a==NULL ) { /* message d'erreur et arret du programme */ printf ("l'allocation a echoue\n") ; exit (1) ; }   char code_msg[]="helloworld\0";   a=code_msg;   printf ("avant %s\n", a); // m'affiche helloworld return a; }   int main (void) { char * b= decodage_msg (); printf ("apres %s\n", b); // m'affiche une chaine de caracteres bizarre   system("pause"); return 0; }

---

Bonjour,

Je te l'avais déjà signalé :

allocation avec malloc, en utilisant pour préciser la taille, une variable non initialisée.

Il faut lire les réponses, sinon il est inutile de demander de l'aide

Je pense comme Emmanuel : tu as manifestement besoin de revoir les bases, en faisant l'effort de comprendre pourquoi les exemples sont écrits comme ils le sont, et pas seulement en te contentant de les copier et vérifier si ça marche.

Citation:

---

des allocations par malloc faites avec une variable non initialisée, PUIS écrasées par des retours de fonction qui ont fait leur propre allocation

---

Ok , j'ai initialisé n à 300 pour 300 caractères , mais ca ne change rien .

J'ai posé une question et je la répète

Je dois faire une allocation mémoire uniquement dans la fonction , uniquement dans le main ou dans les 2? Important pour éviter d'écraser l'allocation

Merci

En théorie, dans la fonction seulement.

Citation:

---

Envoyé par psgman113

Code:

---

1 2 3 4 5 6 7 8

char *a=malloc(n); if (a==NULL ) { /* message d'erreur et arret du programme */ printf ("l'allocation a echoue\n") ; exit (1) ; }

---

---

En supposant que n est correctement intialise, jusque la ca va (enfin la sortie est un peu brutale, mais bon)

Citation:

---

Envoyé par psgman113

Code:

---

1 2 3

char code_msg[]="helloworld\0"; a=code_msg;

---

---

La tu modifies la valeur du pointeur qui ne contient plus l'adresse de la zone alloue mais celle du tableau local code_msg.
Tu n'as donc plus de lien sur la zone memoire allouee ==> fuite memoire.

En passant, le \0 dans "helloworld\0" est inutile car implicite.

Citation:

---

Envoyé par psgman113

Code:

---

1 2 3

return a; }

---

---

a ayant maintenant l'adresse de la variable locale code_msg, tu retournes l'adresse d'un tableau local qui est detruit a la sortie de la fonction. L'adresse que tu retourne est donc invalide.

Rappel: pour copier une chaine de caractere, il faut utiliser strcpy et non =.

En outre la memoire allouee n'est pas liberee. Dans cet exemple ce n'est pas tres grave puique l'on quitte quasi immediatement mais c'est une habitude a prendre, lorsque tu n'as plus besoin de la memoire allouee, il faut la liberer.

Ceci etant, une bonne partie de ces remarques on deja ete faites, il serait peut etre bon de tenir compte des remarques que font les differents contributeurs et de poser des questions si tu ne comprends pas quelque chose dans une explication.
Je t'invite egalement a relire attentivement le chapitre sur les pointeurs et allocation dynamique de ton cours ou de ton livre.

Bonjour,

Citation:

---

Envoyé par psgman113

Je dois faire une allocation mémoire uniquement dans la fonction , uniquement dans le main ou dans les 2? Important pour éviter d'écraser l'allocation

---

Ta question ainsi écrite contient sa propre réponse.

Pourquoi veux-tu allouer de la mémoire à 2 endroits différents pour une même variable ?

Quand tu veux un verre d'eau, le remplis-tu à la cuisine, PUIS à la salle de bain (sans l'avoir vidé au passage, bien entendu) ?

Et pour confirmer gl : il faut libérer la mémoire allouée. Le mieux, c'est de le faire dès qu'on en a plus besoin, et AU MINIMUM, à la sortie du programme.
Si tu ne prends pas cette habitude, ne viens pas plus tard te plaindre de problèmes de mémoire.