[svn-commits] rmudgett: branch 13 r423992 - in /branches/13: ./ res/res_fax.c

[SVN commits to the Digium repositories]

Author: rmudgett
Date: Fri Sep 26 10:21:14 2014
New Revision: 423992

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=423992
Log:
res_fax: Fix out of bounds error in update_modem_bits().

ASTERISK-24357 #close
Reported by: Jeremy Laine
Patches:
      res_fax_bounds.patch (license #6561) patch uploaded by Jeremy Laine
	  Modified patch to not use magic numbers.
........

Merged revisions 423979 from http://svn.asterisk.org/svn/asterisk/branches/1.8
........

Merged revisions 423983 from http://svn.asterisk.org/svn/asterisk/branches/11
........

Merged revisions 423987 from http://svn.asterisk.org/svn/asterisk/branches/12

Modified:
    branches/13/   (props changed)
    branches/13/res/res_fax.c

Propchange: branches/13/
------------------------------------------------------------------------------
Binary property 'branch-12-merged' - no diff available.

Modified: branches/13/res/res_fax.c
URL: http://svnview.digium.com/svn/asterisk/branches/13/res/res_fax.c?view=diff&rev=423992&r1=423991&r2=423992
==============================================================================
--- branches/13/res/res_fax.c (original)
+++ branches/13/res/res_fax.c Fri Sep 26 10:21:14 2014
@@ -809,7 +809,7 @@
 		m[i] = NULL;
 	} else {
 		tok = strtok(v, ", ");
-		while (tok && (i < 5)) {
+		while (tok && i < ARRAY_LEN(m) - 1) {
 			m[i++] = tok;
 			tok = strtok(NULL, ", ");
 		}