for循环中的分段错误[重复]

Question

我遇到了这段代码的分段错误，我一辈子都无法弄清楚原因。我对指针比较陌生，所以它可能是显而易见的。该代码应该从标准输入中获取两行整数并以交替方式打印出来。我还没有完成打印出交替数字的代码的最后一部分。

    int main(){
        char *str1 = NULL;
        char *str2 = NULL;
        size_t sz = 0;
        int i;
        int x;
        char *rp1 = NULL;
        char *rp2 = NULL;
        getline(&str1, &sz, stdin);
        getline(&str2, &sz, stdin);
        char *result1;
        result1 = malloc(sizeof(*str1));
        char *result2; 
        result2 = malloc(sizeof(*str2));
        for(i = 0; (x = sscanf(str1, "%s ", &result1[i])) > 0; i++){
            if(x == EOF){
               return 0;
            }
            if(!isdigit(result1[i])){
               fprintf(stderr, "Error: invalid non-integer input\n");
               return 1;
            }
        }
        rp1 = malloc(i);
        rp1 = result1;
        for(i = 0; sscanf(str2, "%s ", &result2[i]) > 0; i++){
            if(!isdigit(result2[i])){
            fprintf(stderr, "Error: invalid non-integer input\n");
                return 1;
            }
        }
        rp2 = malloc(i);
        rp2 = result2;
        return 0;
  }

当我运行 gdb 时，它说我在第 25 行遇到了分段错误，这是第一个 for 循环。

编辑：所以我已经修复了 for 循环之前的 malloc 函数中的一个问题，但我仍然遇到分段错误。

Answer 1

问题一：

sizeof(*str[1-2]) 与sizeof(char) 相同。您正在为两个结果缓冲区分配一个字符，这显然是不够的。使用strlen(str[1-2]) + 1 这将为您提供一个与字符串中字符数相等的缓冲区，再加上一个用于空终止符的缓冲区。

问题 2：

getline() 分配一个缓冲区，在你完成它之后你应该分配给free()，你没有这样做，这实际上是内存泄漏。这不是您的问题的原因，但值得一提。你所有的malloc()'d 内存也是如此。

Answer 2

这段代码有很多问题。其他人已经指出了malloc 的问题，但总的来说它是不必要的复杂。所有中间变量都可以去掉，直接访问str1和str2。

为了简化，我将把它简化为一个输入字符串，str。如果你想要两个，代码应该放在一个函数中而不是复制。

for + sscanf 循环似乎试图遍历 str 并检查它是否只包含数字。遍历字符数组直到最后遇到空字符会更好地完成此操作。

for(int i = 0; str[i] != '\0'; i++){
    ...do something with str[i]...
}

所以检查输入的每个字符是一个数字...

for(int i = 0; str[i] != '\0'; i++){
    if(!isdigit(str[i])){
        fprintf(stderr, "Error: invalid non-integer input '%c'\n", str[i]);
        return 1;
    }
}

这将失败，因为从标准输入读取的str 末尾有一个换行符。