trafficserver-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From zw...@apache.org
Subject [1/7] trafficserver git commit: TS-3810 traffic_manager leaks that thread stack, make sure to detach the threads
Date Mon, 03 Aug 2015 23:10:12 GMT
Repository: trafficserver
Updated Branches:
  refs/heads/6.0.x 53f150c33 -> 6062188f5


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/6.0.x
Commit: b5c7dbb8a9265e83742a0c56744c9fa4a298c7f0
Parents: 1d1eaa6
Author: Leif Hedstrom <zwoop@apache.org>
Authored: Fri Jul 31 14:28:06 2015 -0600
Committer: Leif Hedstrom <zwoop@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
       }
     }
   }


Mime
View raw message