trafficserver-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From mas...@apache.org
Subject [trafficserver] branch master updated: npn string should be updated on unregistering a protocol as well
Date Fri, 10 Feb 2017 01:21:44 GMT
This is an automated email from the ASF dual-hosted git repository.

maskit 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  3dde26b   npn string should be updated on unregistering a protocol as well
3dde26b is described below

commit 3dde26b8a7c2f3f5f552840942005d0b43f65666
Author: Syeda Persia Aziz <persia.aziz@yahoo.com>
AuthorDate: Wed Feb 8 23:10:11 2017 -0600

    npn string should be updated on unregistering a protocol as well
---
 iocore/net/SSLNextProtocolSet.cc | 12 +++++-------
 1 file changed, 5 insertions(+), 7 deletions(-)

diff --git a/iocore/net/SSLNextProtocolSet.cc b/iocore/net/SSLNextProtocolSet.cc
index 6866349..83df312 100644
--- a/iocore/net/SSLNextProtocolSet.cc
+++ b/iocore/net/SSLNextProtocolSet.cc
@@ -48,7 +48,10 @@ create_npn_advertisement(const SSLNextProtocolSet::NextProtocolEndpoint::list_ty
   const SSLNextProtocolSet::NextProtocolEndpoint *ep;
   unsigned char *advertised;
 
-  *npn = nullptr;
+  if (*npn) {
+    ats_free(*npn);
+    *npn = nullptr;
+  }
   *len = 0;
 
   for (ep = endpoints.head; ep != nullptr; ep = endpoints.next(ep)) {
@@ -100,12 +103,6 @@ SSLNextProtocolSet::registerEndpoint(const char *proto, Continuation
*ep)
   if (!findEndpoint((const unsigned char *)proto, len)) {
     this->endpoints.push(new NextProtocolEndpoint(proto, ep));
 
-    if (npn) {
-      ats_free(npn);
-      npn   = nullptr;
-      npnsz = 0;
-    }
-
     create_npn_advertisement(this->endpoints, &npn, &npnsz);
 
     return true;
@@ -122,6 +119,7 @@ SSLNextProtocolSet::unregisterEndpoint(const char *proto, Continuation
*ep)
       // Protocol must be registered only once; no need to remove
       // any more entries.
       this->endpoints.remove(e);
+      create_npn_advertisement(this->endpoints, &npn, &npnsz);
       return true;
     }
   }

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

Mime
View raw message