diff options
author | Matthew Wild <mwild1@gmail.com> | 2018-03-16 22:26:15 +0000 |
---|---|---|
committer | Matthew Wild <mwild1@gmail.com> | 2018-03-16 22:26:15 +0000 |
commit | a4d4661846349436ba18c0a688f88d7721ea9a96 (patch) | |
tree | 938421cd99e579bdd55b118790b6cd61d04afb92 /util | |
parent | 05c0a961121bc9e04edb225f29c871adaeaf0ecf (diff) | |
download | prosody-a4d4661846349436ba18c0a688f88d7721ea9a96.tar.gz prosody-a4d4661846349436ba18c0a688f88d7721ea9a96.zip |
util.async: Ensure runner is left in correct state after out-of-main-loop error (+tests)
Diffstat (limited to 'util')
-rw-r--r-- | util/async.lua | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/util/async.lua b/util/async.lua index 25236b11..3a75e1c1 100644 --- a/util/async.lua +++ b/util/async.lua @@ -17,13 +17,18 @@ local function runner_continue(thread) end local ok, state, runner = coroutine.resume(thread); if not ok then + local err = state; -- Running the coroutine failed, which means we have to find the runner manually, -- in order to inform the error handler local level = 0; while debug.getinfo(thread, level, "") do level = level + 1; end ok, runner = debug.getlocal(thread, level-1, 1); local error_handler = runner.watchers.error; - if error_handler then error_handler(runner, debug.traceback(thread, state)); end + if error_handler then error_handler(runner, debug.traceback(thread, err)); end + local ready_handler = runner.watchers.ready; + runner.state, runner.thread = "ready", nil; + if ready_handler then ready_handler(runner); end + runner.notified_state = "ready"; elseif state == "ready" then -- If state is 'ready', it is our responsibility to update runner.state from 'waiting'. -- We also have to :run(), because the queue might have further items that will not be |