[incubator-heron] branch master updated: Rate limit needs to be disabled before the bufferevent_free call (#2848)

[ka...@apache.org]

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

karthikz pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-heron.git


The following commit(s) were added to refs/heads/master by this push:
     new c936cf1  Rate limit needs to be disabled before the bufferevent_free call (#2848)
c936cf1 is described below

commit c936cf119dea805c436cbcad53d806c84a72e07a
Author: Ning Wang <nw...@twitter.com>
AuthorDate: Sat Apr 7 09:06:28 2018 -0700

    Rate limit needs to be disabled before the bufferevent_free call (#2848)
---
 heron/common/src/cpp/network/baseconnection.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/heron/common/src/cpp/network/baseconnection.cpp b/heron/common/src/cpp/network/baseconnection.cpp
index e992cd9..28442a6 100644
--- a/heron/common/src/cpp/network/baseconnection.cpp
+++ b/heron/common/src/cpp/network/baseconnection.cpp
@@ -57,8 +57,8 @@ BaseConnection::BaseConnection(ConnectionEndPoint* endpoint, ConnectionOptions*
 
 BaseConnection::~BaseConnection() {
   CHECK(mState == INIT || mState == DISCONNECTED);
-  bufferevent_free(buffer_);
   disableRateLimit();  // To free the config object
+  bufferevent_free(buffer_);
 }
 
 sp_int32 BaseConnection::start() {

-- 
To stop receiving notification emails like this one, please contact
karthikz@apache.org.