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

rmudgett reviewboard at asterisk.org
Tue Feb 17 11:52:20 CST 2015

This is an automatically generated e-mail. To reply, visit:

Review request for Asterisk Developers.

Bugs: ASTERISK-22670, ASTERISK-24689 and ASTERISK-24740

Repository: Asterisk


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/


  /branches/13/main/aoc.c 431915 

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


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.



-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-dev/attachments/20150217/85c9b1d2/attachment.html>

More information about the asterisk-dev mailing list