aboutsummaryrefslogtreecommitdiff
path: root/contrib/epee/src/readline_buffer.cpp
diff options
context:
space:
mode:
authorJethro Grassie <jtg@xtrabass.com>2017-08-16 08:44:52 -0400
committerJethro Grassie <jtg@xtrabass.com>2017-08-16 08:58:12 -0400
commit67dd4933e6ae5517426de93ce8a3a448eeb48797 (patch)
tree9d66d26e9aca00d1a1ad359efe2cac7a09aea027 /contrib/epee/src/readline_buffer.cpp
parentMerge pull request #2297 (diff)
downloadmonero-67dd4933e6ae5517426de93ce8a3a448eeb48797.tar.xz
Add sync lock on stop
Also added and moved two free's hoping to fix leaks.
Diffstat (limited to '')
-rw-r--r--contrib/epee/src/readline_buffer.cpp9
1 files changed, 4 insertions, 5 deletions
diff --git a/contrib/epee/src/readline_buffer.cpp b/contrib/epee/src/readline_buffer.cpp
index ce8260ef6..d4051bcb8 100644
--- a/contrib/epee/src/readline_buffer.cpp
+++ b/contrib/epee/src/readline_buffer.cpp
@@ -61,7 +61,8 @@ void rdln::readline_buffer::start()
void rdln::readline_buffer::stop()
{
- std::unique_lock<std::mutex> lock(process_mutex);
+ std::unique_lock<std::mutex> lock_process(process_mutex);
+ std::unique_lock<std::mutex> lock_sync(sync_mutex);
have_line.notify_all();
if(m_cout_buf == NULL)
return;
@@ -152,9 +153,7 @@ static int process_input()
static void handle_line(char* line)
{
- // This function never gets called now as we are trapping newlines.
- // However, it still needs to be present for readline to know we are
- // manually handling lines.
+ free(line);
rl_done = 1;
return;
}
@@ -166,6 +165,7 @@ static int handle_enter(int x, int y)
line = rl_copy_text(0, rl_end);
std::string test_line = line;
+ free(line);
boost::trim_right(test_line);
rl_crlf();
@@ -191,7 +191,6 @@ static int handle_enter(int x, int y)
add_history(test_line.c_str());
history_set_pos(history_length);
}
- free(line);
if(last_line != "exit" && last_line != "q")
{