出现此 Valgrind 错误的原因是什么？

Question

Valgrind 正在抱怨 substr 调用。

string Message::nextField(string& input) {
    int posSeparator = input.find_first_of(SEPARATOR);
    string temp;
    temp = input.substr(0, posSeparator); //Error points to this line
    input.erase(0, posSeparator + 1);
    return temp;
}

错误如下：
12个块中的290个字节肯定在丢失记录1 of 1中丢失
该函数的作用基本上是解析输入，返回由分隔符字符分隔的字符串部分。该函数是从另一个类的方法中调用的，并具有下一个定义：

void doSomething(string input) {
    input.erase(0,2);
    string temp = nextField(input);
    this->room = atoi(temp.c_str());
    temp = input;
    this->money = atoi(temp.c_str());
}

没有任何其他奇怪或重要的内容可以包含在此处。 我使用 Eclipse Indigo 的 Valgrind 分析中的 Valgrind 默认设置。 有什么想法吗？

Answer 1

这可能不是您的代码中的错误。由于 C++ 标准库的实现细节，可能会报告此错误。要验证这一点，请尝试 Valgrind 常见问题解答中的以下内容：

使用 GCC 2.91、2.95、3.0 和 3.1，使用 STL 编译所有源代码
与-D__USE_MALLOC。提防！这已从 GCC 中删除，开始于
版本3.3。

对于 GCC 3.2.2 及更高版本，您应该导出环境变量
在运行程序之前使用 GLIBCPP_FORCE_NEW。

在 GCC 3.4 及更高版本中，该变量已更改名称为
GLIBCXX_FORCE_NEW。

Answer 2

您的源代码中的其他地方可能有错误。我尝试使用以下代码复制错误：

#include <string>
#include <iostream>
#include <cstdlib>

using namespace std;

const char SEPARATOR = ':';

struct Foo
{
public:
    int room;
    int money;

    void doSomething(string input) {
        input.erase(0,2);
        string temp = nextField(input);
        this->room = atoi(temp.c_str());
        temp = input;
        this->money = atoi(temp.c_str());
    }

    string nextField(string& input) {
        int posSeparator = input.find_first_of(SEPARATOR);
        string temp;
        temp = input.substr(0, posSeparator); //Error points to this line
        input.erase(0, posSeparator + 1);
        return temp;
    }
};

int main()
{
    Foo f;
    f.doSomething("--234:12");
    std::cout << f.room << " - " << f.money << std::endl;
}

然后运行 ​​valgrind:

valgrind --tool=memcheck <executable>

，输出为：

HEAP SUMMARY:
    in use at exit: 0 bytes in 0 blocks
  total heap usage: 2 allocs, 2 frees, 61 bytes allocated

All heap blocks were freed -- no leaks are possible

For counts of detected and suppressed errors, rerun with: -v
ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 4 from 4)

所以，您的问题可能不在这部分代码中

Answer 3

您不检查 posSeparator 是否实际上与 string::npos 不同 - 这可能会导致擦除问题。这是一个疯狂的尝试，但无论如何它可能会修复一个错误。