简单环路上的c分段故障

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 memoryn");
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 %dn",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 memoryn");
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 memoryn");
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为矩阵分配内存，但当返回到主程序时，这些分配是孤立的，其中M仍然是0x0(或NULL(，这就是分配函数通常返回指针的原因。也许你想要这样的东西：

int **MallocX(int row,int col)
{
int **Matrix, i;
Matrix = malloc(row * sizeof(int*));
if(Matrix == NULL) {
fprintf(stderr,"Error allocating memoryn");
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 memoryn");
exit(1);
}

请注意，在原始代码中，当M为NULL时调用free(M)，这本身将导致segfault。所以我也把你的垃圾收集整理了一下。