kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-09-04 20:04:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge "res_fax.c: Fix deadlock potential in FAXOPT(faxdetect) framehook."

Browse Source
This commit is contained in:
zuul
2016-07-21 19:58:55 -05:00
committed by Gerrit Code Review
parent 7f36b79f87 9abbea162c
commit 3abf482393
1 changed files with 13 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

20
res/res_fax.c
View File

@@ -3700,30 +3700,36 @@ static struct ast_frame *fax_detect_framehook(struct ast_channel *chan, struct a
} }
if (result) { if (result) {
const char *target_context = S_OR(ast_channel_macrocontext(chan), ast_channel_context(chan)); const char *target_context;
switch (result) { switch (result) {
case 'f': case 'f':
case 't': case 't':
target_context = S_OR(ast_channel_macrocontext(chan), ast_channel_context(chan));
ast_channel_unlock(chan); ast_channel_unlock(chan);
ast_frfree(f);
f = &ast_null_frame;
if (ast_exists_extension(chan, target_context, "fax", 1, if (ast_exists_extension(chan, target_context, "fax", 1,
S_COR(ast_channel_caller(chan)->id.number.valid, ast_channel_caller(chan)->id.number.str, NULL))) { S_COR(ast_channel_caller(chan)->id.number.valid, ast_channel_caller(chan)->id.number.str, NULL))) {
ast_channel_lock(chan);
ast_verb(2, "Redirecting '%s' to fax extension due to %s detection\n", ast_verb(2, "Redirecting '%s' to fax extension due to %s detection\n",
ast_channel_name(chan), (result == 'f') ? "CNG" : "T38"); ast_channel_name(chan), (result == 'f') ? "CNG" : "T38");
pbx_builtin_setvar_helper(chan, "FAXEXTEN", ast_channel_exten(chan)); pbx_builtin_setvar_helper(chan, "FAXEXTEN", ast_channel_exten(chan));
if (ast_async_goto(chan, target_context, "fax", 1)) { if (ast_async_goto(chan, target_context, "fax", 1)) {
ast_log(LOG_NOTICE, "Failed to async goto '%s' into fax of '%s'\n", ast_channel_name(chan), target_context); ast_log(LOG_NOTICE, "Failed to async goto '%s' into fax of '%s'\n", ast_channel_name(chan), target_context);
} }
ast_frfree(f);
f = &ast_null_frame;
} else { } else {
ast_channel_lock(chan);
ast_log(LOG_NOTICE, "FAX %s detected but no fax extension in context (%s)\n", ast_log(LOG_NOTICE, "FAX %s detected but no fax extension in context (%s)\n",
(result == 'f') ? "CNG" : "T38", target_context); (result == 'f') ? "CNG" : "T38", target_context);
} }
ast_channel_lock(chan);
ast_framehook_detach(chan, details->faxdetect_id);
details->faxdetect_id = -1;
break;
default:
break;
} }
ast_framehook_detach(chan, details->faxdetect_id);
details->faxdetect_id = -1;
} }
return f; return f;