dect
/
asterisk
Archived
13
0
Fork 0
Code Releases Activity

Don't create a Local channel if the target extension does not exist. 

(closes issue #18126)
 Reported by: junky
 Patches: 
       followme.diff uploaded by junky (license 177)
       (partially restructured by me to avoid a possible memory leak)


git-svn-id: http://svn.digium.com/svn/asterisk/branches/1.4@297689 f38db490-d61c-443f-a65b-d21fe96a405b
This commit is contained in:
tilghman 2010-12-07 00:07:37 +00:00
parent 1296b9dcbd
commit 99c687c8c1
1 changed files with 10 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
apps/app_followme.c
View File

@ -801,7 +801,6 @@ static void findmeexec(struct fm_args *tpargs)
break;
while (nm) {
if (option_debug > 1)
ast_log(LOG_DEBUG, "Number %s timeout %ld\n", nm->number,nm->timeout);
@ -815,18 +814,25 @@ static void findmeexec(struct fm_args *tpargs)
rest++;
}
/* We check if that context exists, before creating the ast_channel struct needed */
if (!ast_exists_extension(caller, tpargs->context, number, 1, caller->cid.cid_num)) {
/* XXX Should probably restructure to simply skip this item, instead of returning. XXX */
ast_log(LOG_ERROR, "Extension '%s@%s' doesn't exist\n", number, tpargs->context);
free(findme_user_list);
return;
}
if (!strcmp(tpargs->context, ""))
snprintf(dialarg, sizeof(dialarg), "%s", number);
else
snprintf(dialarg, sizeof(dialarg), "%s@%s", number, tpargs->context);
tmpuser = ast_calloc(1, sizeof(*tmpuser));
if (!tmpuser) {
ast_log(LOG_WARNING, "Out of memory!\n");
free(findme_user_list);
return;
}
outbound = ast_request("Local", ast_best_codec(caller->nativeformats), dialarg, &dg);
if (outbound) {
ast_set_callerid(outbound, caller->cid.cid_num, caller->cid.cid_name, caller->cid.cid_num);