From 8a5941daab398cd7595c5fdfdae4afb2acc271a9 Mon Sep 17 00:00:00 2001
From: Cecylia Bocovich <cohosh@torproject.org>
Date: Fri, 23 Aug 2019 15:45:18 -0400
Subject: [PATCH] Fix to check running status before closing proxy

This fixes a bug reported in #31385. There was an error with the proxy
deadlock fix in #31100 where we close proxies regardless of connection
status.
---
 proxy/snowflake.js | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/proxy/snowflake.js b/proxy/snowflake.js
index 7a242db..0edb42c 100644
--- a/proxy/snowflake.js
+++ b/proxy/snowflake.js
@@ -78,11 +78,13 @@ class Snowflake {
         if (!this.receiveOffer(pair, desc)) {
           return pair.active = false;
         }
-        //set a timeout for channel creation 
+        //set a timeout for channel creation
         return setTimeout((() => {
-          log('proxypair datachannel timed out waiting for open');
-          pair.close();
-          return pair.active = false;
+          if (!pair.running) {
+            log('proxypair datachannel timed out waiting for open');
+            pair.close();
+            return pair.active = false;
+          }
         }), 20000); // 20 second timeout
       } else {
         return pair.active = false;