kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-10-12 15:45:18 +00:00
Code Issues Packages Projects Releases Wiki Activity

app_confbridge: Fix memory leak on reload.

Browse Source 
The config framework options should not be registered multiple times.
Instead the configuration just needs to be reprocessed by the config
framework.
........

Merged revisions 391700 from http://svn.asterisk.org/svn/asterisk/branches/11


git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@391701 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
Richard Mudgett
2013-06-13 19:04:41 +00:00
parent cc06020f23
commit 0e2a9d07ac
3 changed files with 21 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files

26
apps/confbridge/conf_config_parser.c
View File

@@ -1889,12 +1889,10 @@ static int verify_default_profiles(void)
return 0;
}
int conf_load_config(int reload)
int conf_load_config(void)
{
if (!reload) {
if (aco_info_init(&cfg_info)) {
return -1;
}
if (aco_info_init(&cfg_info)) {
return -1;
}
/* User options */
@@ -1944,23 +1942,29 @@ int conf_load_config(int reload)
aco_option_register(&cfg_info, "type", ACO_EXACT, menu_types, NULL, OPT_NOOP_T, 0, 0);
aco_option_register_custom(&cfg_info, "^[0-9A-D*#]+$", ACO_REGEX, menu_types, NULL, menu_option_handler, 0);
if (aco_process_config(&cfg_info, reload) == ACO_PROCESS_ERROR) {
if (aco_process_config(&cfg_info, 0) == ACO_PROCESS_ERROR) {
goto error;
}
if (!reload && ast_cli_register_multiple(cli_confbridge_parser, ARRAY_LEN(cli_confbridge_parser))) {
if (ast_cli_register_multiple(cli_confbridge_parser, ARRAY_LEN(cli_confbridge_parser))) {
goto error;
}
return 0;
error:
/* On a reload, just keep the config we already have in place. */
if (!reload) {
conf_destroy_config();
}
conf_destroy_config();
return -1;
}
int conf_reload_config(void)
{
if (aco_process_config(&cfg_info, 1) == ACO_PROCESS_ERROR) {
/* On a reload, just keep the config we already have in place. */
return -1;
}
return 0;
}
static void conf_user_profile_copy(struct user_profile *dst, struct user_profile *src)
{
*dst = *src;

5
apps/confbridge/include/confbridge.h
View File

@@ -247,7 +247,10 @@ struct confbridge_user {
};
/*! \brief load confbridge.conf file */
int conf_load_config(int reload);
int conf_load_config(void);
/*! \brief reload confbridge.conf file */
int conf_reload_config(void);
/*! \brief destroy the information loaded from the confbridge.conf file*/
void conf_destroy_config(void);