[asterisk-dev] [Code Review] 4373: Manager Action ModuleLoad gives incorrect response when used to reload modules

[Jonathan Rose]

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

(Updated Jan. 27, 2015, 11:22 a.m.)


Status
------

This change has been marked as submitted.


Review request for Asterisk Developers and jbigelow.


Changes
-------

Committed in revision 431153


Repository: Asterisk


Description
-------

ModuleLoad actions give the wrong response when used to reload a module. On success, it will issue a 'no such module' error instead of a success response even though the module was reloaded successfully.


Diffs
-----

  /branches/13/main/manager.c 431112 

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


Testing
-------

Before Patch:

Action: ModuleLoad
LoadType: reload
Module: pbx_config.so

Response: Error
Message: No such module.

CLI output showed the module was actually reloaded successfully.

After Patch:

Action: ModuleLoad
LoadType: reload
Module: pbx_config.so

Response: Success
Message: Module reloaded.


Thanks,

Jonathan Rose

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