[asterisk-commits] russell: trunk r83180 - in /trunk: ./ apps/app_system.c

[SVN commits to the Asterisk project]

Author: russell
Date: Wed Sep 19 14:51:47 2007
New Revision: 83180

URL: http://svn.digium.com/view/asterisk?view=rev&rev=83180
Log:
Merged revisions 83179 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r83179 | russell | 2007-09-19 14:50:48 -0500 (Wed, 19 Sep 2007) | 5 lines

The System() and TrySystem() applications can take a substantial amount of
time to execute while not servicing the channel.  So, put the channel in
autoservice while the command is being executed.
(closes issue #10726, reported by mnicholson)

........

Modified:
    trunk/   (props changed)
    trunk/apps/app_system.c

Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-1.4-merged' - no diff available.

Modified: trunk/apps/app_system.c
URL: http://svn.digium.com/view/asterisk/trunk/apps/app_system.c?view=diff&rev=83180&r1=83179&r2=83180
==============================================================================
--- trunk/apps/app_system.c (original)
+++ trunk/apps/app_system.c Wed Sep 19 14:51:47 2007
@@ -79,6 +79,8 @@
 		return failmode;
 	}
 
+	ast_autoservice_start(chan);
+
 	/* Do our thing here */
 	res = ast_safe_system((char *)data);
 	if ((res < 0) && (errno != ECHILD)) {
@@ -98,6 +100,8 @@
 			pbx_builtin_setvar_helper(chan, chanvar, "SUCCESS");
 		res = 0;
 	} 
+
+	ast_autoservice_stop(chan);
 
 	return res;
 }