Remove superfluous unregisterOutboundConnection call

unregisterOutboundConnection isn't needed after stopMux.
IntersectMBO · Jan 18, 2022 · 92f04cb · 92f04cb
1 parent f50acb5
commit 92f04cb
Showing 1 changed file with 0 additions and 2 deletions.
diff --git a/ouroboros-network/src/Ouroboros/Network/PeerSelection/PeerStateActions.hs b/ouroboros-network/src/Ouroboros/Network/PeerSelection/PeerStateActions.hs
@@ -833,7 +833,6 @@ withPeerStateActions PeerStateActionsArguments {
       case res of
         Nothing -> do
           Mux.stopMux pchMux
-          _ <- unregisterOutboundConnection spsConnectionManager (remoteAddress pchConnectionId)
           atomically (writeTVar pchPeerState (PeerStatus PeerCold))
           traceWith spsTracer (PeerStatusChangeFailure
                                 (HotToWarm pchConnectionId)
@@ -905,7 +904,6 @@ withPeerStateActions PeerStateActionsArguments {
         Nothing -> do
           -- timeout fired
           Mux.stopMux pchMux
-          _ <- unregisterOutboundConnection spsConnectionManager (remoteAddress pchConnectionId)
           atomically (writeTVar pchPeerState (PeerStatus PeerCold))
           traceWith spsTracer (PeerStatusChangeFailure
                                 (WarmToCold pchConnectionId)