You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by zw...@apache.org on 2015/07/31 22:34:01 UTC

trafficserver git commit: Revert "TS-3810 traffic_manager leaks that thread stack, make sure to detach the threads"

Repository: trafficserver
Updated Branches:
  refs/heads/6.0.x c5de4d442 -> 7a336b8bf


Revert "TS-3810 traffic_manager leaks that thread stack, make sure to detach the threads"

This reverts commit c5de4d4423fab4cd930c17f78a5366da69a43dd2.


Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/7a336b8b
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/7a336b8b
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/7a336b8b

Branch: refs/heads/6.0.x
Commit: 7a336b8bf9bc5501634fc0d5881b63a187978463
Parents: c5de4d4
Author: Leif Hedstrom <zw...@apache.org>
Authored: Fri Jul 31 14:33:50 2015 -0600
Committer: Leif Hedstrom <zw...@apache.org>
Committed: Fri Jul 31 14:33:50 2015 -0600

----------------------------------------------------------------------
 cmd/traffic_manager/MgmtHandlers.cc | 2 --
 1 file changed, 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/7a336b8b/cmd/traffic_manager/MgmtHandlers.cc
----------------------------------------------------------------------
diff --git a/cmd/traffic_manager/MgmtHandlers.cc b/cmd/traffic_manager/MgmtHandlers.cc
index 0569d2b..a7f44ce 100644
--- a/cmd/traffic_manager/MgmtHandlers.cc
+++ b/cmd/traffic_manager/MgmtHandlers.cc
@@ -258,8 +258,6 @@ mgmt_synthetic_main(void *)
 
       if (thrId <= 0) {
         mgmt_log(stderr, "[SyntheticHealthServer] Failed to create worker thread");
-      } else {
-        pthread_detach(thrId); // We don't care to join this sucker
       }
     }
   }