为什么我会出现“堆损坏”？

Question

请不要因为这件事而把我钉在十字架上。我认为使用 char* 可能会更好，因为我打算构建的字符串的大小已知。我还知道，如果 timeinfo->tm_hour 返回 2 位数以外的数字，事情就会严重错误。也就是说，当这个函数返回时，VIsual Studio 就开始对我说 HEAP CORRUPTION。出了什么问题？ （另外，我应该使用字符串生成器吗？）

void cLogger::_writelogmessage(std::string Message)
{
    time_t rawtime;
    struct tm* timeinfo = 0;

    time(&rawtime);
    timeinfo = localtime(&rawtime);

    char* MessageBuffer = new char[Message.length()+11];
    char* msgptr = MessageBuffer;

    _itoa(timeinfo->tm_hour, msgptr, 10);
    msgptr+=2;

    strcpy(msgptr, "::");
    msgptr+=2;

    _itoa(timeinfo->tm_min, msgptr, 10);
    msgptr+=2;

    strcpy(msgptr, "::");
    msgptr+=2;

    _itoa(timeinfo->tm_sec, msgptr, 10);
    msgptr+=2;

    strcpy(msgptr, " ");
    msgptr+=1;

    strcpy(msgptr, Message.c_str());

    _file << MessageBuffer;

    delete[] MessageBuffer;
}

Answer 1

您需要再分配一个字节，因为 string 的 .length 返回其长度而没有终止 NUL，为此您还需要 char*< 中的空间/代码>。

即假设 Message.length() 返回 10。您分配了 21 个字节。将11个字节复制到缓冲区中，然后复制消息，这需要10个字节+1个NUL。总计：22 个字节，而您只分配了 21 个字节。

Answer 2

这

char* MessageBuffer = new char[Message.length()+11];

应该是

char* MessageBuffer = new char[Message.length()+12];

因为您要向缓冲区添加 11 个附加字符：

2 for hr
2 for ::
2 for min
2 for ::
2 for sec
1 for " "

您需要一个附加字符作为终止 null 字符。

Answer 3

正如其他人指出的那样，MessageBuffer 的大小需要增加一。

但是，您可以将时间信息直接流式传输到 _file，而不是以这种方式处理原始字符缓冲区，而无需先将其放入中间字符串中。如果您出于某种原因希望将其放在中间字符串中，我建议您使用 ostringstream 类。

void writelogmessage(std::string Message)
{
    time_t rawtime;
    struct tm* timeinfo = 0;

    time(&rawtime);
    timeinfo = localtime(&rawtime);

    std::ostringstream stream;
    stream<<
        timeinfo->tm_hour<<"::"<<
        timeinfo->tm_min<<"::"<<
        timeinfo->tm_sec<<" "<<
        Message;

    _file<<stream.str();
}