c++ 字符串拼接 malloc(): memory corruption

@[TOC](c++ 字符串拼接 malloc(): memory corruption)

发现问题

想把字符串LogDir和LogName拼接到一起，使用如下代码。

char *LogFile = (char *) malloc(strlen(LogDir) + strlen(LogName));
sprintf(LogFile, "%s%s", LogDir, LogName);
free(LogFile);

在一个环境内能够顺利运行，但在另一个环境内报错：

malloc(): memory corruption ...

分析问题

1. 问题一

了解了一下 malloc() 和 calloc() 两个函数的参数区别，如下。
void *malloc(size_t size)
void *calloc(size_t numElements, size_t sizeOfElement)
其中size和numElements*sizeOfElement应该都为分配的内存大小。因此上述代码的第一个错误就是没有乘上sizeOfElement，即sizeof(char)。

2. 问题二

如果仅乘上sizeOfElement，代码仍旧报相同的错，考虑到字符串还有一个结束符'\0'，而strlen()函数返回值并不将该结束符计算在内。因此，在给LogFile分配内存时，还应该分配1字节内存。

解决问题

根据对问题的分析，修改代码如下。问题解决。

char *LogFile = (char *) malloc((strlen(LogDir) + strlen(LogName)) * sizeof(char) + 1);
sprintf(LogFile, "%s%s", LogDir, LogName);
free(LogFile);