[asterisk-commits] jrose: branch 10 r370495 - in /branches/10: ./ res/res_agi.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Wed Jul 25 16:12:54 CDT 2012


Author: jrose
Date: Wed Jul 25 16:12:50 2012
New Revision: 370495

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=370495
Log:
res_agi: Add message indicating need for \n character in verbose message

The while loop responsible for reading AGI messages from a fastAGI service
can end up looping indefinitely when an AGI script fails to indicate the end
of a message with a \n character. This patch adds an indication that we are
expecting a \n character to end the message to make it more clear to users
that this is necessary if they are receiving this warning over and over.

(issue ASTERISK-20061)
Reported by: Eike Kuiper
........

Merged revisions 370494 from http://svn.asterisk.org/svn/asterisk/branches/1.8

Modified:
    branches/10/   (props changed)
    branches/10/res/res_agi.c

Propchange: branches/10/
------------------------------------------------------------------------------
Binary property 'branch-1.8-merged' - no diff available.

Modified: branches/10/res/res_agi.c
URL: http://svnview.digium.com/svn/asterisk/branches/10/res/res_agi.c?view=diff&rev=370495&r1=370494&r2=370495
==============================================================================
--- branches/10/res/res_agi.c (original)
+++ branches/10/res/res_agi.c Wed Jul 25 16:12:50 2012
@@ -3551,7 +3551,7 @@
 					break;
 				len = sizeof(buf) - buflen;
 				if (agidebug)
-					ast_verbose( "AGI Rx << temp buffer %s - errno %s\n", buf, strerror(errno));
+					ast_verbose("AGI Rx << temp buffer %s - errno %s\nNo \\n received, checking again.\n", buf, strerror(errno));
 			}
 
 			if (!buf[0]) {




More information about the asterisk-commits mailing list