summaryrefslogtreecommitdiff
path: root/res/res_pjsip_outbound_registration.c
diff options
context:
space:
mode:
authorScott Griepentrog <sgriepentrog@digium.com>2015-01-22 18:10:13 +0000
committerScott Griepentrog <sgriepentrog@digium.com>2015-01-22 18:10:13 +0000
commit49f405fe4c4c758a0d99d05ed2d35f55dd76b732 (patch)
tree85c40ddaa4c6a2169ffcd9cb2a1deecaf6c5c483 /res/res_pjsip_outbound_registration.c
parent7fcc9ce8bca09810f766a04c0a52be8f83ff7503 (diff)
stasis transfer: fix a race condition on stasis bridge push
After a bridge transfer completes where a local replacement channel is used, a stasis transfer message with the details of the transfer is sent. This is processed by stasis which then sets the stasis app name and replaced channel snapshot on the replacement channel. However, since a separate thread was already started to run stasis on the new replacement channel, a race was on to see if the message processing would be completed before the app name was needed, otherwise the channel would be hung up. This change moves the calls used to set the stasis app name and the replace snapshot to the bridge_stasis_push function callback from the bridge transfer logic, allowing the steps to be completed earlier and more deterministically, and the race elimianted. NOTE: the swap channel parameter to bridge_stasis_push (and thus all bridge push callbacks) must always be present when performing a swap with another channel. ASTERISK-24649 #close Reported by: John Bigelow Review: https://reviewboard.asterisk.org/r/4341/ ........ Merged revisions 430939 from http://svn.asterisk.org/svn/asterisk/branches/13 git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@430940 65c4cc65-6c06-0410-ace0-fbb531ad65f3
Diffstat (limited to 'res/res_pjsip_outbound_registration.c')
0 files changed, 0 insertions, 0 deletions