Boost Asio,异步 UDP 客户端 - 关机时崩溃

Boost Asio, async UDP client - crash on shutdown

我在我的应用程序中为 IPC 使用 UDP 客户端-服务器

它工作正常,但是当尝试关闭客户端时会发生一些竞争情况,这会导致应用程序崩溃或死锁。

UDP 客户端:

// IOServiceBase class contain boost::asio::io_service instance
// it is accessible by service() protected method
class AsyncUDPClient : public IOServiceBase
{
public:

    /// @brief Create a network client
    AsyncUDPClient(const std::string& host, const std::string port)
        : _host(host)
        , _port(port)
        , _reply()
        , _work(service())
        , _sock(service(), ba_ip::udp::endpoint(ba_ip::udp::v4(), 0)) {

        run();
    }

    /// @brief Start async packets processing
    void run(){
        std::thread t([&]{ service().run(); });
        t.detach();
    }

    /// @brief Async request to server
    void send(uint8_t* message, size_t length) {
        std::vector<uint8_t> packet(message, message + length);
        service().post(boost::bind(&AsyncUDPClient::do_send, this, packet));
    }

    /// @brief Cleanup io_service to dismiss already putted tasks
    ~AsyncUDPClient() {
        close();

        // trying to wait until service is stopped, but it does not help
        while (!service().stopped()){
            std::this_thread::sleep_for(std::chrono::milliseconds(100));
        }
    }

    /// @brief Cleanup io_service to dismiss already putted tasks
    void close(){
        std::thread t([&]{ service().stop(); });
        t.join();
    }
protected:

// send-response methods are pretty standard

private:

    std::string _host;
    std::string _port;
    std::array<uint8_t, max_length> _reply;
    ba::io_service::work _work;
    ba_ip::udp::socket _sock;
};

用法示例:

{
    AsyncUDPClient ipc(addr, port);
    ipc.send(&archive_data[0], archive_data.size());
    // it seems client is destroyed before some internal processing is finished?
}

行为不是确定性的,有时工作正常,有时崩溃,有时死机。 Stacktrace 在 boost.asio 内部

的某处显示崩溃点

AsyncUDPClient 的破坏未与线程 运行 和 io_service 正确同步。当处理 io_service 的线程在其生命周期结束后尝试与 AsyncUDPClient 及其 io_service 交互时,这可能会导致调用未定义的行为。

要解决此问题,请不要从处理 io_service 的线程分离,并在 io_service 停止后显式加入线程。

class AsyncUDPClient
 : public IOServiceBase
{
public:

  // ...

  void run()
  {
    _threads.emplace_back([&]{ service().run(); })
  }

  // ...

  ~AsyncUDPClient()
  {
    close();
  }

  void close()
  {
    // Stop the io_service.  This changes its state and return immediately.
    service().stop();

    // Explicitly synchronize with threads running the io_service.
    for (auto& thread: _threads)
    {
      thread.join();
    }
  }

private:

  // ...
  std::vector<std::thread> _threads;
};

正如上面评论中所暗示的,io_service::stop() does not block. It changes the state of io_service to stopped, returns immediately, and causes all invocations of run() and run_one() to return as soon as possible. Calls to io_service::stopped() 立即 return io_service 的状态。这些调用均未指示 run()run_one().

的调用中当前是否存在线程