Index: apps/app_confbridge.c =================================================================== --- apps/app_confbridge.c (revision 404550) +++ apps/app_confbridge.c (working copy) @@ -149,7 +149,7 @@ }); /* Maximum length of a conference bridge name */ -#define MAX_CONF_NAME 32 +#define MAX_CONF_NAME AST_MAX_EXTENSION /* Number of buckets our conference bridges container can have */ #define CONFERENCE_BRIDGE_BUCKETS 53 @@ -735,16 +735,21 @@ AST_APP_ARG(options); ); - 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); return -1; } - /* 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); + return -1; + } - AST_STANDARD_APP_ARGS(args, parse); - if (args.argc == 2) { ast_app_parse_options(app_opts, &conference_bridge_user.flags, conference_bridge_user.opt_args, args.options); }