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:28:34 UTC

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

Repository: trafficserver
Updated Branches:
  refs/heads/master 1d1eaa698 -> b5c7dbb8a


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


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

Branch: refs/heads/master
Commit: b5c7dbb8a9265e83742a0c56744c9fa4a298c7f0
Parents: 1d1eaa6
Author: Leif Hedstrom <zw...@apache.org>
Authored: Fri Jul 31 14:28:06 2015 -0600
Committer: Leif Hedstrom <zw...@apache.org>
Committed: Fri Jul 31 14:28:06 2015 -0600

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


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/b5c7dbb8/cmd/traffic_manager/MgmtHandlers.cc
----------------------------------------------------------------------
diff --git a/cmd/traffic_manager/MgmtHandlers.cc b/cmd/traffic_manager/MgmtHandlers.cc
index a7f44ce..0569d2b 100644
--- a/cmd/traffic_manager/MgmtHandlers.cc
+++ b/cmd/traffic_manager/MgmtHandlers.cc
@@ -258,6 +258,8 @@ 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
       }
     }
   }