我看到一些如下代码段:
std::unique_ptr<uint8_t> mCache;
mCache.reset(new uint8_t[size]);
有人告诉我此代码存在一些问题。 谁能给我一些细节吗?
答案 0 :(得分:10)
鉴于std::unique_ptr<uint8_t> mCache;
,当mCache
被销毁时,其deleter将使用delete
销毁被管理的指针(如果有的话),即为单个对象释放内存。但是在mCache.reset(new uint8_t[size]);
管理的mCache
之后是指向数组的指针,这意味着它应该使用delete[]
来代替;使用delete
为数组取消分配内存会导致UB。
代码可以更改为
std::unique_ptr<uint8_t[]> mCache; // mCache is supposed to manage pointer to array
mCache.reset(new uint8_t[size]); // safe now