锁定专用锁定对象会导致死锁

时间:2014-02-14 09:00:32

标签: c# multithreading deadlock

昨天我遇到了一个奇怪的问题,这让我很头疼。我有一个带有Server类的服务器应用程序,而该类又是从Connection类派生的。 Connection类提供有关连接状态和关闭连接的可能性的信息

public bool Connected
{
    get
    {
        if (connection != null)
        {
            lock (lockObject)
            {
                bool blockingState = connection.Blocking;

                try
                {
                    connection.Blocking = false;
                    connection.Send(new byte[1], 1, 0);
                }
                catch (SocketException e)
                {
                    if (!e.NativeErrorCode.Equals(10035))
                    {
                        return false;
                    }
                     //is connected, but would block

                }
                finally
                {
                    connection.Blocking = blockingState;
                }

                return connection.Connected;

            }

        }

        return false;
   }
}

public virtual void CloseConnection()
{
    if (Connected)
    {
        lock (lockObject)
        {
            connection.Close();
        }
    }
}

Server类对于实际发送数据是合理的

private void ConnectAndPollForData()
{
    try
    {
        TcpListener listener = new TcpListener(Port);

        listener.Start();
        while (true)
        {
            connection = listener.AcceptSocket();

            string currentBuffr = string.Empty;
            const int READ_BUFFER_SIZE = 1024;
            byte[] readBuffr = new byte[READ_BUFFER_SIZE];

            while (Connected)
            {
                int bytesReceived;

                lock (lockObject)
                {
                    bytesReceived = connection.Receive(readBuffr, READ_BUFFER_SIZE, SocketFlags.None);
                }
                currentBuffr += ASCIIEncoding.ASCII.GetString(readBuffr, 0, bytesReceived);

                //do stuff

        }
    }
    catch(ThreadAbortException)
    {
        Thread.ResetAbort();
    }
    finally
    {

    }
}

public void SendString(string stringToSend)
{
    stringToSend += "\r\n";


    if(Connected)
    {
        lock(lockObject)
        {
            connection.Send(ASCIIEncoding.UTF7.GetBytes(stringToSend));
        }
    }

}

没有其他显式访问连接对象。 ConnectAndPollForData函数在单独的线程中执行。每当我在这个版本中运行主机时(我目前使用的是非线程安全的版本,导致其他问题),它会在通过TCP接收到很多行之后挂起。暂停调试器向我显示,一个线程尝试使用Connected锁执行代码,而另一个尝试在ConnectAndPollForData锁中接收数据。这种行为对我来说似乎很奇怪,因为我希望在第一个锁中执行代码,然后执行第二个。在Deadlocking lock() method'Deadlock' with only one locked object?中使用回调时似乎存在类似的问题,但这里的情况有点不同,因为在我的情况下(我认为)锁中的代码不应该发出任何事件他们自己试图获得对象的锁定。

1 个答案:

答案 0 :(得分:4)

让我们假设它首先获得第二种方法的锁定。所以它是持有锁,并等待数据。目前还不清楚这是直接接收第一种方法发送的数据,或者是否正在寻找来自不相关服务器的回复 - 回复发送的消息第一种方法。但不管怎样,我假设在发送出站消息之前不会有数据传入。

现在考虑:出站消息无法发送,因为您持有独占锁。

所以是的,你已经陷入僵局。基本上,不要这样做。即使在同一个套接字上,也无需在入站和出站套接字操作之间进行同步。因为在同一个套接字或并发编写器上同时使用读者是没有意义的,所以我猜你根本不需要那些lock