Index: apps/confbridge/include/confbridge.h =================================================================== --- apps/confbridge/include/confbridge.h (revision 404578) +++ apps/confbridge/include/confbridge.h (working copy) @@ -31,7 +31,7 @@ #include "conf_state.h" /* Maximum length of a conference bridge name */ -#define MAX_CONF_NAME 32 +#define MAX_CONF_NAME AST_MAX_EXTENSION /* Maximum length of a conference pin */ #define MAX_PIN 80 Index: apps/app_confbridge.c =================================================================== --- apps/app_confbridge.c (revision 404578) +++ apps/app_confbridge.c (working copy) @@ -1580,17 +1580,23 @@ ast_answer(chan); } - if (ast_strlen_zero(data)) { + /* We need to make a copy of the input string if we are going to modify it! */ + parse = ast_strdupa(data); + + AST_STANDARD_APP_ARGS(args, parse); + + if (ast_strlen_zero(args.conf_name)) { ast_log(LOG_WARNING, "%s requires an argument (conference name[,options])\n", app); res = -1; goto confbridge_cleanup; } - /* We need to make a copy of the input string if we are going to modify it! */ - parse = ast_strdupa(data); + if (strlen(args.conf_name) >= MAX_CONF_NAME) { + ast_log(LOG_WARNING, "%s does not accept conference names longer than %d\n", app, MAX_CONF_NAME - 1); + res = -1; + goto confbridge_cleanup; + } - AST_STANDARD_APP_ARGS(args, parse); - /* bridge profile name */ if (args.argc > 1 && !ast_strlen_zero(args.b_profile_name)) { b_profile_name = args.b_profile_name;