Merged revisions 324557 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r324557 | twilson | 2011-06-22 22:10:38 -0500 (Wed, 22 Jun 2011) | 5 lines

  Remove tests for parsing address with invalid port

  getaddrinfo on OS X returns with EAI_NONAME error when passed a port
  greater than 65535. Linux throws no error, so remove the tests for now.
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@324558 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 324491 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r324491 | rmudgett | 2011-06-22 14:16:29 -0500 (Wed, 22 Jun 2011) | 1 line

  Use correct variable for text SRTP media.
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@324495 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 324484 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r324484 | twilson | 2011-06-22 13:52:04 -0500 (Wed, 22 Jun 2011) | 20 lines

  Stop sending IPv6 link-local scope-ids in SIP messages

  The idea behind the patch listed below was used, but in a more targeted manner.
  There are now address stringification functions for addresses that are meant to
  be sent to a remote party. Link-local scope-ids only make sense on the machine
  from which they originate and so are stripped in the new functions.

  There is also a host sanitization function added to chan_sip which is used
  for when peer and dialog tohost fields or sip_registry hostnames are used to
  craft a SIP message.

  Also added are some basic unit tests for netsock2 address parsing.

  (closes issue ASTERISK-17711)
  Reported by: ch_djalel
  Patches:
        asterisk-1.8.3.2-ipv6_ll_scope.patch uploaded by ch_djalel (license 1251)

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@324487 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 324481 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

Also fixed a reference leak in an error path in sip_msg_send().

........
  r324481 | rmudgett | 2011-06-22 13:41:20 -0500 (Wed, 22 Jun 2011) | 19 lines

  Timout or error on INFO or MESSAGE transaction causes call to be lost.

  When exchanging INFO messages within a call, 4xx error causes the call to
  be disconnected although RFC 2976 explicitly states that such transactions
  do not modify the state of the dialog.

  When exchanging MESSAGE messages within a call, 4xx error causes the call
  to be disconnected.  To provide least surprise, we should not disconnect
  the call since a MESSAGE is like INFO in this case.  (Implied by RFC 3428
  Section 2)

  (closes issue ASTERISK-17901)
  Reported by: neutrino88

  Review: https://reviewboard.asterisk.org/r/1257/
  Review: https://reviewboard.asterisk.org/r/1258/

  JIRA SWP-3486
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@324482 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 324479 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r324479 | rmudgett | 2011-06-22 13:26:55 -0500 (Wed, 22 Jun 2011) | 1 line

  Comments and whitespace in chan_sip.c
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@324480 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Fixes issue with channel write format being incorrectly restored when MOH is used in confbridge.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@324422 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 324364 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r324364 | dvossel | 2011-06-21 15:11:52 -0500 (Tue, 21 Jun 2011) | 10 lines

  Fixes locking inversion issue in ast_async_goto()

  During this function we can not hold the "chan" lock while
  doing the masquerade, the explicit goto on the tmp chan, or
  the channel alloc.  Instead we need to get the channel lock,
  store off information about the channel that we need, and
  then let the channel lock go for the remainder of the function.

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@324365 65c4cc65-6c06-0410-ace0-fbb531ad65f3

ConfBridge does not handle hangup properly

When playing back a prompt to a channel, confbridge neglects to check for
hangup events causing lockup condititions for hangups that occur before
actually joining the conference.  This change ensures that the user is removed
from the conference in the event of a premature hangup.

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@324304 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Fixes issue with finding correct extension when message context is used.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@324302 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 324241 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r324241 | lmadsen | 2011-06-20 13:12:32 -0500 (Mon, 20 Jun 2011) | 2 lines

  Remove extra 'the'.
  Reported by Vlad Povorozniuc
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@324242 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 324237 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r324237 | twilson | 2011-06-20 12:33:07 -0500 (Mon, 20 Jun 2011) | 12 lines

  Ignore media offers with a port of 0

  Section 5.1 of RFC3264 states:
    A port number of zero in the offer indicates that the stream is offered
    but MUST NOT be used.

  (closes issue ASTERISK-17845)
  Reported by: jacco
  Patches:
        issue19281_2.patch uploaded by jacco (license 1277)
  Tested by: jacco, twilson
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@324238 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 324178 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r324178 | lmadsen | 2011-06-17 14:51:16 -0400 (Fri, 17 Jun 2011) | 2 lines

  Add Username and Secret fields to manager Login action.
  Pointed out by Vlad Povorozniuc
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@324179 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 324176 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r324176 | lmadsen | 2011-06-17 14:38:40 -0400 (Fri, 17 Jun 2011) | 2 lines

  Fix typo in documentation.
  Pointed out by Vlad Povorozniuc
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@324177 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 324174 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r324174 | rmudgett | 2011-06-17 13:23:19 -0500 (Fri, 17 Jun 2011) | 5 lines

  Add header string to libpri debug output.

  Add header string to libpri debug output so the libpri output can be
  found/extracted easier from huge debug trace files.
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@324175 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 324115 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r324115 | lmadsen | 2011-06-17 11:14:54 -0400 (Fri, 17 Jun 2011) | 3 lines

  Fix grammar in documentation for Goto() and GotoIf()
  (closes issue ASTERISK-18023)
  Reported by: Tim Osman
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@324131 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 324048 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r324048 | twilson | 2011-06-16 17:35:41 -0500 (Thu, 16 Jun 2011) | 8 lines

  Lock the channel before calling the setoption callback

  The channel needs to be locked before calling these callback functions. Also,
  sip_setoption needs to lock the pvt and a check p->rtp is non-null before using
  it.

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@324050 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 323990 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r323990 | rmudgett | 2011-06-16 13:12:32 -0500 (Thu, 16 Jun 2011) | 5 lines

  The test_event unit test is occasionally failing.

  Wait for the special posted event to process before adding a new
  subscription.
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323991 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 323932 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r323932 | twilson | 2011-06-16 10:58:22 -0500 (Thu, 16 Jun 2011) | 4 lines

  Don't assume ASTDBDIR exists

  It most likely doesn't on FreeBSD
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323933 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 323866 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r323866 | twilson | 2011-06-15 15:03:58 -0500 (Wed, 15 Jun 2011) | 2 lines

  Remove now-useless cast of ARRAY_LEN
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323867 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 323863 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r323863 | twilson | 2011-06-15 14:58:18 -0500 (Wed, 15 Jun 2011) | 2 lines

  Make ARRAY_LEN() return the same type on x86 and x86_64 systems
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323864 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 323859 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r323859 | twilson | 2011-06-15 14:45:20 -0500 (Wed, 15 Jun 2011) | 2 lines

  Fix more ARRAY_LEN format string issues
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323860 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 323754 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

................
  r323754 | twilson | 2011-06-15 13:21:52 -0500 (Wed, 15 Jun 2011) | 23 lines

  Merged revisions 323733 via svnmerge from
  https://origsvn.digium.com/svn/asterisk/branches/1.6.2

  ................
    r323733 | twilson | 2011-06-15 13:13:00 -0500 (Wed, 15 Jun 2011) | 16 lines

    Merged revisions 323732 via svnmerge from
    https://origsvn.digium.com/svn/asterisk/branches/1.4

    ........
      r323732 | twilson | 2011-06-15 13:06:24 -0500 (Wed, 15 Jun 2011) | 9 lines

      Fix DYNAMIC_FEATURES

      DYNAMIC_FEATURES were broken by a recent DTMF change. This patch makes
      sure that dynamic features are also checked when deciding whether or not
      to pass DTMF through or store it for interpreting.

      (closes issue ASTERISK-17914)
      Reported by: vrban
    ........
  ................
................

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323760 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Blocked revisions 323730 via svnmerge

........
  r323730 | jrose | 2011-06-15 12:42:42 -0500 (Wed, 15 Jun 2011) | 11 lines

  Adds locking to find_table in res_configure_pgsql to prevent a crash.

  Bryonclark described the problem as occuring during this function because of multiple
  simultaneous database operations causing corruption against a pgsqlConn object.

  (closes issue ASTERISK-17811)
  Reported by: byronclark
  Patches:
        pgsql_find_table_locking.patch uploaded by byronclark (license 1200)
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323731 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 323672 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r323672 | twilson | 2011-06-15 10:09:51 -0700 (Wed, 15 Jun 2011) | 5 lines

  Cast ARRAY_LEN to size_t for ast_logging

  32-bit and 64-bit machines return different types for ARRAY_LEN(), so cast
  it before using in a format string.
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323673 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 323669-323670 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r323669 | rmudgett | 2011-06-15 11:43:18 -0500 (Wed, 15 Jun 2011) | 21 lines

  [regression] Voicemail MWI is no longer sent.

  When leaving a voicemail, the MWI message is never sent.  The same thing
  happens when checking a voicemail and marking it as read.

  If you restart Asterisk, everything comes up at that state correctly, but
  changes to the messages in voicemail causes the light to not be set
  appropriately.  Very easy to reproduce.

  * Made ast_event_check_subscriber() return TRUE if there are ANY
  subscribers to an event type when there are no restricting ie values
  passed.  This allows an event being queued to be queued.

  (closes issue ASTERISK-18002)
  Reported by: lmadsen
  Tested by: lmadsen, irroot
  Patches:
       jira_asterisk_18002_v1.8.patch uploaded by rmudgett (License #5621)

  (closes issue ASTERISK-18019)
........
  r323670 | rmudgett | 2011-06-15 11:43:31 -0500 (Wed, 15 Jun 2011) | 7 lines

  Add a test to the event unit tests to catch ASTERISK-18002.

  The new tests check to see if there are ANY subscribers to the event type
  when ast_event_check_subscriber() is not passed any specific ie values.

  (issue ASTERISK-18002)
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323671 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 323610 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r323610 | jrose | 2011-06-15 11:09:24 -0500 (Wed, 15 Jun 2011) | 7 lines

  Adds PQclear calls on result to various parts of res_conf_pgsql

  (closes issue ASTERISK-17812)
  Reported by: byronclark
  Patches:
        pgsql_pqclear.patch uploaded by byronclark (license 1200)
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323621 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 323608 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

................
  r323608 | seanbright | 2011-06-15 11:31:53 -0400 (Wed, 15 Jun 2011) | 39 lines

  Merged revisions 323579 via svnmerge from
  https://origsvn.digium.com/svn/asterisk/branches/1.6.2

  ................
    r323579 | seanbright | 2011-06-15 11:22:50 -0400 (Wed, 15 Jun 2011) | 32 lines

    Merged revisions 323559 via svnmerge from
    https://origsvn.digium.com/svn/asterisk/branches/1.4

    ........
      r323559 | seanbright | 2011-06-15 11:15:30 -0400 (Wed, 15 Jun 2011) | 25 lines

      Resolve a segfault/bus error when we try to map memory that falls on a page
      boundary.

      The fix for ASTERISK-15359 was incorrect in that it added 1 to the length of the
      mmap'd region.  The problem with this is that reading/writing to that extra byte
      outside of the bounds of the underlying fd causes a bus error.

      The real issue is that we are working with both a FILE * and the raw fd
      underneath it and not synchronizing between them.  The code that was removed in
      ASTERISK-15359 was correct, but we weren't flushing the FILE * before mapping
      the fd.

      Looking at the manager code in 1.4 reveals that the FILE * in 'struct
      mansession' is never used except to create a temporary file that we immediately
      fdopen.  This means we just need to write a 0 byte to the fd and everything will
      just work.  The other branches require a call to fflush() which, while not a
      guaranteed fix, should reduce the likelihood of a crash.

      This all makes sense in my head.

      (closes issue ASTERISK-16460)
      Reported by: Ravelomanantsoa Hoby (hoby)
      Patches:
       issue17747_1.4_svn_markII.patch uploaded by Sean Bright (license #5060)
    ........
  ................
................

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323609 65c4cc65-6c06-0410-ace0-fbb531ad65f3

CONFBRIDGE_INFO function to get conference data

Added the CONFBRIDGE_INFO dialplan function to get information about a
conference bridge including locked status and number of parties, admins, and
marked users.

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323517 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 323456 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r323456 | rmudgett | 2011-06-14 19:50:20 -0500 (Tue, 14 Jun 2011) | 1 line

  Add missing break in ast_event_get_cached().
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323457 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 323392,323394 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r323392 | rmudgett | 2011-06-14 12:21:24 -0500 (Tue, 14 Jun 2011) | 6 lines

  Add more strict hostname checking to ast_dnsmgr_lookup().

  Change suggested in review.

  Review: https://reviewboard.asterisk.org/r/1240/
........
  r323394 | rmudgett | 2011-06-14 12:21:39 -0500 (Tue, 14 Jun 2011) | 2 lines

  Made ast_sockaddr_split_hostport() port warning msgs more meaningful.
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323397 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 323370 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r323370 | twilson | 2011-06-14 09:33:55 -0700 (Tue, 14 Jun 2011) | 10 lines

  Add rtpkeepalives back to 1.8

  The RTP-engine conversion left out support for handling rtpkeepalives.
  This patch adds them back.

  (closes issue ASTERISK-17304)
  Reported by: lmadsen

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323374 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 323371 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r323371 | jrose | 2011-06-14 11:38:43 -0500 (Tue, 14 Jun 2011) | 12 lines

  Changes contact use in build_peer to use the FORCE_RPORT flag instead of RPORT_PRESENT

  It turned out that this was causing NAT=Yes to always use rport when present which was
  against 1.6.2 behavior and the check itself was redundant since the only way this
  segment of code could be reached was if RPORT_PRESENT was already evaluated as true
  earlier.

  (closes issue ASTERISK-17789)
  Reported by: byronclark
  Patches:
        use_sip_nat_force_rport.patch uploaded by byronclark (license 1200)
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323372 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Store sip peer name as var data on a outofcall msg.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323325 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Config inheritance doesn't work with ConfBridge() menu definitions

Current behavior in ConfBridge menu definitions is that first definition takes
precedence, even in templated situations.  This change allows inheritance and
overriding to work as expected so that the last definition takes precedence.

(closes ASTERISK-17986)
Review: https://reviewboard.asterisk.org/r/1267/

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323272 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 323213 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r323213 | lmadsen | 2011-06-13 15:51:52 -0400 (Mon, 13 Jun 2011) | 6 lines

  Avoid dividing by zero with L() option to Dial()

  Reported by: nicolasom
  Patches:

  issue-17995.patch - nicolasom (License #5994)
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323214 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Addition of "outofcall_message_context" sip.conf option.

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323212 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 323154 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r323154 | lmadsen | 2011-06-13 15:00:41 -0400 (Mon, 13 Jun 2011) | 6 lines

  Tweak documentation for AGI Hangup command.

  (closes issue ASTERISK-17999)
  Reported by: Ben Klang
  Patches:
       hangup-doc.diff - uploaded by Ben Klang (License #5876)
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323155 65c4cc65-6c06-0410-ace0-fbb531ad65f3

MOH for only user not working with ConfBridge

This adds the playing_moh flag to the conference_bridge_user struct that
signifies when MOH should be playing so code doesn't have to guess whether
MOH is playing.

This change also adds the necessary checking to ensure that MOH continues
playing for a single user in a conference after the join sound is played when
configured to do so.

(closes ASTERISK-17988)
Review: https://reviewboard.asterisk.org/r/1263/

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323107 65c4cc65-6c06-0410-ace0-fbb531ad65f3

ConfBridge: Use of bridge or user profiles that don't exist

Bridge and user profiles are not checked for existence before use.  The lack
of a fully formed bridge profile can cause a segfault when sounds are accessed.
This change ensures that bridge and user profiles exist prior to usage
attempts.

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323106 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 323040 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r323040 | mnicholson | 2011-06-10 14:20:41 -0500 (Fri, 10 Jun 2011) | 5 lines

  Unlock the sip channel during fax detection like chan_dahdi does to prevent a deadlock with ast_autoservice_stop.

  (closes issue ASTERISK-17798)
  tested by mnicholson
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@323041 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 322981 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r322981 | twilson | 2011-06-10 08:29:00 -0700 (Fri, 10 Jun 2011) | 11 lines

  Avoid a DB1 infinite loop bug

  Explicity check the last entry in the DB and make sure that we don't iterate
  past it. Since there can be no duplicates, this just makes sure that we stop
  after matching the last key.

  This patch also refactors the code to get away from some code duplication. A
  previous patch added many astdb tests and this patch passed them.

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@322982 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 322923 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r322923 | twilson | 2011-06-09 19:33:23 -0700 (Thu, 09 Jun 2011) | 2 lines

  Add some astdb unit tests
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@322940 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 322865 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r322865 | twilson | 2011-06-09 15:29:20 -0700 (Thu, 09 Jun 2011) | 4 lines

  Correct ast_db_deltree documentation

  ast_db_deltree returns -1 on error, otherwise the number of deletions
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@322866 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 322807 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r322807 | mnicholson | 2011-06-09 12:37:07 -0500 (Thu, 09 Jun 2011) | 5 lines

  don't drop any voice frames when checking for T.38 during early media

  (closes issue ASTERISK-17705)
  Review: https://reviewboard.asterisk.org/r/1186/
  patch by oej
  reported by oej
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@322808 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 322749 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r322749 | rmudgett | 2011-06-09 11:31:53 -0500 (Thu, 09 Jun 2011) | 15 lines

  Remove potential deadlock in call pickup race.

  Deadlock is possible in ast_do_pickup() when holding the target channel
  lock and trying to get the chan channel lock.  Also, holding the target
  lock when calling ast_channel_masquerade() is not a good idea because that
  routine does deadlock avoidance.

  * Removed the need to hold the target lock after marking the target with a
  datastore and getting the connected line data off of the target channel.

  * Moved can_pickup() to ast_can_pickup() in features.c.  Now all the call
  pickup methods use the same basic call pickup availability check.

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@322750 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Blocked revisions 322585 via svnmerge

........
  r322585 | jrose | 2011-06-09 09:06:42 -0500 (Thu, 09 Jun 2011) | 11 lines

  Adds ast_escape_encoded utility to properly handle escaping of quoted field before uri.

  This commit backports a feature in trunk affecting initreqprep so that display name won't
  be encoded improperly. Also includes unit tests for the ast_escape_quoted function.
  This patch gives 1.8 a much improved outlook in countries which don't use standard
  ASCII characters.

  (closes issue ASTERISK-16949)
  Reported by: Örn Arnarson
  Review: https://reviewboard.asterisk.org/r/1235/
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@322586 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Add autoanswer to skinny.

Autoanswer added to skinny based on incoming chan var SKINNY_AUTOANSWER.
Initial value must be the time to autoanswer in ms, then optionally :BEEP
to play a tone when answered and :MUTE to mute the mic when answering.
eg 3000:MUTE:BEEP will ring for 3 secs, then answer, mute the mic, and
play a beep. just 3000 would answer afer 3 secs of ringing with no
beep and full two way audio.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@322544 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 322484 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r322484 | rmudgett | 2011-06-08 15:46:55 -0500 (Wed, 08 Jun 2011) | 15 lines

  Ring all queue with more than 255 agents will cause crash.

  1. Create a ring-all queue with 500 permanent agents.
  2. Call it.
  3. Asterisk will crash.

  The watchers array in app_queue.c has a hard limit of 255.  Bounds
  checking is not done on this array.  No sane person should put 255 people
  in a ring-all queue, but we should not crash anyway.

  * Added bounds checking to the watchers array.

  JIRA AST-464
  JIRA SWP-2903
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@322485 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 322425 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r322425 | rmudgett | 2011-06-08 13:46:30 -0500 (Wed, 08 Jun 2011) | 16 lines

  SRV lookup attempted for SIP peers listed as an IP address.

  Asterisk attempts to SRV lookup a host name even if the host name is an IP
  address.  Regression introduced when IPv6 support was added.

  * Restored the check in ast_dnsmgr_lookup() to see if the given host name
  is an IP address.  The IP address could be in either IPv4 or IPv6 formats.

  (closes issue ASTERISK-17815)
  Reported by: Byron Clark
  Tested by: Byron Clark, Richard Mudgett
  Patches:
       issue19248_v1.8.patch - uploaded by Richard Mudgett (License #5621)

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@322426 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Remove skinny do_monitor and use ast_sched_start instead

The do_monitor seemed to be there for task scheduling and network monitoring. However, the network monitoring has a dedicated thread so the ast_io_wait was basically just a usleep as it didn't actually seem to be monitoring anything.

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@322381 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 322322 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r322322 | irroot | 2011-06-08 08:18:38 +0200 (Wed, 08 Jun 2011) | 18 lines

    Make handle_request_publish do dialog expiration and destruction.

    This patch fixes handle_request_publish so that it does dialog expiration and destruction.

    Without this patch the incoming PUBLISH requests will get stuck in the dialog list.
    Restarting asterisk is the only way to remove them.

    Personal observation on one system the server hung up while looping through the channels
    rendering asterisk unusable and all sip phones unregisterd when they try reregister
    more requests are added.

    (closes issue #18898)
    Reported by: gareth
    Tested by: loloski, Chainsaw, wimpy, se, kuj, irroot

    Jira: https://issues.asterisk.org/jira/browse/ASTERISK-17915
    Review: https://reviewboard.asterisk.org/r/1253
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@322323 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Correct some whitespace and a reference debug message.

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@322284 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Actually check the "sendtodialplan" option setting for xmpp.

(closes issue ASTERISK-17978)
Reported by: elguero
Patches:
    stop_messages_going_to_dialplan.patch (license #5026)

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@322244 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 322189 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r322189 | pabelanger | 2011-06-07 13:59:13 -0400 (Tue, 07 Jun 2011) | 4 lines

  Use correct syntax for 'sip notify snom-reboot'

  (closes issue ASTERISK-17915)
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@322190 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Remove Unused Var Warning rt_handle_member_record

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@322128 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Refactor rt_handle_member_record

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@322111 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 322069 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r322069 | jrose | 2011-06-06 14:07:56 -0500 (Mon, 06 Jun 2011) | 8 lines

  Fixes level toggling for logger set levels since it was reversed

  (closes issue ASTERISK-17850)
  Reported by: Luke H
  Tested by: jrose, Luke H

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@322070 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 321926 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r321926 | rmudgett | 2011-06-03 17:09:36 -0500 (Fri, 03 Jun 2011) | 18 lines

  Asterisk crash when unloading cdr_radius/cel_radius.

  The rc_openlog() API call is passed a string that is used by openlog() to
  format log messages.  The openlog() does not copy the string it just keeps
  a pointer to it.  When the module is unloaded, the string is gone from
  memory.  Depending upon module load order and if the other module then has
  an error, a crash happens.

  * Pass rc_openlog() a strdup'd string with the understanding that there
  will be a small memory leak if the cdr_radius/cel_radius modules are
  unloaded.

  * Call rc_destroy() to free the rc handle memory when the module is
  unloaded.

  JIRA AST-483
  JIRA SWP-3062
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321927 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 321924 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r321924 | rmudgett | 2011-06-03 16:49:17 -0500 (Fri, 03 Jun 2011) | 5 lines

  Be more explicit for CCSS generic device state event subscription.

  Make CCSS generic device state event subscription specify the
  AST_EVENT_IE_STATE ie exists to be safe.
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321925 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 321871 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r321871 | rmudgett | 2011-06-03 15:58:13 -0500 (Fri, 03 Jun 2011) | 27 lines

  Event subscription fixes.

  Must commit the subscription fixes together with the integration
  subscription tests.  The subscription fixes cause an erroneously passing
  test to fail.  The new subscription tests detect errors without the
  subscription fixes.

  * Added missing event_names[] table entry.

  * Reworked ast_event_check_subscriber()/match_sub_ie_val_to_event() to
  correctly detect if a subscriber exists for the proposed event.

  * Made match_ie_val() and match_sub_ie_val_to_event() check the buffer
  length for RAW payload types.

  * Fixed error handling memory leak in ast_event_sub_activate(),
  ast_event_unsubscribe(), and ast_event_queue().

  * Made ast_event_new() and ast_event_check_subscriber() better protect
  themselves from an invalid payload type.

  * Added container lock protection between removing old cache events and
  adding the new cached event in
  ast_event_queue_and_cache()/event_update_cache().

  * Added new event subscription tests.
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321872 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 321812-321813 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r321812 | rmudgett | 2011-06-03 14:55:21 -0500 (Fri, 03 Jun 2011) | 1 line

  Correct IAX2 and SIP event subscription description string.
........
  r321813 | rmudgett | 2011-06-03 14:56:09 -0500 (Fri, 03 Jun 2011) | 1 line

  Constify subscription description parameter string.
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321814 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Blocked revisions 321753 via svnmerge

........
  r321753 | russell | 2011-06-03 13:32:45 -0500 (Fri, 03 Jun 2011) | 2 lines

  Backport an astobj2 unit test so that it runs on 1.8 as well.
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321754 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Fix some astobj2 iterator breakage, add another unit test.

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321752 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 321685 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r321685 | lmadsen | 2011-06-03 08:17:50 -0500 (Fri, 03 Jun 2011) | 5 lines

  Also document the 'queue-minute' option.

  (closes issue #19386)
  Reported by: juanmol
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321689 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Fix message destination extension.

Don't send all messages to 's'.  Get the destination from the request URI.
(Found using automated test cases).

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321617 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 321547 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r321547 | rmudgett | 2011-06-01 18:11:55 -0500 (Wed, 01 Jun 2011) | 1 line

  CDR comment tweaks.
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321548 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Support routing text messages outside of a call.

Asterisk now has protocol independent support for processing text messages
outside of a call.  Messages are routed through the Asterisk dialplan.
SIP MESSAGE and XMPP are currently supported.  There are options in sip.conf
and jabber.conf that enable these features.

There is a new application, MessageSend().  There are two new functions,
MESSAGE() and MESSAGE_DATA().  Documentation will be available on
the project wiki, wiki.asterisk.org.

Thanks to Terry Wilson for the assistance with development and to David Vossel
for helping with some additional testing.

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321546 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 321537 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r321537 | bbryant | 2011-06-01 16:10:02 -0400 (Wed, 01 Jun 2011) | 8 lines

  This patch fixes an issue with using the wrong voicemail folders with greetings.

  (closes issue #17871)
  Reported by: edhorton
  Patches:
        digium_bug_17871_2 uploaded by fhackenberger (license 592)
  Tested by: edhorton, fhackenberger
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321538 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 321528 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r321528 | may | 2011-06-01 14:40:19 +0400 (Wed, 01 Jun 2011) | 14 lines

  Fix double alerting, add forced alerting before answer

  Fix double alerting (it wasn't fixed here by issue #18542)
  Add forced alerting before connect (if it wasn't before)
  Try to send all packets from outgoing queue rather than one only
  Call goes into clearing state when disconnect command is received

  (closes issue #19361)
  Reported by: vmikhelson
  Patches:
        issue19361-3.patch uploaded by may213 (license 454)
  Tested by: vmikhelson
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321529 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 321517 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r321517 | rmudgett | 2011-05-31 15:54:35 -0500 (Tue, 31 May 2011) | 1 line

  Update some comments.
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321518 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 321515 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r321515 | dvossel | 2011-05-31 13:52:54 -0500 (Tue, 31 May 2011) | 12 lines

  Chan_local locking cleanup.

  This patch removes all of the unnecessary deadlock
  avoidance loops that occur in chan_local.  It also
  resolves an issue with a deadlock triggered by
  local channel optimizations.

  (issue #18028)

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321516 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 321511 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r321511 | lmadsen | 2011-05-31 12:04:47 -0400 (Tue, 31 May 2011) | 8 lines

  Enhance NOTICE message to know who couldn't access the dialplan.

  (closes issue #19390)
  Reported by: lmadsen
  Patches:
        __20110531-sip-notice-tweak.txt uploaded by lmadsen (license 10)
  Tested by: russell
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321512 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 321436 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r321436 | rmudgett | 2011-05-27 19:27:52 -0500 (Fri, 27 May 2011) | 4 lines

  Some hagi launch cleanup.

  Inspired by issue 19256.  This patch would also fix the crash.
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321445 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 321392 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r321392 | rmudgett | 2011-05-27 18:45:41 -0500 (Fri, 27 May 2011) | 12 lines

  Crash when using hagi and no servers are available.

  When none of the servers returned by the SRV querey respond, asterisk
  crashes.  The problem is that if the loop over all the SRV entries
  finishes then the srv_context has already been cleaned up.

  * Make ast_srv_cleanup() check to see if the context is already cleaned
  up.

  (closes issue #19256)
  Reported by: byronclark
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321393 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 321337 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

Also revert -r321331 and -r321332.

........
  r321337 | rmudgett | 2011-05-27 17:06:43 -0500 (Fri, 27 May 2011) | 7 lines

  The app_privacy args have undocumented "options" position, interferes with "context" position.

  * Add documention for unused "options" position to match existing code.

  (closes issue #19273)
  Reported by: mdavenport
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321338 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Blocked revisions 321335 via svnmerge

........
  r321335 | lmadsen | 2011-05-27 17:54:54 -0400 (Fri, 27 May 2011) | 7 lines

  Fix issue with playback of H.261 video.

  (closes issue #19379)
  Reported by: neutrino88
  Patches:
        videoprompt.patch uploaded by neutrino88 (license 297)
  (changes by russell)
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321336 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 321333 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r321333 | lmadsen | 2011-05-27 17:40:23 -0400 (Fri, 27 May 2011) | 7 lines

  Allow parking lot hints and musicclass to be set.

  (closes issue #19378)
  Reported by: sboily_proformatique
  Patches:
        pf_parkinghint_music_fix uploaded by sboily proformatique (license 206)
  Tested by: russell
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321334 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Add note about PrivacyManager to UPGRADE.txt

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321332 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 321330 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r321330 | rmudgett | 2011-05-27 16:31:25 -0500 (Fri, 27 May 2011) | 8 lines

  The app_privacy args have undocumented "options" position, interferes with "context" position.

  * Add documention for unused "options" position to match existing code.
  The trunk(v1.10) version will remove the unused options position.

  (closes issue #19273)
  Reported by: mdavenport
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321331 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 321273 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r321273 | jrose | 2011-05-27 09:59:34 -0500 (Fri, 27 May 2011) | 3 lines

  markm committed a patch I was working on yesterday, this fixes it to mesh up with suggestions by mnicholson.
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321289 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 321211 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r321211 | alecdavis | 2011-05-27 20:31:15 +1200 (Fri, 27 May 2011) | 16 lines

  Fix *8 directed pickup locks system during pickupsound play out

  move playout from sip_pickup_thread to bridge using BRIDGE_PLAY_SOUND method,
  This stop the clash of 2 threads trying to write audio to same channel.
  In addition fixes choppy audio beep in issue 19177.

   (issue #18654)
   (issue #19177)
   Reported by: Docent
   Patches:
        review1232-1.8.diff.txt alecdavis (license 585)
   Tested by: alecdavis

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321212 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 321155 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r321155 | markm | 2011-05-26 17:48:45 -0400 (Thu, 26 May 2011) | 10 lines

  Fixed build problem with dev mode enabled, which was caused by commit 321100.  Reformulated patch to be more generic.

  Moved the sip uri parse variable initalization to parse_uri_full in reqresp_parser.c.  This will ensure that any use of parse uri will have null output variables if the parse fails.

  (closes issue #19346)
  Reported by: kobaz
  Tested by: kobaz,JonathanRose

  Review: [full review board URL with trailing slash]
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321156 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 321100 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r321100 | markm | 2011-05-26 16:09:35 -0400 (Thu, 26 May 2011) | 11 lines

  ast_sockaddr_resolve() in netsock2.c may deref a null pointer

  Added a null check in netsock2 ast_sockaddr_resolve() as well as added default initalizers in chan_sip parse_uri_legacy_check() to make sure that invalid uris will make null (and not undefined) user,pass,domain,transport variables

  (closes issue #19346)
  Reported by: kobaz
  Patches:
        netsock2.patch uploaded by kobaz (license 834)
  Tested by: kobaz, Marquis
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321101 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 321044 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r321044 | rmudgett | 2011-05-26 13:10:17 -0500 (Thu, 26 May 2011) | 1 line

  Update ast_sockaddr comment with an important note.
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321045 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 321042 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r321042 | twilson | 2011-05-26 10:29:54 -0700 (Thu, 26 May 2011) | 6 lines

  Initialize stack-allocated ast_sockaddrs before use

  It is important to always initialize ast_sockaddrs before use--even if they
  are passed to ast_sockaddr_copy as the underlying storage could be bigger
  than what ends up being copied--leaving part of the data unitialized.
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321043 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 320947 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r320947 | russell | 2011-05-26 10:57:13 -0500 (Thu, 26 May 2011) | 2 lines

  Remove some variables that were set but unused.
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@321003 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Use va_copy for stringfields

The ast_string_field_build_va functions were written to take to separate
va_lists to work around FreeBSD 4 not having va_copy defined.

In the end, we don't support anything using gcc < 3 anyway because we use
va_copy all over the place anyway. This patch just simplifies things by
removing the second va_list function arguments in favor of va_copy.

Review: https://reviewboard.asterisk.org/r/1233/
--This line, and those below, will be ignored--

M    include/asterisk/stringfields.h
M    main/utils.c
M    main/channel.c

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@320946 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 320883 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r320883 | rmudgett | 2011-05-25 17:25:18 -0500 (Wed, 25 May 2011) | 17 lines

  Native SIP CCSS sends bad CC cancel SUBSCRIBE message.

  The SUBSCRIBE message used to cancel a CC request has incorrect To/From
  SIP headers.  They are reversed and the dialog tags are the same when they
  should not be.  If pedantic mode was disabled, then the cancel would have
  succeeded despite the incorrect message.

  * The SIP_OUTGOING flag was not set correctly for the dialog and I had to
  move some CC subscribe handling code as a result.

  * Initialized the dialog subscribed type to CALL_COMPLETION earlier.  If a
CC request SUBSCRIBE message comes in and the CC instance is not found,
  the 404 response was duplicated.

  JIRA AST-568
  JIRA SWP-3493
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@320884 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 320823 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r320823 | rmudgett | 2011-05-25 12:06:38 -0500 (Wed, 25 May 2011) | 18 lines

  The AMI Newstate event contains different information between v1.4 and v1.8.

  The addition of connected line support in v1.8 changes the behavior of the
  channel caller ID somewhat.  The channel caller ID value no longer time
  shares with the connected line ID on outgoing call legs.  The timing of
  some AMI events/responses output the connected line ID as caller ID.
  These party ID's are now separate.

  * The ConnectedLineNum and ConnectedLineName headers were added to many
  AMI events/responses if the CallerIDNum/CallerIDName headers were also
  present.

  (closes issue #18252)
  Reported by: gje
  Tested by: rmudgett

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@320825 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 320796 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r320796 | rmudgett | 2011-05-25 11:23:11 -0500 (Wed, 25 May 2011) | 17 lines

  Give zombies a safe channel driver to use.

  Recent crashes from zombie channels suggests that they need a safe home to
  goto.  When a masquerade happens, the physical part of the zombie channel
  is hungup.  The hangup normally sets the channel private pointer to NULL.
  If someone then blindly does a callback to the channel driver, a crash is
  likely because the private pointer is NULL.

  The masquerade now sets the channel technology of zombie channels to the
  kill channel driver.

  Related to the following issues:
  (issue #19116)
  (issue #19310)

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@320820 65c4cc65-6c06-0410-ace0-fbb531ad65f3

  CHANNEL(pickupgroup)

  Allow Setting / Reading the pickupgroup of a channel with func_channel.c

  (closes issue #19045)
  Reported by: irroot

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@320772 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 320716 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r320716 | twilson | 2011-05-24 17:49:10 -0700 (Tue, 24 May 2011) | 4 lines

  Cast data as char * before using S_OR

  This is required for compiling successfully under dev mode
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@320717 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 320650 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r320650 | rmudgett | 2011-05-23 12:53:44 -0500 (Mon, 23 May 2011) | 16 lines

  Add ConnectedLineNum/Name headers to output of AMI action Status.

  * Add ConnectedLineNum and ConnectedLineName headers to the output of the
  AMI action Status.  This makes it easier to find out who the channel is
  connected to without having to lookup BridgedChannel or when they are
  connected to an application (e.g.: VoiceMail) which has no bridged
  channel.

  * Bridged channels with no CallerID had "" instead of "<unknown>" output,
  that might be a bug as "<unknown>" was what older versions used.

  (closes issue #18158)
  Reported by: gareth
  Patches:
        svn-292308.diff uploaded by gareth (license 208)
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@320651 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 320568 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

................
  r320568 | dvossel | 2011-05-23 11:18:33 -0500 (Mon, 23 May 2011) | 14 lines

  Merged revisions 320562 via svnmerge from
  https://origsvn.digium.com/svn/asterisk/branches/1.6.2

  ........
    r320562 | dvossel | 2011-05-23 11:15:18 -0500 (Mon, 23 May 2011) | 9 lines

    Adds missing part to the ast_tcptls_server_start fails second attempt to bind patch.

    (closes issue #19289)
    Reported by: wdoekes
    Patches:
          issue19289_delay_old_address_setting_tcptls_2.patch uploaded by wdoekes (license 717)
  ........
................

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@320606 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 320573 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r320573 | tilghman | 2011-05-23 11:19:32 -0500 (Mon, 23 May 2011) | 7 lines

  GNU libiconv uses symbol "libiconv_open" instead of "iconv_open".

  (closes issue #19344)
   Reported by: rohanl
   Patches:
         iconv-check.patch uploaded by rohanl (license 1284)
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@320579 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 320560 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r320560 | kpfleming | 2011-05-23 10:47:14 -0500 (Mon, 23 May 2011) | 4 lines

  Don't generate spurious "No: command not found" messages when running the
  configure script on a system that has neither gmime-config nor pkg-config.
........

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@320561 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 320504 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

........
  r320504 | jrose | 2011-05-23 09:33:20 -0500 (Mon, 23 May 2011) | 10 lines

  Fixes segfault occuring in chan_sip.c at __set_address_from_contact

  Checks to see if domain contains anything before sending it off to ast_sockaddr_resolve
  which is where the segfault was occuring due to null str.

  (closes issue #18857)
  Reported by: sybasesql

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

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@320505 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 320445 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

................
  r320445 | tilghman | 2011-05-22 18:34:57 -0500 (Sun, 22 May 2011) | 15 lines

  Merged revisions 320444 via svnmerge from
  https://origsvn.digium.com/svn/asterisk/branches/1.6.2

  ........
    r320444 | tilghman | 2011-05-22 18:25:51 -0500 (Sun, 22 May 2011) | 8 lines

    Don't crash when the connection fails.

    (closes issue #19250)
     Reported by: seadweller
     Patches:
           20110514__issue19250.diff.txt uploaded by tilghman (license 14)
     Tested by: seadweller, sum
  ........
................

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@320446 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 320338 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

................
  r320338 | dvossel | 2011-05-20 16:39:36 -0500 (Fri, 20 May 2011) | 14 lines

  Merged revisions 320271 via svnmerge from
  https://origsvn.digium.com/svn/asterisk/branches/1.6.2

  ........
    r320271 | dvossel | 2011-05-20 16:24:48 -0500 (Fri, 20 May 2011) | 8 lines

    Fixes issue with ast_tcptls_server_start failing on second attempt to bind.

    (closes issue #19289)
    Reported by: wdoekes
    Patches:
          issue19289_delay_old_address_setting_tcptls.patch uploaded by wdoekes (license 717)
  ........
................

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@320340 65c4cc65-6c06-0410-ace0-fbb531ad65f3

Merged revisions 320237 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.8

................
  r320237 | rmudgett | 2011-05-20 15:49:03 -0500 (Fri, 20 May 2011) | 27 lines

  Merged revisions 320236 via svnmerge from
  https://origsvn.digium.com/svn/asterisk/branches/1.6.2

  ................
    r320236 | rmudgett | 2011-05-20 15:44:54 -0500 (Fri, 20 May 2011) | 20 lines

    Merged revisions 320235 via svnmerge from
    https://origsvn.digium.com/svn/asterisk/branches/1.4

    ........
      r320235 | rmudgett | 2011-05-20 15:38:22 -0500 (Fri, 20 May 2011) | 13 lines

      The meetme CLI command completion leaves conferences mutex locked.

      When issuing a meetme kick CLI command and an invalid (non-existent)
      conference number is specified, pressing Tab leaves the conferences mutex
      locked and, therefore, all conferences deadlock.

      Add missing unlock.

      (closes issue #19336)
      Reported by: zvision
      Patches:
            app_meetme.diff uploaded by zvision (license 798)
    ........
  ................
................

git-svn-id: https://origsvn.digium.com/svn/asterisk/trunk@320238 65c4cc65-6c06-0410-ace0-fbb531ad65f3