0

这是我的示例代码：

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，对吧？

4

1 回答 1

5

您没有分配足够的空间来包含终止零。最后一次调用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);
}
于 2013-07-23T13:14:12.857 回答