Merged revisions 269271 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r269271 | dvossel | 2010-06-09 10:09:25 -0500 (Wed, 09 Jun 2010) | 15 lines

  fixes crash in moh when cachertclasses flag is used

  The result for moh_register was not verified to guarantee
  the mohclass as added to the container.

  (closes issue #16993)
  Reported by: dmitri
  Patches:
        res_musiconhold_rtclass2.patch uploaded by dmitri (license 1001)
        moh_crash2.diff uploaded by dvossel (license 671)
  Tested by: dmitri
........

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

Merged revisions 269083 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r269083 | mnicholson | 2010-06-08 13:50:45 -0500 (Tue, 08 Jun 2010) | 9 lines

  Don't pass null to manager_event()

  (closes issue #17087)
  Reported by: bklang
  Patches:
        app-fax-null-sprintf1.diff uploaded by mnicholson (license 96)
  Tested by: bklang
........

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

Merged revisions 269007 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r269007 | seanbright | 2010-06-08 11:39:52 -0400 (Tue, 08 Jun 2010) | 18 lines

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

  ........
    r269006 | seanbright | 2010-06-08 11:28:49 -0400 (Tue, 08 Jun 2010) | 11 lines

    Reduce startup time for cdr_tds with large CDR tables.

    Since we are just checking for table existence, add a WHERE clause that will
    return no rows but will raise an error if the table doesn't exist.

    (closes issue #17380)
    Reported by: kkwong
    Patches:
          issue17380-01.patch uploaded by seanbright (license 71)
    Tested by: kkwong
  ........
................

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

Merged revisions 269008 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r269008 | russell | 2010-06-08 10:41:23 -0500 (Tue, 08 Jun 2010) | 5 lines

  Ensure CONFIG_FLAGS makes it into the build rules when doing out of tree builds.

  (closes issue #16685)
  Reported by: pprindeville
........

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

Merged revisions 268988 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r268988 | lmadsen | 2010-06-08 10:23:20 -0500 (Tue, 08 Jun 2010) | 8 lines

  Update note in sip.conf.sample.
  Update note in sip.conf.sample about externip and externhost with STUN.

  (closes issue #16323)
  Reported by: klaus3000
  Patches:
        sip.conf.sample-patch.txt uploaded by klaus3000 (license 65)
........

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

Merged revisions 268933 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r268933 | tilghman | 2010-06-08 01:57:24 -0500 (Tue, 08 Jun 2010) | 2 lines

  Release list lock before returning on error.
........

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

Merged revisions 268817 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r268817 | tilghman | 2010-06-07 17:47:13 -0500 (Mon, 07 Jun 2010) | 9 lines

  Mailbox list would previously grow at each reload, containing duplicates.

  Also, optimize the allocation of mailboxes to avoid additional memory structures.

  (closes issue #16320)
   Reported by: Marquis
   Patches:
         20100525__issue16320.diff.txt uploaded by tilghman (license 14)
........

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

Fix some chan_oss build system problems (these changes were already in trunk).

This is what caused a bunch of tests to fail on 1.6.2.  They expected a console
channel driver, but chan_oss was failing to load.

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

Merged revisions 268653,268731 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r268653 | tilghman | 2010-06-07 12:14:40 -0500 (Mon, 07 Jun 2010) | 2 lines

  Avoid unloading res_smdi twice.

  (closes issue #17237)
  Reported by: pabelanger
........
  r268731 | tilghman | 2010-06-07 13:59:27 -0500 (Mon, 07 Jun 2010) | 4 lines

  Event well was going dry.

  (issue #17234)
........

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

Merged revisions 268690 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r268690 | pabelanger | 2010-06-07 13:34:45 -0400 (Mon, 07 Jun 2010) | 11 lines

  Set threshold for silence detection defaults to 256

  (closes issue #15685)
  Reported by: david_s5
  Patches:
        dsp-silence-threshold-init.diff uploaded by dant (license 670)
        issue15685.patch.v5 uploaded by pabelanger (license 224)
  Tested by: danti

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

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

Merged revisions 268534 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r268534 | tilghman | 2010-06-06 00:29:50 -0500 (Sun, 06 Jun 2010) | 8 lines

  Take advantage of variable substitution already in the Makefile to specify the correct location for files in init.d.

  (closes issue #16979)
   Reported by: jw-asterisk

  (issue #15691)
   Reported by: itamarjp
........

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

Merged revisions 268495 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r268495 | tilghman | 2010-06-05 19:37:30 -0500 (Sat, 05 Jun 2010) | 2 lines

  Finally track down and eliminate the "FRACK! warnings from chan_iax2".
........

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

Merged revisions 268456 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r268456 | tilghman | 2010-06-05 12:55:28 -0500 (Sat, 05 Jun 2010) | 14 lines

  Fix crash in DTMF detection.

  What I did not originally see in my previous commit was that even though the
  next digit could be detected before the previous was considered ended, the
  detection of the next digit effectively ends the detection of the previous.
  Therefore, the length moves in lockstep with the digit, and no separate counter
  is needed for the length alone.

  (closes issue #17371)
   Reported by: alecdavis

  (closes issue #17474)
   Reported by: kenner
........

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

Merged revisions 268454 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r268454 | tilghman | 2010-06-05 12:27:12 -0500 (Sat, 05 Jun 2010) | 5 lines

  Verify event is not NULL before attempting to lower its usecount.

  (closes issue #17234)
   Reported by: mav3rick
........

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

Merged revisions 268321 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r268321 | tilghman | 2010-06-04 21:51:34 -0500 (Fri, 04 Jun 2010) | 10 lines

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

  ........
    r268320 | tilghman | 2010-06-04 21:49:52 -0500 (Fri, 04 Jun 2010) | 3 lines

    Rest In Peace
    http://www.outandaboutnewspaper.com/article/4061
  ........
................

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

Get rid of warning when the console is configured without logger levels

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

Merged revisions 266735 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r266735 | tilghman | 2010-06-01 13:02:24 -0500 (Tue, 01 Jun 2010) | 7 lines

  Don't register functions until the last possible point, so they're not unloaded unnecessarily.

  (closes issue #15996)
   Reported by: junky
   Patches:
         sdmi_wait.diff uploaded by junky (license 177)
........

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

Merged revisions 268127 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r268127 | tilghman | 2010-06-04 15:42:27 -0500 (Fri, 04 Jun 2010) | 9 lines

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

  ........
    r268126 | tilghman | 2010-06-04 15:41:24 -0500 (Fri, 04 Jun 2010) | 2 lines

    AC_CONFIG_SUBDIRS has a bad side-effect on cross-compiles.
  ........
................

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

Merged revisions 268051 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r268051 | tilghman | 2010-06-04 14:40:00 -0500 (Fri, 04 Jun 2010) | 13 lines

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

  ........
    r268050 | tilghman | 2010-06-04 14:38:57 -0500 (Fri, 04 Jun 2010) | 6 lines

    Build menuselect with the build environment's compiler, not the host (target)'s compiler.

    (closes issue #17464)
     Reported by: pprindeville
     Tested by: tilghman
  ........
................

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

Merged revisions 267972 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r267972 | tilghman | 2010-06-04 11:31:25 -0500 (Fri, 04 Jun 2010) | 9 lines

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

  ........
    r267971 | tilghman | 2010-06-04 11:27:02 -0500 (Fri, 04 Jun 2010) | 2 lines

    As-fixiate the build process
  ........
................

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

Merge fail.

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

Merged revisions 267862,267877 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r267862 | tilghman | 2010-06-03 21:58:55 -0500 (Thu, 03 Jun 2010) | 5 lines

  As signed linear audio data is accessed as 16-bit values, certain processors require the values to be aligned in memory.

  (closes issue #16912)
   Reported by: michaelevdokimov
........
  r267877 | tilghman | 2010-06-03 22:20:47 -0500 (Thu, 03 Jun 2010) | 8 lines

  As signed linear audio data is accessed as 16-bit values, certain processors require the values to be aligned in memory.

  (closes issue #16912)
   Reported by: michaelevdokimov
   Patches:
         asterisk.patch uploaded by michaelevdokimov (license 997)
   Tested by: michaelevdokimov
........

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

Merged revisions 263724,267819 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r263724 | tilghman | 2010-05-17 18:49:15 -0500 (Mon, 17 May 2010) | 8 lines

  Cache sound tarfiles in a common directory, such that a clean reinstall does not force a re-download of the tarballs.

  (closes issue #15370)
   Reported by: pprindeville
   Patches:
         asterisk-trunk-bugid15370.patch uploaded by pprindeville (license 347)
   Tested by: pprindeville, tilghman, seanbright
........
  r267819 | tilghman | 2010-06-03 20:36:46 -0500 (Thu, 03 Jun 2010) | 2 lines

  If there's a default, turn it on, even when the option isn't specified.
........

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

Merged revisions 267775 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r267775 | tilghman | 2010-06-03 20:20:17 -0500 (Thu, 03 Jun 2010) | 14 lines

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

  ........
    r267759 | tilghman | 2010-06-03 20:16:26 -0500 (Thu, 03 Jun 2010) | 7 lines

    Make the default install path appear to be /usr on Linux, instead of /usr/local.

    Also, reorganize the options, so that they're more alphabetical.

    (closes issue #17013)
     Reported by: klaus3000
  ........
................

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

Merged revisions 267669 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r267669 | tilghman | 2010-06-03 14:46:42 -0500 (Thu, 03 Jun 2010) | 8 lines

  Handle OOM errors more gracefully.

  (closes issue #17084)
   Reported by: falves11
   Patches:
         issue17084_162_A.diff uploaded by falves11 (license 374)
   Tested by: falves11
........

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

Merged revisions 267624 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r267624 | lmadsen | 2010-06-03 13:53:24 -0500 (Thu, 03 Jun 2010) | 7 lines

  Update UPGRADE.txt and CHANGE for CDR functionality changes.
  Updated the UPGRADE.txt and CHANGES file stating that CDR records will not be explicity
  written unless cdr.conf exists and is configured.

  (closes issue #17373)
  Reported by: wdoekes
  Tested by: pabelanger
........

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

Merged revisions 267622 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r267622 | rmudgett | 2010-06-03 13:38:00 -0500 (Thu, 03 Jun 2010) | 1 line

  Make compile again.
........

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

Merged revisions 267537 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r267537 | russell | 2010-06-03 12:31:41 -0500 (Thu, 03 Jun 2010) | 2 lines

  Don't stop Asterisk if chan_usbradio isn't configured.
........

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

Add fields back to ast_translator for ABI reasons.

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

Merged revisions 267492 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r267492 | mmichelson | 2010-06-03 12:09:11 -0500 (Thu, 03 Jun 2010) | 6 lines

  Remove unnecessary code relating to PLC.

  The logic for handling generic PLC is now handled in ast_write in
  channel.c instead of in translation code.
........

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

Merged revisions 267490 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r267490 | russell | 2010-06-03 12:05:30 -0500 (Thu, 03 Jun 2010) | 2 lines

  Remove a line that was killing Asterisk on startup.
........

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

Merged revisions 267445 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r267445 | russell | 2010-06-03 09:48:09 -0500 (Thu, 03 Jun 2010) | 2 lines

  Comment out a rule that likes to run implicitly unnecessarily, breaking builds
........

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

Merged revisions 267352 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r267352 | russell | 2010-06-02 17:46:37 -0500 (Wed, 02 Jun 2010) | 7 lines

  try to fix some random chan_h323 compilation failures

  After some debugging, the random chan_h323 build failures appear to be due
  to complications introduced by some chan_h323 specific build stuff getting
  triggered during a clean.  Simplify this by moving the h323 clean commands
  down into channels/makefile.
........

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

Merged revisions 267305 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r267305 | russell | 2010-06-02 16:44:06 -0500 (Wed, 02 Jun 2010) | 2 lines

  Fix a build error on mac.
........

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

Merged revisions 267303 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r267303 | russell | 2010-06-02 16:41:54 -0500 (Wed, 02 Jun 2010) | 6 lines

  Ensure the -Wno-strict-aliasing flag makes it, even if ASTCFLAGS has been specified.

  When ASTCFLAGS was specified with the make command, Makefile.rules was using
  the specified value from the command line and not the one here, making it so this
  flag would go missing.
........

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

Merged revisions 267138 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r267138 | russell | 2010-06-02 13:53:38 -0500 (Wed, 02 Jun 2010) | 4 lines

  Add a CLI command that blocks until Asterisk has fully booted.

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

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

Merged revisions 267065 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r267065 | jpeeler | 2010-06-02 12:29:35 -0500 (Wed, 02 Jun 2010) | 12 lines

  Fix infinite loop when loading codec speex

  This changes the sample slinear frame data to contain non-zero data so that
  translation calculations for speex works when preprocessing and VAD is turned
  on. The encoder expects samples to be returned, but when attempted with the
  mentioned two options and silent sample frames everything was discarded.

  (closes issue #17240)
  Reported by: seandarcy

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

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

Merged revisions 267041 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r267041 | pabelanger | 2010-06-02 13:25:05 -0400 (Wed, 02 Jun 2010) | 14 lines

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

  ........
    r267009 | pabelanger | 2010-06-02 13:14:37 -0400 (Wed, 02 Jun 2010) | 7 lines

    Cleanup error/warning messages in AEL2 parser

    (closes issue #16684)
    Reported by: Silmaril
    Patches:
          patch_ael2_logmsg.diff uploaded by Silmaril (license 979)
  ........
................

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

Merged revisions 266877 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r266877 | pabelanger | 2010-06-02 09:32:22 -0400 (Wed, 02 Jun 2010) | 10 lines

  pthread_join to assure the thread is really gone

  (closes issue #15465)
  Reported by: fnordian
  Patches:
        bridging.patch uploaded by fnordian (license 110)
  Tested by: lmadsen, fnordian, peterh

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

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

Merged revisions 266682 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r266682 | tilghman | 2010-06-01 11:41:00 -0500 (Tue, 01 Jun 2010) | 16 lines

  Eliminate stale manager events after a set interval, even if AMI clients don't query for them.

  Actions (or failures to act) by external clients should not cause memory leaks
  in Asterisk, especially when those continued leaks could cause Asterisk to
  misbehave later.

  (closes issue #17234)
   Reported by: mav3rick
   Patches:
         20100510__issue17234.diff.txt uploaded by tilghman (license 14)
         20100517__issue17234__trunk.diff.txt uploaded by tilghman (license 14)
   Tested by: mav3rick, davidw

  (closes issue #17365)
   Reported by: davidw
........

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

Merged revisions 266592 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r266592 | tilghman | 2010-06-01 10:18:59 -0500 (Tue, 01 Jun 2010) | 18 lines

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

  ........
    r266585 | tilghman | 2010-06-01 10:17:46 -0500 (Tue, 01 Jun 2010) | 11 lines

    Prevent CLI prompt from distorting output of lines shorter than the prompt.

    Uses the VT100 method of clearing the line from the cursor position to the
    end of the line:  Esc-0K

    (closes issue #17160)
     Reported by: coolmig
     Patches:
           20100531__issue17160.diff.txt uploaded by tilghman (license 14)
     Tested by: coolmig
  ........
................

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

Fix typo in documentation

(closes issue #17395)
Reported by: pabelanger
Patches:
      res_agi.c.patch uploaded by pabelanger (license 224)

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

Merged revisions 266438 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r266438 | tilghman | 2010-05-29 23:44:28 -0500 (Sat, 29 May 2010) | 9 lines

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

  ........
    r266437 | tilghman | 2010-05-29 23:43:28 -0500 (Sat, 29 May 2010) | 2 lines

    Reverting patch and reopening issue #16784, as patch breaks color display.
  ........
................

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

Merged revisions 266337 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r266337 | tilghman | 2010-05-28 15:53:04 -0500 (Fri, 28 May 2010) | 1 line

  Only report swap on platforms which can examine those statistics
........

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

Merged revisions 266292 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r266292 | dvossel | 2010-05-28 12:55:38 -0500 (Fri, 28 May 2010) | 9 lines

  fixes crash when creation of UDPTL fails

  (closes issue #17264)
  Reported by: falves11
  Patches:
        issue_17264_reviewboard_fix.diff uploaded by dvossel (license 671)
        issue_17264_1.6.2_reviewboard_fix.diff uploaded by dvossel (license 671)
  Tested by: falves11
........

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

Merged revisions 266146 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r266146 | tilghman | 2010-05-26 16:17:46 -0500 (Wed, 26 May 2010) | 21 lines

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

  ........
    r266142 | tilghman | 2010-05-26 16:11:44 -0500 (Wed, 26 May 2010) | 14 lines

    Use sigaction for signals which should persist past the initial trigger, not signal.

    If you call signal() in a Solaris signal handler, instead of just resetting
    the signal handler, it causes the signal to refire, because the signal is not
    marked as handled prior to the signal handler being called.  This effectively
    causes Solaris to immediately exceed the threadstack in recursive signal
    handlers and crash.

    (closes issue #17000)
     Reported by: rmcgilvr
     Patches:
           20100526__issue17000.diff.txt uploaded by tilghman (license 14)
     Tested by: rmcgilvr
  ........
................

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

Merged revisions 266006 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r266006 | dvossel | 2010-05-26 13:32:51 -0500 (Wed, 26 May 2010) | 8 lines

  fixes failed SIP Directed pickup resulting in dead channel

  (closes issue #17339)
  Reported by: one47
  Patches:
        sip_magic_pickup2 uploaded by one47 (license 23)
  Tested by: one47, dvossel
........

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

Merged revisions 265923 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r265923 | tilghman | 2010-05-26 11:23:28 -0500 (Wed, 26 May 2010) | 14 lines

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

  ........
    r265910 | tilghman | 2010-05-26 11:21:00 -0500 (Wed, 26 May 2010) | 7 lines

    Not finding rows in the DB does not rise to the level of a warning.

    (closes issue #17062)
     Reported by: drookie
     Patches:
           20100525__issue17062.diff.txt uploaded by tilghman (license 14)
  ........
................

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

Merged revisions 265894 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r265894 | tilghman | 2010-05-26 11:14:48 -0500 (Wed, 26 May 2010) | 8 lines

  Construct socket name, according to the Postgres docs, and document as such.

  (closes issue #17392)
   Reported by: dps
   Patches:
         20100525__issue17392.diff.txt uploaded by tilghman (license 14)
   Tested by: dps
........

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

Recorded merge of revisions 265842 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r265842 | mmichelson | 2010-05-26 09:41:55 -0500 (Wed, 26 May 2010) | 9 lines

  Re-enable "always" option for videosupport option in sip.conf.

  (closes issue #17016)
  Reported by: twilson
  Patches:
        17016.patch uploaded by mmichelson (license 60)
     Tested by: devmod
........

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

Merged revisions 265747 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r265747 | tilghman | 2010-05-25 19:29:40 -0500 (Tue, 25 May 2010) | 8 lines

  Use configure to determine the prefixes and include directories properly.

  This ensures cross-platform compatibility, even among Linux distributions,
  which don't always put headers in the same place.

  (closes issue #17391)
   Reported by: loloski
........

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

Merged revisions 265698 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r265698 | mmichelson | 2010-05-25 15:59:04 -0500 (Tue, 25 May 2010) | 12 lines

  Properly use peer's outboundproxy for outbound REGISTERs.

  The logic used in transmit_register to get the outboundproxy for a peer
  was flawed since this value would be overridden shortly afterwards when
  create_addr was called.

  In addition, this also fixes some logic used when parsing users.conf so
  that the peer name is placed in the internally-generated register string
  so that an outboundproxy set in the Asterisk GUI will be used for outbound
  REGISTERs.
........

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

fixes build issue with zaptel

(closes issue 0017394)
Reported by: aragon
Patches:
      half_buffer_fix.diff uploaded by dvossel (license 671)
Tested by: aragon

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

Merged revisions 265611 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r265611 | mnicholson | 2010-05-25 12:00:11 -0500 (Tue, 25 May 2010) | 15 lines

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

  ........
    r265610 | mnicholson | 2010-05-25 11:48:19 -0500 (Tue, 25 May 2010) | 8 lines

    Don't mark the cdr records of unanswered queue calls with "NOANSWER".  This restores the behavior prior to r258670.

    (closes issue #17334)
    Reported by: jvandal
    Patches:
          queue-cdr-fixes1.diff uploaded by mnicholson (license 96)
    Tested by: aragon, jvandal
  ........
................

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

Merged revisions 265320,265467 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r265320 | twilson | 2010-05-24 14:06:40 -0500 (Mon, 24 May 2010) | 14 lines

  Add the FullyBooted AMI event

  It is possible to connect to the manager interface before all Asterisk modules
  are loaded. To ensure that an application does not send AMI actions that might
  require a module that has not yet loaded, the application can listen for the
  FullyBooted manager event. It will be sent upon connection if all modules have
  been loaded, or as soon as loading is complete. The event:

     Event: FullyBooted
     Privilege: system,all
     Status: Fully Booted

  Review: https://reviewboard.asterisk.org/r/639/
........
  r265467 | twilson | 2010-05-24 17:21:58 -0500 (Mon, 24 May 2010) | 1 line

  Merge the rest of the FullyBooted patch
........

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

Merged revisions 265451 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r265451 | mmichelson | 2010-05-24 17:05:15 -0500 (Mon, 24 May 2010) | 8 lines

  Print openh323 log to the Asterisk console.

  (closes issue #17109)
  Reported by: under
  Patches:
        logstream.diff uploaded by under (license 914)
........

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

Merged revisions 265449 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r265449 | mmichelson | 2010-05-24 16:44:30 -0500 (Mon, 24 May 2010) | 11 lines

  Allow type=user SIP endpoints to be loaded properly from realtime.

  (closes issue #16021)
  Reported by: Guggemand
  Patches:
        realtime-type-fix.patch uploaded by Guggemand (license 897)
     (altered by me slightly to avoid ref leaks)
  Tested by: Guggemand
........

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

Merged revisions 265273 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r265273 | dvossel | 2010-05-24 11:10:09 -0500 (Mon, 24 May 2010) | 2 lines

  fixes segfault when using generic plc
........

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

Blocked revisions 265317 via svnmerge

........
  r265317 | twilson | 2010-05-24 13:21:20 -0500 (Mon, 24 May 2010) | 15 lines

  Calendaring support for Exchange Server 2007+ via EWS

  This commit adds support for calendaring with Exchange Server 2007+ via
  Exchange Web Services. Full write support and for querying attendees. Many
  thanks to Jan Kaláb for the feature.

  (closes issue #17022)
  Reported by: pitel
  Patches:
        res_calendar_ews.c uploaded by pitel (license 1008)
  Tested by: pitel, twilson

  Review: https://reviewboard.asterisk.org/r/557/
  Review: https://reviewboard.asterisk.org/r/668/
........

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

Merged revisions 265316 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r265316 | tilghman | 2010-05-24 13:19:08 -0500 (Mon, 24 May 2010) | 7 lines

  On systems with a LOT of RAM, a signed integer sometimes printed negative.

  (closes issue #16837)
   Reported by: jlpedrosa
   Patches:
         20100504__issue16837.diff.txt uploaded by tilghman (license 14)
........

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

Fix memory hogging behavior of app_queue.

From reviewboard:
This review request is for the patch on issue 17081.

A user reported that he saw increasing numbers of allocations stemming
from app_queue.c when he would run the "queue show" CLI command. The
user reported that he was using approximately 40 realtime queues and
as he ran the CLI command more and more, the memory usage would shoot up.
As it turns out, there was a memory leak and a separate usage of memory
that, while not really a leak, was very irresponsible.

Both memory problems can be attributed to the function init_queue(). When
the "queue show" command is run, all realtime queues have the init_queue()
function called on the in-memory queue. The idea is to place the queue in
its default state and then overwrite options specified in the realtime backend
as we read them.

The first problem, the memory leak, had to do with the fact that the string
field for the name of the first periodic announcement file was being re-created
every time init_queue was called. This patch corrects the behavior by only
calling ast_str_create if the memory has not already been allocated.

The other problem is a bit more complicated. The majority of the strings
in the call_queue structure were changed to use the ast_string_fields API
for 1.6.0 and beyond. init_queue resets all string fields on the queue to
their default values. Then, later in the realtime queue loading process,
these string fields are set to their configured values.

For those unfamiliar with string fields, frequent resizing of a string like
this is not what the string fields API is designed for. The result of this
constant resizing is that as the queue gets loaded, eventually space for
the string runs out and so a new memory pool, at twice the size of the
previously allocated one, is created for the string fields. The reporter
of issue 17081 wrote a script that ran the "queue show" CLI command 2100
times. By the end, each of his 40 queues was taking about a megabyte of
memory apiece just for their string fields.

My fix for this problem is to revert the call_queue structure from using
string fields. In my patch here, I have moved the queue back to using
fixed-sized buffers. I ran the script provided by the reporter of 17081
and determined that I no longer saw the steadily-increasing memory usage
that I had seen before applying the patch.

(closes issue #17081)
Reported by: wliegel
Patches:
      17081v2.patch uploaded by mmichelson (license 60)
Tested by: wliegel, mmichelson

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

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

Merged revisions 265090 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r265090 | mmichelson | 2010-05-21 16:08:51 -0500 (Fri, 21 May 2010) | 15 lines

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

  ........
    r265089 | mmichelson | 2010-05-21 15:59:14 -0500 (Fri, 21 May 2010) | 8 lines

    Don't hang up on a queue caller if the file we attempt to play does not exist.

    This also fixes a documentation mistake in file.h that made my original attempt
    to correct this problem not work correctly.

    (closes issue #17061)
    Reported by: RoadKill
  ........
................

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

Merged revisions 265087 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r265087 | mmichelson | 2010-05-21 15:38:14 -0500 (Fri, 21 May 2010) | 7 lines

  Be sure to set the sin_family on the proxy when allocating.

  (closes issue #17157)
  Reported by: stuarth
........

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

Merged revisions 265000 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r265000 | mmichelson | 2010-05-21 11:54:21 -0500 (Fri, 21 May 2010) | 9 lines

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

  ........
    r264999 | mmichelson | 2010-05-21 11:53:53 -0500 (Fri, 21 May 2010) | 3 lines

    Fix grammatical error in comment.
  ........
................

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

Merged revisions 264997 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r264997 | mmichelson | 2010-05-21 11:44:27 -0500 (Fri, 21 May 2010) | 38 lines

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

  ........
    r264996 | mmichelson | 2010-05-21 11:28:34 -0500 (Fri, 21 May 2010) | 32 lines

    Allow ast_safe_sleep to defer specific frames until after the sleep has concluded.

    From reviewboard

    Background:
    A Digium customer discovered a somewhat odd bug. The setup is that parties A
    and B are bridged, and party A places party B on hold. While party B is
    listening to hold music, he mashes a bunch of DTMF. Party A takes party
    B off hold while this is happening, but party B continues to hear hold
    music. I could reproduce this about 1 in 5 times.

    The issue:
    When DTMF features are enabled and a user presses keys, the channel that
    the DTMF is streamed to is placed in an ast_safe_sleep for 100 ms, the
    duration of the emulated tone. If an AST_CONTROL_UNHOLD frame is read
    from the channel during the sleep, the frame is dropped. Thus the
    unhold indication is never made to the channel that was originally placed
    on hold.

    The fix:
    Originally, I discussed with Kevin possible ways of fixing the specific
    problem reported. However, we determined that the same type of problem
    could happen in other situations where ast_safe_sleep() is used. Using
    autoservice as a model, I modified ast_safe_sleep_conditional() to
    defer specific frame types so they can be re-queued once the sleep has
    finished. I made a common function for determining if a frame should
    be deferred so that there are not two identical switch blocks to
    maintain.

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

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

Merged revisions 264828 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r264828 | rmudgett | 2010-05-20 18:29:43 -0500 (Thu, 20 May 2010) | 13 lines

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

  ........
    r264820 | rmudgett | 2010-05-20 18:23:21 -0500 (Thu, 20 May 2010) | 6 lines

    ast_callerid_parse() had a path that left name uninitialized.

    Several callers of ast_callerid_parse() do not initialize the name
    parameter before calling thus there is the potential to use an
    uninitialized pointer.
  ........
................

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

Merged revisions 264779 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r264779 | tilghman | 2010-05-20 17:23:32 -0500 (Thu, 20 May 2010) | 8 lines

  Let ExtensionState resolve dynamic hints.

  (closes issue #16623)
   Reported by: tilghman
   Patches:
         20100116__issue16623.diff.txt uploaded by tilghman (license 14)
   Tested by: lmadsen
........

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

Merged revisions 264752 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r264752 | tilghman | 2010-05-20 16:28:53 -0500 (Thu, 20 May 2010) | 7 lines

  Error message fix.

  (closes issue #17356)
   Reported by: kenner
   Patches:
         app_stack.c.diff uploaded by kenner (license 1040)
........

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

Merged revisions 264452 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r264452 | mmichelson | 2010-05-19 16:29:08 -0500 (Wed, 19 May 2010) | 86 lines

  Fix transcode_via_sln option with SIP calls and improve PLC usage.

  From reviewboard:
  The problem here is a bit complex, so try to bear with me...

  It was noticed by a Digium customer that generic PLC (as configured in
  codecs.conf) did not appear to actually be having any sort of benefit when
  packet loss was introduced on an RTP stream. I reproduced this issue myself
  by streaming a file across an RTP stream and dropping approx. 5% of the
  RTP packets. I saw no real difference between when PLC was enabled or disabled
  when using wireshark to analyze the RTP streams.

  After analyzing what was going on, it became clear that one of the problems
  faced was that when running my tests, the translation paths were being set
  up in such a way that PLC could not possibly work as expected. To illustrate,
  if packets are lost on channel A's read stream, then we expect that PLC will
  be applied to channel B's write stream. The problem is that generic PLC can
  only be done when there is a translation path that moves from some codec to
  SLINEAR. When I would run my tests, I found that every single time, read
  and write translation paths would be set up on channel A instead of channel
  B. There appeared to be no real way to predict which channel the translation
  paths would be set up on.

  This is where Kevin swooped in to let me know about the transcode_via_sln
  option in asterisk.conf. It is supposed to work by placing a read translation
  path on both channels from the channel's rawreadformat to SLINEAR. It also
  will place a write translation path on both channels from SLINEAR to the
  channel's rawwriteformat. Using this option allows one to predictably set up
  translation paths on all channels. There are two problems with this, though.
  First and foremost, the transcode_via_sln option did not appear to be working
  properly when I was placing a SIP call between two endpoints which did not
  share any common formats. Second, even if this option were to work, for PLC
  to be applied, there had to be a write translation path that would go from
  some format to SLINEAR. It would not work properly if the starting format
  of translation was SLINEAR.

  The one-line change presented in this review request in chan_sip.c fixed the
  first issue for me. The problem was that in sip_request_call, the
  jointcapability of the outbound channel was being set to the format passed to
  sip_request_call. This is nativeformats of the inbound channel. Because of this,
  when ast_channel_make_compatible was called by app_dial, both channels already
  had compatibly read and write formats. Thus, no translation path was set up at
  the time. My change is to set the jointcapability of the sip_pvt created during
  sip_request_call to the intersection of the inbound channel's nativeformats and
  the configured peer capability that we determined during the earlier call to
  create_addr. Doing this got the translation paths set up as expected when using
  transcode_via_sln.

  The changes presented in channel.c fixed the second issue for me. First and
  foremost, when Asterisk is started, we'll read codecs.conf to see the value of
  the genericplc option. If this option is set, and ast_write is called for a
  frame with no data, then we will attempt to fill in the missing samples for
  the frame. The implementation uses a channel datastore for maintaining the
  PLC state and for creating a buffer to store PLC samples in. Even when we
  receive a frame with data, we'll call plc_rx so that the PLC state will have
  knowledge of the previous voice frame, which it can use as a basis for when
  it comes time to actually do a PLC fill-in.

  So, reviewers, now I ask for your help. First off, there's the one line change
  in chan_sip that I have put in. Is it right? By my logic it seems correct, but
  I'm sure someone can tell me why it is not going to work. This is probably the
  change I'm least concerned about, though. What concerns me much more is the
  set of changes in channel.c. First off, am I even doing it right? When I run
  tests, I can clearly see that when PLC is activated, I see a significant increase
  in RTP traffic where I would expect it to be. However, in my humble opinion, the
  audio sounds kind of crappy whenever the PLC fill-in is done. It sounds worse to
  me than when no PLC is used at all. I need someone to review the logic I have used
  to be sure that I'm not misusing anything. As far as I can see my pointer arithmetic
  is correct, and my use of AST_FRIENDLY_OFFSET should be correct as well, but I'm
  sure someone can point out somewhere where I've done something incorrectly.

  As I was writing this review request up, I decided to give the code a test run under
  valgrind, and I find that for some reason, calls to plc_rx are causing some invalid
  reads. Apparently I'm reading past the end of a buffer somehow. I'll have to dig around
  a bit to see why that is the case. If it's obvious to someone reviewing, speak up!

  Finally, I have one other proposal that is not reflected in my code review. Since
  without transcode_via_sln set, one cannot predict or control where a translation
  path will be up, it seems to me that the current practice of using PLC only when
  transcoding to SLINEAR is not useful. I recommend that once it has been determined
  that the method used in this code review is correct and works as expected, then
  the code in translate.c that invokes PLC should be removed.

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

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

Merged revisions 264400 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r264400 | dvossel | 2010-05-19 15:30:33 -0500 (Wed, 19 May 2010) | 11 lines

  fixes infinite loop during udptl.c's decode_open_type

  When decode_length returns the length there is a check to see if that
  length is negative, if so the decode loop breaks as this means the
  limit has been reached.  The problem here is that length is an
  unsigned int, so length can never be negative.  This resulted in
  an infinite loop.

  (issue #17352)
........

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

Merged revisions 264379 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r264379 | mnicholson | 2010-05-19 15:26:27 -0500 (Wed, 19 May 2010) | 4 lines

  Cast an unsigned int to a signed int when comparing it with 0.

  (AST-377)
........

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

Merged revisions 264335 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r264335 | mnicholson | 2010-05-19 15:02:57 -0500 (Wed, 19 May 2010) | 12 lines

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

  ........
    r264334 | mnicholson | 2010-05-19 15:01:38 -0500 (Wed, 19 May 2010) | 5 lines

    Set quieted flag when receiving a dtmf tone during playback in speechbackground.

    (closes issue #16966)
    Reported by: asackheim
  ........
................

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

Merged revisions 264331 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r264331 | dvossel | 2010-05-19 14:21:04 -0500 (Wed, 19 May 2010) | 13 lines

  fixes crash in check_rtp_timeout

  During deadlock avoidance the sip dialog pvt is locked and
  unlocked.  When this occurs we have no guarantee the pvt's owner
  is still valid.  We were trying to access the pvt's owner after
  this without checking to see if it still existed first.

  (closes issue #17271)
  Reported by: under
  Patches:
        check_rtp_timeout.diff uploaded by under (license 914)
  Tested by: dvossel
........

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

Merged revisions 264249 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r264249 | tilghman | 2010-05-19 12:48:31 -0500 (Wed, 19 May 2010) | 24 lines

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

  ........
    r264248 | tilghman | 2010-05-19 12:41:29 -0500 (Wed, 19 May 2010) | 17 lines

    Internal timing is now on by default, if you're using DAHDI 2.3 or above.

    The reason for ensuring DAHDI 2.3 or above is that this version ensures that
    a timer is always available, whereas in previous versions, it was possible
    for DAHDI to be loaded, but have no drivers to actually generate timing.  If
    internal_timing was turned on in this circumstance, a complete lack of audio
    would result.  This is the reason why internal_timing was not on by default.
    However, now that DAHDI ensures the availability of a timer, there is no
    reason for this setting to be off (and in fact, it solves a great many initial
    user problems).

    (closes issue #15932)
     Reported by: dimas
     Patches:
           20100519__issue15932.diff.txt uploaded by tilghman (license 14)
     Tested by: tilghman
  ........
................

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

Merged revisions 264204 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r264204 | tilghman | 2010-05-19 11:42:20 -0500 (Wed, 19 May 2010) | 9 lines

  Keep track of digit duration, when we're decoding inband to pass DTMF frames.

  (closes issue #17235)
   Reported by: frawd
   Patches:
         new_dtmf_dsp_len.patch uploaded by frawd (license 610)
         20100518__issue17235.diff.txt uploaded by tilghman (license 14)
   Tested by: frawd
........

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

Merged revisions 264114 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r264114 | dvossel | 2010-05-19 09:38:02 -0500 (Wed, 19 May 2010) | 13 lines

  fixes crash during dtmf

  During the processing of Cisco dtmf the dtmf samples were
  not being calculated correctly.  In an attempt to determine
  what sample rate was being used, a NULL frame was processed
  which caused a crash.  This patch resolves this.

  (closes issue #17248)
  Reported by: falves11
  Patches:
        issue_17248.diff uploaded by dvossel (license 671)
........

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

Merged revisions 264031 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r264031 | alecdavis | 2010-05-19 20:09:14 +1200 (Wed, 19 May 2010) | 8 lines

  fix incorrectly typed indications for [nz] stutter and dialrecall

  (closes issue #17359)
  Reported by: alecdavis
  Patches:
        bug17359.diff.txt uploaded by alecdavis (license 585)
........

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

Merged revisions 263950 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r263950 | tilghman | 2010-05-19 01:41:04 -0500 (Wed, 19 May 2010) | 15 lines

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

  ........
    r263949 | tilghman | 2010-05-19 01:32:27 -0500 (Wed, 19 May 2010) | 8 lines

    Because progress is called multiple times, across several frames, we must persist states when detecting multitone sequences.

    (closes issue #16749)
     Reported by: dant
     Patches:
           dsp.c-bug16749-1.patch uploaded by dant (license 670)
     Tested by: dant
  ........
................

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

Merged revisions 263904 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r263904 | dvossel | 2010-05-18 17:48:51 -0500 (Tue, 18 May 2010) | 9 lines

  fixes segfault on logging

  (closes issue #17331)
  Reported by: under
  Patches:
        utils.diff uploaded by under (license 914)
        segfault_on_logging.diff uploaded by dvossel (license 671)
  Tested by: under, dvossel
........

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

Merged revisions 263807 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r263807 | jpeeler | 2010-05-18 14:27:34 -0500 (Tue, 18 May 2010) | 17 lines

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

  ........
    r263769 | jpeeler | 2010-05-18 13:54:58 -0500 (Tue, 18 May 2010) | 10 lines

    Modify directory name reading to be interrupted with operator or pound escape.

    In the case of accidentally entering the wrong first three letters for the
    reading, users could be very frustrated if the name listing is very long. This
    allows interrupting the reading by pressing 0 or #. 0 will attempt to execute
    a configured operator (o) extension and # will exit and proceed in the
    dialplan.

    ABE-2200
  ........
................

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

Merged revisions 263640 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r263640 | mmichelson | 2010-05-17 17:08:01 -0500 (Mon, 17 May 2010) | 16 lines

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

  ........
    r263639 | mmichelson | 2010-05-17 17:00:28 -0500 (Mon, 17 May 2010) | 10 lines

    Fix logic error when checking for a devstate provider.

    When using strsep, if one of the list of specified separators is not found,
    it is the first parameter to strsep which is now NULL, not the pointer returned
    by strsep.

    This issue isn't especially severe in that the worst it is likely to do is waste
    some cycles when a device with no '/' and no ':' is passed to ast_device_state.
  ........
................

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

Merged revisions 263589 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r263589 | tilghman | 2010-05-17 14:31:15 -0500 (Mon, 17 May 2010) | 9 lines

  With IMAP backend, messages in INBOX were counted twice for MWI.

  (closes issue #17135)
   Reported by: edhorton
   Patches:
         20100513__issue17135.diff.txt uploaded by tilghman (license 14)
         17135_2.diff uploaded by ebroad (license 878)
   Tested by: edhorton, ebroad
........

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

Don't close 'n', just close 'above_n'.
(closes issue #17345)
 Reported by: wdoekes

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

Merged revisions 263457 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r263457 | lmadsen | 2010-05-17 09:37:35 -0500 (Mon, 17 May 2010) | 19 lines

  Recorded merge of revisions 263456 via svnmerge from
  https://origsvn.digium.com/svn/asterisk/branches/1.4

  ........
    r263456 | lmadsen | 2010-05-17 09:35:18 -0500 (Mon, 17 May 2010) | 11 lines

    Manager cookies are not compatible with RFC2109.

    The Version field in the cookies we're setting contain quotes around the version
    number which is not compatible with RFC2109 and breaks some implementations.

    (closes issue #17231)
    Reported by: ecarruda
    Patches:
          manager_rfc2109-trunk-v1.patch uploaded by ecarruda (license 559)
          manager_rfc2109-1.6.2-v1.patch uploaded by ecarruda (license 559)
    Tested by: ecarruda, russell
  ........
................

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

Merged revisions 263375 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r263375 | lmadsen | 2010-05-17 09:05:33 -0500 (Mon, 17 May 2010) | 16 lines

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

  ........
    r263374 | lmadsen | 2010-05-17 09:04:57 -0500 (Mon, 17 May 2010) | 8 lines

    Update link to new version of core sounds.

    The latest version of the core sounds files 1.4.19 now includes the missing
    queue-minute sound file which is called by app_queue but which has been
    missing.

    (closes issue #17123)
    Reported by: n8ideas
  ........
................

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

backport of DAHDI dynamic buffer policy dialstring option

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

Merged revisions 252488 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r252488 | tilghman | 2010-03-15 12:27:08 -0400 (Mon, 15 Mar 2010) | 9 lines

  Make the Makefile logic more explicit and move the Snow Leopard logic down to where it's not executed on non-Darwin systems.

  (closes issue #17028)
   Reported by: pabelanger
   Patches:
         issue17028_20100315.patch uploaded by seanbright (license 71)
         20100315__issue17028.diff.txt uploaded by tilghman (license 14)
   Tested by: tilghman, pabelanger
........

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

Merged revisions 263069 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r263069 | rmudgett | 2010-05-13 17:01:36 -0500 (Thu, 13 May 2010) | 1 line

  Fix inverted logic in cli command: ss7 set debug on/off
........

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

Merged revisions 262897 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r262897 | russell | 2010-05-13 10:36:12 -0500 (Thu, 13 May 2010) | 4 lines

  Fix an off by one error that causes a crash.

  Thanks to Raymond Burke for pointing it out.
........

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

Merged revisions 262800 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r262800 | pabelanger | 2010-05-12 15:59:16 -0400 (Wed, 12 May 2010) | 8 lines

  Notify CLI when modules is loaded / unloaded

  (closes issue #17308)
  Reported by: pabelanger
  Patches:
        cli.modules.patch uploaded by pabelanger (license 224)
  Tested by: pabelanger, russell
........

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

Merged revisions 262798 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r262798 | lmadsen | 2010-05-12 14:53:10 -0500 (Wed, 12 May 2010) | 7 lines

  Revert previous WARNING message removal.

  Marquis42 suggested a better method of doing what I wanted because I ended up
  removing the WARNING message for all instances when really I just wanted to
  remove it for the 'return' keyword, not everything.

  (issue #17145)
........

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

Merged revisions 262796 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r262796 | lmadsen | 2010-05-12 14:31:42 -0500 (Wed, 12 May 2010) | 4 lines

  Remove unnecessary WARNING message in ael/pval.c

  (closes issue #17145)
  Reported by: okrief
........

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

Merged revisions 262744 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r262744 | dvossel | 2010-05-12 13:01:20 -0500 (Wed, 12 May 2010) | 17 lines

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

  ........
    r262662 | dvossel | 2010-05-12 12:00:04 -0500 (Wed, 12 May 2010) | 11 lines

    fixes app_meetme dsp error

    We attempted to detect silence after translating a frame
    from signed linear.  This caused a flooding of errors.  To
    resolve this the code to detect silence was moved before the
    translation.

    (closes issue #17133)
    Reported by: jsdyer
  ........
................

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

Merged revisions 262656 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r262656 | tilghman | 2010-05-12 11:23:26 -0500 (Wed, 12 May 2010) | 8 lines

  Ensure the arguments are initialized.  Also miscellaneous CG cleanup.

  (closes issue #16576)
   Reported by: uxbod
   Patches:
         20100505__issue16576.diff.txt uploaded by tilghman (license 14)
   Tested by: uxbod
........

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

Merged revisions 262513 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r262513 | tilghman | 2010-05-11 16:25:05 -0500 (Tue, 11 May 2010) | 7 lines

  Move cause 200 to cause 26, as specified in Q.850.

  Also cleanup the formatting and add a few more that seem like good candidates.

  (closes issue #16157)
   Reported by: wimpy
........

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

Merged revisions 262422 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r262422 | qwell | 2010-05-11 14:57:24 -0500 (Tue, 11 May 2010) | 18 lines

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

  ........
    r262421 | qwell | 2010-05-11 14:55:42 -0500 (Tue, 11 May 2010) | 11 lines

    Use a less silly method for modifying a flex-generated file.

    The sed syntax that was used wasn't actually valid, causing some versions to
    choke.  This is the method that is used in 1.6.x+ for similar changes.

    (closes issue #16696)
    Reported by: bklang
    Patches:
          16696-sedfix.diff uploaded by qwell (license 4)
    Tested by: qwell
  ........
................

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

Merged revisions 262419 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r262419 | pabelanger | 2010-05-11 15:40:37 -0400 (Tue, 11 May 2010) | 8 lines

  Improve logging by displaying line number

  (closes issue #16303)
  Reported by: dant
  Patches:
        issue16303.patch.v2 uploaded by pabelanger (license 224)
  Tested by: dant, lmadsen, pabelanger
........

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

Merged revisions 262414 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

........
  r262414 | pabelanger | 2010-05-11 15:26:17 -0400 (Tue, 11 May 2010) | 8 lines

  Improve logging information for misconfigured contexts

  (closes issue #17238)
  Reported by: pprindeville
  Patches:
        chan_sip-bug17238.patch uploaded by pprindeville (license 347)
  Tested by: pprindeville
........

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

Merged revisions 262330 via svnmerge from
https://origsvn.digium.com/svn/asterisk/trunk

................
  r262330 | tilghman | 2010-05-11 12:23:51 -0500 (Tue, 11 May 2010) | 9 lines

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

  ........
    r262321 | tilghman | 2010-05-11 12:22:07 -0500 (Tue, 11 May 2010) | 2 lines

    Fix issue #17302 a slightly different way (mad props to Qwell)
  ........
................

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

Blocked revisions 262299 via svnmerge

........
  r262299 | qwell | 2010-05-11 11:43:07 -0500 (Tue, 11 May 2010) | 12 lines

  Allow bootstrap script to work on Solaris.

  As usual, the way they do things is different, so we need to account for that.
  automake is versioned ala BSD/Linux, but autoconf is not.  We don't actually
  need to specify a version there, since AC_PREREQ will cover it for us.  Things
  will fail pretty loudly if AC_PREREQ isn't met.

  (closes issue #16341)
  Reported by: bklang
  Patches:
        opensolaris_bootstrap.sh uploaded by bklang (license 919)
........

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