You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@trafficserver.apache.org by ma...@apache.org on 2018/05/21 05:32:52 UTC

[trafficserver] branch quic-latest updated: Fix debug message when storing frame

This is an automated email from the ASF dual-hosted git repository.

masaori pushed a commit to branch quic-latest
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/quic-latest by this push:
     new a0f579e  Fix debug message when storing frame
a0f579e is described below

commit a0f579e882224e8945b29f898b83fe900250512a
Author: Masaori Koshiba <ma...@apache.org>
AuthorDate: Mon May 21 14:29:18 2018 +0900

    Fix debug message when storing frame
---
 iocore/net/QUICNetVConnection.cc | 14 +++++++++-----
 1 file changed, 9 insertions(+), 5 deletions(-)

diff --git a/iocore/net/QUICNetVConnection.cc b/iocore/net/QUICNetVConnection.cc
index 525f129..5d659b6 100644
--- a/iocore/net/QUICNetVConnection.cc
+++ b/iocore/net/QUICNetVConnection.cc
@@ -1117,23 +1117,27 @@ QUICNetVConnection::_store_frame(ats_unique_buf &buf, size_t &len, bool &retrans
     buf = ats_unique_malloc(max_size);
   }
 
-  size_t l = 0;
+  ink_assert(max_size > len);
 
-  // TODO: check debug build
   char msg[1024];
-  QUICConDebug("[TX] %s", msg);
-
-  ink_assert(max_size > len);
+  size_t l = 0;
   size_t n = frame->store(buf.get() + len, &l, max_size - len);
   if (n > 0) {
+    // TODO: check debug build
     frame->debug_msg(msg, sizeof(msg));
+    QUICConDebug("[TX] %s", msg);
+
     len += l;
     return;
   }
 
   // split frame
   auto new_frame = QUICFrameFactory::split_frame(frame.get(), max_size - len);
+
+  // TODO: check debug build
   frame->debug_msg(msg, sizeof(msg));
+  QUICConDebug("[TX] %s", msg);
+
   ink_assert(frame->store(buf.get() + len, &l, max_size - len) > 0);
   ink_assert(new_frame != nullptr);
 

-- 
To stop receiving notification emails like this one, please contact
masaori@apache.org.