Fallo segmentacion con fread

[mikeW]

Buenas, ¿sabéis por qué me da un fallo segmentación con la función fread?

Código:

#include <stdio.h>
int main() {

    FILE *fichero1;
    char buffer[255];

    fichero1=fopen("hola.txt", "r");
    
    while(feof(fichero1)==0) {
        fread(&buffer, sizeof(buffer), 1, fichero1);
    }
    fclose(fichero1);
}

Uso GCC como compilador y linux

[sam90]

Acabo de probar el codigo. Con el archivo creado no falla. Pero si borras el archivo si falla. Comproba que el archivo este exista. Por cuando abris un archivo en modo lectura que no existe, es no es creado, asi que la variable fichero1 no esta seteada correctamente.

[Fw190]

Ese fallo de segmentación tiene dos posibles causas:

1.- Como te dice sam90, si el archivo no existe el programa fallará. Comprueba el valor de tu variable fichero nada más abrir el archivo, si vale NULL es que ha fallado la operación de apertura, y cualquier intento de operar con ese archivo te puede provocar ese error de segmentación. Deberías poner algo así como

Código C++:

fichero = fopen(....);
if (fichero == NULL)
{
// Tratar el fallo en la apertura
}
else
{
// Hacer el tratamiento del fichero
}
 

2.- Cuando llamas a fread, estás pasando la dirección de la variable buffer:

fread(&buffer,....);

Ten en cuenta que buffer ya ha sido declarada como array, con lo que el nombre del array es un puntero al primer elemento. La llamada deberá ser:

fread(buffer, ...);

Si el archivo ya existía, ésta es la causa más probable del error.

Un par de comentarios adicionales:

1.- Tal como has puesto la apertura del fichero, ésta se va a hacer en modo secuencial (texto), pero luego utilizas fread, que es para acceso directo (binario).Si vas a leerlo en modo texto, utiliza fscanf, fgets, etc, o ábrelo con "rb" si vas a leerlo en modo binario.

2.- La marca de final de fichero (código ASCII 26) no tiene sentido cuando lees un fichero en modo binario, ya que puedes tener un byte con ese valor que no tenga nada que ver con el final de fichero. Lo digo porque el bucle de lectura del fichero lo controlas con la marca de fin de fichero, como secuencial (que es como lo has abierto), pero luego lo estás leyendo como binario (con fread), así no detectará el final de fichero y puede entrar un bucle infinito.