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

rmudgett reviewboard at asterisk.org
Tue Dec 6 16:03:39 CST 2011


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

Review request for Asterisk Developers.


Summary
-------

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.


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


Diffs
-----

  /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/a86e5231/attachment.htm>


More information about the asterisk-dev mailing list