[asterisk-dev] [Code Review] Fix FollowMe CallerID on outgoing calls.

rmudgett reviewboard at asterisk.org
Tue Dec 6 16:11:31 CST 2011


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

(Updated Dec. 6, 2011, 4:11 p.m.)


Review request for Asterisk Developers.


Changes
-------

Fix red blobs.


Summary (updated)
-------

The addition of the Connected Line support changed how CallerID is passed to outgoing calls.  The FollowMe application was not updated to pass CallerID to the outgoing calls.

* Fix FollowMe CallerID on outgoing calls.
* Restructured findmeexec() to fix several memory leaks and eliminate some duplicated code.
* Made check the return value of create_followme_number().  Putting a NULL into the numbers list is bad if create_followme_number() fails.
* Fixed a couple uses of ast_strdupa() inside loops.

The changes to bridge_builtin_features.c fix a similar CallerID issue with the bridging API attended and blind transfers.


This addresses bug ASTERISK-17557.
    https://issues.asterisk.org/jira/browse/ASTERISK-17557


Diffs (updated)
-----

  /branches/1.8/apps/app_followme.c 347238 
  /branches/1.8/bridges/bridge_builtin_features.c 347238 

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


Testing
-------

FollowMe now supplies the CallerID on outgoing calls.


Thanks,

rmudgett

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20111206/c3bbc451/attachment-0001.htm>


More information about the asterisk-dev mailing list