C/Visual C - Liberar memoria para una struct entera

Estoy intentando liberar memoria para una struct de la siguiente forma:

*agenda=(struct persona *)malloc(sizeof(struct persona));

En el código tengo declarado lo siguiente:

struct persona
{
char *nombre;
int telefono;
char *direccion;
};

void main()
{
struct persona *agenda;

Me gustaría saber en que me estoy equivocando y a poder ser porque me estoy equivocando o algún link de donde consultarlo ya que en todo sitio donde miré yo no encontré lo que buscaba. Gracias

Creo que hay cierte imprecisión en las declaraciones:
char *nombre es un punter a un caracter, no a una cadena
Debes emplear char nombre[50] o realizar reserva dinámica de memoria para 50 caracteres. Lo mismo para dirección
si quieres guardar un número de telefóno (9 dígitos) como valor int, creo que se producirá overflow (emplea datos ma´s amplios o tratalo como cadena de carateres.
Sslaudos
Ismael

Ismael tiene razón en lo del número de teléfono. Debes tratarlo como caracter. Si es entero tendrás overflow. Si lo pones como float a double, igual guardas 918452356 como teléfono y te devuelve 9.1845e8.

En cuanto a punteros: Primero reservar memoria para la estructura.

agenda = (struct persona *)malloc (sizeof(struct persona));

Luego reservar memoria para los campos de la estructura que son punteros

agenda->nombre = (char *)malloc (25); /* 24 letras para el nombre más largo más el fin de cadena = 25 caracteres */
agenda->direccion = (char *)malloc (25); /*Idem para la dirección*/

Cuando termines de usar la estructura, hay que liberar todo en orden inverso. Primero los campos

free (agenda->nombre);
free (agenda->dirección);

y finalmente la estructura

free (agenda);

También puedes hacer como te dijo Ismael. Hacer que el campo nombre, en vez de un puntero, sea un array
struct persona
{
char nombre[25];
char telefono [15];
char direccion[25];
};

Es más cómodo, aunque el tamaño ya queda igual para todas las personas.

Se bueno.