Question

我在这里处理一个很小的问题，我的代码几乎可以工作。

现在这是我的问题。

我试图用这样的浮点数分配一个字段树：

float ** PermLaster;
float ** VarLaster;

然后我使用calloc

PermLaster= (float **) calloc(AntPermLast, sizeof(float*));
VarLaster= (float **) calloc(AntVarLast, sizeof(float*));

然后在嵌套for：

中再次调用calloc

for (StegLastAnt= 0; StegLastAnt <  sizeLastM;StegLastAnt++)
{
    AktLast = inLastP[StegLastAnt];
    if (StegLastAnt > 0)
        {
            OldAktLast=inLastP[StegLastAnt-1];
        }

    if (((int(inLastP[7*(sizeLastM-1)+StegLastAnt+7])== 0)||(int(inLastP[7*(sizeLastM-1)+StegLastAnt+6])== 0)) && (AktLast != OldAktLast))
        {
            PermLaster[NumPerm] = (float *) calloc(AntAktuell*6, sizeof(float));
            AntLastVekt[NumPerm]=AntAktuell;
            NumPerm++;
            AntAktuell=0;
        }


    if ((int(inLastP[7*(sizeLastM-1)+StegLastAnt+7])== 1) && (AktLast != OldAktLast) && (int(inLastP[7*(sizeLastM-1)+StegLastAnt+6]) != 0))
    {
            VarLaster[NumVar] = (float *) calloc(AntAktuell*6, sizeof(float));
            AntLastVekt[NumPerm+NumVar]=AntAktuell;
            NumVar++;
            AntAktuell=0;
    }
    AntAktuell++;
}

到目前为止没有错误。

然后分配所有元素：

  for (StegLastAnt= 0; StegLastAnt < sizeLastM;StegLastAnt++)
        {
        AktLast = inLastP[StegLastAnt];
        if (StegLastAnt > 0)
            {
                OldAktLast=inLastP[StegLastAnt-1];
            }

        if (((int(inLastP[7*(sizeLastM-1)+StegLastAnt+7])== 0)||(int(inLastP[7*(sizeLastM-1)+StegLastAnt+6])== 0)) && (AktLast != OldAktLast))
            {
                PLastP = PermLaster[NumPerm];
                RadPos=StegLastAnt-AntAktuell;

                for (StegLastAnt2 = 0;StegLastAnt2<AntAktuell;StegLastAnt2++)
                {
                    PLastP[StegLastAnt2*6+0]=float(inLastP[1*(sizeLastM-1)+RadPos+1]);PLastP[StegLastAnt2*6+1]=float(inLastP[2*(sizeLastM-1)+RadPos+2]);PLastP[StegLastAnt2*6+2]=float(inLastP[3*(sizeLastM-1)+RadPos+3]);PLastP[StegLastAnt2*6+3]=float(inLastP[4*(sizeLastM-1)+RadPos+4]);PLastP[StegLastAnt2*6+4]=float(inLastP[5*(sizeLastM-1)+RadPos+5]);PLastP[StegLastAnt2*6+5]=float(inLastP[6*(sizeLastM-1)+RadPos+6]);
                    RadPos++;
                }
                NumPerm++;
                AntAktuell=0;
                PLastP=nullptr;
            }
        if ((int(inLastP[7*(sizeLastM-1)+StegLastAnt+7])== 1) && (AktLast != OldAktLast) && (int(inLastP[7*(sizeLastM-1)+StegLastAnt+6]) != 0))
        {
                VLastP = VarLaster[NumVar];
                RadPos=StegLastAnt-AntAktuell;
                for (StegLastAnt2 = 0;StegLastAnt2<AntAktuell;StegLastAnt2++)
                {
                    VLastP[StegLastAnt2*6+0]=float(inLastP[1*(sizeLastM-1)+RadPos+1]);VLastP[StegLastAnt2*6+1]=float(inLastP[2*(sizeLastM-1)+RadPos+2]);VLastP[StegLastAnt2*6+2]=float(inLastP[3*(sizeLastM-1)+RadPos+3]);VLastP[StegLastAnt2*6+3]=float(inLastP[4*(sizeLastM-1)+RadPos+4]);VLastP[StegLastAnt2*6+4]=float(inLastP[5*(sizeLastM-1)+RadPos+5]);VLastP[StegLastAnt2*6+5]=float(inLastP[6*(sizeLastM-1)+RadPos+6]);
                    RadPos++;
                }
                NumVar++;
                AntAktuell=0;
                VLastP=nullptr;
        }
    AntAktuell++;
}

它从输入读取，我一直在调试它，它工作正常。我故意避免在那里额外的for循环，以确保发生什么，并使其更容易调试（虽然看起来不太好）。它工作正常，我一直在查看正在进行的值，它们是正确的。没有与地址损坏有关的问题。

然后经过大量的代码来释放它。

for (StegLastAnt= 0; StegLastAnt < AntPermLast;StegLastAnt++)
{
    free(PermLaster[StegLastAnt]);
}

这里有香蕉和抱怨堆腐败。

for (StegLastAnt= 0; StegLastAnt < AntVarLast-1;StegLastAnt++)
{
    free(VarLaster[StegLastAnt]);
}
free(PermLaster), free(VarLaster);

字段所经过的函数都有“PermLaster”作为const指针const指针指向const float。没有更改地址，内部字段中也没有值。记忆似乎很好，为什么会发生这种情况呢？可能是因为我的子场使用了不同的尺寸（我必须为我的应用而无法解决这个问题）？使用realloc会更好吗？用new切换到c ++并删除？

还有一件事，指向PermLaster的指针通过它传递的函数声明为restrict。这可能有问题吗？

我相信你们中的一些人会说使用更多的结构或类，但我不想让问题变得更大。

我非常感谢这里的好建议。

约翰

Answer 1

您没有向我们提供所有代码。我想假设NumPerm和NumVar在循环之间重置为0。

顺便说一下，这里还有另一个问题：

        AntLastVekt[NumPerm]=AntAktuell;

除非所有“Perms”都出现在所有“Vars”之前，否则这应该是AntLastVekt [NumPerm + NumVar]。否则，如果你有序列Perm / Var / Perm，第二个Perm将覆盖AntLastVekt [1]。

但是，这不是问题的原因，因为AntLastVekt在第二个循环中未使用。

错误在第二个循环中，即使操作系统在释放数组之前没有检测到它。为了证明这一点，请注释掉中间的所有代码。您可以使用valgrind进行调试。

使用calloc堆损坏

1 个答案: