Valgrind在定义char *数组时报告内存泄漏

Question

我还有另一个无法识别的valgrind错误。 我有一些代码加载脚本，然后使用strtok（）标记已加载到缓冲区的脚本中的数据。 我使用以下方法在堆上为缓冲区分配内存：

char *pPngStr = new char[4096];

然后在函数结尾，我使用以下命令释放内存：

delete[] pPngStr;

Valgrind如此报告分配：

173 bytes in 1 blocks are definitely lost in loss record 3,753 of 4,627

Valgrind还报告了我释放内存的行：

Invalid free() / delete / delete[] / realloc()

这全部在同一个函数中，变量仅在该函数而不是类中是局部的。 我不知道valgrind如何认为这是内存泄漏。 我分配了我需要的所有内存，然后在完成后释放它。 我看不出问题是什么。

这是完整的功能（可能有点混乱）：

void CSprite::LoadSprite()
{
FILE *pF = fopen("Data/Art/coin/coin.sprite", "rb");

if(!pF)
{
    return;
}

fseek(pF, 0, SEEK_END);
int length = ftell(pF);
fseek(pF, 0, SEEK_SET);

char *aData = new char[length];

fread(aData, 1, length, pF);

CheckGLError();

/**
  * The scripts are setup like so:
  ***********************************************
  * Pngs:
  * 4; // This is the number of frames.
  * <pngname>_001.png;
  * <pngname>_002.png;
  * <pngname>_003.png;
  * <pngname>_004.png;
  *
  * Properties:
  * name=<obj name>;
  * width=###;
  * height=###;
  * framerate=0.### // rate in seconds
  ************************************************
  */

mpSprite = new SGfxSprite;

char *pPngStr = new char[4096];
pPngStr = strtok(aData, "\n");

pPngStr = strtok(NULL, "\n");
int iNumFrames = atoi(pPngStr);
mpSprite->numFrames = iNumFrames;

mpSprite->textures = new GLuint[iNumFrames];
glGenTextures(iNumFrames, mpSprite->textures);

int i = 0; // used for loop only!
while(pPngStr != NULL)
{
    pPngStr = strtok(NULL, "\n");

    if(!strcmp(pPngStr, "Properties:") || i >= iNumFrames)
    {
        break;
    }

    if(mpPng->LoadPng(pPngStr))
    {
        CheckGLError();

        glBindTexture(GL_TEXTURE_2D, mpSprite->textures[i]);
        CheckGLError();

        glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MIN_FILTER, GL_LINEAR_MIPMAP_LINEAR);
        CheckGLError();

        glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_MAG_FILTER, GL_LINEAR);
        CheckGLError();

        glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_S, GL_CLAMP);
        CheckGLError();

        glTexParameteri(GL_TEXTURE_2D, GL_TEXTURE_WRAP_T, GL_CLAMP);
        CheckGLError();

        glTexParameteri(GL_TEXTURE_2D, GL_GENERATE_MIPMAP, GL_TRUE);
        CheckGLError();

        glTexImage2D(GL_TEXTURE_2D, 0, GL_RGBA, mpPng->miWidth, mpPng->miHeight, 0, GL_RGBA, GL_UNSIGNED_BYTE, mpPng->mpData);
        CheckGLError();
    }

    ++i;
}

pPngStr = strtok(NULL, "=\n");
pPngStr = strtok(NULL, "\n");
mpSprite->name = pPngStr;

pPngStr = strtok(NULL, "=\n");
pPngStr = strtok(NULL, "\n");
mpSprite->width = atoi(pPngStr);

pPngStr = strtok(NULL, "=\n");
pPngStr = strtok(NULL, "\n");
mpSprite->height = atoi(pPngStr);

pPngStr = strtok(NULL, "=\n");
pPngStr = strtok(NULL, "\n");
mpSprite->framerate = (atof(pPngStr) * 1000);

mpSprite->x = 0;
mpSprite->y = 0;

mpSprite->currentFrame = 0;
mpSprite->paintTimer = 0;

fclose(pF);

delete[] pPngStr, aData;

}

*请注意：此函数包含OpenGL代码，并且还会从此处未定义的数据结构中调用成员。 使用Qt Creator 2.4.1在带有g ++（gcc 4.6）的Xubuntu 12.04中编译代码时也不会出现任何警告或错误。

Answer 1

char *pPngStr = new char[4096];
pPngStr = strtok(aData, "\n");

在新代码之后的一行中，您忘记了要使用的指针，而是使用该变量存储其他数据。 毫无疑问，内存泄漏。

Answer 2

您拥有的行：

 delete[] pPngStr, aData; 

不按照您的想法去做。

尝试以下方法：

delete[] pPngStr;
delete[] aData;

注意 ：在修复SJuan76识别的错误之前，我的回答还没有定论。