1

おはよう

valgrind 言います:

==9735== 24,976 bytes in 446 blocks are definitely lost in loss record 9 of 9
==9735==    at 0x100012362: malloc (vg_replace_malloc.c:266)
==9735==    by 0x1000016F2: interpolate (in ./a.out)
==9735==    by 0x100000CFA: main (in ./a.out)

うわぁ。これが私の補間関数です：

void interpolate(PDouble* evaluated, int doubleCount){
int i, j, k;
int boxCount = 0;
int frameCount = 0;

for(i=0; i<doubleCount; i++){
  boxCount = evaluated[i]->first->numBoxes;
  frameCount = evaluated[i]->gap;
  evaluated[i]->changeMatrix = (int***)malloc(boxCount*sizeof(int**));

for(j=0; j < boxCount; j++){
  evaluated[i]->changeMatrix[j] = (int **)malloc(ATTR * sizeof(int*));

  for(k=0; k < ATTR; k++){
    evaluated[i]->changeMatrix[j][k] = (int*)malloc(sizeof(int)*frameCount);
    if(evaluated[i]->differenceMatrix[j][k] > 200 ||
       evaluated[i]->differenceMatrix[j][k] < -200){
      generateRotationSequence(evaluated[i]->changeMatrix[j][k],
                               evaluated[i]->first->boxes[j]->o,
                               evaluated[i]->second->boxes[j]->o,
                               frameCount);
    }
    else{
      evaluated[i]->changeMatrix[j][k] = (int*)malloc(sizeof(int)*frameCount);
      generateSequence(evaluated[i]->changeMatrix[j][k], 
                       evaluated[i]->differenceMatrix[j][k], frameCount);
    }
   }
  }
 }
}

しかし、PDoubles のすべてのポインターを解放する C 関数があります。必要に応じてコードを提供することはできますが、これについて漏れたり、ポインターがぶら下がったりするような何かが目立つことはありますか?

編集：説明に奇妙な用語を使用していたことに気付きました。また、今回はデストラクタを追加しています

void killDouble(PDouble marked){
int i, j, k;
int gap = marked->gap;
for(i=0; i < marked->first->numBoxes; i++){
  for(j=0; j < ATTR; j++){
    free(marked->changeMatrix[i][j]);
  }
  free(marked->changeMatrix[i]);
  free(marked->differenceMatrix[i]);
}
for(i=0; i < gap; i++){
  killFrame(marked->intFrames[i]);
}
killFrame(marked->first);
killFrame(marked->second);
free(marked->changeMatrix);
free(marked->differenceMatrix);
free(marked->intFrames);
free(marked);
}
4

2 に答える 2

5

あなたが割り当てますevaluated[i]->changeMatrix[j][k]

evaluated[i]->changeMatrix[j][k] = (int*)malloc(sizeof(int)*frameCount);
if(evaluated[i]->differenceMatrix[j][k] > 200 ||
   evaluated[i]->differenceMatrix[j][k] < -200){
  generateRotationSequence(evaluated[i]->changeMatrix[j][k],
                           evaluated[i]->first->boxes[j]->o,
                           evaluated[i]->second->boxes[j]->o,
                           frameCount);
}
else{
  evaluated[i]->changeMatrix[j][k] = (int*)malloc(sizeof(int)*frameCount);

無条件で、条件が満たされない場合は解放せずにもう一度。それはあなたの漏れです。

両方の割り当てはまったく同じなので、elseブロック内の割り当てを削除する必要があります。これは無意味です。

于 2012-06-11T15:53:50.043 に答える
2 
evaluated[i]->changeMatrix[j][k] = (int*)malloc(sizeof(int)*frameCount);

これが漏れです。メモリを割り当てますが、決して解放しません

于 2012-06-11T15:53:27.513 に答える