sysmocom
/
asterisk
11
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

res_fax: Handle fax gateway being started more than once. 

The T.38 fax gateway state machine can cause the fax gateway
to be started more than once on a channel depending on the
responses of the remote endpoint. This would previously leak
the channel name, channel unique id, and underlying fax engine
state. This change instead makes it so that if the fax gateway
session is already present and not reserved the fax gateway
is not started again.

ASTERISK-27981

Change-Id: I552d95086860cb18f2522ee40ef47b13b6da2e0e
This commit is contained in:
Joshua Colp 2018-08-29 07:18:08 -03:00
parent a52b56b4d1
commit 390d0b42ca
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
res/res_fax.c
View File

@ -2914,6 +2914,11 @@ static int fax_gateway_start(struct fax_gateway *gateway, struct ast_fax_session
struct ast_fax_session *s;
int start_res;
/* if the fax gateway is already started then do nothing */
if (gateway->s && gateway->s->state != AST_FAX_STATE_RESERVED) {
return 0;
}
/* create the FAX session */
if (!(s = fax_session_new(details, chan, gateway->s, gateway->token))) {
gateway->token = NULL;