[svn-commits] pabelanger: trunk r270936 - in /trunk: CHANGES res/res_agi.c

[SVN commits to the Digium repositories]

Author: pabelanger
Date: Wed Jun 16 13:43:22 2010
New Revision: 270936

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=270936
Log:
MSG_OOB flag on HANGUP packet removed.

Per Tilghman's request on IRC (#asterisk-bugs).

(closes issue #17506)
Reported by: brycebaril
Tested by: pabelanger, tilghman

Modified:
    trunk/CHANGES
    trunk/res/res_agi.c

Modified: trunk/CHANGES
URL: http://svnview.digium.com/svn/asterisk/trunk/CHANGES?view=diff&rev=270936&r1=270935&r2=270936
==============================================================================
--- trunk/CHANGES (original)
+++ trunk/CHANGES Wed Jun 16 13:43:22 2010
@@ -1468,7 +1468,7 @@
 AGI Changes
 -----------
   * Added SPEECH commands for speech recognition. A complete listing can be found
-     using agi show.
+    using agi show.
   * If app_stack is loaded, GOSUB is a native AGI command that may be used to
     invoke subroutines in the dialplan.  Note that calling EXEC with Gosub
     does not behave as expected; the native command needs to be used, instead.
@@ -1477,6 +1477,7 @@
     of the URI parameter to the AGI function call in your dial plan. Also note
     that specifying a port number in the AGI URI will disable SRV lookups,
     even if you use the hagi: protocol.
+  * No longer support MSG_OOB flag on HANGUP.
 
 Logger changes
 --------------

Modified: trunk/res/res_agi.c
URL: http://svnview.digium.com/svn/asterisk/trunk/res/res_agi.c?view=diff&rev=270936&r1=270935&r2=270936
==============================================================================
--- trunk/res/res_agi.c (original)
+++ trunk/res/res_agi.c Wed Jun 16 13:43:22 2010
@@ -814,9 +814,9 @@
 			signals a desire to stop (either by exiting or, in the case of a net script, by
 			closing the connection). A locally executed AGI script will receive SIGHUP on
 			hangup from the channel except when using DeadAGI. A fast AGI server will
-			correspondingly receive a HANGUP in OOB data. Both of these signals may be disabled
-			by setting the <variable>AGISIGHUP</variable> channel variable to <literal>no</literal>
-			before executing the AGI application.</para>
+			correspondingly receive a HANGUP inline with the command dialog. Both of theses
+			signals may be disabled by setting the <variable>AGISIGHUP</variable> channel
+			variable to <literal>no</literal> before executing the AGI application.</para>
 			<para>Use the CLI command <literal>agi show commands</literal> to list available agi
 			commands.</para>
 			<para>This application sets the following channel variable upon completion:</para>
@@ -3345,7 +3345,7 @@
 				if (pid > -1) {
 					kill(pid, SIGHUP);
 				} else if (agi->fast) {
-					send(agi->ctrl, "HANGUP\n", 7, MSG_OOB);
+					send(agi->ctrl, "HANGUP\n", 7, 0);
 				}
 			}
 		}
@@ -3443,7 +3443,7 @@
 			}
 			waitpid(pid, status, WNOHANG);
 		} else if (agi->fast) {
-			send(agi->ctrl, "HANGUP\n", 7, MSG_OOB);
+			send(agi->ctrl, "HANGUP\n", 7, 0);
 		}
 	}
 	fclose(readf);