summaryrefslogtreecommitdiff
path: root/src/hook_manager.cc
diff options
context:
space:
mode:
authorMaxime Coste <frrrwww@gmail.com>2016-06-27 20:55:07 +0100
committerMaxime Coste <frrrwww@gmail.com>2016-06-27 20:55:07 +0100
commit089eaa52e06c5e12ed1cc965ec4088f37f78aaaa (patch)
treea6e908fdf13dbf2c604921dc41ab896afeed5dff /src/hook_manager.cc
parent014cd5088316aec308e57544f29daa1ff7949e19 (diff)
Detect recursive hook run and abort in that case
Fixes #709
Diffstat (limited to 'src/hook_manager.cc')
-rw-r--r--src/hook_manager.cc10
1 files changed, 10 insertions, 0 deletions
diff --git a/src/hook_manager.cc b/src/hook_manager.cc
index 2a29d5cc..bd83b976 100644
--- a/src/hook_manager.cc
+++ b/src/hook_manager.cc
@@ -51,6 +51,16 @@ void HookManager::run_hook(StringView hook_name,
if (hook_list_it == m_hook.end())
return;
+ if (contains(m_running_hooks, std::make_pair(hook_name, param)))
+ {
+ auto error = format("recursive call of hook {}/{}, aborting", hook_name, param);
+ write_to_debug_buffer(error);
+ throw runtime_error(std::move(error));
+ }
+
+ m_running_hooks.emplace_back(hook_name, param);
+ auto pop_running_hook = on_scope_end([this]{ m_running_hooks.pop_back(); });
+
using Clock = std::chrono::steady_clock;
using TimePoint = Clock::time_point;