Commit Graph

5577 Commits

Author SHA1 Message Date
Jeff Peeler
ab6510ebf1 Revert r207636, this approach could potentially block for an unacceptable
amount of time.


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@207783 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-21 17:11:21 +00:00
Kevin P. Fleming
69255bd210 Merged revisions 207680 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r207680 | kpfleming | 2009-07-21 08:28:04 -0500 (Tue, 21 Jul 2009) | 18 lines
  
  Merged revisions 207647 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r207647 | kpfleming | 2009-07-21 08:04:44 -0500 (Tue, 21 Jul 2009) | 12 lines
    
    Ensure that user-provided CFLAGS and LDFLAGS are honored.
    
    This commit changes the build system so that user-provided flags (in ASTCFLAGS
    and ASTLDFLAGS) are supplied to the compiler/linker *after* all flags provided
    by the build system itself, so that the user can effectively override the
    build system's flags if desired. In addition, ASTCFLAGS and ASTLDFLAGS can now
    be provided *either* in the environment before running 'make', or as variable
    assignments on the 'make' command line. As a result, the use of COPTS and LDOPTS
    is no longer necessary, so they are no longer documented, but are still supported
    so as not to break existing build systems that supply them when building Asterisk.
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@207683 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-21 13:39:44 +00:00
Jeff Peeler
55a51b9194 Wait for wink before dialing when using E&M wink signaling
This patch adds a new dahdi_wait function to specifically wait for the wink
event. If the wink is not eventually received the channel is hung up. 

(closes issue #14434)
Reported by: araasch
Patches:
      emwinkmod uploaded by araasch (license 693)


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@207636 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-21 04:38:57 +00:00
Mark Michelson
935f33e481 Merged revisions 207424 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r207424 | mmichelson | 2009-07-20 14:48:12 -0500 (Mon, 20 Jul 2009) | 39 lines
  
  Merged revisions 207423 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r207423 | mmichelson | 2009-07-20 14:39:59 -0500 (Mon, 20 Jul 2009) | 33 lines
    
    Answer video SDP offers properly when videosupport is not enabled.
    
    Copied from Review board:
    
    In issue 12434, the reporter describes a situation in which audio and video 
    is offered on the call, but because videosupport is disabled in sip.conf, 
    Asterisk gives no response at all to the video offer. According to RFC 3264, 
    all media offers should have a corresponding answer. For offers we do not 
    intend to actually reply to with meaningful values, we should still reply 
    with the port for the media stream set to 0.
    
    In this patch, we take note of what types of media have been offered and 
    save the information on the sip_pvt. The SDP in the response will take into 
    account whether media was offered. If we are not otherwise going to answer 
    a media offer, we will insert an appropriate m= line with the port set to 0.
    
    It is important to note that this patch is pretty much a bandage being 
    applied to a broken bone. The patch *only* helps for situations where video 
    is offered but videosupport is disabled and when udptl_pt is disabled but 
    T.38 is offered. Asterisk is not guaranteed to respond to every media offer. 
    Notable cases are when multiple streams of the same type are offered. 
    The 2 media stream limit is still present with this patch, too.
    
    In trunk and the 1.6.X branches, things will be a bit different since Asterisk 
    also supports text in SDPs as well.
    
    (closes issue #12434)
    Reported by: mnnojd
    
    Review: https://reviewboard.asterisk.org/r/311
    Review: https://reviewboard.asterisk.org/r/313
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@207425 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-20 19:55:28 +00:00
Richard Mudgett
c305a6d0a3 Merged revisions 145293,158010 from
https://origsvn.digium.com/svn/asterisk/branches/1.4
to make merging easier.  These changes are already on trunk.

................
  r145293 | rmudgett | 2008-09-30 18:55:24 -0500 (Tue, 30 Sep 2008) | 54 lines

  channels/chan_misdn.c
  channels/misdn/isdn_lib.c
  *  Miscellaneous other fixes from trunk to make merging easier later.

  ........
  r145200 | rmudgett | 2008-09-30 16:00:54 -0500 (Tue, 30 Sep 2008) | 7 lines

  *  Miscellaneous formatting changes to make v1.4 and trunk
  more merge compatible in the mISDN area.

  channels/chan_misdn.c
  *  Eliminated redundant code in cb_events() EVENT_SETUP

  ........
  r144257 | crichter | 2008-09-24 03:42:55 -0500 (Wed, 24 Sep 2008) | 9 lines

  improved helptext of misdn_set_opt.
  ........
  r142181 | rmudgett | 2008-09-09 12:30:52 -0500 (Tue, 09 Sep 2008) | 1 line

  Cleaned up comment

  ........
  r138738 | rmudgett | 2008-08-18 16:07:28 -0500 (Mon, 18 Aug 2008) | 30 lines

  channels/chan_misdn.c
  *  Made bearer2str() use allowed_bearers_array[]
  *  Made use the causes.h defines instead of hardcoded numbers.
  *  Made use Asterisk presentation indicator values if either of the
  mISDN presentation or screen options are negative.
  *  Updated the misdn_set_opt application option descriptions.
  *  Renamed the awkward Caller ID presentation misdn_set_opt
  application option value not_screened to restricted.
  Deprecated the not_screened option value.

  channels/misdn/isdn_lib.c
  *  Made use the causes.h defines instead of hardcoded numbers.
  *  Fixed some spelling errors and typos.
  *  Added all defined facility code strings to fac2str().

  channels/misdn/isdn_lib.h
  *  Added doxygen comments to struct misdn_bchannel.

  channels/misdn/isdn_lib_intern.h
  *  Added doxygen comments to struct misdn_stack.

  channels/misdn_config.c
  configs/misdn.conf.sample
  *  Updated the mISDN presentation and screen parameter descriptions.

  doc/misdn.txt (doc/tex/misdn.tex)
  *  Updated the misdn_set_opt application option descriptions.
  *  Fixed some spelling errors and typos.
................
  r158010 | rmudgett | 2008-11-19 19:46:09 -0600 (Wed, 19 Nov 2008) | 9 lines

  Merged revision 157977 from
  https://origsvn.digium.com/svn/asterisk/team/group/issue8824

  ........
  Fixes JIRA ABE-1726

  The dial extension could be empty if you are using MISDN_KEYPAD
  to control ISDN provider features.
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@207286 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-18 01:35:06 +00:00
Jeff Peeler
08fb833859 Merged revisions 207156 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r207156 | jpeeler | 2009-07-17 14:37:38 -0500 (Fri, 17 Jul 2009) | 14 lines
  
  Merged revisions 207155 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r207155 | jpeeler | 2009-07-17 14:36:19 -0500 (Fri, 17 Jul 2009) | 7 lines
    
    Fix format specifier to print out an unsigned long long.
    
    Yep, it's even ifdefed out code. But it made it to the RR list...
    
    (closes issue #14726)
    Reported by: lmadsen
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@207157 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-17 19:38:54 +00:00
David Vossel
5f6fa4990f Merged revisions 207029 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r207029 | dvossel | 2009-07-17 12:51:44 -0500 (Fri, 17 Jul 2009) | 6 lines
  
  sip option flags handled incorrectly
  
  (closes issue #15376)
  Reported by: Takehiko Ooshima
  Tested by: dvossel, Takehiko_Ooshima
........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@207032 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-17 17:53:50 +00:00
David Vossel
263df0044d Merged revisions 206939 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r206939 | dvossel | 2009-07-17 11:13:22 -0500 (Fri, 17 Jul 2009) | 20 lines
  
  Merged revisions 206938 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r206938 | dvossel | 2009-07-17 11:05:06 -0500 (Fri, 17 Jul 2009) | 14 lines
    
    SIP incorrect From: header information when callpres is prohib
    
    Some ITSP make use of the "Anonymous" display name to detect a
    requirement to withhold caller id across the PSTN. This does
    not work if the display name is "Unknown".
    
    (closes issue #14465)
    Reported by: Nick_Lewis
    Patches:
          chan_sip.c-callerpres.patch uploaded by Nick (license 657)
          chan_sip.c-callerpres_trunk.patch uploaded by dvossel (license 671)
    Tested by: Nick_Lewis, dvossel
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@206947 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-17 16:18:49 +00:00
David Vossel
0faed3d459 Merged revisions 206768 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r206768 | dvossel | 2009-07-15 17:04:13 -0500 (Wed, 15 Jul 2009) | 8 lines
  
  Session timer were not activated if Supported header field in INVITE had both "timer" and other options.
  
  (closes issue #15403)
  Reported by: makoto
  Patches:
        sip-session-timer.patch uploaded by makoto (license
........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@206775 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-15 22:06:36 +00:00
Richard Mudgett
57f664c8f4 Merged revisions 206707 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r206707 | rmudgett | 2009-07-15 16:14:41 -0500 (Wed, 15 Jul 2009) | 33 lines
  
  Merged revisions 206706 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ................
    r206706 | rmudgett | 2009-07-15 15:44:55 -0500 (Wed, 15 Jul 2009) | 26 lines
    
    Merged revision 206700 from
    https://origsvn.digium.com/svn/asterisk/be/branches/C.2-...
    
    ..........
      Fixed chan_misdn crash because mISDNuser library is not thread safe.
    
      With Asterisk the mISDNuser library is driven by two threads concurrently:
      1. channels/misdn/isdn_lib.c::manager_event_handler()
      2. channels/misdn/isdn_lib.c::misdn_lib_isdn_event_catcher()
    
      Calls into the library are done concurrently and recursively from
      isdn_lib.c.
    
      Both threads can fiddle with the master/child layer3_proc_t lists.  One
      thread may traverse the list when the other interrupts it and then removes
      the list element which the first thread was currently handling.  This is
      exactly what caused the crash.  About 60 calls were needed to a Gigaset
      CX475 before it occurred once.
    
      This patch adds locking when calling into the mISDNuser library.
      This also fixes some cb_log calls with wrong port parameter.
    
      JIRA ABE-1913
          Patches: misdn-locking.patch (Modified with mostly cosmetic changes)
    ..........
  ................
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@206762 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-15 21:34:28 +00:00
David Vossel
f84624e23d Merged revisions 206702 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r206702 | dvossel | 2009-07-15 15:20:01 -0500 (Wed, 15 Jul 2009) | 10 lines
  
  callerid(num) is wrong when username is missing 
  
  A domain only sip uri <sip:123.123.123.123> would return
  123.123.123.123 as callid num.  Now, if the username is
  missing from a uri, the callerid num field is left empty.
  
  (closes issue #15476)
  Reported by: viraptor
........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@206705 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-15 20:21:34 +00:00
Richard Mudgett
587c202b8c Merged revisions 206489 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r206489 | rmudgett | 2009-07-14 12:01:48 -0500 (Tue, 14 Jul 2009) | 35 lines
  
  Merged revisions 206487 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r206487 | rmudgett | 2009-07-14 11:44:47 -0500 (Tue, 14 Jul 2009) | 28 lines
    
    Fixes several call transfer issues with chan_misdn.
    
    *  issue #14355 - Crash if attempt to transfer a call to an application.
    Masquerade the other pair of the four asterisk channels involved in the
    two calls.  The held call already must be a bridged call (not an
    applicaton) or it would have been rejected.
    
    *  issue #14692 - Held calls are not automatically cleared after transfer.
    Allow the core to initate disconnect of held calls to the ISDN port.  This
    also fixes a similar case where the party on hold hangs up before being
    transferred or taken off hold.
    
    *  JIRA ABE-1903 - Orphaned held calls left in music-on-hold.
    Do not simply block passing the hangup event on held calls to asterisk
    core.
    
    *  Fixed to allow held calls to be transferred to ringing calls.
    Previously, held calls could only be transferred to connected calls.
    *  Eliminated unused call states to simplify hangup code.
    *  Eliminated most uses of "holded" because it is not a word.
    
    (closes issue #14355)
    (closes issue #14692)
    Reported by: sodom
    Patches:
          misdn_xfer_v14_r205839.patch uploaded by rmudgett (license 664)
    Tested by: rmudgett
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@206555 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-14 18:17:15 +00:00
Russell Bryant
f54c70ea66 Merged revisions 206386 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r206386 | russell | 2009-07-14 09:51:44 -0500 (Tue, 14 Jul 2009) | 20 lines
  
  Merged revisions 206385 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ................
    r206385 | russell | 2009-07-14 09:48:00 -0500 (Tue, 14 Jul 2009) | 13 lines
    
    Merged revisions 206384 via svnmerge from 
    https://origsvn.digium.com/svn/asterisk/branches/1.2
    
    ........
      r206384 | russell | 2009-07-14 09:45:47 -0500 (Tue, 14 Jul 2009) | 6 lines
      
      Ensure apathetic replies are sent out on the proper socket.
      
      chan_iax2 supports multiple address bindings.  The send_apathetic_reply()
      function did not attempt to send its response on the same socket that the
      incoming message came in on.
    ........
  ................
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@206387 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-14 14:54:47 +00:00
Richard Mudgett
3d3e165752 Merged revisions 206341 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r206341 | rmudgett | 2009-07-13 19:48:59 -0500 (Mon, 13 Jul 2009) | 11 lines
  
  Merged revisions 206284 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r206284 | rmudgett | 2009-07-13 19:17:28 -0500 (Mon, 13 Jul 2009) | 4 lines
    
    Fix some memory leaks in chan_misdn.
    
    JIRA ABE-1911
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@206369 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-14 01:25:27 +00:00
David Vossel
23705acc5e Merged revisions 205985 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r205985 | dvossel | 2009-07-10 16:42:10 -0500 (Fri, 10 Jul 2009) | 16 lines
  
  SIP register not using peer's outbound proxy
  
  If callbackextension is defined for a peer it successfully causes
  a registration to occur, but the registration ignores the
  outboundproxy settings for the peer.  This patch allows the
  peer to be passed to obproxy_get() in transmit_register().
  
  (closes issue #14344)
  Reported by: Nick_Lewis
  Patches:
        callbackextension_peer_trunk.diff uploaded by dvossel (license 671)
  Tested by: dvossel
  
  Review: https://reviewboard.asterisk.org/r/294/
........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@206017 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-10 22:50:51 +00:00
Mark Michelson
d2c214e042 Fix build.
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@205880 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-10 17:44:34 +00:00
Mark Michelson
b3c7b4fa2d Merged revisions 205878 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r205878 | mmichelson | 2009-07-10 12:39:57 -0500 (Fri, 10 Jul 2009) | 30 lines
  
  Merged revisions 205877 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ................
    r205877 | mmichelson | 2009-07-10 12:39:13 -0500 (Fri, 10 Jul 2009) | 23 lines
    
    Merged revisions 205776 via svnmerge from 
    https://origsvn.digium.com/svn/asterisk/trunk
    
    ................
      r205776 | mmichelson | 2009-07-10 10:56:45 -0500 (Fri, 10 Jul 2009) | 16 lines
      
      Merged revisions 205775 via svnmerge from 
      https://origsvn.digium.com/svn/asterisk/branches/1.4
      
      ........
        r205775 | mmichelson | 2009-07-10 10:51:36 -0500 (Fri, 10 Jul 2009) | 10 lines
        
        Ensure that outbound NOTIFY requests are properly routed through stateful proxies.
        
        With this change, we make note of Record-Route headers present in any SUBSCRIBE
        request that we receive so that our outbound NOTIFY requests will have the proper
        Route headers in them.
        
        (closes issue #14725)
        Reported by: ibc
      ........
    ................
  ................
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@205879 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-10 17:42:19 +00:00
David Vossel
6e6557cb04 Merged revisions 205840 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r205840 | dvossel | 2009-07-10 11:42:04 -0500 (Fri, 10 Jul 2009) | 37 lines
  
  Merged revisions 205804 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r205804 | dvossel | 2009-07-10 11:23:59 -0500 (Fri, 10 Jul 2009) | 31 lines
    
    SIP registration auth loop caused by stale nonce
    
    If an endpoint sends two registration requests in a very short
    period of time with the same nonce, both receive 401 responses
    from Asterisk, each with a different nonce (the second 401
    containing the current nonce and the first one being stale).
    If the endpoint responds to the first 401, it does not match
    the current nonce so Asterisk sends a third 401 with a newly
    generated nonce (which updates the current nonce)... Now if
    the endpoint responds to the second 401, it does not match the
    current nonce either and Asterisk sends a fourth 401 with a
    newly generated nonce... This loop goes on and on.
    
    There appears to be a simple fix for this.  If the nonce from
    the request does not match our nonce, but is a good response
    to a previous nonce, instead of sending a 401 with a newly
    generated nonce, use the current one instead.  This breaks
    the loop as the nonce is not updated until a response is
    received. Additional logic has been added to make sure no
    nonce can be responded to twice though.
    
    (closes issue #15102)
    Reported by: Jamuel
    Patches:
          patch-bug_0015102 uploaded by Jamuel (license 809)
          nonce_sip.diff uploaded by dvossel (license 671)
    Tested by: Jamuel
    
    Review: https://reviewboard.asterisk.org/r/289/
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@205843 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-10 16:48:56 +00:00
Mark Michelson
966a316fac Merged revisions 205776 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r205776 | mmichelson | 2009-07-10 10:56:45 -0500 (Fri, 10 Jul 2009) | 16 lines
  
  Merged revisions 205775 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r205775 | mmichelson | 2009-07-10 10:51:36 -0500 (Fri, 10 Jul 2009) | 10 lines
    
    Ensure that outbound NOTIFY requests are properly routed through stateful proxies.
    
    With this change, we make note of Record-Route headers present in any SUBSCRIBE
    request that we receive so that our outbound NOTIFY requests will have the proper
    Route headers in them.
    
    (closes issue #14725)
    Reported by: ibc
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@205777 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-10 15:57:08 +00:00
Richard Mudgett
35dbf93676 Merged revisions 205728 via svn merge from
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
  r205728 | rmudgett | 2009-07-09 18:37:53 -0500 (Thu, 09 Jul 2009) | 21 lines
  
  No audio on calls from Asterisk to various ISDN devices until DTMF sent by caller.
  
  Add missing clearing of the dialing flag when the ISDN call is CONNECTED.
  (i.e. When libpri generates the event PRI_EVENT_ANSWER.)
  
  (closes issue #15420)
  Reported by: scottbmilne
  Patches:
        bug15420-1.4.25.1-diff2.txt uploaded by alecdavis (license 585)
  Tested by: scottbmilne, alecdavis
  
  (closes issue #15416)
  Reported by: avinoash
  
  (closes issue #15389)
  Reported by: alecdavis
  
  This patch should also fix the following issue:
  (issue #15205)
  Reported by: vinsik
........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@205729 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-09 23:46:22 +00:00
Kevin P. Fleming
b2e3c3e436 Merged revisions 205696 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r205696 | kpfleming | 2009-07-09 16:20:23 -0500 (Thu, 09 Jul 2009) | 16 lines
  
  Repair ability of SendFAX/ReceiveFAX to respond to T.38 switchover.
  
  Recent changes in T.38 negotiation in Asterisk caused these applications to
  not respond when the other endpoint initiated a switchover to T.38; this
  resulted in the T.38 switchover failing, and the FAX attempt to be made
  using an audio connection, instead of T.38 (which would usually cause the
  FAX to fail completely).
  
  This patch corrects this problem, and the applications will now correctly
  respond to the T.38 switchover request. In addition, the response will include
  the appopriate T.38 session parameters based on what the other end offered
  and what our end is capable of.
  
  (closes issue #14849)
  Reported by: afosorio
........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@205697 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-09 21:26:00 +00:00
David Vossel
f22cf5c484 Merged revisions 205479 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r205479 | dvossel | 2009-07-08 18:19:09 -0500 (Wed, 08 Jul 2009) | 16 lines
  
  Merged revisions 205471 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r205471 | dvossel | 2009-07-08 18:15:54 -0500 (Wed, 08 Jul 2009) | 10 lines
    
    Fixes 8khz assumptions
    
    Many calculations assume 8khz is the codec rate. This
    is not always the case.  This patch only addresses chan_iax.c
    and res_rtp_asterisk.c, but I am sure there are other areas
    that make this assumption as well.
    
    Review: https://reviewboard.asterisk.org/r/306/
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@205597 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-09 15:57:28 +00:00
David Vossel
6547190dd4 SIP Dialog ref counting
This patch adds reference counting for sip dialogs into 1.6.0.
When proc_session_timer() is called from the scheduler thread
it has no guarantee the session timer's dialog won't be freed
from underneath it.  Now the session timer holds a reference
to the dialog, preventing it from being destroyed during the
middle of proc_session_timer().

(closes issue #13623)
Reported by: Nik Soggia

Review: https://reviewboard.asterisk.org/r/302/



git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@205117 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-08 14:35:57 +00:00
Richard Mudgett
0a0144c4a0 Merged revisions 204835 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r204835 | rmudgett | 2009-07-02 17:01:28 -0500 (Thu, 02 Jul 2009) | 17 lines
  
  Merged revisions 204834 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r204834 | rmudgett | 2009-07-02 16:59:43 -0500 (Thu, 02 Jul 2009) | 10 lines
    
    Removed confusing warning message "Got Busy in Connected State"
    
    If an incoming mISDN call is answered with the Answer application and a
    subsequent Dial gets a busy endpoint then it is valid for that already
    connected channel to get the busy indication.  Asterisk will play the busy
    tones until the dialplan plays something else or hangs up the call.
    
    (closes issue #11974)
    Reported by: fvdb
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@204836 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-02 22:03:25 +00:00
David Vossel
8d1643655b removes fake dialog_unref and dialog_ref function calls.
dialog_unref() and dialog_ref() in 1.6.0 where only place holders
for reference counting once it was implemented.  The functions
did nothing but return the pointer on ref and NULL on unref.  These
calls have been removed to make way for a patch that actually does
dialog ref counting.



git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@204652 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-07-01 18:48:50 +00:00
Mark Michelson
ae065d0125 Merged revisions 204301 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r204301 | mmichelson | 2009-06-29 17:50:35 -0500 (Mon, 29 Jun 2009) | 15 lines
  
  Merged revisions 204300 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r204300 | mmichelson | 2009-06-29 17:45:34 -0500 (Mon, 29 Jun 2009) | 9 lines
    
    Add error message so that it is clear why a SIP peer was not processed when
    a DNS lookup fails on a host or outboundproxy.
    
    (closes issue #13432)
    Reported by: p_lindheimer
    Patches:
          outboundproxy.patch uploaded by p (license 558)
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@204302 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-29 22:52:39 +00:00
Mark Michelson
0889af49c6 Merged revisions 204247 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r204247 | mmichelson | 2009-06-29 16:48:54 -0500 (Mon, 29 Jun 2009) | 32 lines
  
  Merged revisions 204243,204246 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r204243 | mmichelson | 2009-06-29 16:23:43 -0500 (Mon, 29 Jun 2009) | 22 lines
    
    Fix a problem where chan_sip would ignore "old" but valid responses.
    
    chan_sip has had a problem for quite a long time that would manifest when
    Asterisk would send multiple SIP responses on the same dialog before receiving
    a response. The problem occurred because chan_sip only kept track of the highest
    outgoing sequence number used on the dialog. If Asterisk sent two requests out,
    and a response arrived for the first request sent, then Asterisk would ignore
    the response. The result was that Asterisk would continue retransmitting the
    requests and ignoring the responses until the maximum number of retransmissions
    had been reached.
    
    The fix here is to rearrange the code a bit so that instead of simply comparing
    the sequence number of the response to our latest outgoing sequence number, we
    walk our list of outstanding packets and determine if there is a match. If there is,
    we continue. If not, then we ignore the response.
    
    In doing this, I found a few completely useless variables that I have now removed.
    
    (closes issue #11231)
    Reported by: flefoll

    Review: https://reviewboard.asterisk.org/r/298
  ........
    r204246 | mmichelson | 2009-06-29 16:37:05 -0500 (Mon, 29 Jun 2009) | 3 lines
    
    Fix build oops.
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@204248 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-29 21:51:27 +00:00
Richard Mudgett
4730b56a50 Merged revisions 203909 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r203909 | rmudgett | 2009-06-26 20:07:52 -0500 (Fri, 26 Jun 2009) | 23 lines
  
  Merged revisions 203908 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r203908 | rmudgett | 2009-06-26 19:55:12 -0500 (Fri, 26 Jun 2009) | 16 lines
    
    The ISDN CPE side should not exclusively pick B channels normally.
    
    Before this patch, Asterisk unconditionally picked B channels exclusively
    on the CPE side and normally allowed alternative B channels on the network
    side.  Now Asterisk does the opposite.
    
    Reasons for the CPE side to normally not pick B channels exclusively:
    *  For CPE point-to-multipoint mode (i.e. phone side), the CPE side does
    not have enough information to exclusively pick B channels.  (There may be
    other devices on the line.)
    *  Q.931 gives preference to the network side picking B channels.
    *  Some telcos require the CPE side to not pick B channels exclusively.
    
    (closes issue #14383)
    Reported by: mbrancaleoni
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@203910 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-27 01:14:18 +00:00
Jeff Peeler
21af4c4acb Merged revisions 203853 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r203853 | jpeeler | 2009-06-26 17:11:31 -0500 (Fri, 26 Jun 2009) | 12 lines
  
  Merged revisions 203848 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r203848 | jpeeler | 2009-06-26 17:09:19 -0500 (Fri, 26 Jun 2009) | 5 lines
    
    Make sure to recreate the dahdi pseudo channel after dahdi restart
    
    (closes issue #14477)
    Reported by: timking
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@203855 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-26 22:12:48 +00:00
Russell Bryant
3be09ad7e9 Merged revisions 203779 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r203779 | russell | 2009-06-26 15:45:00 -0500 (Fri, 26 Jun 2009) | 5 lines
  
  Ensure the TCP read buffer is fully initialized before handling each packet.
  
  (closes issue #14452)
  Reported by: umberto71
........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@203780 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-26 20:46:55 +00:00
Jeff Peeler
cb69985027 reverse whitespace change 203711 that was based on looking at sig_analog (which has about a 1000 line indentation change that is not worth doing here)
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@203717 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-26 19:54:51 +00:00
David Vossel
9fa99fc5b7 Merged revisions 203710 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r203710 | dvossel | 2009-06-26 14:47:11 -0500 (Fri, 26 Jun 2009) | 7 lines
  
  moving debug message from level 0 to 1.
  
  (closes issue #15404)
  Reported by: leobrown
  Patches:
        iax_codec_debug.patch uploaded by leobrown (license 541)
........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@203716 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-26 19:49:33 +00:00
Jeff Peeler
a184348797 whitespace fix
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@203711 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-26 19:47:45 +00:00
Joshua Colp
fc33f7b57e Merged revisions 203699 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r203699 | file | 2009-06-26 16:27:24 -0300 (Fri, 26 Jun 2009) | 2 lines
  
  Improve T.38 negotiation by exchanging session parameters between application and channel.
........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@203701 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-26 19:29:02 +00:00
Jeff Peeler
7f0af9cf29 Merged revisions 203672 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r203672 | jpeeler | 2009-06-26 14:03:25 -0500 (Fri, 26 Jun 2009) | 16 lines
  
  Check if polarityonanswerdelay has elapsed before setting a channel as answered
  after a polarity reversal.
  
  Previously on a polarity switch event chan_dahdi would set the channel
  immediately as answered. This would cause problems if a polarity reversal
  occurred when the line was picked up as the dial would not have yet occurred. 
  Now if the polarity reversal occurs before delay has elapsed after coming off
  hook or an answer, it is ignored. Also, some refactoring was done in
  _handle_event.
  
  (closes issue #13917)
  Reported by: alecdavis
  Patches:
        chan_dahdi.bug13917.feb09.diff2.txt uploaded by alecdavis (license 585)
  Tested by: alecdavis
........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@203698 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-26 19:25:51 +00:00
Russell Bryant
8ee2d538bd Merged revisions 203116 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r203116 | russell | 2009-06-25 11:04:10 -0500 (Thu, 25 Jun 2009) | 18 lines
  
  Merged revisions 203115 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r203115 | russell | 2009-06-25 11:02:16 -0500 (Thu, 25 Jun 2009) | 11 lines
    
    Resolve a crash related to a T.38 reinvite race condition.
    
    This change resolves a crash observed locally during some T.38 testing.
    A call was set up using a call file, and when the T.38 reinvite came in,
    the channel state was still AST_STATE_DOWN.  The reason is explained by
    a comment in the code that previously lived in the handling of
    AST_STATE_RINGING.  This change modifies the logic to handle the same
    race condition for any channel state that is not UP.
    
    (closes ABE-1895)
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@203117 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-25 16:05:36 +00:00
Richard Mudgett
db7b94fafb Merged revisions 203037 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r203037 | rmudgett | 2009-06-24 16:08:55 -0500 (Wed, 24 Jun 2009) | 15 lines
  
  Merged revisions 203036 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r203036 | rmudgett | 2009-06-24 16:01:43 -0500 (Wed, 24 Jun 2009) | 8 lines
    
    Improved chan_dahdi.conf pritimer error checking.
    
    Valid format is: pritimer=timer_name,timer_value
    
    *  Fixed segfault if the ',' is missing.
    *  Completely check the range returned by pri_timer2idx() to prevent
    possible access outside array bounds.
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@203044 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-24 21:18:48 +00:00
Mark Michelson
a228e74faa Merged revisions 202967 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r202967 | mmichelson | 2009-06-24 13:29:10 -0500 (Wed, 24 Jun 2009) | 9 lines
  
  Merged revisions 202966 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r202966 | mmichelson | 2009-06-24 13:28:47 -0500 (Wed, 24 Jun 2009) | 3 lines
    
    Use the handy UNLINK macro instead of hand-coding the same thing in-line.
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@202968 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-24 18:29:43 +00:00
Joshua Colp
03914b9a4a Merged revisions 202925 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r202925 | file | 2009-06-24 15:08:17 -0300 (Wed, 24 Jun 2009) | 2 lines
  
  Ensure the default settings are applied for T.38 when we set it up for a peer.
........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@202926 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-24 18:09:31 +00:00
Matthew Fredrickson
d9ab6e17c1 Merged revisions 202761 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
r202761 | mattf | 2009-06-23 17:08:43 -0500 (Tue, 23 Jun 2009) | 1 line

I could have sworn I committed this patch ages ago, but... bug fix with setting NAI properly on linksets in certain situations.
........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@202763 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-23 22:09:53 +00:00
David Vossel
cbdbf23bdc Merged revisions 202672 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r202672 | dvossel | 2009-06-23 11:31:30 -0500 (Tue, 23 Jun 2009) | 18 lines
  
  Merged revisions 202671 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r202671 | dvossel | 2009-06-23 11:28:46 -0500 (Tue, 23 Jun 2009) | 12 lines
    
    MWI NOTIFY contains a wrong URI if Asterisk listens to non-standard port and transport
    
    (closes issue #14659)
    Reported by: klaus3000
    Patches:
          patch_chan_sip_fixMWIuri_1.4.txt uploaded by klaus3000 (license 65)
          mwi_port-transport_trunk.diff uploaded by dvossel (license 671)
    Tested by: dvossel, klaus3000
    
    Review: https://reviewboard.asterisk.org/r/288/
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@202675 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-23 16:40:15 +00:00
Mark Michelson
b535dda70c Merged revisions 202603 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r202603 | mmichelson | 2009-06-23 10:23:00 -0500 (Tue, 23 Jun 2009) | 8 lines
  
  Blocked revisions 202601 via svnmerge
  
  ........
    r202601 | mmichelson | 2009-06-23 10:22:35 -0500 (Tue, 23 Jun 2009) | 3 lines
    
    Fix more memory leaks that may result if rtp is not successfully allocated.
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@202612 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-23 15:25:03 +00:00
Mark Michelson
436dce6109 Recorded merge of revisions 202574 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r202574 | mmichelson | 2009-06-23 10:11:47 -0500 (Tue, 23 Jun 2009) | 8 lines
  
  Blocked revisions 202572 via svnmerge
  
  ........
    r202572 | mmichelson | 2009-06-23 10:08:27 -0500 (Tue, 23 Jun 2009) | 3 lines
    
    Fix potential memory leak in chan_sip when video rtp is not allocated properly.
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@202577 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-23 15:16:06 +00:00
Russell Bryant
eca12f6e13 Merged revisions 202415 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r202415 | russell | 2009-06-22 11:05:08 -0500 (Mon, 22 Jun 2009) | 9 lines
  
  Merged revisions 202414 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r202414 | russell | 2009-06-22 11:00:00 -0500 (Mon, 22 Jun 2009) | 2 lines
    
    Make Polycom subscription type override check more explicit.
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@202416 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-22 16:06:43 +00:00
Mark Michelson
25b0edc60a Merged revisions 202343 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r202343 | mmichelson | 2009-06-22 09:58:24 -0500 (Mon, 22 Jun 2009) | 36 lines
  
  Merged revisions 202341-202342 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r202341 | mmichelson | 2009-06-22 09:42:55 -0500 (Mon, 22 Jun 2009) | 26 lines
    
    Fix a situation in which Asterisk would not stop retransmitting 487s.
    
    If a CANCEL were received by Asterisk, we would send a 487 in response
    to the original INVITE and a 200 OK for the CANCEL. If there were a network
    hiccup which caused the 200 OK and the 487 to be lost, then the UA communicating
    with Asterisk may try to retransmit its CANCEL. Asterisk's response to this used
    to be to try sending another 487 to the canceled INVITE and another 200 OK to the
    CANCEL.
    
    The problem here is that the originally-sent 487 was sent "reliably" meaning that
    it will be retransmitted until it is received properly. So when we receive the second
    CANCEL it is likely that the first batch of 487s we sent is still going strong and
    reaches the UA. The result was that the second set of 487s would be retransmitted
    constantly until the maximum number of retries had been reached.
    
    The fix for this is that if we receive a second CANCEL for an INVITE, then we cancel
    the retransmission of the first set of 487s and start a second set. This causes the
    dialog to be terminated reasonably.
    
    (closes issue #14584)
    Reported by: klaus3000
    Patches:
          14584_v2.patch uploaded by mmichelson (license 60)
    Tested by: klaus3000
  ........
    r202342 | mmichelson | 2009-06-22 09:44:58 -0500 (Mon, 22 Jun 2009) | 3 lines
    
    Remove an extra debug line left from previous commit.
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@202344 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-22 15:05:00 +00:00
Mark Michelson
03f46e7a81 Merged revisions 202337 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r202337 | mmichelson | 2009-06-22 09:35:09 -0500 (Mon, 22 Jun 2009) | 31 lines
  
  Merged revisions 202336 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r202336 | mmichelson | 2009-06-22 09:34:05 -0500 (Mon, 22 Jun 2009) | 25 lines
    
    Fix a possible infinite loop in SDP parsing during glare situation.
    
    There was a while loop in get_ip_and_port_from_sdp which was controlled
    by a call to get_sdp_iterate. The loop would exit either if what we were
    searching for was found or if the return was NULL. The problem is that
    get_sdp_iterate never returns NULL. This means that if what we were searching
    for was not present, the loop would run infinitely. This modification of the
    loop fixes the problem.
    
    (closes issue #15213)
    Reported by: schmidts
    
    (closes issue #15349)
    Reported by: samy
    
    (closes issue #14464)
    Reported by: pj
    
    (closes issue #15345)
    Reported by: aragon
    Patches:
          sip_inf_loop.patch uploaded by mmichelson (license 60)
    Tested by: aragon
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@202338 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-22 14:35:35 +00:00
Matthew Nicholson
93017afcc8 Added deadlock protection to try_suggested_sip_codec in chan_sip.c.
Review: https://reviewboard.asterisk.org/r/287/


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@202006 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-19 21:07:53 +00:00
David Vossel
ac6ab2899d Merged revisions 201994 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r201994 | dvossel | 2009-06-19 15:24:37 -0500 (Fri, 19 Jun 2009) | 14 lines
  
  Merged revisions 201993 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r201993 | dvossel | 2009-06-19 15:22:02 -0500 (Fri, 19 Jun 2009) | 8 lines
    
    timestamp was being converted to host order as a short rather than a long
    
    (closes issue #15361)
    Reported by: ffloimair
    Patches:
          ts_issue.diff uploaded by dvossel (license 671)
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@201997 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-19 20:27:31 +00:00
David Vossel
986be7d2a2 Merged revisions 201678 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r201678 | dvossel | 2009-06-18 11:37:42 -0500 (Thu, 18 Jun 2009) | 11 lines
  
  fixes some memory leaks and redundant conditions
  
  (closes issue #15269)
  Reported by: contactmayankjain
  Patches:
        patch.txt uploaded by contactmayankjain (license 740)
        memory_leak_stuff.trunk.diff uploaded by dvossel (license 671)
  Tested by: contactmayankjain, dvossel
........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@201682 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-18 16:58:03 +00:00
Mark Michelson
82f2aa293d Merged revisions 201462 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r201462 | mmichelson | 2009-06-17 15:10:01 -0500 (Wed, 17 Jun 2009) | 12 lines
  
  Fix problem with no audio due to ignoring the SDP.
  
  A recent change to our SDP version comparison made audio not function
  on some calls. This was because of a test wherein we were trying to
  see if an unsigned value was less than 0. This is a dumb comparison
  and arguably the compiler should have warned about it. Alas, though,
  it slipped past. Now it's fixed by changing the variable to be a
  signed type.
  
  Found by several developers. Tested by mnicholson and dbrooks.
........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.6.0@201463 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-06-17 20:10:50 +00:00