Escribir una matriz de estructuras en un archivo binario en C

Question

Tengo una matriz de estructuras que me gustaría escribir en un archivo binario. Tengo un programa write.c y un programa read.c. El programa write.c parece funcionar correctamente, pero cuando ejecuto el programa read.c recibo un error de segmentación. Soy nuevo en C, por lo que sería genial si alguien pudiera revisar mi código para detectar cualquier error obvio. Prometo que no es demasiado largo :)

write.c:

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

struct Person 
{ 
    char f_name[256]; 
    char l_name[256]; 
    int age; 
}; 

int main(int argc, char* argv[]) 
{ 
    struct Person* people; 
    int people_count; 

    printf("How many people would you like to create: "); 
    scanf("%i", &people_count); 
    people = malloc(sizeof(struct Person) * people_count); 

    int n; 
    for (n = 0; n < people_count; n++) 
    { 
     printf("Person %i's First Name: ", n); 
     scanf("%s", people[n].f_name); 

     printf("Person %i's Last Name: ", n); 
     scanf("%s", people[n].l_name); 

     printf("Person %i's Age: ", n); 
     scanf("%i", &people[n].age); 
    } 

    FILE* data; 
    if ((data = fopen("data.bin", "wb")) == NULL) 
    { 
     printf("Error opening file\n"); 
     return 1; 
    } 

    fwrite(people, sizeof(struct Person) * people_count, 1, data); 
    fclose(data); 

    return 0; 
} 

read.c:

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

struct Person 
{ 
    char f_name[256]; 
    char l_name[256]; 
    int age; 
}; 

int main(int argc, char* argv[]) 
{ 
    FILE* data; 
    if ((data = fopen("data.bin", "rb")) == NULL) 
    { 
     printf("Error opening file\n"); 
     return 1; 
    } 

    struct Person* people; 

    fread(people, sizeof(struct Person) * 1/* Just read one person */, 1, data); 
    printf("%s\n", people[0].f_name); 

    fclose(data); 

    return 0; 
} 

Gracias por la ayuda!

Answer 1

struct Person* people; 

Esto asigna solo un puntero a struct, pero no tiene ningún espacio asignado para el contenido de la estructura real. O bien utilizar malloc de manera similar a su programa de escritura, o intentar algo como:

struct Person people; 
fread(&people, sizeof(people), 1, data); 

Answer 2

o hay que hacer malloc de memoria en el puntero variable de people antes de hacer la fread, o (más fácil) acaba de leer directamente en una variable local:

struct Person people; 

fread(&people, sizeof(struct Person) * 1/* Just read one person */, 1, data); 

Answer 3

es necesario asignar espacio para los datos que está leyendo en:

people = malloc(sizeof(*people)*numPeople); 

Answer 4

Primero debe asignar memoria a la persona. Cambio: struct Person* people; en struct Person* people = malloc(sizeof(struct Person));. Y no se olvide de liberar su memoria al final: free(people);.