Не удается найти причину ошибки сегментации (сбрасывание ядра) в C

Question

Я пытаюсь получить вход пользователя, записать ввод в файл, а затем использовать данные для структуры. Файл компилируется нормально, но после первого ввода я получаю сообщение об ошибке:

$ gcc student.c -o student 
$ ./student 
Student First Name: 
alex 
Student Last Name: 
Segmentation fault (core dumped) 

Код:

/* student.h is to include the structure for stundent info */ 
#include <stdio.h> 
#include <stdlib.h> 

typedef struct { 
    char sID[9]; 
    char firstN[30]; 
    char lastN[30]; 
    }student[100]; 

#include "student.h"  
int main() 
{ 
    FILE *sIf; 
    int i; 
    int num; 
    student *sI; 
    char *c; 
    char *b; 
    char *a; 

    sIf = fopen("student.txt", "a+"); 
    if(!sIf) 
    { 
     printf("File could not be opened\n\a\a"); 
     getchar(); 
     return -1; 
    } 
    { 
     printf("Student First Name: \n"); 
     scanf("%s", b); 
     printf("Student Last Name: \n"); 
     scanf("%s", a); 

     printf("Student ID Number: \n"); 
     scanf("%s", c); 

     fprintf(sIf, "%s, %s\t%s\n", a, b, c); 
    } 

    fclose(sIf); 
    return 0; 
} 

Answer 1

1. Всегда использовать правильный отступ при кодировании, это делает код легко читать.
2. Вы не выделяете память для своих char указателей a, b, c.
3. Вместо scanf() использовать fgets(), чтобы принять входные строки.

Попробуйте этот фиксированный код (при необходимости, я добавил комментарии).

/* student.h is to include the structure for stundent info */ 

#include <stdio.h> 
#include <stdlib.h> 
#include <string.h> 
//#include "student.h" 
typedef struct { 
    char sID[9]; 
    char firstN[30]; 
    char lastN[30]; 
    }student[100]; 
int main() 
{ 
    FILE *sIf; 
    student *sI;//Why are you using it? 
    char *ptr_pos; 
    //Fix 1 Allocate space for a, b and c 
    char *c=malloc(sizeof(char)*256); 
    if(c==NULL) 
    { 
     puts("Failed to allocate memory for variable c"); 
     exit(1); 
    } 
    char *b=malloc(sizeof(char)*256); 
    if(b==NULL) 
    { 
     puts("Failed to allocate memory for variable b"); 
     exit(1); 
    } 
    char *a=malloc(sizeof(char)*256); 
    if(a==NULL) 
    { 
     puts("Failed to allocate memory for variable a"); 
     exit(1); 
    } 

    sIf = fopen("student.txt", "a+"); 
    if(!sIf) 
    { 
     printf("File could not be opened\n\a\a"); 
     return EXIT_FAILURE; 
    } 
    printf("Student First Name: \n"); 
    //Accept string using fgets, it prevents from overflow 
    fgets(b,256,stdin); 
    b[strlen(b)-1]='\0'; //Remove \n from the input string 
    // Or you can use the following to remove `\n` from input string 
    //if((ptr_pos=strchr(b, '\n')) != NULL) 
    // *ptr_pos = `\0`; 


    printf("Student Last Name: \n"); 
    fgets(a,256,stdin); 
    a[strlen(a)-1]='\0'; 

    printf("Student ID Number: \n"); 
    fgets(c,256,stdin); 
    c[strlen(c)-1]='\0'; 

    fprintf(sIf, "%s, %s\t%s\n", a, b, c);  

    fclose(sIf); 
    //Free allocated memory 
    free(a); 
    free(b); 
    free(c); 

    return EXIT_SUCCESS; 
}