diff options
| author | Maxime Coste <frrrwww@gmail.com> | 2016-06-27 20:55:07 +0100 |
|---|---|---|
| committer | Maxime Coste <frrrwww@gmail.com> | 2016-06-27 20:55:07 +0100 |
| commit | 089eaa52e06c5e12ed1cc965ec4088f37f78aaaa (patch) | |
| tree | a6e908fdf13dbf2c604921dc41ab896afeed5dff /src/hook_manager.cc | |
| parent | 014cd5088316aec308e57544f29daa1ff7949e19 (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.cc | 10 |
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; |
