[asterisk-dev] [Code Review] IAX2 Transfer Fix

[David Vossel]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.digium.com/r/140/
-----------------------------------------------------------

(Updated 2009-02-03 14:02:50.959517)


Review request for Asterisk Developers and Russell Bryant.


Changes
-------

Added remove_by_transfercallno(pvt) to destructor. fixed white space issues. 


Summary
-------

Fixes issue with IAX2 transfers not taking place.  As it was, a call that was being transfered would never be handed off correctly to the call ends because of how call numbers were stored in a hash table.  The hash table, "iax_peercallno_pvt", storing all the current call numbers did not take into account the complications associated with transferring a call, so a separate hash table was required.  This second hash table "iax_transfercallno_pvt" handles calls being transfered, once the call transfer is complete the call is removed from the transfer hash table and added to the peer hash table resuming normal operations. Addition functions were created to handle storing, removing, and comparing items in the iax_transfercallno_pvt table. 


This addresses bug 0013468.
    http://bugs.digium.com/view.php?id=0013468


Diffs (updated)
-----

  /branches/1.6.0/channels/chan_iax2.c 173167 

Diff: http://reviewboard.digium.com/r/140/diff


Testing
-------

this patch has been tested on both 1.6.0 and 1.4.  1.6.1 and trunk have issues of there own that must be fixed before this patch can be applied to them. 


Thanks,

David