Merged revisions 372391 via svnmerge from

file:///srv/subversion/repos/asterisk/branches/10

................
  r372391 | mjordan | 2012-09-05 19:56:47 -0500 (Wed, 05 Sep 2012) | 24 lines
  
  Allow configured numbers for FollowMe to be greater than 90 characters
  
  When parsing a 'number' defined in followme.conf, FollowMe previously parsed
  the number in the configuration file into a buffer with a length of 90
  characters.  This can artificially limit some parallel dial scenarios.  This
  patch allows for numbers of any length to be defined in the configuration
  file.
  
  Note that Clod Patry originally wrote a patch to fix this problem and received
  a Ship It! on the JIRA issue.  The patch originally expanded the buffer to 256
  characters.  Instead, the patch being committed duplicates the string in the
  config file on the stack before parsing it for consumption by the application.
  
  (closes issue ASTERISK-16879)
  Reported by: Clod Patry
  Tested by: mjordan
  patches:
    followme_no_limit.diff uploaded by Clod Patry (license #5138)
  
  Slightly modified for this commit.
  ........
  
  Merged revisions 372390 from http://svn.asterisk.org/svn/asterisk/branches/1.8
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/10-digiumphones@372416 65c4cc65-6c06-0410-ace0-fbb531ad65f3
10-digiumphones
Automerge script 14 years ago
parent 827d446b4b
commit cf2a598cab

@ -351,7 +351,7 @@ static int reload_followme(int reload)
char *cat = NULL, *tmp;
struct ast_variable *var;
struct number *cur, *nm;
char numberstr[90];
char *numberstr;
int timeout;
int numorder;
const char *takecallstr;
@ -467,7 +467,7 @@ static int reload_followme(int reload)
int idx = 0;
/* Add a new number */
ast_copy_string(numberstr, var->value, sizeof(numberstr));
numberstr = ast_strdupa(var->value);
if ((tmp = strchr(numberstr, ','))) {
*tmp++ = '\0';
timeout = atoi(tmp);

Loading…
Cancel
Save