[asterisk-commits] mjordan: branch 11 r379179 - in /branches/11: ./ addons/chan_mobile.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Tue Jan 15 22:13:37 CST 2013


Author: mjordan
Date: Tue Jan 15 22:13:33 2013
New Revision: 379179

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=379179
Log:
Fix parsing SMSSRC for SMS messages

The parser for SMS messages would incorrectly parse out the from number.
The parsing would incorrectly start scanning for the from number at the
same index as the first double quote ("); this would inadvertently cause
it to treat the first double quote as the terminating double quote for
the from number as well.

The SMSSRC should now populate correctly.

(closes issue ASTERISK-16822)
Reported by: menschentier
Tested by: Jonas Falck
patches:
 fixSMSSRC.patch uploaded by jonax (license 6320)

(closes issue ASTERISK-19153)
Reported by: Panos Gkikakis
patches:
  sms-sender-fix.diff uploaded by roeften (license 5884)  
........

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

Modified:
    branches/11/   (props changed)
    branches/11/addons/chan_mobile.c

Propchange: branches/11/
------------------------------------------------------------------------------
--- branch-1.8-merged (original)
+++ branch-1.8-merged Tue Jan 15 22:13:33 2013
@@ -1,1 +1,1 @@
-/branches/1.8:1-378147,378164,378356,378375,378427,378455-378456,378486,378514,378554,378591,378733,378776,378933,378967,379001,379145
+/branches/1.8:1-378147,378164,378356,378375,378427,378455-378456,378486,378514,378554,378591,378733,378776,378933,378967,379001,379145,379178

Modified: branches/11/addons/chan_mobile.c
URL: http://svnview.digium.com/svn/asterisk/branches/11/addons/chan_mobile.c?view=diff&rev=379179&r1=379178&r2=379179
==============================================================================
--- branches/11/addons/chan_mobile.c (original)
+++ branches/11/addons/chan_mobile.c Tue Jan 15 22:13:33 2013
@@ -2229,6 +2229,7 @@
 			if (buf[i] == '"') {
 				state++;
 			}
+			break;
 		case 2: /* mark the start of the number */
 			if (from_number) {
 				*from_number = &buf[i];




More information about the asterisk-commits mailing list