[test-results] [Bamboo] Asterisk > Asterisk Unit Tests > #809 has FAILED (4 tests failed, no failures were new). Change made by Corey Farrell and Matt Jordan.

[Bamboo]

-----------------------------------------------------------------------
Asterisk > Asterisk Unit Tests > #809 failed.
-----------------------------------------------------------------------
This build occurred because it is a dependant of AST-ATRUNKFULLBUILD-837.
2/2 jobs failed, with 4 failing tests, no failures were new.

https://bamboo.asterisk.org/bamboo/browse/AST-ATRUNKUNIT-809/

---------------------
Currently Responsible
---------------------

Corey Farrell (Automatically assigned)
John Bigelow (Automatically assigned)
Richard Mudgett (Automatically assigned)



--------------
Failing Jobs
--------------
  - CentOS 6 32-Bit Unit Testing (Unit Testing): 2 of 457 tests failed.
  - CentOS 6 64-Bit Unit Testing (Unit Testing): 2 of 457 tests failed.



--------------
Code Changes
--------------
Matt Jordan (427612):

>addons/chan_mobile: Increase buffer size of UCS2 encoded SMS messages
>
>When UCS2 character encoding is used, one symbol in national language can be
>expanded to 4 bytes. The current buffer used for receiving message in
>do_monitor_phone is 256 bytes, which is not large enough for incoming messages.
>
>For example:
>* AT+CMGR phone response prefix
>  '+CMGR: "REC UNREAD","+7**********",,"14/10/29,13:31:39+12"\r\n' - 60 bytes
>* SMS body with UCS2 encoding (max) - 280 bytes
>* AT+CMGR phone response suffix '\r\n\r\nOK\r\n' - 8 bytes
>* Terminating null character - 1 byte
>
>This results in a needed buffer size of 349 bytes. Hence, this patch opts for a
>350 byte buffer.
>
>ASTERISK-24468 #close
>Reported by: Dmitriy Bubnov
>patches:
>  chan_mobile-1_8.diff uploaded by Dmitriy Bubnov (License 6651)
>  chan_mobile-trunk.diff uploaded by Dmitry Bubnov (License 6651)
>........
>
>Merged revisions 427607 from http://svn.asterisk.org/svn/asterisk/branches/11
>........
>
>Merged revisions 427610 from http://svn.asterisk.org/svn/asterisk/branches/12
>........
>
>Merged revisions 427611 from http://svn.asterisk.org/svn/asterisk/branches/13
>

Matt Jordan (427586):

>app_voicemail: Fix enhancement that allowed multiple recipients in To: header
>
>An issue existed in r420577, which added multiple recipients to voicemail
>emails. The patch, when looking at the intended recipients, looked ahead for
>the '|' character inside a while loop which already had pulled out the
>appropriate field parsing on the '|' character. This would cause it to skip
>the recipients.
>
>This patch fixes it such that it relies completely on the while loop to parse
>through the e-mail fields.
>
>Note that the original author of the patch looked at this fix and approved it.
>
>ASTERISK-24250 #close
>Reported by: abelbeck
>patches:
>  voicemail-420577-to-comma-fix.diff uploaded by abelbeck (License 5903)
>........
>
>Merged revisions 427585 from http://svn.asterisk.org/svn/asterisk/branches/13
>

Matt Jordan (427616):

>channels/chan_mgcp: Fix regression which causes gateways to be skipped
>
>In r227276, a while loop was turned into a for loop. Unfortunately, a portion
>of the while loop was left in the code such that, when a static gateway is
>encountered in the list of MGCP gateways, the next gateway would be skipped.
>At best, we would simply flip past a gateway; at worst, this could lead to a
>crash.
>
>ASTERISK-24500 #close
>Reported by: Xavier Hienne
>patches:
>  chan_mgcp.patch uploaded by Xavier Hienne (License 6657)
>........
>
>Merged revisions 427613 from http://svn.asterisk.org/svn/asterisk/branches/11
>........
>
>Merged revisions 427614 from http://svn.asterisk.org/svn/asterisk/branches/12
>........
>
>Merged revisions 427615 from http://svn.asterisk.org/svn/asterisk/branches/13
>



--------------
Tests
--------------
Existing Test Failures (4)
   - AsteriskUnitTests: /res/parking/dynamic parking variables
   - AsteriskUnitTests: /channels/features/test features channel dtmf
   - AsteriskUnitTests: /res/parking/dynamic parking variables
   - AsteriskUnitTests: /channels/features/test features channel dtmf

--
This message is automatically generated by Atlassian Bamboo
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/test-results/attachments/20141110/09626965/attachment-0001.html>