You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by br...@apache.org on 2014/01/15 23:43:23 UTC
git commit: [TS-2497] Failed post results in tunnel buffers being
returned to freelist prematurely,
Reporter: Thomas Jackson
Updated Branches:
refs/heads/master 3fa5b0319 -> 5eab1b838
[TS-2497] Failed post results in tunnel buffers being returned to freelist prematurely, Reporter: Thomas Jackson <th...@linkedin.com>
Project: http://git-wip-us.apache.org/repos/asf/trafficserver/repo
Commit: http://git-wip-us.apache.org/repos/asf/trafficserver/commit/5eab1b83
Tree: http://git-wip-us.apache.org/repos/asf/trafficserver/tree/5eab1b83
Diff: http://git-wip-us.apache.org/repos/asf/trafficserver/diff/5eab1b83
Branch: refs/heads/master
Commit: 5eab1b838c0bc449f6e0ed881da40e6e832c8ee5
Parents: 3fa5b03
Author: Brian Geffon <br...@apache.org>
Authored: Wed Jan 15 14:43:14 2014 -0800
Committer: Brian Geffon <br...@apache.org>
Committed: Wed Jan 15 14:43:14 2014 -0800
----------------------------------------------------------------------
CHANGES | 4 ++++
proxy/http/HttpSM.cc | 1 -
2 files changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/5eab1b83/CHANGES
----------------------------------------------------------------------
diff --git a/CHANGES b/CHANGES
index a688aea..38ebf8d 100644
--- a/CHANGES
+++ b/CHANGES
@@ -1,6 +1,10 @@
-*- coding: utf-8 -*-
Changes with Apache Traffic Server 4.2.0
+ *) [TS-2497] Failed post results in tunnel buffers being returned to
+ freelist prematurely.
+ Reporter: Thomas Jackson <th...@linkedin.com>
+
*) [TS-1668] Added HSTS configuration options to ATS
*) [TS-2495] Reduce the size of HttpVCTableEntry.
http://git-wip-us.apache.org/repos/asf/trafficserver/blob/5eab1b83/proxy/http/HttpSM.cc
----------------------------------------------------------------------
diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index eff3260..2d6324a 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -4957,7 +4957,6 @@ HttpSM::handle_post_failure()
}
ua_entry->in_tunnel = false;
server_entry->in_tunnel = false;
- tunnel.deallocate_buffers();
// disable redirection in case we got a partial response and then EOS, because the buffer might not
// have the full post and it's deallocating the post buffers here