6

我有以下代码：

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 上出现段错误。

这是否可以被其他人重现，我在使用 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");
}

因此，您已到达调用堆栈的顶部。

4

1 回答 1

8

您对输入（可能还有高度）数组的声明应该是int inputs[4];您想要一个整数数组，而不是指向整数的指针数组。然后，您希望仅使用inputs和heights作为参数调用函数。

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

于 2011-02-28T01:11:29.563 回答