[asterisk-commits] mmichelson: trunk r75406 - in /trunk: ./ apps/app_dial.c

[SVN commits to the Asterisk project]

Author: mmichelson
Date: Tue Jul 17 15:05:19 2007
New Revision: 75406

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

........
r75405 | mmichelson | 2007-07-17 15:03:48 -0500 (Tue, 17 Jul 2007) | 6 lines

Fixing an error I made earlier. ast_fileexists can return -1 on failure, so I need to be sure that we only enter the if
statement if it is successful.

Related to my fix to issue #10186


........

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

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

Modified: trunk/apps/app_dial.c
URL: http://svn.digium.com/view/asterisk/trunk/apps/app_dial.c?view=diff&rev=75406&r1=75405&r2=75406
==============================================================================
--- trunk/apps/app_dial.c (original)
+++ trunk/apps/app_dial.c Tue Jul 17 15:05:19 2007
@@ -1917,7 +1917,7 @@
 		if (res == 0) {
 			if (ast_test_flag(&peerflags, OPT_DTMF_EXIT)) {
 				if (!ast_strlen_zero(announce)) {
-					if (ast_fileexists(announce, NULL, chan->language)) {
+					if (ast_fileexists(announce, NULL, chan->language) > 0) {
 						if(!(res = ast_streamfile(chan, announce, chan->language)))								
 							ast_waitstream(chan, AST_DIGIT_ANY);
 					} else
@@ -1930,7 +1930,7 @@
 				}
 			} else {
 				if (!ast_strlen_zero(announce)) {
-					if (ast_fileexists(announce, NULL, chan->language)) {
+					if (ast_fileexists(announce, NULL, chan->language) > 0) {
 						if (!(res = ast_streamfile(chan, announce, chan->language)))
 							res = ast_waitstream(chan, "");
 					} else