[英]Buffer Overrun using a Vector
unsigned short* myClass::get_last(size_t _last) const
{
if (_last == 0) _last = last.size();
if (_last <= last.size())
{
unsigned short* temp = new unsigned short [_last] {};
while (_last > 0) temp[_last] = last[last.size() - _last--]; //I get a warning HERE
return temp;
}
throw std::runtime_error("Error!");
}
它说:
写入 'temp' 时缓冲区溢出:可写大小为 '_last*2' 字节,但可能会写入 '_last' 字节。
这是什么意思? 我肯定知道_last并不比temp.size()大,因为if
这样我该怎么办?
它在运行时完美运行,但我讨厌有警告,这会使其他用户或我将来难以理解我的代码。
编辑: _last是用户在运行时给出的参数,因此它最终可能具有任何值,但如果他的值超出范围,则会出现异常(在另一个函数中管理)。
我在标题中提到的向量是last , whis 是myClass的成员。
我知道数组的元素从0到_last - 1 ,这就是为什么我在第一次使用它之前递减_last (你可能知道赋值关联性是从右到左)。
我希望我回答了你所有的评论;)
问题是 C++ 索引从 0 开始的数组。所以大小为 4 的数组具有有效的索引 0、1、2 和 3。
但是您正在分配一个大小为 _last 的数组:
unsigned short* temp = new unsigned short [_last] {};
然后写入temp[_last]
。 那是超出数组大小的一个。
使用向量解决!
std::vector<unsigned short> Tombola::get_last(size_t _last) const
{
if (_last == 0) _last = last.size();
if (_last <= last.size())
{
std::vector<unsigned short> temp(_last);
while (_last > 0) temp[_last] = last[last.size() - _last--];
return temp;
}
throw std::runtime_error("Error!");
}
出于某种原因,即使您不知道如何,他们也总是能解决所有问题;)
声明:本站的技术帖子网页,遵循CC BY-SA 4.0协议,如果您需要转载,请注明本站网址或者原文地址。任何问题请咨询:yoyou2525@163.com.