[trafficserver] branch master updated: Within UDPConnection::Release(), it should calls the ep.stop() only if the UDPConnection will be closed

[ok...@apache.org]

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

oknet 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 0b52a1a  Within UDPConnection::Release(), it should calls the ep.stop() only if the UDPConnection will be closed
0b52a1a is described below

commit 0b52a1abfb1d19b7405f2f62006485db662af075
Author: Oknet Xu <xu...@skyguard.com.cn>
AuthorDate: Mon Oct 30 12:16:32 2017 +0800

    Within UDPConnection::Release(), it should calls the ep.stop() only if the UDPConnection will be closed
---
 iocore/net/UnixUDPConnection.cc | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/iocore/net/UnixUDPConnection.cc b/iocore/net/UnixUDPConnection.cc
index d372e07..9cbab85 100644
--- a/iocore/net/UnixUDPConnection.cc
+++ b/iocore/net/UnixUDPConnection.cc
@@ -140,9 +140,9 @@ UDPConnection::Release()
 {
   UnixUDPConnection *p = (UnixUDPConnection *)this;
 
-  p->ep.stop();
-
   if (ink_atomic_increment(&p->refcount, -1) == 1) {
+    p->ep.stop();
+
     ink_assert(p->callback_link.next == nullptr);
     ink_assert(p->callback_link.prev == nullptr);
     ink_assert(p->polling_link.next == nullptr);

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