[asterisk-dev] [Code Review] 3144: Protect manager data structures during shutdown.

[rmudgett]

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

(Updated Jan. 24, 2014, 12:29 p.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers.


Bugs: AST-1242
    https://issues.asterisk.org/jira/browse/AST-1242


Repository: Asterisk


Description
-------

Occasionally, the manager module would get an "INTERNAL_OBJ: bad magic number" error on a "core restart gracefully" command if an AMI connection is established.

* Added ao2_global_obj protection to the sessions global container.

* Fixed the order of unreferencing a session object in session_destroy().

* Removed unnecessary container traversals of the white/black filters during session_destructor().


Diffs
-----

  /branches/11/main/manager.c 406034 

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


Testing
-------

Since I was not able to reprocuce the INTERNAL_OBJ error reported, I just tested to see that things still worked.
* The CLI "manager show connected" still works.
* The system can still restart and shutdown.


Thanks,

rmudgett

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