C 中大型连续二维数组（具有动态内存分配）的分段错误

Question

下面的代码应该创建两个具有连续内存位置的二维数组（posa 和 posb），然后使用 memmove 函数将 posa 复制到 posb 上。令人惊讶的是，它适用于“小”数组大小（例如：size1=size2=100），但对于较大的数组（例如：size1=size2=300），我会遇到分段错误（核心转储）。 当我使用动态内存分配时，我认为这不是像here 这样的堆栈溢出问题... 有人可以解释我做错了什么吗？

#include <stdio.h>
#include <stdlib.h>
#include <string.h>

int ** Allocation_array_x2(int size1, int size2){
    int **ptr;
    int i;
    ptr = malloc(size1 * sizeof(*ptr));
    if(ptr == NULL)  {        
        fprintf(stderr,"Allocation error");
        exit(1);
    }
    ptr[0] = malloc(size1 * size2 * sizeof(**ptr));
    for(i=1 ; i < size1 ; i++){
        ptr[i]=ptr[0]+(i * size2);
        if( ptr[i] == NULL) {
            fprintf(stderr,"Allocation error");
            exit(1);
        }
    }
    return ptr;                         
}

void Free_array_x2(int **ptr){
    free(ptr);
    ptr=NULL;
}

int main(){
    int size1=300, size2=300;
    int **posa, **posb;
    posa=Allocation_array_x2(size1,size2);
    posb=Allocation_array_x2(size1,size2);

    /* array_filling */
    for( int j = 0 ; j<size1 ; j++ ) {
        for( int k = 0 ; k<size2 ; k++ ) {
            posa[j][k] = 2*j+3*k+1;     
            posb[j][k] = 1000;
        }
    }

    memmove(posb,posa,size1*size2*sizeof(int));

    for(int i = 0 ; i<size1 ; i++ ) {
        for(int j = 0 ; j<size2 ; j++ ) {
            printf("%d\t%d\n",posa[i][j],posb[i][j]);
        }
    }


    Free_array_x2(posa);
    Free_array_x2(posb);

}

Answer 1

将代码更改为此为我解决了问题：

memmove(*posb,*posa,size1*size2*sizeof(int));

问题在于您的主要内存块实际上位于posa[0] 和posb[0]，而不是posa 和posb。 posa 和 posb 只是指向二级分配。

说到这里，应该指出您正在泄漏内存——您的 create 函数包含 2 个 malloc 调用，但您的 free 函数只释放其中一个。

Answer 2

错误在这里：

memmove(posb,posa,size1*size2*sizeof(int));

因为您将指针传递给指向整数的指针，但您应该将指针传递给整数：

memmove(*posb, *posa, size1 * size2 * sizeof(int));

Free_array_x2 方法中也有错误，因为您没有释放使用 malloc 从系统中获取的所有内存

void Free_array_x2(int **ptr){
    free(ptr);
    ptr=NULL;
}

这应该回到你在分配例程中所做的事情：

void Free_array_x2(int **ptr)
{
    free(ptr[0]);
    ptr[0] = NULL;
    free(ptr);
    ptr = NULL;
}