这是我的示例代码:
int main()
{
const wchar_t *envpath = L"hello\\";
const wchar_t *dir = L"hello2\\";
const wchar_t *core = L"hello3";
wchar_t *corepath = new wchar_t[
wcslen(envpath) +
wcslen(dir) +
wcslen(core)
];
wcscpy_s(corepath, wcslen(corepath) + wcslen(envpath) + 1, envpath);
wcscat_s(corepath, wcslen(corepath) + wcslen(dir) + 1, dir);
wcscat_s(corepath, wcslen(corepath) + wcslen(core) + 1, core);
delete []corepath;
return 0;
}
在
delete []corepath
命令上,将触发一个断点。可能是什么原因?
另外,如果我这样重写代码:
wcscpy_s(corepath, wcslen(envpath) + 1, envpath);
wcscat_s(corepath, wcslen(corepath) + wcslen(dir) + 1, dir);
wcscat_s(corepath, wcslen(corepath) + wcslen(core) + 1, core);
删除指针时检测到堆损坏。
编辑:
我想我也应该用+1分配corepath来存储结尾\ 0,对吗?
最佳答案
您没有分配足够的空间来包含终止零。对wcscat_s
的最后一次调用将在'\0'
指向的缓冲区末尾之外写入corepath
。
您还要对wcscat_s
说谎,以了解缓冲区的容量。容量为wcslen(envpath) + wcslen(dir) + wcslen(core)
,但您正在传递wcslen(corepath) + wcslen(core) + 1
。
您还要在wcslen(corepath)
初始化之前调用corepath
。
固定代码应如下所示:
int main()
{
const wchar_t *envpath = L"hello\\";
const wchar_t *dir = L"hello2\\";
const wchar_t *core = L"hello3";
size_t cap = wcslen(envpath) +
wcslen(dir) +
wcslen(core) + 1;
wchar_t *corepath = new wchar_t[cap];
wcscpy_s(corepath, cap, envpath);
wcscat_s(corepath, cap, dir);
wcscat_s(corepath, cap, core);
delete[] corepath;
return 0;
}
实际上,固定代码应如下所示:
#include <string>
int main()
{
const wchar_t *envpath = L"hello\\";
const wchar_t *dir = L"hello2\\";
const wchar_t *core = L"hello3";
std::wstring corepath = envpath;
corepath.append(dir);
corepath.append(core);
}