Ошибка сегментации с помощью fscanf-not allocate memory issue

Question

Я пытаюсь сделать этот код, чтобы читать греческие буквы из файла и печатать английские буквы, которые звучат одинаково с помощью fscanf. Проблема в том, что я продолжаю получать ошибку сегментации. Что мне не хватает?

#include <stdio.h> 

int main() 
{    
    char s[100];  
    int i; 
    FILE * myfile; 

    myfile = fopen("/home/angelos/Downloads/λεξικο2.txt", "r"); 

    while(fscanf(myfile, "%s", s) == 1) 
     { 
      for (i=0 ; i<100 ; i++) 
       { 
        if (s[i] == 'Α') 
         { printf("A") ; } 
        else 
        if (s[i] == 'Β') 
         { printf("V") ; } 
     } 
} 

Answer 1

У вас есть 3 серьезных проблемы с вашим кодом.

1) Вы никогда не проверяете, успешно ли fopen.

2) Вы можете читать неинициализированные подписанные данные.

3) Вы можете переполнить ваш входной буфер

Все три вещи могут привести к вашей программе на провал.

Попробуйте эти изменения:

#include <stdio.h> 
#include <string.h> 

int main() 
{    
    char s[100];  
    int i; 
    FILE * myfile; 

    myfile = fopen("/home/angelos/Downloads/λεξικο2.txt", "r"); 

    // Check that fopen went fine 
    if (!myfile) 
    { 
     printf("Failed to open file\n"); 
     return 1; 
    } 

    while(fscanf(myfile, "%99s", s) == 1) 
         // ^^ 
         // Never read more than 99 chars (i.e. 99 + a terminating null byte) 
     { 
      for (i=0 ; i<strlen(s) ; i++) 
         // ^^^^^^ 
         // Only iterate over the valid chars 
       { 
        if (s[i] == 'Α') 
         { printf("A") ; } 
        else 
        if (s[i] == 'Β') 
         { printf("V") ; } 
     } 
}