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

paziz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/master by this push:
     new d1fdba4  coverity 1382795: Pointer to local outside scope
d1fdba4 is described below

commit d1fdba4dcea9f57a1c40f287391c73c1d15f5101
Author: Bryan Call <bc...@apache.org>
AuthorDate: Wed Nov 22 09:03:05 2017 -0800

    coverity 1382795: Pointer to local outside scope
---
 iocore/net/SSLNetVConnection.cc | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/iocore/net/SSLNetVConnection.cc b/iocore/net/SSLNetVConnection.cc
index a0cf7b0..61bc0aa 100644
--- a/iocore/net/SSLNetVConnection.cc
+++ b/iocore/net/SSLNetVConnection.cc
@@ -976,6 +976,8 @@ SSLNetVConnection::sslStartHandShake(int event, int &err)
 
   case SSL_EVENT_CLIENT:
 
+    char buff[INET6_ADDRSTRLEN];
+
     if (this->ssl == nullptr) {
       // Making the check here instead of later, so we only
       // do this setting immediately after we create the SSL object
@@ -983,7 +985,6 @@ SSLNetVConnection::sslStartHandShake(int event, int &err)
       int8_t clientVerify = 0;
       cchar *serverKey    = this->options.sni_servername;
       if (!serverKey) {
-        char buff[INET6_ADDRSTRLEN];
         ats_ip_ntop(this->get_remote_addr(), buff, INET6_ADDRSTRLEN);
         serverKey = buff;
       }

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

Reply via email to