[asterisk-commits] bbryant: branch 1.6.2 r285639 - in /branches/1.6.2: ./ res/res_musiconhold.c

[SVN commits to the Asterisk project]

Author: bbryant
Date: Thu Sep  9 12:22:25 2010
New Revision: 285639

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

........
  r285638 | bbryant | 2010-09-09 13:20:17 -0400 (Thu, 09 Sep 2010) | 7 lines
  
  Fixes an issue with MOH where it doesn't recover cleanly when it can't play a file and would just stop, instead of continuing to find the next playable file in the MOH class.
  
  (closes issue #17807)
  Reported by: kshumard
  
  Review: https://reviewboard.asterisk.org/r/910/
........

Modified:
    branches/1.6.2/   (props changed)
    branches/1.6.2/res/res_musiconhold.c

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

Modified: branches/1.6.2/res/res_musiconhold.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.6.2/res/res_musiconhold.c?view=diff&rev=285639&r1=285638&r2=285639
==============================================================================
--- branches/1.6.2/res/res_musiconhold.c (original)
+++ branches/1.6.2/res/res_musiconhold.c Thu Sep  9 12:22:25 2010
@@ -294,10 +294,17 @@
 		state->samples = 0;
 	}
 
-	if (!ast_openstream_full(chan, state->class->filearray[state->pos], chan->language, 1)) {
+	for (tries = 0; tries < state->class->total_files; ++tries) {
+		if (ast_openstream_full(chan, state->class->filearray[state->pos], chan->language, 1)) {
+			break;
+		}
+
 		ast_log(LOG_WARNING, "Unable to open file '%s': %s\n", state->class->filearray[state->pos], strerror(errno));
 		state->pos++;
 		state->pos %= state->class->total_files;
+	}
+
+	if (tries == state->class->total_files) {
 		return -1;
 	}