C++ shared_ptr 继承内存泄漏

Question

我有一个 shared_ptr 指向子类基础的情况。

当shared_ptr去删除指针时，只有父析构函数被调用。

父母析构函数是虚拟的，孩子不是，尽管我已经尝试过所有组合。

我在 valgrind 中有该程序，它显示在创建对象时在 new 语句中创建了内存。而且我知道父析构函数被调用，但子析构函数没有。

这是孩子：

class NetworkUserAgent : public bbs::UserAgent
{
friend class Server;

public:
    NetworkUserAgent(boost::asio::io_service &ioService, size_t _szBuffer=512u);
    ~NetworkUserAgent();

    void asyncRead();
    void doneRead(std::shared_ptr< std::vector<char> > pBuf,
                    const boost::system::error_code &error, size_t byTrans);

    void writeTo(const std::string &msg);
    void doneWrite(const boost::system::error_code &error, size_t byTrans);

void close();

private:
    boost::asio::ip::tcp::socket socket_;
    const size_t szBuffer;
};

父母：

class UserAgent
{
public:
    //'structors
    UserAgent();
    virtual ~UserAgent();

    //commication
    virtual void writeTo(const std::string &msg)=0;
    std::function<void(std::string&)> dataRead;

    //user management
    void login(AccessLevel _accessLevel, int userId, const std::string &_userName);
    void logout();

    //Accessors
    AccessLevel UserAccessLevel() const;
    const std::string &UserName() const;
    const int &UserId() const;
    bool LoggedIn() const;

    //shared to allow reference to child type
    std::shared_ptr<ContextAgentData> contextAgentData;
private:
    std::string userName;
    int userId;

    AccessLevel accessLevel;
};

用法：

void Server::reset()
{   
    shared_ptr<NetworkUserAgent> client (new NetworkUserAgent(ioService));
    acceptor_.async_accept(client->socket_,
        [=] (const boost::system::error_code &error)
            { this->clientAccepted(client, error); }
        );
}

void Server::clientAccepted(shared_ptr<NetworkUserAgent> client,
                                const boost::system::error_code &error)
{
    if(error) return;
    cout << "[] New client has connected" << endl;

    //Generalise to Network useragent
    shared_ptr<UserAgent> uaClientPtr=client;
    context->receiveUserAgent(uaClientPtr);
    client->asyncRead();
    reset();
}

The rest of the code can be seen here.

谢谢。

另外请注意，上面的代码仍在进行中。

编辑：我错了，正在调用子析构函数，

NetworkUserAgent::~NetworkUserAgent()
{   
    this->close();
} 

void NetworkUserAgent::close()
{
    if(!socket_.is_open()) return; //socket is already closed
    //one or more of these functions are probably redundant  
    cout << "send request" <<endl;
    socket_.shutdown(ip::tcp::socket::shutdown_send);
    cout << "cancel" <<endl;
    socket_.cancel();
    cout <<"close"<<endl;
    socket_.close();
    cout << "done" <<endl;
}

编辑： 我做了更多的测试，恐怕问题比我希望的要复杂。销毁项目时会调用析构函数，但是问题是一旦 UserAgent 进入系统，它就不会被销毁。有些东西正在停止被破坏。

如果它与用户代理的 shared_ptr 的几个容器不同，当一个包含被销毁时，是否调用了内部元素的析构函数？

请让我知道我还能提供什么来解决问题。

Answer 1

UserAgent 中有一个 dataRead std::function，它最终被设置为一个包含 std::shared_ptr 的 lambda，以阻止它自毁。我添加了一个 close() 方法并将 std::function 设置为默认值。

现在一切都很好，删除也很好

感谢您的所有帮助

Answer 2

我怀疑您的对象正在被切片，尽管我没有要测试的环境。

尝试以下方法之一：

shared_ptr<UserAgent> uaClientPtr = boost::static_pointer_cast<UserAgent>(client);
shared_ptr<UserAgent> uaClientPtr = boost::dynamic_pointer_cast<UserAgent>(client);

Answer 3

在void Server::reset() auto_ptr 就足够了。

在第二种情况下，我会做以下事情：

void Server::clientAccepted(shared_ptr<NetworkUserAgent> client, const boost::system::error_code error)
{
  if(error) return;
  cout << "[] New client has connected" << endl;
  context->receiveUserAgent(client);
  client->asyncRead();
  this->reset();
}