简单循环上的分段错误

Question

我正在尝试编写一个程序来计算给定矩阵的元素数量，例如 M[2][3] = [1,2,3][3,4,5] 应该给出 nxm 数组中从 0 到 9 的元素数。该算法在第 34 行（M[i][j] = i + 2;）之前都是正确的，其中我遇到了分段错误。我做错了什么？

#include<stdio.h>
#include<stdlib.h>
#define ROW 10
#define COL 10
#define MAX 10
void Print_Matrix(int **M,int row,int col);
int MallocX(int **M,int row,int col);
int main(void)
{
    int **M = 0x0;
    int count[MAX] = {0};
    int i,j;
    
    
    
    if(MallocX(M,ROW,COL)){
        fprintf(stdout,"Could not allocate memory\n");
        exit(1);
    }

    for(i = 0;i<ROW;i++){
        for(j = 0;j<COL;j++){
            M[i][j] = i + 2;
        }
    }
    Print_Matrix(M,ROW,COL);
    for(i = 0;i<ROW;i++){
        for(j = 0;j<COL;j++){
            ++count[M[i][j]];
        }
    }
    
    for(j = 0;j<MAX;j++){
        if(count[j]){
            printf("%d %d\n",j,count[j]);
        }
    }
    for(i = 0;i<ROW;i++){
        free(M[i]);
    }
    free(M);
}
int MallocX(int **M,int row,int col)
{
    int i;
    M = (int **) malloc(row * sizeof(int *));
    if(M == NULL){
        fprintf(stderr,"Error allocating memory\n");
        free(M);
        return 1;
    }

    for(i = 0;i<row;i++){   
        M[i] = (int *) malloc(col * sizeof(int));
        if(M[i] == NULL){
            fprintf(stderr,"Error allocating memory\n");
            free(M[i]);
            return 1;
        }
    }
    return 0;
}
void Print_Matrix(int **M,int row,int col)
{
    int i,j;
    for(i = 0;i<row;i++){
        for(j = 0;j<col;j++){
            printf("%d ",M[i][j]);
        }
        printf("\n");
    }
}

Answer 1

这是因为您按值而非引用传递 M。您的 MallocX 为您的矩阵分配内存，但是当您 return 到主程序时，这些分配是孤立的，其中 M 仍然是 0x0（或 NULL），这就是为什么分配函数通常 return 一个指针。也许你想要这样的东西：

int **MallocX(int row,int col)
{
    int **Matrix, i;

    Matrix = malloc(row * sizeof(int*));
    if(Matrix == NULL) {
        fprintf(stderr,"Error allocating memory\n");
        return NULL;
    }

    for (i = 0; i < row; i++) {
        Matrix[i] = (int*) malloc(col * sizeof(int));
        if(Matrix[i] == NULL){
            fprintf(stderr,"Error allocating memory (%d)\n",i);
            for (int j = 0; j < i; ++j)
                free(Matrix[j]);
            free(Matrix);
            return NULL;
        }
    }
    return Matrix;
}

然后在main中调用它：

if (!(M = MallocX(ROW,COL)) {
    fprintf(stdout,"Could not allocate memory\n");
    exit(1);
}

请注意，在您的原始代码中，您在 M 为 NULL 时调用了 free(M)，这本身会导致段错误。所以我也稍微整理了一下你的垃圾收集。

简单循环上的分段错误

segmentation fault on simple loop

c

matrix

segmentation-fault