Merged revisions 331316 via svnmerge from
authorKinsey Moore <kmoore@digium.com>
Wed, 10 Aug 2011 13:49:31 +0000 (13:49 +0000)
committerKinsey Moore <kmoore@digium.com>
Wed, 10 Aug 2011 13:49:31 +0000 (13:49 +0000)
https://origsvn.digium.com/svn/asterisk/branches/10

................
  r331316 | kmoore | 2011-08-10 08:48:41 -0500 (Wed, 10 Aug 2011) | 15 lines

  Merged revisions 331315 via svnmerge from
  https://origsvn.digium.com/svn/asterisk/branches/1.8

  ........
    r331315 | kmoore | 2011-08-10 08:47:46 -0500 (Wed, 10 Aug 2011) | 8 lines

    AMI action ModuleReload returns Error if Module: missing or empty

    An empty string was not being checked for properly causing identification of
    the module to be reloaded to fail and return an Error with message
    "No such module."

    (closes issue AST-616)
  ........
................

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@331317 65c4cc65-6c06-0410-ace0-fbb531ad65f3

main/manager.c

index 9478241..319b5ba 100644 (file)
@@ -4586,7 +4586,7 @@ static int manager_moduleload(struct mansession *s, const struct message *m)
                        astman_send_ack(s, m, "Module unloaded.");
                }
        } else if (!strcasecmp(loadtype, "reload")) {
-               if (module != NULL) {
+               if (!ast_strlen_zero(module)) {
                        res = ast_module_reload(module);
                        if (res == 0) {
                                astman_send_error(s, m, "No such module.");