Fix a case where callbacks could be called more than once in error conditions.

bdarnell · bdarnell · commit 6fa950ceba51 · 2011-02-19T13:58:56.000-08:00
diff --git a/tornado/simple_httpclient.py b/tornado/simple_httpclient.py
@@ -240,15 +240,16 @@ def cleanup(self):
         except Exception, e:
             logging.warning("uncaught exception", exc_info=True)
             if self.callback is not None:
-                self.callback(HTTPResponse(self.request, 599, error=e))
+                callback = self.callback
                 self.callback = None
+                callback(HTTPResponse(self.request, 599, error=e))
 
     def _on_close(self):
         if self.callback is not None:
-            self.callback(HTTPResponse(self.request, 599,
-                                       error=HTTPError(599, 
-                                                       "Connection closed")))
+            callback = self.callback
             self.callback = None
+            callback(HTTPResponse(self.request, 599,
+                                  error=HTTPError(599, "Connection closed")))
 
     def _on_headers(self, data):
         first_line, _, header_data = data.partition("\r\n")
