[asterisk-dev] [Code Review] 2613: IAX2: Transfer Reject: Lock bridgecallno before touching it, refactor

[svnbot]

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

(Updated June 11, 2013, 3:03 a.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers and Matt Jordan.


Changes
-------

Committed in revision 391333


Repository: Asterisk


Description
-------

1). When touching the bridgecallno, we need to lock it.

2). Remove magic number '0' and replace with TRANSFER_NONE.

3). Exit early if no bridgecallno.

4). Reduce indentation.


Diffs
-----

  branches/1.8/channels/chan_iax2.c 391332 

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


Testing
-------

Test calls to the queue app up the trunk, which inturn make a call back down same trunk.
These queue calls don't have have a bridgecallno.
Verified that 'unable to transfer' was still emiited.  


Thanks,

Alec Davis

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20130611/8b0367ca/attachment.htm>