35

有时 boost::asio 似乎在我想要它之前断开连接,即在服务器正确处理断开连接之前。我不确定这是怎么可能的,因为客户端似乎认为它完全发送了消息,但是当服务器发出错误时,它甚至没有读取消息头......在测试期间,这可能只发生五分之一,服务器收到客户端关闭消息,并干净地断开客户端。

错误:“现有连接被远程主机强行关闭”

客户端断开连接:

void disconnect()
{
    boost::system::error_code error;
    //just creates a simple buffer with a shutdown header
    boost::uint8_t *packet = createPacket(PC_SHUTDOWN,0);
    //sends it
    if(!sendBlocking(socket,packet,&error))
    {
        //didnt get here in my tests, so its not that the write failed...
        logWrite(LOG_ERROR,"server",
            std::string("Error sending shutdown message.\n")
            + boost::system::system_error(error).what());
    }

    //actaully disconnect
    socket.close();
    ioService.stop();
}
bool sendBlocking(boost::asio::ip::tcp::socket &socket,
    boost::uint8_t *data, boost::system::error_code* error)
{
    //get the length section from the message
    boost::uint16_t len = *(boost::uint16_t*)(data - 3);
    //send it
    asio::write(socket, asio::buffer(data-3,len+3),
        asio::transfer_all(), *error);
    deletePacket(data);
    return !(*error);
}

服务器:

void Client::clientShutdown()
{
    //not getting here in problem cases
    disconnect();
}
void Client::packetHandler(boost::uint8_t type, boost::uint8_t *data,
    boost::uint16_t len, const boost::system::error_code& error)
{
    if(error)
    {
        //error handled here
        delete[] data;
        std::stringstream ss;
        ss << "Error recieving packet.\n";
        ss << logInfo() << "\n";
        ss << "Error: " << boost::system::system_error(error).what();
        logWrite(LOG_ERROR,"Client",ss.str());

        disconnect();
    }
    else
    {
        //call handlers based on type, most will then call startRead when
        //done to get the next packet. Note however, that clientShutdown
        //does not
        ...
    }
}



void startRead(boost::asio::ip::tcp::socket &socket, PacketHandler handler)
{
    boost::uint8_t *header = new boost::uint8_t[3];
    boost::asio::async_read(socket,boost::asio::buffer(header,3),
        boost::bind(&handleReadHeader,&socket,handler,header, 
        boost::asio::placeholders::bytes_transferred,boost::asio::placeholders::error));
}
void handleReadHeader(boost::asio::ip::tcp::socket *socket, PacketHandler handler,
    boost::uint8_t *header, size_t len, const boost::system::error_code& error)
{
    if(error)
    {
        //error "thrown" here, len always = 0 in problem cases...
        delete[] header;
        handler(0,0,0,error);
    }
    else
    {
        assert(len == 3);
        boost::uint16_t payLoadLen  = *((boost::uint16_t*)(header + 0));
        boost::uint8_t  type        = *((boost::uint8_t*) (header + 2));
        delete[] header;
        boost::uint8_t *payLoad = new boost::uint8_t[payLoadLen];

        boost::asio::async_read(*socket,boost::asio::buffer(payLoad,payLoadLen),
            boost::bind(&handleReadBody,socket,handler,
            type,payLoad,payLoadLen,
            boost::asio::placeholders::bytes_transferred,boost::asio::placeholders::error));
    }
}
void handleReadBody(ip::tcp::socket *socket, PacketHandler handler,
    boost::uint8_t type, boost::uint8_t *payLoad, boost::uint16_t len,
    size_t readLen, const boost::system::error_code& error)
{
    if(error)
    {
        delete[] payLoad;
        handler(0,0,0,error);
    }
    else
    {
        assert(len == readLen);
        handler(type,payLoad,len,error);
        //delete[] payLoad;
    }
}
4

5 回答 5

34

我认为您可能应该在致电 tosocket.shutdown(boost::asio::ip::tcp::socket::shutdown_both, ec)之前致电 to socket.close()

basic_stream_socket::close的boost::asio 文档指出:

对于与优雅关闭连接的套接字有关的可移植行为,请在关闭套接字之前调用 shutdown()。

这应该确保套接字上的任何未决操作都被正确取消,并且在调用 socket.close 之前刷新任何缓冲区。

于 2010-06-18T08:23:23.143 回答
13

我尝试使用 close() 方法和 shutdown() 方法来做到这一点

socket.shutdown(boost::asio::ip::tcp::socket::shutdown_both, ec)

关机方法是两者中最好的。但是,我发现使用 ASIO 套接字的析构函数是一种干净的方法,因为 ASIO 会为您处理这一切。所以你的目标是让套接字超出范围。现在,您可以使用 shared_ptr 轻松完成此操作,并将 shared_ptr 重置为新的套接字或 null。这将调用 ASIO 套接字的析构函数并且生活很好。

于 2013-01-07T23:53:11.580 回答
5

也许这就是正在发生的事情:

  • 客户端发送断开数据包
  • 客户端关闭套接字
  • 服务器读取处理程序被调用,但由于套接字已关闭,因此与关闭数据包相关联的错误。

我在您的读取处理程序中看到,如果有错误,您永远不会检查您的关闭数据包是否存在。也许是的。基本上我要说的是,也许您的客户端有时能够在服务器有机会单独处理它们之前发送关闭和关闭数据包。

于 2010-01-03T01:13:22.087 回答
4

使用 async_write() 并将 socket.close() 放入写入处理程序中。这将确保数据包由 boost asio 处理,并且在处理过程中不会被忽略(因为 close() 调用)。

于 2010-08-30T21:38:55.357 回答
4

我有一个非常相似的问题。我相信这与 Windows 回收连接有关。下面是不是很熟悉?

  • 您在启动程序时立即收到此错误,但在建立连接后没有?
  • 如果您在重新启动应用程序之前等待超过 4 分钟,该错误永远不会发生?

tcp 规范指定默认情况下,当 tcp 连接关闭时,它应该等待 4 分钟以等待最终确认。您可以使用 netstat 查看这些处于 FIN_WAIT 状态的连接。Windows 操作系统会检测您何时尝试连接到完全相同的系统,并采用这些部分关闭的连接并回收它们。您对程序的第二次调用获得了第一次运行留下的“关闭”连接。它得到下一个确认,然后真正关闭。

于 2010-08-30T21:48:03.047 回答