Fix a use-after-free in quic_client.
gfe-relnote: (n/a) Tools only.
PiperOrigin-RevId: 248303570
Change-Id: I7c666a928347d23b34c6b7b7c66f74c77971669a
diff --git a/quic/quartc/quartc_factory.cc b/quic/quartc/quartc_factory.cc
index 93bb03f..cb457ff 100644
--- a/quic/quartc/quartc_factory.cc
+++ b/quic/quartc/quartc_factory.cc
@@ -73,7 +73,7 @@
// TODO(b/117157454): Perform version negotiation for Quartc outside of
// QuicSession/QuicConnection. Currently default of
- // quic_restart_flag_quic_no_server_conn_ver_negotiation2 is false,
+ // gfe2_restart_flag_quic_no_server_conn_ver_negotiation2 is false,
// but we fail blueprint test that sets all QUIC flags to true.
//
// Forcing flag to false to pass blueprint tests, but eventually we'll have
diff --git a/quic/tools/quic_client_bin.cc b/quic/tools/quic_client_bin.cc
index ff711a5..62bc0db 100644
--- a/quic/tools/quic_client_bin.cc
+++ b/quic/tools/quic_client_bin.cc
@@ -331,8 +331,8 @@
header_block[":path"] = url.PathParamsQuery();
// Append any additional headers supplied on the command line.
- for (QuicStringPiece sp :
- QuicTextUtils::Split(GetQuicFlag(FLAGS_headers), ';')) {
+ const std::string headers = GetQuicFlag(FLAGS_headers);
+ for (QuicStringPiece sp : QuicTextUtils::Split(headers, ';')) {
QuicTextUtils::RemoveLeadingAndTrailingWhitespace(&sp);
if (sp.empty()) {
continue;
