stasis transfer: fix a race condition on stasis bridge push
authorScott Griepentrog <sgriepentrog@digium.com>
Thu, 22 Jan 2015 18:10:13 +0000 (18:10 +0000)
committerScott Griepentrog <sgriepentrog@digium.com>
Thu, 22 Jan 2015 18:10:13 +0000 (18:10 +0000)
commit49f405fe4c4c758a0d99d05ed2d35f55dd76b732
tree85c40ddaa4c6a2169ffcd9cb2a1deecaf6c5c483
parent7fcc9ce8bca09810f766a04c0a52be8f83ff7503
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
res/stasis/app.c
res/stasis/stasis_bridge.c