C - fgets segfault

时间:2011-02-28 00:57:04

标签: c segmentation-fault fgets

我有以下代码:

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

它给了我一个标记的段错误。我完全不知道问题是什么,因为我在不同的程序中有以下代码:

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

它工作正常。我100%肯定它不是atoi上的segfaulting。

这是否可以被其他人重现,我使用GCC 4.4.5在Linux amd64上。它编译并输出没有警告。

自请求以来,这里是调用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 个答案:

答案 0 :(得分:8)

你对输入(可能是高度)数组的声明应该是int inputs[4];你需要一个整数数组,而不是一个指向整数的指针数组。然后,您只需使用inputsheights作为参数调用函数。

您现在正在做的是创建一个包含4个整数指针的数组。然后使用*inputs调用函数只会将该数组中的第一个指针传递给函数get_intget_int函数将尝试将第一个指向整数的指针用作数组,这不是您想要的行为。