[test-results] [Bamboo] Asterisk - 1.6.2 - Linux - x86_64 build 136 has FAILED (1 tests failed). Change made by 9 authors

[Bamboo]

-------------- next part --------------
-----------------------------------------------------------
AST-162-136 failed.
-----------------------------------------------------------
Code has been updated by twilson, Russell Bryant, rmudgett, alecdavis, bbryant, Paul Belanger, dvossel, Matthew Nicholson, Tilghman Lesher.
1/59 tests failed.

http://bamboo.asterisk.org/browse/AST-162-136/        


--------------
Code Changes
--------------
rmudgett (288193):

>Merged revisions 288192 via svnmerge from 
>https://origsvn.digium.com/svn/asterisk/branches/1.4
>
>........
>  r288192 | rmudgett | 2010-09-21 18:55:58 -0500 (Tue, 21 Sep 2010) | 26 lines
>  
>  In chan_iax2.c:schedule_delivery() calls ast_bridged_channel() on an unlocked channel.
>  
>  Near the beginning of schedule_delivery(), ast_bridged_channel() is called
>  on iaxs[fr->callno]->owner.  However, the channel is not locked, which can
>  result in ast_bridged_channel() crashing should owner->tech change to a
>  technology that doesn't implement bridged_channel.
>  
>  I also fixed the other calls to ast_bridged_channel() in chan_iax2.c since
>  the owner lock was not held there either.
>  
>  Converted the existing channel deadlock avoidance to use
>  iax2_lock_owner().  Using the new function simplified some awkward code.
>  
>  In the process of fixing the locking on ast_bridged_channel(), I also
>  found a memory leak in socket_process() for v1.6.2 and v1.8.  The local
>  struct variable ies.vars is not freed on early/abnormal function exits.
>  
>  (closes issue #17919)
>  Reported by: rain
>  Patches:
>        issue17919_v1.4.patch uploaded by rmudgett (license 664)
>        issue17919_w_leak_v1.6.2.patch uploaded by rmudgett (license 664)
>        issue17919_w_leak_v1.8.patch uploaded by rmudgett (license 664)
>  
>  Review: https://reviewboard.asterisk.org/r/926/
>........
>

bbryant (287759):

>Merged revisions 287758 via svnmerge from 
>https://origsvn.digium.com/svn/asterisk/branches/1.4
>
>........
>  r287758 | bbryant | 2010-09-20 19:57:08 -0400 (Mon, 20 Sep 2010) | 16 lines
>  
>  Fix misvalidation of meetme pins in conjunction with the 'a' MeetMe flag.
>  
>  When using the 'a' MeetMe flag and having a user and admin pin setup for your
>  conference, using the user pin would gain you admin priviledges. Also, when no
>  user pin was set, an admin pin was, the 'a' MeetMe flag wasn't used, and the
>  user tried to enter a conference then they were still prompted for a pin and
>  forced to hit #.
>  
>  (closes issue #17908)
>  Reported by: kuj
>  Patches:
>        pins_2.patch uploaded by kuj (license 1111)
>        Tested by: kuj
>  
>        Review: [full review board URL with trailing slash]
>........
>

Tilghman Lesher (288637):

>Merged revisions 288636 via svnmerge from 
>https://origsvn.digium.com/svn/asterisk/branches/1.4
>
>........
>  r288636 | tilghman | 2010-09-23 22:20:24 -0500 (Thu, 23 Sep 2010) | 2 lines
>  
>  Solaris compatibility fixes
>........
>


--------------
Tests
--------------
Failed Tests (1)
   - AsteriskTestSuite: Feature attended transfer (New)


--
This message is automatically generated by Atlassian Bamboo

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.digium.com/pipermail/test-results/attachments/20100927/3ac798e5/attachment.htm