El fallo está claro.
char *line;
Estas declarando un puntero
que apunta a algo, no sabemos que.
Cambialo por
char line[MAX_LINE];
y todo debería ir bien.
En C, un código del tipo char NOMBRE[123]
lo que hace es reservar memoria para un grupo de 123 elementos, cada uno de los cuales es un char; pueden ser char
o int
o cualquier otro tipo.
Una curiosidad de lo anterior es que, por decirlo de alguna forma, la expresión anterior devuelve un puntero; de echo, un char *
y un char[]
son intercambiables entre si. Es por ello que puedes seguir llamando a tu función read_line
con el mismo argumento, aunque hayas cambiado su tipo.
Cambiando un poco el tema, el Segmentation Fault
que te muestra, indica que has intentado acceder a memória fuera de tu alcance. Normalmente, memória que no pertenece a tu proceso. En este caso, al hacer char *line;
, el puntero resultante apuntaba a alguna lugar desconocido, que muy difícilmente estaría a tu disposición.