C - Fgets Segfault - PullRequest
Новая
       16

C - Fgets Segfault

5 голосов
/ 28 февраля 2011

У меня есть следующий код: 

int get_int(void) {
    char input[10];
    fgets(input, 10, stdin); // Segfault here
    return atoi(input);
}

Это дает мне segfault, где отмечен.Я понятия не имею, в чем проблема, потому что у меня есть следующий код в другой программе: 

int main(void) {
    char card[17];
    printf("Number: ");
    fgets(card, 17, stdin);
    printf("%s\n", card_type(card));
    return 0;
}

И он отлично работает.Я на 100% уверен, что это не segfaulting на Atoi.

Является ли это воспроизводимым другими, я на Linux amd64 с использованием GCC 4.4.5.Он скомпилирован и не выводит предупреждений.

Так как это было запрошено, вот код, который вызывает get_int: 

void get_input(int *inputs) { // Stop cluttering up my main
    printf("M spotting F: ");
    inputs[0] = get_int();
    printf("F spotting M: ");
    inputs[1] = get_int();
    printf("F spotting F: ");
    inputs[2] = get_int();
    printf("M spotting M: ");
    inputs[3] = get_int();
}

Код, который вызывает это: 

int main(void) {
    int *inputs[4];
    int *heights[4];
    get_input(*inputs);
    get_heights(*inputs, *heights);

    print_bars(*heights);

    printf("M4F  F4M  F4F  M4M\n");
}

И, таким образом, вы достигли вершины стека вызовов.

1 Ответ

8 голосов
/ 28 февраля 2011

Ваши объявления для входных (и, вероятно, высотных) массивов должны быть int inputs[4]; Вы хотите массив целых чисел, а не массив указателей на целые числа. Затем вы хотите вызвать функции, используя в качестве параметров только inputs и heights.

Что вы делаете сейчас, это создание массива из 4 указателей на целые числа. И затем вызов функции с *inputs передаст только первый указатель в этом массиве функции get_int. Функция get_int попытается использовать первый указатель на целое как массив, что не соответствует желаемому поведению.

...