You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by bc...@apache.org on 2015/07/08 19:26:45 UTC

[2/2] trafficserver git commit: TS-3595: Cookie header split into multiple lines with H2 Changed to not be recursive

TS-3595: Cookie header split into multiple lines with H2
Changed to not be recursive

(cherry picked from commit 63ae314a50dcc0b0811f6fb044b1ca1afbc72a31)


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

Branch: refs/heads/6.0.x
Commit: bb998134e2053285753ca6b692ac4c9f81c3175f
Parents: 45b90f5
Author: Bryan Call <bc...@apache.org>
Authored: Tue Jul 7 18:52:02 2015 -0700
Committer: Bryan Call <bc...@apache.org>
Committed: Wed Jul 8 10:26:21 2015 -0700

----------------------------------------------------------------------
 proxy/hdrs/MIME.h | 14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/trafficserver/blob/bb998134/proxy/hdrs/MIME.h
----------------------------------------------------------------------
diff --git a/proxy/hdrs/MIME.h b/proxy/hdrs/MIME.h
index 4d9b2b6..f55da24 100644
--- a/proxy/hdrs/MIME.h
+++ b/proxy/hdrs/MIME.h
@@ -1329,22 +1329,22 @@ MIMEHdr::field_value_append(MIMEField *field, const char *value_str, int value_l
   field->value_append(m_heap, m_mime, value_str, value_len, prepend_comma, separator);
 }
 
-
-
 inline void
 MIMEHdr::field_combine_dups(MIMEField *field, bool prepend_comma, const char separator)
 {
-  if (field->has_dups()) {
-    MIMEField *duplicate = field->m_next_dup;
-    field_combine_dups(duplicate, prepend_comma, separator);
+  MIMEField *current = field->m_next_dup;
+  MIMEField *next = NULL;
 
+  while (current) {
+    next = current->m_next_dup;
 
     int value_len = 0;
-    const char *value_str = duplicate->value_get(&value_len);
+    const char *value_str = current->value_get(&value_len);
     if (value_len > 0) {
       field->value_append(m_heap, m_mime, value_str, value_len, prepend_comma, separator);
     }
-    field_delete(duplicate);
+    field_delete(current, false); // don't delete duplicates
+    current = next;
   }
 }