diff options
| author | Alexander Traud <pabstraud@compuserve.com> | 2018-01-05 14:44:55 +0100 |
|---|---|---|
| committer | Richard Mudgett <rmudgett@digium.com> | 2018-01-06 20:13:07 -0600 |
| commit | 7e9781c25e59be8462ac6cc4d3a8914aca42d16f (patch) | |
| tree | 5b264511e7eaadb4c5c2bc1e43802e345c40e845 /res/res_curl.c | |
| parent | 09f339bda56d08a1b78359bcabaf2f9cc3234b03 (diff) | |
General: Silence modules on (un)load.
Some (normally optional) modules created notices, warnings, and even errors
in normal situations like (un)load. This cluttered the command-line interface
(CLI) on start and while stopping gracefully. However, when an user went for
the script './contrib/scripts/install_prereq', those modules get compiled-in
because their prerequisites were met at compile time. Furthermore, because of
ASTERISK_27475, the former talkative module 'res_curl' is built as side-effect.
ASTERISK-27553
Change-Id: I9f105f46d72553994e820679bfde3478a551b281
Diffstat (limited to 'res/res_curl.c')
| -rw-r--r-- | res/res_curl.c | 16 |
1 files changed, 4 insertions, 12 deletions
diff --git a/res/res_curl.c b/res/res_curl.c index 2551efcea..e0f753cc3 100644 --- a/res/res_curl.c +++ b/res/res_curl.c @@ -60,7 +60,9 @@ static int unload_module(void) /* If the dependent modules are still in memory, forbid unload */ for (i = 0; i < ARRAY_LEN(dependents); i++) { if (ast_module_check(dependents[i])) { - ast_log(LOG_ERROR, "%s (dependent module) is still loaded. Cannot unload res_curl.so\n", dependents[i]); + if (!ast_shutting_down()) { + ast_log(LOG_WARNING, "%s (dependent module) is still loaded. Cannot unload res_curl.so\n", dependents[i]); + } res = -1; } } @@ -73,19 +75,9 @@ static int unload_module(void) return res; } -/*! - * \brief Load the module - * - * Module loading including tests for configuration or dependencies. - * This function can return AST_MODULE_LOAD_FAILURE, AST_MODULE_LOAD_DECLINE, - * or AST_MODULE_LOAD_SUCCESS. If a dependency or environment variable fails - * tests return AST_MODULE_LOAD_FAILURE. If the module can not load the - * configuration file or other non-critical problem return - * AST_MODULE_LOAD_DECLINE. On success return AST_MODULE_LOAD_SUCCESS. - */ static int load_module(void) { - int res = 0; + int res = AST_MODULE_LOAD_SUCCESS; if (curl_global_init(CURL_GLOBAL_ALL)) { ast_log(LOG_ERROR, "Unable to initialize the cURL library. Cannot load res_curl.so\n"); |