[C] Funzione free

[D4rkAng3l]

Ho scritto questo semplice programmino che alloca dinamicamente un array con calloc, poi lo riempie e infine dovrebbe liberare la memoria precedentemente allocata con free ma quest'ultimo punto mi crea qualche problema....come mai non mi libera la memoria?

#include <stdlib.h>
#include <stdio.h>

/* Non riceve alcun parametro ed emette un puntatore a float */
int * get_mem(void);

int main(void){

int i;
int *puntatore; // Puntatore al primo valore float dell'array allocato

/* Alloca la memoria dinamicamente e metti il valore del puntatore uscente
dalla funzione get_mem nellaa variabile puntatore */

puntatore = get_mem();


printf("\nValore contenuto in puntatore: %p\n\n\n\n", puntatore);

for(i=0; i<100; i++){
*(puntatore + i) = i; // Inizializza l'array con il valore di i
}

/* Visualizza l'array allocato dinamicamente */

for(i=0; i<100; i++){
printf("Indirizzo: %p Valore: %d\n", &puntatore[i],puntatore[i]);
}
printf("\n\n\n\n\nValore contenuto in puntatore: %p\n\n\n\n", puntatore);

/* Libera la memoria precendetemente allocata */

for(i=0; i<100; i++)
free(puntatore[i]);

printf("%d", puntatore[2]);

system("PAUSE");
return 0;
}

/* La funzione get_mem non riceve alcun parametro in ingresso e restituisce un
puntatore al primo valore di tipo float dell'array di 100 elementi allocato
dinamicamente */

int *get_mem(void){

int *p; // Puntatore al primo elemento di tipo float dell'array

/* Alloca memoria per un array di 100 elementi di tipo float */
p = calloc(100, sizeof(int));

if(!p){ // Se calloc restituisce un puntatore NULL
printf("Errore di allocazione della memoria\n");
exit(1);
}
return p;
}

[Ziosilvio]

Non vorrei sbagliare, ma quando usi malloc e calloc devi sempre convertire a puntatore al tipo che vuoi.
Prova a usare questa:

int *get_mem(void)
{
    int *p;

    p = (int*)calloc(100*sizeof(int));
    if (p==NULL)
        exit(1);

    return p;
}

o questa:

int *get_mem(void)
{
    void *p;

    p = calloc(100*sizeof(int));
    if (p==NULL)
        exit(1);

    return (int*)p;
}

[AnonimoVeneziano]

prima di tutto la sintassi è sbagliata.

A free() devi passare un puntatore , non un dato.

facendo "free(puntatore[i]);" passi il valore contenuto in "puntatore[i]" alla funzione free() che cercherà di liberare la memoria all' indirizzo di memoria indicato da puntatore[i] (e quindi 1, 2, 3 , 4 , 5 ,6 ... strano che non ti sia andato in segmentation fault) .

per liberare tutto l'array è sufficiente dare "free(puntatore);" e dovrebbe andare.

In ogni caso c'è la possibilità che i valori rimangano in memoria e che quindi visualizzi ugualmente il valore. Infatti free() non cancella la memoria, semplicemente la rende disponibile per un altro programma che ne abbia necessità. Comunque accedere a una locazione di memoria non allocata come fai nella riga :

printf("%d", puntatore[2]);

a seconda del sistema operativo potrebbe portare a un segmentation fault.

Ciao

Ciao

[D4rkAng3l]

Originariamente inviato da AnonimoVeneziano

prima di tutto la sintassi è sbagliata.

A free() devi passare un puntatore , non un dato.

facendo "free(puntatore[i]);" passi il valore contenuto in "puntatore[i]" alla funzione free() che cercherà di liberare la memoria all' indirizzo di memoria indicato da puntatore[i] (e quindi 1, 2, 3 , 4 , 5 ,6 ... strano che non ti sia andato in segmentation fault) .

per liberare tutto l'array è sufficiente dare "free(puntatore);" e dovrebbe andare.

In ogni caso c'è la possibilità che i valori rimangano in memoria e che quindi visualizzi ugualmente il valore. Infatti free() non cancella la memoria, semplicemente la rende disponibile per un altro programma che ne abbia necessità. Comunque accedere a una locazione di memoria non allocata come fai nella riga :

printf("%d", puntatore[2]);

a seconda del sistema operativo potrebbe portare a un segmentation fault.

Ciao

Ciao

mmm grazie...ora provo...la cosa strana è che sul McGrowHill faceva come avevo scritto io...poi vi posto il suo esempio

[Ziosilvio]

Ancora un altro paio di cose:

    free(puntatore[i]);

puntatore[i] è un int, non un puntatore a int; e non dovresti potergli applicare free.
Il compilatore ti dovrebbe dare quantomeno un warning.
Devi chiamare direttamente:

    free(puntatore);

che libera tutta la memoria che hai allocato a partire da lì.
Solo che, a quel punto, la chiamata a printf non sarebbe lecita in quanto fai riferimento a una zona della memoria che non è più allocata.

EDIT: stavo scrivendo questo post quando AnonimoVeneziano ha inserito la sua replica...

[Ziosilvio]

Originariamente inviato da D4rkAng3l

mmm grazie...ora provo...la cosa strana è che sul McGrowHill faceva come avevo scritto io...poi vi posto il suo esempio

Di quale McGraw-Hill parli? Sarà mica il Bullschildt?

[D4rkAng3l]

perchè non è buono?io mi ci trovo bene...sicuramente meglio del daitel & daitle...che ha che non và?

[Ziosilvio]

Originariamente inviato da D4rkAng3l

che ha che non và?

Se hai seguito il link che ho inserito nel mio post precedente, dovresti avere già un indizio.
L'articolo all'origine di tutto è questo.
Anche su ACCU.org (Association of C and C++ Users) non lo trattano certo bene.

[AnonimoVeneziano]

Originariamente inviato da D4rkAng3l

perchè non è buono?io mi ci trovo bene...sicuramente meglio del daitel & daitle...che ha che non và?

Il deitel & deitel per me è un buon libro

[D4rkAng3l]

Originariamente inviato da AnonimoVeneziano

Il deitel & deitel per me è un buon libro

si per carità però ripete sempre le stesse cose, per spiegarti una cosa ti fà i discorsi lunghissimi....

[anx721]

ma almeno ti dice le cose corrette e non ti scrive cose sbagliate