[asterisk-commits] jrose: branch 1.8 r370494 - /branches/1.8/res/res_agi.c

[SVN commits to the Asterisk project]

Author: jrose
Date: Wed Jul 25 16:00:00 2012
New Revision: 370494

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=370494
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

Modified:
    branches/1.8/res/res_agi.c

Modified: branches/1.8/res/res_agi.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/res/res_agi.c?view=diff&rev=370494&r1=370493&r2=370494
==============================================================================
--- branches/1.8/res/res_agi.c (original)
+++ branches/1.8/res/res_agi.c Wed Jul 25 16:00:00 2012
@@ -3539,7 +3539,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]) {