summaryrefslogtreecommitdiff
path: root/main/jitterbuf.c
diff options
context:
space:
mode:
authorMark Michelson <mmichelson@digium.com>2015-04-30 15:20:43 -0500
committerMark Michelson <mmichelson@digium.com>2015-04-30 15:20:43 -0500
commitdc23204acae2bc3cff01c1ef57efd88cfe6bd825 (patch)
tree1b038fadcd62fff821fe48f6b2f2f65df733884e /main/jitterbuf.c
parentbb6ddb3dc825ed94ac53de64758761372d2ead18 (diff)
Prevent potential crash on blond transfer.
Scenario: Alice calls Bob. Bob performs a blond transfer to Carol. Carol rejects the incoming call (or some other immediate circumstance causes Carol not to answer the call) What occurs in this case is that when the bridge between Alice and Bob breaks, Alice is told to masquerade into Bob's channel that had placed the call to Carol. The actual masquerade goes down without a hitch. However, a channel fixup callback that attempts to publish dial events over Stasis has a crash. The reason for this crash is that the datastore on Bob's channel that placed the outbound call to Carol only had a bare pointer to Carol's channel. Since Carol rejected the incoming call, Carol's channel has been hung up and freed, meaning accessing her channel results in a crash. The fix here is simple. The dial fixup code has been altered to hold references to the involved channels and to drop those references when freeing data. ASTERISK-25025 #close Reported by Chet Stevens Change-Id: I54eedda207b8ec7a69263353b43abe5746aea197
Diffstat (limited to 'main/jitterbuf.c')
0 files changed, 0 insertions, 0 deletions