trafficserver-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From mas...@apache.org
Subject [trafficserver] 02/05: Propagate errors
Date Fri, 01 Sep 2017 07:49:11 GMT
This is an automated email from the ASF dual-hosted git repository.

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

commit 08c398d158c6877f98fa0c981cc955536800dfd4
Author: Masakazu Kitajo <maskit@apache.org>
AuthorDate: Wed Aug 30 11:41:22 2017 +0900

    Propagate errors
---
 iocore/net/QUICNetVConnection.cc     | 2 +-
 iocore/net/quic/QUICStreamManager.cc | 4 ++--
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/iocore/net/QUICNetVConnection.cc b/iocore/net/QUICNetVConnection.cc
index 2b772b7..5d85ac3 100644
--- a/iocore/net/QUICNetVConnection.cc
+++ b/iocore/net/QUICNetVConnection.cc
@@ -505,7 +505,7 @@ QUICNetVConnection::_state_handshake_process_initial_client_packet(std::unique_p
     // Check integrity (QUIC-TLS-04: 6.1. Integrity Check Processing)
     if (packet->has_valid_fnv1a_hash()) {
       bool should_send_ack;
-      this->_frame_dispatcher->receive_frames(packet->payload(), packet->payload_size(),
should_send_ack);
+      error = this->_frame_dispatcher->receive_frames(packet->payload(), packet->payload_size(),
should_send_ack);
     } else {
       DebugQUICCon("Invalid FNV-1a hash value");
       return QUICError(QUICErrorClass::QUIC_TRANSPORT, QUICErrorCode::CRYPTOGRAPHIC_ERROR);
diff --git a/iocore/net/quic/QUICStreamManager.cc b/iocore/net/quic/QUICStreamManager.cc
index c1aed64..6b62ebd 100644
--- a/iocore/net/quic/QUICStreamManager.cc
+++ b/iocore/net/quic/QUICStreamManager.cc
@@ -113,7 +113,7 @@ QUICStreamManager::_handle_frame(const std::shared_ptr<const QUICMaxStreamDataFr
 {
   QUICStream *stream = this->_find_stream(frame->stream_id());
   if (stream) {
-    stream->recv(frame);
+    return stream->recv(frame);
   } else {
     // TODO: connection error?
   }
@@ -126,7 +126,7 @@ QUICStreamManager::_handle_frame(const std::shared_ptr<const QUICStreamBlockedFr
 {
   QUICStream *stream = this->_find_stream(frame->stream_id());
   if (stream) {
-    stream->recv(frame);
+    return stream->recv(frame);
   } else {
     // TODO: connection error?
   }

-- 
To stop receiving notification emails like this one, please contact
"commits@trafficserver.apache.org" <commits@trafficserver.apache.org>.

Mime
View raw message