构造函数中错误使用memset引发的惨案

仲夏夜之梦~

于 2021-11-15 15:33:15 发布

阅读量877

点赞数

CC 4.0 BY-SA版权

文章标签： c++ 开发语言后端

本文链接：https://blog.youkuaiyun.com/challenglistic/article/details/121331693

本文讨论了在C++中如何正确使用new和delete防止内存泄漏，并强调了在使用memset时，为何应当用sizeof代替strlen。错误地使用strlen可能导致填充长度的不确定性，而sizeof能确保正确填充对象的大小。修复这个问题可以提高代码的健壮性和内存安全性。

摘要生成于 C知道，由 DeepSeek-R1 满血版支持，前往体验 >

在类成员函数中使用new以后，需要delete来释放空间，同时将指针设置为NULL来避免内存泄漏，而且new创建的对象必须要和delete的对象一致，并且不能重复delete，否则有可能出现下面这种情况

在delete释放空间的时候，我的写法如下

尽管我是这么操作的，但是依然出了问题，经过各种渠道了解，我发现 vacTransformDataType的构造函数里有这么一句话

memset(cTempBuf, '0', strlen(cTempBuf));

这里应该将strlen换成sizeof

strlen(const char*) 在遇到'\0'以后才会停止计算元素个数，否则返回值是一个随机值

char* str = "hello";
char* str1 = {'h','e','l','l','o'};
printf("%d\n",strlen(str));
printf("%d\n",strlen(str1));         //错误，返回值是随机值
printf("%d\n",sizeof(str1));         //正确，返回的是数组的字节数
printf("%d\n",sizeof(str1)/sizeof(char));     //返回数组的元素个数

正确的应该是

memset(cTempBuf, '0', sizeof(cTempBuf));

memset将前sizeof(cTempBuf)个字节替换成 '0'