0

我有一堆方法:

bool PGConnection::connect() {
    try
    {
        conn = new pqxx::connection(
                            "user=temp "
                            "host=xxxx "
                            "password=xx "
                            "dbname=temp");
    }
    catch (const std::exception &e)
    {
        std::cerr << e.what() << std::endl;
        return false;
    }
    return true;
}

//Disconnect from db
bool PGConnection::disconnect() {
        if ( conn->is_open()) {
            std::cout<<"try disconnect"<<std::endl;
            conn->disconnect();
            return true;
        }
    return false;
}

PGConnection::~PGConnection() {
    if ( conn != NULL) {
        delete conn;
   }
}

调用断开连接或类析构函数时,会导致分段错误。(当我注释掉下面的断开连接部分时,它会在调用析构函数时发生。)

int main () {
 PGConnection pgConn("xxx","xxx");
 pgConn.connect();
 pgConn.disconnect();
 return 0;
 }

带有 disconnect() 的 gdb 调用

Program received signal SIGSEGV, Segmentation fault.
0x00007ffff7ba4852 in pqxx::connection_base::is_open() const () from /usr/lib/libpqxx-3.1.so
(gdb) bt
#0  0x00007ffff7ba4852 in pqxx::connection_base::is_open() const () from /usr/lib/libpqxx-3.1.so
#1  0x00000000004019d3 in PGConnection::disconnect (this=0x7fffffffe600) at pgconnection.cpp:42
#2  0x000000000040269e in main () at main.cpp:8
(gdb) frame 2
#2  0x000000000040269e in main () at main.cpp:8
8       pgConn.disconnect();
(gdb) print pgConn
$1 = {conn = 0x3}

gdb 没有调用 disconnect

(gdb) bt
#0  0x00007ffff7ba5fdb in pqxx::connection_base::close() () from /usr/lib/libpqxx-3.1.so
#1  0x0000000000401d3e in pqxx::basic_connection<pqxx::connect_direct>::~basic_connection (this=0x3,
    __in_chrg=<optimized out>) at /usr/include/pqxx/basic_connection.hxx:74
#2  0x0000000000401a3d in PGConnection::~PGConnection (this=0x7fffffffe600, __in_chrg=<optimized out>)
    at pgconnection.cpp:57
#3  0x00000000004026a3 in main () at main.cpp:11
(gdb) frame 2
#2  0x0000000000401a3d in PGConnection::~PGConnection (this=0x7fffffffe600, __in_chrg=<optimized out>)
    at pgconnection.cpp:57
57          delete conn;
(gdb) print conn
$1 = (pqxx::connection *) 0x3
(gdb)
4

1 回答 1

2

没有看到构造函数就很难说PGConnection. 特别是,构造函数应始终设置conn=NULL.

我还建议当前说的那一行

    if ( conn->is_open()) {

应该改为

    if (conn && conn->is_open()) {

我强烈怀疑您的conn成员未正确初始化。

更好的办法是重新设计这个类,这样你就可以完全消除使用,new如果delete可以的话。如果你这样做,这将是解决问题的更好方法。PGConnection除非它即将连接到某个东西,否则不太可能存在存在的理由,因此构造函数还可以包含用于实例化pqxx::connection.

于 2014-04-12T16:57:16.753 回答