wangshao1
commented
2 weeks ago 是redis.cc scankeynum里resize大小不对吗？
Closed chenbt-hz closed 2 weeks ago
wangshao1
commented
2 weeks ago 是redis.cc scankeynum里resize大小不对吗？
Issues-translate-bot
commented
2 weeks ago Bot detected the issue body's language is not English, translate it automatically.
Is the resize size in redis.cc scankeynum wrong?
chenbt-hz
commented
2 weeks ago 是redis.cc scankeynum里resize大小不对吗？
是吧。你看是改成resize(6)还是删除ScanStreamsKeyNum(&((*key_infos)[5]))？
Status Redis::ScanKeyNum(std::vector<KeyInfo>* key_infos) {
key_infos->resize(5);
rocksdb::Status s;
s = ScanStringsKeyNum(&((*key_infos)[0]));
if (!s.ok()) {
return s;
}
s = ScanHashesKeyNum(&((*key_infos)[1]));
if (!s.ok()) {
return s;
}
s = ScanListsKeyNum(&((*key_infos)[2]));
if (!s.ok()) {
return s;
}
s = ScanZsetsKeyNum(&((*key_infos)[3]));
if (!s.ok()) {
return s;
}
s = ScanSetsKeyNum(&((*key_infos)[4]));
if (!s.ok()) {
return s;
}
s = ScanStreamsKeyNum(&((*key_infos)[5]));
if (!s.ok()) {
return s;
}
return Status::OK();
}嗯，我提了pr，你帮忙看下。我本地测试没有问题了。
Issues-translate-bot
commented
2 weeks ago Bot detected the issue body's language is not English, translate it automatically.
Well, I submitted a PR, please help me take a look at it. My local test has no problem.
Is this a regression?
Yes
Description
在mac环境中，执行dbsize会导致进程崩溃，而且ut大概率无法通过会触发IO 异常：
错误日志如下：
Please provide a link to a minimal reproduction of the bug
No response
Screenshots or videos
No response
Please provide the version you discovered this bug in (check about page for version information)
No response
Anything else?
AI日志分析：
错误类型: heap-buffer-overflow 表示在堆上分配的缓冲区被写入了超出其大小的数据。
错误位置: 错误发生在 redis_streams.cc 文件的第 358 行，函数 storage::Redis::ScanStreamsKeyNum 中。
触发点: 写入操作的地址是 0x60e000030040，这个地址正好位于之前分配的 160 字节区域的末尾，即 [0x60e00002ffa0, 0x60e000030040)。
分配信息: 这个区域是由线程 T109 分配的，分配的调用栈显示，分配是通过 std::1::vector<storage::KeyInfo, std::1::allocator>::resize 函数进行的，这是 C++ STL 标准库中的 vector 类型的一个成员函数，用于改变 vector 的大小。
错误上下文: 日志显示，这个写入操作是在执行 info keyspace 命令时发生的，这通常是一个用于获取 Redis 数据库状态信息的命令。
调用栈: 调用栈显示了从命令执行到发生错误的函数调用路径，包括 PikaClientConn::DoCmd、InfoCmd::InfoKeyspace 等函数。
错误处理: 日志的最后部分显示 ABORTING，表示程序因为检测到这个错误而终止执行。
针对这个问题，你需要检查 storage::Redis::ScanStreamsKeyNum 函数在第 358 行的代码，找出为什么会出现写入超出缓冲区大小的数据。可能的原因包括：
错误的数组索引或循环条件，导致写入超出了数组的界限。 错误的内存分配或释放逻辑，可能导致了内存越界。 修复这个问题通常需要确保所有对缓冲区的写入操作都不超过其分配的大小，并且循环和条件语句正确地限制了访问范围。在修复后，重新编译并运行程序，以确保问题不再出现。
AI修改建议
key_infos->resize(6); // 确保vector至少有6个元素