C中的分段错误,似乎无法找到原因

时间:2013-03-21 08:49:15

标签: c segmentation-fault

我似乎无法在下面的代码中找到分段错误的原因。从evaluate()跳回main()后,代码在注释行中崩溃。我用gdb查看了这段代码,似乎在evaluate函数中一切正常 - 输出正确填充,但是在我返回main之后,内存中的任何内容都无法访问。 gdb中的“p j”返回内存不可访问错误,同时尝试打印任何其他变量。我检查了堆栈是否有许多值,但增加堆栈大小并没有帮助。

疯狂的是我可以解决这个错误,但我不知道它为什么会改变任何东西。如果我在evaluate()中添加一个int声明(例如int iAmNotUsedEver;),那么代码突然不会导致分段错误并且在gdb中完美运行。 编辑:在评估(int *node = malloc(116*sizeof(int));)中动态分配节点[116]也解决了问题,但我又不知道为什么?

评估函数:(我删除了部分代码,否则会过长)

void evaluate(int *parameter, int output[][16]) {
    int node[116];
    node[0] = 0;
    output[0][0] = !node[0];
    node[1] = parameter[0];
    output[0][1] = !node[1];
    output[0][2] = !node[0];
    ...
    node[34] = !node[114] && !node[45];
    node[45] = !node[34] && !node[105];
    output[11][15] = node[45];
}

主要功能:

int main(int argc, char *argv[]) {
    int i;
    int j;
    int k;
    int ret;
    int parameter[8];
    int output[12][16];
    FILE *paramFile;
    FILE *outFile;

    paramFile = fopen(argv[1], "r");
    if (paramFile == NULL) {
        printf("I couldn't open the parameter file.\n");
        exit(0);
    }
    outFile = fopen(argv[2], "ab");
    if (outFile == NULL) {
        printf("Something went wrong with the output file.\n");
        exit(0);
    }
    while(1){
        for(i=0;i<8;i++){
            ret=fscanf(paramFile, "%d", &parameter[i]);
            printf("parameter: %d\n", parameter[i]);
        }
        if(ret!=1){
            break;
        }
        for(j=0;j<12;j++){
            for(k=0;k<16;k++){
                output[j][k] = 2;
            }
        }
        evaluate(parameter,output);
        printf("Evaluation is done. \t %d\n",i);
        for(j=0;j<12;j++){ //SEG FAULT HERE
            for(k=0;k<16;k++){
                fprintf(outFile, "%d", output[j][k]);
            }
            fprintf(outFile,"\n");
        }
        fprintf(outFile,"\n\n\n");
    }
    printf("Closing files\n");  
    fclose(paramFile);
    fclose(outFile);
    return 0;
}

1 个答案:

答案 0 :(得分:0)

首先,您应该检查是否至少有两个参数。这也可能会引发分段错误。

if (argc < 3)
    return -1;

然后,在评估中,因为它们是fixe数组,所以你应该传入参数

void evaluate(int parameter[8], int output[12][16]) {
    ...
}

你也可以这样做

void evaluate(int *parameter, int **output) {
    ...
}

我认为段错误可以避免在这两个声明中使用。