From 96692d24c8cdf0fe88260949b67f8580e0c70696 Mon Sep 17 00:00:00 2001 From: Greg Farnum Date: Mon, 18 Oct 2010 11:23:50 -0700 Subject: [PATCH] messenger: Make sure to unlock existing->pipe_lock. There are a few cases in the "open" section where we can go to fail_unlocked while still holding existing->pipe_lock. So unlock it. --- src/msg/SimpleMessenger.cc | 2 ++ 1 file changed, 2 insertions(+) diff --git a/src/msg/SimpleMessenger.cc b/src/msg/SimpleMessenger.cc index 12a42097af934..eb487f609362d 100644 --- a/src/msg/SimpleMessenger.cc +++ b/src/msg/SimpleMessenger.cc @@ -897,6 +897,8 @@ int SimpleMessenger::Pipe::accept() fail_unlocked: + if (existing) + existing->pipe_lock.Unlock(); pipe_lock.Lock(); state = STATE_CLOSED; fault(); -- 2.39.5