Correctly handle disconnections while sending short responses in Done()

xrootd · Oct 6, 2017 · 01db447 · 01db447
1 parent 0e841d1
commit 01db447
Showing 1 changed file with 8 additions and 4 deletions.
diff --git a/src/XrdHttp/XrdHttpReq.cc b/src/XrdHttp/XrdHttpReq.cc
@@ -531,10 +531,14 @@ bool XrdHttpReq::Done(XrdXrootd::Bridge::Context & info) {
   TRACE(REQ, " XrdHttpReq::Done");
 
   xrdresp = kXR_ok;
-  //this->iovN = 0;
-
-  if (PostProcessHTTPReq(true)) reset();
-
+
+
+  int r = PostProcessHTTPReq(true);
+  // Beware, we don't have to reset() if the result is 0
+  if (r) reset();
+  if (r < 0) return false; 
+
+
   return true;
 };