[svn-commits] mmichelson: trunk r149279 - in /trunk: CHANGES apps/app_dial.c

[SVN commits to the Digium repositories]

Author: mmichelson
Date: Tue Oct 14 18:57:46 2008
New Revision: 149279

URL: http://svn.digium.com/view/asterisk?view=rev&rev=149279
Log:
When specifying an invalid timeout to Dial, take it
to mean that no timeout is desired.

(closes issue #13625)
Reported by: atis


Modified:
    trunk/CHANGES
    trunk/apps/app_dial.c

Modified: trunk/CHANGES
URL: http://svn.digium.com/view/asterisk/trunk/CHANGES?view=diff&rev=149279&r1=149278&r2=149279
==============================================================================
--- trunk/CHANGES (original)
+++ trunk/CHANGES Tue Oct 14 18:57:46 2008
@@ -145,6 +145,8 @@
    'UNSUPPORTED'.  This change makes SendImage() more consistent with other
    applications.
  * Park has a new option, 's', which silences the announcement of the parking space number.
+ * A non-numeric, zero, or negative timeout specified to Dial() will now be interpreted as
+   invalid input and will be assumed to mean that no timeout is desired.
 
 SIP Changes
 -----------

Modified: trunk/apps/app_dial.c
URL: http://svn.digium.com/view/asterisk/trunk/apps/app_dial.c?view=diff&rev=149279&r1=149278&r2=149279
==============================================================================
--- trunk/apps/app_dial.c (original)
+++ trunk/apps/app_dial.c Tue Oct 14 18:57:46 2008
@@ -1589,8 +1589,10 @@
 		to = atoi(args.timeout);
 		if (to > 0)
 			to *= 1000;
-		else
-			ast_log(LOG_WARNING, "Invalid timeout specified: '%s'\n", args.timeout);
+		else {
+			ast_log(LOG_WARNING, "Invalid timeout specified: '%s'. Setting timeout to infinite\n", args.timeout);
+			to = -1;
+		}
 	}
 
 	if (!outgoing) {