Fix a crash that occurred when a BYE was received on a replaced dialog.
authorMark Michelson <mmichelson@digium.com>
Fri, 15 Feb 2013 18:51:40 +0000 (18:51 +0000)
committerMark Michelson <mmichelson@digium.com>
Fri, 15 Feb 2013 18:51:40 +0000 (18:51 +0000)
commit8a7dd2f40880f988ea017294ad8001bf0e1825b5
treee4245e9648b86df1f761c2a8505b5ad75773e545
parente123ee2d77280e784d237dfc55740e96493819b7
Fix a crash that occurred when a BYE was received on a replaced dialog.

Reference counting for the channel and its tech_pvt got messed up at
some point between 1.8 and 11. The result was that if a BYE for a dialog
that had been replaced (via an INVITE with Replaces) was received, Asterisk
would crash due to trying to access data on a channel that was no longer there.

The fix I introduced is to remove code that both unrefs the sip_pvt and sets
the channel's tech_pvt to NULL when an INVITE with Replaces is handled. This
way when a BYE is received, the tech_pvt will be non-NULL and so the BYE can
be processed and not cause a crash.

(closes issue ASTERISK-20929)
reported by Kristopher Lalletti
patches:
ASTERISK-20929.patch uploaded by Mark Michelson (License #5049)
........

Merged revisions 381566 from http://svn.asterisk.org/svn/asterisk/branches/11

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@381568 65c4cc65-6c06-0410-ace0-fbb531ad65f3
channels/chan_sip.c