[asterisk-dev] [Code Review] 4037: Release AMI connections on shutdown

Corey Farrell reviewboard at asterisk.org
Fri Oct 3 15:39:03 CDT 2014


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

(Updated Oct. 3, 2014, 4:39 p.m.)


Review request for Asterisk Developers and opticron.


Changes
-------

Prevent fake event from being published to manager hooks.


Bugs: ASTERISK-24378
    https://issues.asterisk.org/jira/browse/ASTERISK-24378


Repository: Asterisk


Description
-------

Release AMI connections on shutdown.


I've looked further into this, I believe it has no effect on events generated after manager_shutdown.  manager_shutdown free's mgr_sessions, preventing __ast_manager_event_multichan from reaching any active sessions.  Note this wouldn't be the case with Asterisk 1.8 as the sessions container is never freed, yet another reason I'm proposing this fix for 11+.


Diffs (updated)
-----

  /branches/11/main/manager.c 424175 

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


Testing
-------

Tested with tests/channels/SIP/route.  Verified the AMI session is leaked without this patch, released with this patch.


Thanks,

Corey Farrell

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


More information about the asterisk-dev mailing list