2013-07-23 115 views
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); 

堆损坏。

编辑:

我想我也应该分配corepath与+1储存结束\ 0,对不对?

+1

“我想我也应该分配corepath与+1存储结局\ 0,对吗?“是。更好的是,使用'std :: wstring'来正确处理所有的内存分配。 –

+1

wcslen返回字符串的长度,没有NULL终止字符,所以当你分配corepath时,你应该考虑NULL终止字符。 – TheDarkKnight

回答

5

您没有分配足够的空间来包含终止零。 wcscat_s的最后一次调用将在corepath指向的缓冲区末尾写入。

你也在说谎缓冲区的容量wcscat_s。容量为wcslen(envpath) + wcslen(dir) + wcslen(core),但您改为通过wcslen(corepath) + wcslen(core) + 1

corepath被初始化之前,您还打电话给wcslen(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); 
} 
+0

+1为最终版本。 –