Question

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

我开始设计得相当差，所以为了修复内存泄漏，我的想法是创建一个全局工厂来添加我的对象，以便以后删除它们。我用工厂方法替换了每次出现的“new”来创建它。在这种情况下，它是类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
}

我花了好几个小时试图解决这个问题而且我只是不确定我在哪里丢失内存......程序在处理数据时可以运行几分钟甚至几小时，因此它确实会建立起来。

Answer 1

解决此问题（未知来源错误）的好方法，尝试下载GCC并针对“the Wall”抛出代码。

Answer 2

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

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

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

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

virtual ~SQLData() { }

bam，修好了。我不需要为任何派生类声明声明析构函数。

感谢您的帮助！

找不到Valgrind检测到的内存泄漏

2 个答案: