Fix a use-after-free in quic_client.

gfe-relnote: (n/a) Tools only.
PiperOrigin-RevId: 248303570
Change-Id: I7c666a928347d23b34c6b7b7c66f74c77971669a

quic/quartc/quartc_factory.cc

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

quic/tools/quic_client_bin.cc

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;