[asterisk-dev] [Code Review] sig_pri and chan_misdn do not update connected line after call transfer

[rmudgett at digium.com]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviewboard.asterisk.org/r/958/
-----------------------------------------------------------

(Updated 2010-10-04 19:16:46.726651)


Review request for Asterisk Developers and Russell Bryant.


Changes
-------

Fixed some double unlocks in ast_indicate_data().


Summary
-------

This patch corrects an oversight and is against the 1.8 branch.  The sig_pri and chan_misdn channel drivers did not update the connected line information after a call transfer.


Diffs (updated)
-----

  /branches/1.8/channels/chan_misdn.c 290256 
  /branches/1.8/channels/sig_pri.c 290256 
  /branches/1.8/main/channel.c 290256 

Diff: https://reviewboard.asterisk.org/r/958/diff


Testing
-------

Call transfers still work and the connected line information is updated.  Also the expected caller/callee connected line interception macros are run on each channel.

Basic transfer actions:
A calls B
A places B on hold
A calls C
A can optionally swap between B and C by flash hook.
A transfers B to C

Test cases:
A transfers B to C using hangup.
A transfers B to C using hangup while C ringing.
A transfers C to B using hangup.
A transfers B to C using ECTExecute message.
A transfers B to C using ECTExecute message while C ringing.
A transfers C to B using ECTExecute message.

Note that chan_misdn does not support the ECTExecute method.


Thanks,

rmudgett