commit 9b3cba4056fc1f2acf7a315b73958771f2f2f993
author nharper <nharper@google.com> Thu Apr 16 08:17:20 2020 -0700
committer Copybara-Service <copybara-worker@google.com> Thu Apr 16 08:17:51 2020 -0700
tree 5fbaf71fece52d02afddcdcd039fe1f587ade3e2
parent 82ee4d30bf5a1e8bfd0c2751b19af151a4b203bd

Check for nullptr before calling OnProofVerifyDetailsAvailable

gfe-relnote: add nullptr check to QUIC's TlsClientHandshaker, protected by ietf quic draft version flags.
PiperOrigin-RevId: 306849131
Change-Id: I3e73186d70e7778fa49096b4c96710902c4e5f7c

quic/core/tls_client_handshaker.cc

diff --git a/quic/core/tls_client_handshaker.cc b/quic/core/tls_client_handshaker.cc
index 09b9097..f3e1b13 100644
--- a/quic/core/tls_client_handshaker.cc
+++ b/quic/core/tls_client_handshaker.cc
@@ -472,7 +472,9 @@
       std::unique_ptr<ProofVerifierCallback>(proof_verify_callback));
   switch (verify_result) {
     case QUIC_SUCCESS:
-      proof_handler_->OnProofVerifyDetailsAvailable(*verify_details_);
+      if (verify_details_) {
+        proof_handler_->OnProofVerifyDetailsAvailable(*verify_details_);
+      }
       return ssl_verify_ok;
     case QUIC_PENDING:
       proof_verify_callback_ = proof_verify_callback;