diff options
author | markster <markster@f38db490-d61c-443f-a65b-d21fe96a405b> | 2004-03-19 06:36:25 +0000 |
---|---|---|
committer | markster <markster@f38db490-d61c-443f-a65b-d21fe96a405b> | 2004-03-19 06:36:25 +0000 |
commit | 205ada92871891b9d8ce647da8fa6992639aa099 (patch) | |
tree | c3d39fbccb8e867075f64f0670c65fd4cb0e4598 /pbx.c | |
parent | 4e91b615f042041af68571c6497c66778040a643 (diff) |
Fix LEN segfault (bug #1228)
git-svn-id: http://svn.digium.com/svn/asterisk/trunk@2469 f38db490-d61c-443f-a65b-d21fe96a405b
Diffstat (limited to 'pbx.c')
-rwxr-xr-x | pbx.c | 26 |
1 files changed, 14 insertions, 12 deletions
@@ -751,7 +751,20 @@ static void pbx_substitute_variables_temp(struct ast_channel *c,const char *var, headp=&c->varshead; *ret=NULL; /* Now we have the variable name on cp3 */ - if ((first=strchr(var,':'))) { + if (!strncasecmp(var,"LEN(",4)) { + int len=strlen(var); + int len_len=4; + if (strrchr(var,')')) { + char cp3[80]; + strncpy(cp3, var, sizeof(cp3) - 1); + cp3[len-len_len-1]='\0'; + sprintf(workspace,"%d",strlen(cp3)); + *ret = workspace; + } else { + /* length is zero */ + *ret = "0"; + } + } else if ((first=strchr(var,':'))) { strncpy(tmpvar, var, sizeof(tmpvar) - 1); first = strchr(tmpvar, ':'); if (!first) @@ -929,17 +942,6 @@ static void pbx_substitute_variables_temp(struct ast_channel *c,const char *var, } } } - if (!(*ret) && !strncasecmp(var,"LEN(",4)) { - int len=strlen(var); - int len_len=4; - if (len > (len_len+1) && !strncasecmp(var,"LEN(",len_len) && strchr(var+len_len+2,')')) { - char cp3[80]; - strncpy(cp3, var, sizeof(cp3) - 1); - cp3[len-len_len-1]='\0'; - sprintf(workspace,"%d",strlen(cp3)); - *ret = workspace; - } else ast_log(LOG_NOTICE, "Wrong use of LEN(VARIABLE)\n"); - } } } |