You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by ig...@apache.org on 2013/01/14 17:39:01 UTC

git commit: [TS-1652] Mutex leak fix

Updated Branches:
  refs/heads/3.2.x 3aa39e572 -> 9f27803fa


[TS-1652]  Mutex leak fix

test/review: amc, zym, jpeach, igalic
backport: igalic


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

Branch: refs/heads/3.2.x
Commit: 9f27803fa434e1027798dff17f5454cd56cc6621
Parents: 3aa39e5
Author: Igor Galić <i....@brainsware.org>
Authored: Mon Jan 14 17:22:31 2013 +0100
Committer: Igor Galić <i....@brainsware.org>
Committed: Mon Jan 14 17:22:31 2013 +0100

----------------------------------------------------------------------
 CHANGES                          |    4 +++-
 STATUS                           |    7 -------
 iocore/net/UnixNetVConnection.cc |    1 +
 3 files changed, 4 insertions(+), 8 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/9f27803f/CHANGES
----------------------------------------------------------------------
diff --git a/CHANGES b/CHANGES
index 91764db..5d30e4f 100644
--- a/CHANGES
+++ b/CHANGES
@@ -1,5 +1,7 @@
                                                          -*- coding: utf-8 -*-
-Changes with Apache Traffic Server 3.2.3
+Changes with Apache Traffic Server 3.3.1
+
+  *) [TS-1652] Mutex leak plugged.
 
   *) [TS-1585] plugins need memory barriers for ARM
 

http://git-wip-us.apache.org/repos/asf/trafficserver/blob/9f27803f/STATUS
----------------------------------------------------------------------
diff --git a/STATUS b/STATUS
index 4c6be50..3aeee83 100644
--- a/STATUS
+++ b/STATUS
@@ -41,13 +41,6 @@ A list of all bugs open for the next development release can be found at
 
 PATCHES ACCEPTED TO BACKPORT FROM TRUNK:
 
-  Mutex leak fix
-  Patch: db0cfd44222e5e280db5b832496691ec7722e28e
-  Jira: https://issues.apache.org/jira/browse/TS-1652
-  +1: amc, zym, jpeach, igalic
-    this patch have been in our own tree for weeks --zym
-	why didn't you propose it yourself then! --igalic
-	
   raw disk cache disabled when system start
   Jira: https://issues.apache.org/jira/browse/TS-1651
   Patch: 41754c7e5a23f3785384afd0eea5f50486e407d9

http://git-wip-us.apache.org/repos/asf/trafficserver/blob/9f27803f/iocore/net/UnixNetVConnection.cc
----------------------------------------------------------------------
diff --git a/iocore/net/UnixNetVConnection.cc b/iocore/net/UnixNetVConnection.cc
index bad8746..9bfb137 100644
--- a/iocore/net/UnixNetVConnection.cc
+++ b/iocore/net/UnixNetVConnection.cc
@@ -1128,6 +1128,7 @@ UnixNetVConnection::free(EThread *t)
   NET_SUM_GLOBAL_DYN_STAT(net_connections_currently_open_stat, -1);
   // clear variables for reuse
   this->mutex.clear();
+  action_.mutex.clear();
   got_remote_addr = 0;
   got_local_addr = 0;
   read.vio.mutex.clear();