Hi Khanh,

Ack from me.

I will push this patch tomorrow on your behalf if no more comment from reviewers.

Regards, Vu

On 9/27/19 4:58 PM, khanh.h.dang wrote:
If a stream is deleted, free the allocated memory for that stream and
decrease the no_of_log_streams_.
---
  src/dtm/transport/log_server.cc | 2 ++
  1 file changed, 2 insertions(+)

diff --git a/src/dtm/transport/log_server.cc b/src/dtm/transport/log_server.cc
index 43fa86a..2cff9e5 100644
--- a/src/dtm/transport/log_server.cc
+++ b/src/dtm/transport/log_server.cc
@@ -251,6 +251,8 @@ std::string LogServer::ExecuteCommand(const std::string& 
command,
      if (current_stream_ == stream) {
        current_stream_ = log_streams_.begin()->second;
      }
+    delete stream;
+    --no_of_log_streams_;
      return std::string{"!delete " + argument};
    } else if (command == "?flush") {
      for (const auto& s : log_streams_) {



_______________________________________________
Opensaf-devel mailing list
Opensaf-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/opensaf-devel

Reply via email to