liberar memoria correctamente con free

[vangodp]

¡Hola! >_<

Tengo una duda que me quita el sueño

Si uso esta forma de reserva de memoria:

Código C:

int linhas = 10;
    int colunas = 10;
    int **matrix = ( int** ) malloc ( sizeof (int) * linhas * colunas );
 

¿Seria correcto liberarlo de esta forma?:

Código C:

free ( matrix );
 

tengo entendido que malloc ( sizeof (int) * linhas * colunas ) reserva la memoria sin fragmentar. ¿O sea que reservando así, todas las casillas quedan adyacentes, cierto? Luego acontece el cast a 2d o algo así.
El caso es que usando malloc para reservar primero el vector de punteros, y luego un loop para reservar cada uno de los vectores que será apuntado por el vector de punteros reservado previamente, puede pasar que cada vector quede desperdigado donde le parezca a malloc. ¿No es eso? Para que hagan una idea de lo que digo si reservo así:

Código C:

int **m = (int**)malloc(sizeof(int)*10);
for(int i = 0; i < 10; i++)
m[i]=(int*)malloc(sizeof(int)*20);
 

¿En ese caso si que debo usar un loop para liberar correcto? ¿Pero en el caso de usar malloc como dije antes, se puede prescindir de usar el loop para ir liberando cada uno de los vetores apuntados por el vector de punteros y usar free como si de un array se tratara?. No estoy del todo seguro. ¿Alguien me lo confirma o me lo niega?

Esa es mi duda mayor vamos.

¡Gracias y buen finde!

[eferion]

malloc sirve para reservar un bloque de memoria de tamaño X. Una vez hecha la reserva te devuelve un puntero al inicio de dicha zona... El como uses esa memoria es irrelevante para él. Cuando tu llamas a free le pasas el puntero al inicio de la memoria a liberar y él se encarga del resto.

Lo anterior quiere decir que cada reserva que hagas debe ser liberada de forma individual. El sistema no sabe que dos regiones van a estar relacionadas e ninguna manera, luego no va a tomar la decisión e hacer liberaciones en cascada... Tu reservas y tu liberas.

En tu ejemplo, para reservar espacio para toda la matriz de una pasada tendrías que hacer:

Código C:

int linhas = 10;
int colunas = 10;
int* inmatrix = ( int* ) malloc ( sizeof (int) * linhas * colunas );
 

La reserva de tu ejemplo original sería útil si necesitas guardar una lista de valores en cada posición de la matriz... Que no creo que sea el caso.

Un saludo

[enrieto]

Creo que tendrías que revisar esto:

int **matrix = ( int** ) malloc ( sizeof (int) * linhas * colunas );

malloc ( sizeof (int) * linhas * colunas ); va a reservar espacio en memoria para una cantidad (linhas * colunas) de ints, cuando en realidad deberías reservar esa misma cantidad (linhas * colunas) pero para punteros a int.

En una arquitectura donde ints y punteros a ints tienen el mismo tamaño (32 bits cada uno, por ejemplo) no habría diferencia, pero donde sizeof(int) != sizeof(int*) eso no va a andar. Entonces, tendría que ser

int** matrix = ( int** ) malloc ( sizeof (int*) * linhas * colunas );

[vangodp]

Creo que tendrías que revisar esto:
malloc ( sizeof (int) * linhas * colunas ); va a reservar espacio en memoria para una cantidad (linhas * colunas) de ints, cuando en realidad deberías reservar esa misma cantidad (linhas * colunas) pero para punteros a int.

Sí... tienes razón. Gracias por recordar.