Add DCHECKs to quic_framer These DCHECKs helped debug some Chromium test issues. gfe-relnote: n/a, dcheck-only PiperOrigin-RevId: 255500147 Change-Id: I123f715b139e8bb374a77445afe54cd11b969166
diff --git a/quic/core/quic_framer.cc b/quic/core/quic_framer.cc index 6a6046c..cdb941f 100644 --- a/quic/core/quic_framer.cc +++ b/quic/core/quic_framer.cc
@@ -2096,7 +2096,9 @@ QuicConnectionIdIncluded server_connection_id_included = GetServerConnectionIdIncludedAsSender(header, perspective_); DCHECK_EQ(CONNECTION_ID_ABSENT, - GetClientConnectionIdIncludedAsSender(header, perspective_)); + GetClientConnectionIdIncludedAsSender(header, perspective_)) + << ENDPOINT << ParsedQuicVersionToString(version_) + << " invalid header: " << header; switch (server_connection_id_included) { case CONNECTION_ID_ABSENT: @@ -2228,6 +2230,10 @@ if (QuicVersionHasLongHeaderLengths(transport_version()) && header.version_flag) { if (header.long_packet_type == INITIAL) { + DCHECK_NE(VARIABLE_LENGTH_INTEGER_LENGTH_0, + header.retry_token_length_length) + << ENDPOINT << ParsedQuicVersionToString(version_) + << " bad retry token length length in header: " << header; // Write retry token length. if (!writer->WriteVarInt62(header.retry_token.length(), header.retry_token_length_length)) {