[SOLUCIONADO] Liberar array reservado con malloc

[albondi]

Hola a todos, tengo un programa que crea un vector de registros:

Código:

typedef struct fecha {
    int dia;
    int mes;
    int anio;
} tipoFecha;

typedef struct empleados {
    char nombre[15];
    tipoFecha fechaNacimiento;
} tipoEmpleado;

Creo el vector pidiendo al usuario cuantos registros de empleados quiere crear (numEmpleados) y reservo la memoria:

Código:

    tipoEmpleado *registros;
//Pido numEmpleados
    registros = (tipoEmpleado *)malloc(numRegistros*sizeof(tipoEmpleado));

El problema lo tengo al pasar los registros a una función para operar con ellos, y luego deseo liberarlos dentro de esa función:

Código:

//llamada a función:
funcion(registros, numRegistros);

Código:

funcion(tipoEmpleado *regEmp, int nEmpleados,) {

...

//Libero array
    for (i=0; i<nEmpleados; i++) {
        free(&regEmp[i]);
    }
    free(regEmp);
    regEmp = NULL;

}

Pero me da un error, y haciendo solo free(regEmp) no se libera...
Como debo liberar correctamente el array de registros?

[eferion]

Código C:

for (i=0; i<nEmpleados; i++) {
        free(&regEmp[i]);
    }
 

El bucle anterior sobra. Cuando hiciste la reserva de memoria, únicamente usaste un malloc, luego el bloque de memoria se ha de liberar únicamente con un free.

Por otro lado, "free(regEmp)" SI que libera la memoria. Liberar la memoria únicamente sirve para decirle al Sistema Operativo que esa región de memoria puede ser utilizada por quien la pida... liberar la memoria no implica sobreescribir lo que ya está almacenado en la misma. Lo que sucede es que la siguiente instrucción "regEmp = NULL;" implica únicamente un cambio local, por lo que el puntero registro no se entera.

Dicho de otra forma: regEmp es un puntero, sí, pero también es una variable local de funcion. Todos los cambios que realices en la memoria apuntada se verán reflejados fuera de la función... pero si haces que regEmp apunte a otra región de memoria, entonces ese cambio es local.

Si quieres que registro acabe apuntando a NULL necesitas usar un puntero doble tal que:

Código C:

funcion(&registros, numRegistros);
 
// Nota que nEmpleados no es necesaria en la función
funcion(tipoEmpleado **regEmp, int nEmpleados) {
    //Libero array
    free(&regEmp);
    *regEmp = NULL;
}
 

Un saludo

[albondi]

De acuerdo, entonces con free(regEmp) libero la posición en memoria del array de registros de empleados, pero éste sigue existiendo no? Porque para comprobarlo después de liberar la memoria, intente imprimir de nuevo los registros de empleados y seguir apareciendo. Es normal?

Era un ejercicio de examen que me pedía liberar los registros con función. Está bien resuelto?

[eferion]

Iniciado por albondi

De acuerdo, entonces con free(regEmp) libero la posición en memoria del array de registros de empleados, pero éste sigue existiendo no? Porque para comprobarlo después de liberar la memoria, intente imprimir de nuevo los registros de empleados y seguir apareciendo. Es normal?

Como te he comentado, free únicamente marca esa memoria como disponible. Los datos ahí almacenados seguirán existiendo hasta que esa memoria vuelva a ser reservada, momento a partir del cual esos datos pueden acabar sobreescritos en cualquier momento. En cualquier caso no es para nada seguro acceder a memoria liberada, así que asegúrate de liberar la memoria una vez que sepas que ya no necesitas lo que ahí está almacenado.

Iniciado por albondi

Era un ejercicio de examen que me pedía liberar los registros con función. Está bien resuelto?

Si haces X=malloc(), entonces has de hacer free(X). Esto implica que free se ha de ejecutar exactamente el mismo número de veces que malloc.

Mientras cumplas con estas premisas el ejercicio estará correctamente resuelto.

Un saludo