1

我正在开发一个程序，但我无法确定内存泄漏。我对 C/C++ 也不是很有经验。我将发布一个 valgrind 错误以及相关的类定义和函数......如果我忘记了什么，请问我会更新:) 我没有发布所有 valgrind 报告的原因是有很多它们，但它们相似......唯一的区别是它具有的堆栈跟踪。

我一开始设计的很差，所以为了修复内存泄漏，我的想法是创建一个全局工厂来添加我的对象，以便以后删除它们。我用工厂方法替换了每次出现的“新”来创建它。在这种情况下，它是 Column 类。我很肯定 makeColumn 创建的每个对象都被删除了，因为我使用向量来存储指针。遍历向量并删除其中每个项目的函数在程序结束之前被调用。

这个 valgrind 报告让我觉得不知何故，一个字符串没有被分配。我设置了 GLIBCXX_FORCE_NEW 变量，它对检测到的泄漏没有影响。我正在使用 gcc 4.7.2。

另外，是的，我正在从 ANTLR 生成的解析器接收信息……这可能无关紧要，因为 ANTLR 处理它自己的内存。char 指针是来自 ANTLR 的唯一数据。

==23168== 15 bytes in 1 blocks are definitely lost in loss record 10 of 30
==23168==    at 0x4ACE73C: operator new(unsigned int) (in /usr/lib/valgrind/vgpreload_memcheck-x86-linux.so)
==23168==    by 0x4BA62A3: std::string::_Rep::_S_create(unsigned int, unsigned int, std::allocator<char> const&) (in /usr/lib32/libstdc++.so.6.0.17)
==23168==    by 0x4BA75EE: std::string::_Rep::_M_clone(std::allocator<char> const&, unsigned int) (in /usr/lib32/libstdc++.so.6.0.17)
==23168==    by 0x4BA7F3F: std::string::assign(std::string const&) (in /usr/lib32/libstdc++.so.6.0.17)
==23168==    by 0x4BA7F92: std::string::operator=(std::string const&) (in /usr/lib32/libstdc++.so.6.0.17)
==23168==    by 0x82E0B57: GenericFactory::makeColumn(char const*, char const*, char const*) (global.cpp:246)
==23168==    by 0x82DEA23: addTable (helper.cpp:93)
==23168==    by 0x810DE1F: query_table_expression (OracleSQLParser.c:165181)
==23168==    by 0x8108F66: table_reference (OracleSQLParser.c:162767)
==23168==    by 0x81154B1: join_clause (OracleSQLParser.c:168172)
==23168==    by 0x82A0845: synpred349_OracleSQL_fragment (OracleSQLParser.c:460632)
==23168==    by 0x82AFA48: synpred349_OracleSQL (OracleSQLParser.c:469414)

helper.cpp:addTable - putValue 只是将指针添加到地图。

void addTable(char* schema, char* table) {
    ::gbl_info->tables->putValue(::gbl_info->factory.makeColumn(schema,table,""),NULL);
}

GenericFactory::makeColumn

Column* GenericFactory::makeColumn(const char* schema,const char* table, const char* column) {
    this->count++;
    Column* col = new Column(schema,table,column);
    this->allocated_objects.push_back(col);
    return col;
}

列::列

Column::Column(const char* schema, const char* table, const char* column) {
    string temp = schema;
    this->schema = normalize(temp);
    temp = table;
    this->table = normalize(temp);
    temp = column;
    this->column = normalize(temp);
    temp = schema;
    temp = temp + "." + table + "." + column;
    this->text = normalize(temp);
}

标准化

string& normalize(string& str) {
    for (string::iterator p=str.begin(); p != str.end(); p++)
        *p = toupper(*p);
    str.erase(remove(str.begin(),str.end(),'"'),str.end()); // erase double quotes
    return str;
}

列定义：SQLData 没有成员或构造函数

class Column : public SQLData {
    std::string text;
    std::string schema;
    std::string table;
    std::string column;
public:
    std::string alias; // TABLE alias
    Column(const char*,const char*,const char*);
    Column(const std::string qn);
    //Has functions too, but probably irrelevant
}

我已经花了几个小时试图解决这个问题，但我只是不确定我在哪里失去了记忆......程序可以在处理数据时运行几分钟甚至几个小时，所以它真的建立起来了。

4

2 回答 2

4

解决诸如此类问题（未知来源的错误）的好方法，尝试下载 GCC 并将您的代码扔到“墙”上。

于 2013-07-25T18:47:58.060 回答
2

好的，所以我想通了。当我使用 -Wall 运行它时，问题就出现了，所以我会接受这个答案:) 这就是问题所在。

虚拟析构函数。不熟悉 C++，我昨天才听说它们......我认为我不需要使用它们，因为我的派生类中实际上没有任何清理代码。错误的！为了在派生类上调用默认析构函数，必须将超类（在本例中为 SQLData）的析构函数声明为虚拟的。因此，我的派生类中没有清理字符串成员，即使它们不是指针。

有趣的是，我不推荐 Cygwin 进行调试。-Wall 选项在安装的 gcc Cygwin 上没有警告我，但是当我在 Ubuntu 机器上编译时，它告诉我每个类。

所以，我为修复这个巨大的内存泄漏所做的只是将以下内容添加到我的 SQLData 类定义中：

virtual ~SQLData() { }

砰，固定。我不需要为我的任何派生类声明析构函数。

谢谢您的帮助！

于 2013-07-26T13:55:59.380 回答