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