]> git.ipfire.org Git - thirdparty/asterisk.git/commit
app_dial.c: Moved channel lock to prevent deadlock
authorAlexey Khabulyak <khabalex@gmail.com>
Mon, 4 Aug 2025 14:48:57 +0000 (17:48 +0300)
committerAlexey Khabulyak <khabalex@gmail.com>
Mon, 11 Aug 2025 14:37:12 +0000 (14:37 +0000)
commit4b766d24d078dff701805f18cb41fb59bd6033ce
tree4c83b38dd61b592541ff91b3c161eb9e91b023e0
parent8dce57f9ada76580cb6d5ea12779f2a0301069a2
app_dial.c: Moved channel lock to prevent deadlock

It's reproducible with pbx_lua, not regular dialplan.

deadlock description:
1. asterisk locks a channel
2. calls function onedigit_goto
3. calls ast_goto_if_exists funciton
4. checks ast_exists_extension -> pbx_extension_helper
5. pbx_extension_helper calls pbx_find_extension
6. Then asterisk starts autoservice in a new thread
7. autoservice run tries to lock the channel again

Because our channel is locked already, autoservice can't lock.
Autoservice can't lock -> autoservice stop is waiting forever.
onedigit_goto waits for autoservice stop.

Resolves: https://github.com/asterisk/asterisk/issues/1335
apps/app_dial.c