Segmentation fault on fopen C

Question

guy's well I have this code

#include <stdio.h>

typedef struct
{
    int numero, notaF, notaE;
    char nome[100];
} ALUNO;

void lerFicheiro(char path[], ALUNO alunos[], int *i);
void escreverFicheiro(char path[], ALUNO alunos[], int tamanho);

int main()
{
    //Declarações 
    int i=0,t=0;
    char path[999], wpath[999];
    ALUNO alunos[999];
    FILE *f;

    //Introdução do nome do ficheiro para leitura e para escrita
    printf("Introduza a localização do ficheiro para leitura: ");
    fgets(path,999,stdin); //segmentation fault e o fopen dá null (apenas no read)
    printf("Introduza a localização do ficheiro para escrita: ");
    fgets(wpath,999,stdin);

    //Leitura do ficheiro
        lerFicheiro(path,alunos,&t);

    //Escrita do ficheiro
    escreverFicheiro(wpath, alunos, t);

    return 0;
}

void lerFicheiro(char path[], ALUNO alunos[],int *i)
{
    FILE *f = fopen("dados1.txt","r");
    if(f!=NULL)
    {
        while(fscanf(f,"%d\n",&alunos[*i].numero)==1)
        {
            fgets(alunos[*i].nome,100,f);
            fscanf(f,"%d\n",&alunos[*i].notaF);
            fscanf(f,"%d\n",&alunos[*i].notaE);
            *i=*i+1;
        }
    }
    else
    {
        printf("Erro ao abrir o ficheiro\n");
    }
    fclose(f);
}

void escreverFicheiro(char path[], ALUNO alunos[], int tamanho)
{
    FILE *f = fopen(path,"w+");
    int i = 0, notaFinal = 0;
    for(i=0;i<tamanho;i++)
    {
        if(alunos[i].notaF>alunos[i].notaE)
            notaFinal = alunos[i].notaF;
        else
            notaFinal = alunos[i].notaE;
        if(notaFinal>=10)
        {
            fprintf(f,"%d\n",alunos[i].numero);
            fputs(alunos[i].nome,f);
            fprintf(f,"%d\n",notaFinal);
        }   
    }
    fclose(f);
}

But on the lerFicheiro function, on the fopen if I replace the "dados1.txt" by path I will get the error "Erro ao abrir o ficheiro" in english "Unable to open file" and right after segmentation fault I can't find the error anywere

You need to strip the trailing `\n` from the file name returned by `fgets`. — Paul R, Mar 05 '15 at 15:35
Shouldn't it be `void lerFicheiro(char *path,` instead of `void lerFicheiro(char path[],`? — maganap, Mar 05 '15 at 15:36
Move the `fclose(f);`, you are trying to close a file you did not open. — Weather Vane, Mar 05 '15 at 15:36
nop It should't... The second function is working just fine with the path but I'll give it a shot — Pedro Cavaleiro, Mar 05 '15 at 15:39

score 2 · Accepted Answer · answered Mar 05 '15 at 15:43

You should strip the trailing newline from the file name

char *sptr = strchr(path, '\n');
if (sptr) *sptr = '\0';

Also move the fclose(f);, you are trying to close a file you did not open.

void lerFicheiro(char path[], ALUNO alunos[],int *i)
{
    FILE *f = fopen("dados1.txt","r");
    if(f!=NULL)
    {
        while(fscanf(f,"%d\n",&alunos[*i].numero)==1)
        {
            fgets(alunos[*i].nome,100,f);
            fscanf(f,"%d\n",&alunos[*i].notaF);
            fscanf(f,"%d\n",&alunos[*i].notaE);
            *i=*i+1;
        }
        fclose(f);
    }
    else
    {
        printf("Erro ao abrir o ficheiro\n");
    }
}

Segmentation fault on fopen C

1 Answers1