In QuicConnection class, move stats_ to be above uber_received_packet_manager_, because the later is initialized from the former. In other words, if uber_received_packet_manager_'s constructor reads anything inside stats_ it might read garbage.
PiperOrigin-RevId: 337567677
Change-Id: Ib9914edc7338b585c48479a21476ebfb8f7f3e04
diff --git a/quic/core/quic_connection.h b/quic/core/quic_connection.h
index afcbe13..9531581 100644
--- a/quic/core/quic_connection.h
+++ b/quic/core/quic_connection.h
@@ -1605,6 +1605,9 @@
// When > 0, close the QUIC connection after this number of RTOs.
size_t num_rtos_for_blackhole_detection_;
+ // Statistics for this session.
+ QuicConnectionStats stats_;
+
UberReceivedPacketManager uber_received_packet_manager_;
// Indicates how many consecutive times an ack has arrived which indicates
@@ -1655,9 +1658,6 @@
QuicPacketCreator packet_creator_;
- // Statistics for this session.
- QuicConnectionStats stats_;
-
// The time that a packet is received for this connection. Initialized to
// connection creation time.
// This does not indicate the packet was processed.
