manager.c: Fix erroneous reloads in UpdateConfig.

Currently, a reload will always occur if the
Reload header is provided for the UpdateConfig
action. However, we should not be doing a reload
if the header value has a falsy value, per the
documentation, so this makes the reload behavior
consistent with the existing documentation.

Resolves: #551
(cherry picked from commit d50d981543)
This commit is contained in:
Naveen Albert 2024-01-25 12:44:29 -05:00 committed by Asterisk Development Team
parent 309ec5e762
commit faf4c3b5d3
1 changed files with 3 additions and 2 deletions

View File

@ -4329,9 +4329,10 @@ static int action_updateconfig(struct mansession *s, const struct message *m)
astman_send_ack(s, m, NULL);
if (!ast_strlen_zero(rld)) {
if (ast_true(rld)) {
rld = NULL;
ast_module_reload(NULL); /* Reload everything */
} else if (!ast_false(rld)) {
ast_module_reload(rld); /* Reload the specific module */
}
ast_module_reload(rld);
}
} else {
ast_config_destroy(cfg);