Question

我的代码中有一段时间内有一个我无法弄清楚的错误。也许我只是盲目，但这个bug没有多大意义，因为从我所看到的，这段代码是完美的。我猜测其他地方可能存在腐败，但我想我会在这里粘贴我的问题代码以防万一。

我知道存在堆损坏，因为我收到标准的错误消息，后跟free（）无效的下一个大小消息。我已经包含了我认为相关的内容。堆损坏发生在第二个free（FileBuf）;

while ((DirPtr = readdir(ProcDir)))
{
    if (AllNumeric(DirPtr->d_name) && atoi(DirPtr->d_name) >= InObj->ObjectPID &&
        atoi(DirPtr->d_name) <= InObj->ObjectPID + 10) /*Search 10 PIDs forward.*/
    {
        char TChar;

        snprintf(FileName, sizeof FileName, "/proc/%s/cmdline", DirPtr->d_name);

        if (!(Descriptor = fopen(FileName, "r")))
        {
            closedir(ProcDir);
            return 0;
        }

        FileBuf = malloc(MAX_LINE_SIZE);
        /*Scroll further down to find the free()s.*/
        for (Inc = 0; (TChar = getc(Descriptor)) != EOF && Inc < MAX_LINE_SIZE - 1; ++Inc)
        {
            FileBuf[Inc] = TChar;
        }
        FileBuf[Inc] = '\0';

        fclose(Descriptor);

        for (Inc = 0, Inc2 = NumSpaces; Inc2 != 0; ++Inc)
        { /*We need to replace the NUL characters with spaces.*/
            if (FileBuf[Inc] == '\0')
            {
                --Inc2;
                FileBuf[Inc] = ' ';
            }
        }

        if (!strcmp(FileBuf, InObj->ObjectStartCommand))
        {
            unsigned long RealPID;

            free(FileBuf);
            FileBuf = NULL;
            snprintf(FileName, sizeof FileName, "%s", DirPtr->d_name);
            closedir(ProcDir);

            RealPID = atoi(FileName);

            if (UpdatePID)
            {
                InObj->ObjectPID = RealPID;
            }

            return RealPID;

        }
        /*And here is the problem.*/
        free(FileBuf);
    }
}
closedir(ProcDir);

return 0;

Answer 1

FOUND！ AdvancedPIDFind（）的工作原理是计算start命令中的空格数，以便从它的NUL分隔符重新组合/ proc / PID / cmdline，但是如果添加多个由＆符号分隔的命令或添加额外的空格，则不包含在/ PROC / CMDLINE。当我们飞离分配的内存末尾时，这会导致内存损坏错误。我现在正在写修复。谢谢你忍受我。好像我在点击'post'之后总能找到答案。

神秘堆腐败，为什么？

1 个答案: