path: root/addons/ooh323c/src
diff options
authorrussell <russell@f38db490-d61c-443f-a65b-d21fe96a405b>2010-03-06 14:16:20 +0000
committerrussell <russell@f38db490-d61c-443f-a65b-d21fe96a405b>2010-03-06 14:16:20 +0000
commit5e8a340e01fa2bf47fb3f07aac63d59207b8965c (patch)
tree2debb1f980d3c2bb4a51d51b27ce98aa55ef243e /addons/ooh323c/src
parent77cb4d4a4c4cf6221ec7a73ad16d5da5a4dc0ce2 (diff)
Fix a crash in SIP blind transfer handling found by an automated external test.
The first real test added to the external test suite found a pretty nasty crash that occurred in Asterisk trunk. The crash was due to a race condition between the REFER handling and channel destruction in the channel thread. After the transfer has been completed, we go back to the transferrer channel and try to lock it so we can fire off a CEL event. However, there was no guarantee that the channel was still around at that point since it's racing against the channel thread. Since ast_channel is a reference counted object, the fix is simple. The code unlocks the transferrer channel before finally completing the transfer with an async goto. At this point the channel thread is going to start call tear down and the channel will eventually be destroyed. To ensure that the channel is valid when we want to fire off the CEL event, increase the channel's reference count. git-svn-id: http://svn.digium.com/svn/asterisk/trunk@251137 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'addons/ooh323c/src')
0 files changed, 0 insertions, 0 deletions