Merged revisions 287308 via svnmerge from

https://origsvn.digium.com/svn/asterisk/branches/1.6.2

................
  r287308 | mnicholson | 2010-09-17 08:36:07 -0500 (Fri, 17 Sep 2010) | 12 lines
  
  Merged revisions 287307 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r287307 | mnicholson | 2010-09-17 08:34:34 -0500 (Fri, 17 Sep 2010) | 5 lines
    
    Use ast_strdup() instead of ast_strdupa() while processing in ast_hint_state_changed().
    
    (related to issue #17928)
    Reported by: mdu113
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@287309 65c4cc65-6c06-0410-ace0-fbb531ad65f3
This commit is contained in:
Matthew Nicholson
2010-09-17 13:37:10 +00:00
parent ada7a489cf
commit c293fa192f

View File

@@ -4225,8 +4225,10 @@ static int handle_statechange(void *datap)
AST_RWLIST_TRAVERSE(&hints, hint, list) { AST_RWLIST_TRAVERSE(&hints, hint, list) {
struct ast_state_cb *cblist; struct ast_state_cb *cblist;
char *parse = ast_strdupa(ast_get_extension_app(hint->exten)); /* can't use ast_strdupa() here because we may run out of stack
char *cur; * space while looping over a large number of large strings */
char *dup = ast_strdup(ast_get_extension_app(hint->exten));
char *cur, *parse = dup;
int state; int state;
while ( (cur = strsep(&parse, "&")) ) { while ( (cur = strsep(&parse, "&")) ) {
@@ -4234,6 +4236,9 @@ static int handle_statechange(void *datap)
break; break;
} }
} }
ast_free(dup);
if (!cur) { if (!cur) {
continue; continue;
} }