[trafficserver] branch master updated: check if any of client cert path or file is null

[zw...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

zwoop pushed a commit to branch master
in repository https://git-dual.apache.org/repos/asf/trafficserver.git

The following commit(s) were added to refs/heads/master by this push:
       new  0798fe4   check if any of client cert path or file is null
0798fe4 is described below

commit 0798fe4c594d2cca34d0a56b097cd67124fec362
Author: Persia Aziz <pe...@yahoo-inc.com>
AuthorDate: Mon Jan 30 16:35:52 2017 -0600

    check if any of client cert path or file is null
---
 proxy/http/HttpSM.cc | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

diff --git a/proxy/http/HttpSM.cc b/proxy/http/HttpSM.cc
index 6d611b9..8d77b1c 100644
--- a/proxy/http/HttpSM.cc
+++ b/proxy/http/HttpSM.cc
@@ -5050,10 +5050,13 @@ HttpSM::do_http_server_open(bool raw)
     if (host && len > 0) {
       opt.set_sni_servername(host, len);
     }
-
-    ats_scoped_str clientCert(
-      (Layout::relative_to(t_state.txn_conf->client_cert_filepath, t_state.txn_conf->client_cert_filename)));
-    opt.set_client_certname(clientCert);
+    if (t_state.txn_conf->client_cert_filepath && t_state.txn_conf->client_cert_filename) {
+      ats_scoped_str clientCert(
+        (Layout::relative_to(t_state.txn_conf->client_cert_filepath, t_state.txn_conf->client_cert_filename)));
+      if (clientCert != nullptr) {
+        opt.set_client_certname(clientCert);
+      }
+    }
     connect_action_handle = sslNetProcessor.connect_re(this,                                 // state machine
                                                        &t_state.current.server->dst_addr.sa, // addr + port
                                                        &opt);

-- 
To stop receiving notification emails like this one, please contact
['"commits@trafficserver.apache.org" <co...@trafficserver.apache.org>'].