[Asterisk-code-review] pbx config: Register manager actions with module version of ... (asterisk[13])

Matt Jordan asteriskteam at digium.com
Tue May 5 21:05:10 CDT 2015


Matt Jordan has submitted this change and it was merged.

Change subject: pbx_config: Register manager actions with module version of macro.
......................................................................


pbx_config: Register manager actions with module version of macro.

Switch manager actions in pbx_config to use the registration macro that
passes the module pointer, allowing pbx_config reference to be bumped
while the manager actions run.

ASTERISK-25061 #close
Reported by: Corey Farrell

Change-Id: I422c50dd74814616ac10c5e9c6598a0b1bc2c44e
---
M pbx/pbx_config.c
1 file changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Matt Jordan: Looks good to me, approved; Verified
  George Joseph: Looks good to me, but someone else must approve



diff --git a/pbx/pbx_config.c b/pbx/pbx_config.c
index 79260bb..b036425 100644
--- a/pbx/pbx_config.c
+++ b/pbx/pbx_config.c
@@ -2080,9 +2080,9 @@
 		ast_cli_register(&cli_dialplan_save);
 	ast_cli_register_multiple(cli_pbx_config, ARRAY_LEN(cli_pbx_config));
 
-	res = ast_manager_register_xml_core(AMI_EXTENSION_ADD,
+	res = ast_manager_register_xml(AMI_EXTENSION_ADD,
 		EVENT_FLAG_SYSTEM, manager_dialplan_extension_add);
-	res |= ast_manager_register_xml_core(AMI_EXTENSION_REMOVE,
+	res |= ast_manager_register_xml(AMI_EXTENSION_REMOVE,
 		EVENT_FLAG_SYSTEM, manager_dialplan_extension_remove);
 
 	if (res) {

-- 
To view, visit https://gerrit.asterisk.org/360
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I422c50dd74814616ac10c5e9c6598a0b1bc2c44e
Gerrit-PatchSet: 1
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-Owner: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: George Joseph <george.joseph at fairview5.com>
Gerrit-Reviewer: Matt Jordan <mjordan at digium.com>



More information about the asterisk-code-review mailing list