[asterisk-dev] [Code Review] 4430: ISDN AOC: Fix crash from an AOC-E message that doesn't have a channel association.

[rmudgett]

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

(Updated Feb. 19, 2015, 6:25 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers.


Changes
-------

Committed in revision 431974


Bugs: ASTERISK-22670, ASTERISK-24689 and ASTERISK-24740
    https://issues.asterisk.org/jira/browse/ASTERISK-22670
    https://issues.asterisk.org/jira/browse/ASTERISK-24689
    https://issues.asterisk.org/jira/browse/ASTERISK-24740


Repository: Asterisk


Description
-------

Processing an AOC-E event that does not or no longer has a channel
association causes a crash.

The problem with posting AOC events to the channel topic is that AOC-E
events don't always have a channel association and posting the event to
the all channels topic is just wrong.  AOC-E events do however have their
own charging association method to refer to the agreement with the
charging entity.

* Changed the AOC events to post to the AMI manager topic instead of the
channel topics.  If a channel is associated with the event then channel
snapshot information is supplied with the AMI event.

* Eliminated RAII_VAR() usage in aoc_to_ami() and ast_aoc_manager_event().

This patch supercedes the patch on Review: https://reviewboard.asterisk.org/r/4427/


Diffs
-----

  /branches/13/main/aoc.c 431915 

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


Testing
-------

Created a dummy AOC-E event with and without a channel association when an
ISDN call came in.  The AMI AOC-E events were generated as expected and
didn't crash.


Thanks,

rmudgett

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20150219/1235627e/attachment.html>