尝试调整2D数组的大小时出现分段错误

时间:2016-12-03 16:32:05

标签: c arrays matrix realloc calloc

这是代码:

int **mat(int nl, int nc) {
  int i;
  int **v = malloc(nl * sizeof(int *));

  for (i = 0; i < nl; i++) {
    v[i] = calloc(nc, sizeof(int));
  }
  return v;
}


void crop(int **v,int *nl,int *nc,int l1,int c1,int l2,int c2)

{

int i,j;

for(i=0;i<(l2-l1);i++)
    for(j=0;j<(c2-c1)*3;j++)
        v[i][j]=v[l1+i][c1*3+j];

for(i=l2-l1;i<*nl;i++)
    free(v[i]);

v=realloc(v,(l2-l1)*sizeof(int *));

for(i=0;i<*nl;i++)
    v[i]=realloc(v[i],(c2-c1)*3*sizeof(int));

int x=l2-l1,y=c2-c1;
*nl=x;
*nc=y;

}


void resize(int **v,int *nl,int *nc,int nw,int nh)
{
int i,j,h=*nl,w=*nc,x=nh,y=nw;

if(nh>h && nw<=w)
    {
        crop(v,&w,&h,0,0,*nl,nw);
        v=realloc(v,nh*sizeof(int *));
        for(i=*nl;i<nh;i++)
        {
        v[i]=calloc(nw*3,sizeof(int));
        for(j=0;j<=nw*3;j++)
        v[i][j]=255;
        }

    }
    if(nh<=h && nw>w)
    {
        crop(v,&w,&h,0,0,nh,*nc);
        for(i=0;i<nh;i++)
        {
        v[i]=realloc(v[i],nw*3*sizeof(int));
        for(j=*(nc)*3;j<=nw*3;j++)
        v[i][j]=255;
        }
    }
    *nl=x;
    *nc=y;
}

int main(){

int nl,nc,i,j;
scanf("%d%d",&nc,&nl);
int **p = mat(nl,nc*3);

for(i=0;i<nl;i++)
    for(j=0;j<nc*3;j++)
        p[i][j]=i+j;

resize(p,&nl,&nc,2,4);

for(i=0;i<nl;i++)
{
    for(j=0;j<nc*3;j++)
    printf("%d ",p[i][j]);
    printf("\n");
}

nc=2,nl=3

因此,当我在主resize(p,&nl,&nc,4,2)中调用时,如果第一个裁剪矩阵的底部,则调整大小函数会在第二个调整中导致新高度小于旧高度(nh&gt; nl),在裁剪之后,它会继续显示剩余的线条(2)和realloc的内存,以便它可以用{255,255,255}填充新的宽度,一切顺利。

然而,当我调用resize(p,&nl,&nc,2,4)时,它继续第一个if,调试器显示没有错误并且它正在填充2D数组,但当它到达主要的printf时,我得到分段故障。可能是什么问题?

1 个答案:

答案 0 :(得分:0)

正如BLUEPIXY用 void crop暗示的那样 - &gt; int **cropreturn v; ,我们必须考虑realloc可能会移动指向的区域,因此为您的行指针数组调用realloc的函数返回可能更改的内存指针,调用函数是用返回的值重新分配它。这不仅适用于函数crop(),也适用于resize(),因此必要的更改包括:

int **crop(int **v, int *nl, int *nc, int l1, int c1, int l2, int c2)
{
    …
    return v;
}

int **resize(int **v, int *nl, int *nc, int nw, int nh)
{
    …
        v = crop(v, &w, &h, 0, 0, *nl, nw);
    …
        v = crop(v, &w, &h, 0, 0, nh, *nc);
    …
    return v;
}

int main()
{
    …
    p = resize(p, &nl, &nc, 2, 4);
    …
}