Cleanup T.38 negotiation changes.
Convert LOG_NOTICE messages about T.38 negotiation in debug level 1 messages, clean up some looping logic, and correct an improper use of ast_free() for freeing an ast_frame. git-svn-id: http://svn.digium.com/svn/asterisk/trunk@209279 f38db490-d61c-443f-a65b-d21fe96a405b
This commit is contained in:
parent
930c5befcc
commit
067e38493a
|
@ -394,7 +394,7 @@ static int transmit_audio(fax_session *s)
|
|||
unsigned int timeout = 5000;
|
||||
int ms;
|
||||
|
||||
ast_log(LOG_NOTICE, "Negotiating T.38 for receive on %s\n", s->chan->name);
|
||||
ast_debug(1, "Negotiating T.38 for receive on %s\n", s->chan->name);
|
||||
while (timeout > 0) {
|
||||
ms = ast_waitfor(s->chan, 1000);
|
||||
if (ms < 0) {
|
||||
|
@ -421,9 +421,9 @@ static int transmit_audio(fax_session *s)
|
|||
|
||||
switch (parameters->request_response) {
|
||||
case AST_T38_NEGOTIATED:
|
||||
ast_log(LOG_NOTICE, "Negotiated T.38 for receive on %s\n", s->chan->name);
|
||||
ast_free(inf);
|
||||
return 1;
|
||||
ast_debug(1, "Negotiated T.38 for receive on %s\n", s->chan->name);
|
||||
res = 1;
|
||||
break;
|
||||
case AST_T38_REFUSED:
|
||||
ast_log(LOG_WARNING, "channel '%s' refused to negotiate T.38\n", s->chan->name);
|
||||
break;
|
||||
|
@ -432,7 +432,11 @@ static int transmit_audio(fax_session *s)
|
|||
break;
|
||||
}
|
||||
ast_frfree(inf);
|
||||
break;
|
||||
if (res == 1) {
|
||||
return 1;
|
||||
} else {
|
||||
break;
|
||||
}
|
||||
}
|
||||
ast_frfree(inf);
|
||||
}
|
||||
|
|
Reference in New Issue