Prevent unload of modules which implement an Optional API.
authorCorey Farrell <git@cfware.com>
Thu, 2 Nov 2017 00:46:11 +0000 (20:46 -0400)
committerCorey Farrell <git@cfware.com>
Thu, 2 Nov 2017 00:46:11 +0000 (20:46 -0400)
Once an Optional API module is loaded it should stay loaded.  Unloading
an optional API module runs the risk of a crash if something else is
using it.  This patch causes all optional API providers to tell the
module loader not to unload except at shutdown.

ASTERISK-27389

Change-Id: Ia07786fe655681aec49cc8d3d96e06483b11f5e6

funcs/func_periodic_hook.c
res/res_agi.c
res/res_http_websocket.c
res/res_monitor.c
res/res_pktccops.c
res/res_smdi.c
res/res_statsd.c

index f8e79b3..0ab3d6b 100644 (file)
@@ -488,6 +488,11 @@ static int load_module(void)
 
        res = ast_custom_function_register_escalating(&hook_function, AST_CFE_BOTH);
 
+       if (!res) {
+               /* For Optional API. */
+               ast_module_shutdown_ref(AST_MODULE_SELF);
+       }
+
        return res ? AST_MODULE_LOAD_DECLINE : AST_MODULE_LOAD_SUCCESS;
 }
 
index 4660635..91f270c 100644 (file)
@@ -4713,6 +4713,10 @@ static int load_module(void)
                unload_module();
                return AST_MODULE_LOAD_DECLINE;
        }
+
+       /* For Optional API. */
+       ast_module_shutdown_ref(AST_MODULE_SELF);
+
        return AST_MODULE_LOAD_SUCCESS;
 }
 
index 60332f5..8e9aae9 100644 (file)
@@ -1446,6 +1446,9 @@ static int load_module(void)
        ast_http_uri_link(&websocketuri);
        websocket_add_protocol_internal("echo", websocket_echo_callback);
 
+       /* For Optional API. */
+       ast_module_shutdown_ref(AST_MODULE_SELF);
+
        return 0;
 }
 
index 3e3611b..aed5a26 100644 (file)
@@ -984,6 +984,9 @@ static int load_module(void)
        ast_manager_register_xml("PauseMonitor", EVENT_FLAG_CALL, pause_monitor_action);
        ast_manager_register_xml("UnpauseMonitor", EVENT_FLAG_CALL, unpause_monitor_action);
 
+       /* For Optional API. */
+       ast_module_shutdown_ref(AST_MODULE_SELF);
+
        return AST_MODULE_LOAD_SUCCESS;
 }
 
index 4f69448..e8d266c 100644 (file)
@@ -1472,6 +1472,10 @@ static int load_module(void)
        }
        ast_cli_register_multiple(cli_pktccops, sizeof(cli_pktccops) / sizeof(struct ast_cli_entry));
        restart_pktc_thread();
+
+       /* For Optional API. */
+       ast_module_shutdown_ref(AST_MODULE_SELF);
+
        return 0;
 }
 
index 4d72401..0edabb8 100644 (file)
@@ -1405,6 +1405,10 @@ static int _unload_module(int fromload)
        }
 
        smdi_loaded = 0;
+
+       /* For Optional API. */
+       ast_module_shutdown_ref(AST_MODULE_SELF);
+
        return 0;
 }
 
index 3d7dd16..aee0bcd 100644 (file)
@@ -353,6 +353,9 @@ static int load_module(void)
                return AST_MODULE_LOAD_DECLINE;
        }
 
+       /* For Optional API. */
+       ast_module_shutdown_ref(AST_MODULE_SELF);
+
        return AST_MODULE_LOAD_SUCCESS;
 }