diff options
author | kpfleming <kpfleming@f38db490-d61c-443f-a65b-d21fe96a405b> | 2009-07-27 21:43:36 +0000 |
---|---|---|
committer | kpfleming <kpfleming@f38db490-d61c-443f-a65b-d21fe96a405b> | 2009-07-27 21:43:36 +0000 |
commit | 067e38493ab79afa9bef3c7a3c0126a3f035aa8e (patch) | |
tree | 95326103faa3e92510c071956e9d79078d9d07b2 /apps/app_fax.c | |
parent | 930c5befcc26901cc059150a8c6a53e5e76f5ba4 (diff) |
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
Diffstat (limited to 'apps/app_fax.c')
-rw-r--r-- | apps/app_fax.c | 14 |
1 files changed, 9 insertions, 5 deletions
diff --git a/apps/app_fax.c b/apps/app_fax.c index c386b7dab..87ef18ed9 100644 --- a/apps/app_fax.c +++ b/apps/app_fax.c @@ -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); } |