[svn-commits] lmadsen: tag 1.6.2.9-rc1 r266656 - /tags/1.6.2.9-rc1/

SVN commits to the Digium repositories svn-commits at lists.digium.com
Tue Jun 1 11:16:15 CDT 2010


Author: lmadsen
Date: Tue Jun  1 11:16:13 2010
New Revision: 266656

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=266656
Log:
Importing files for 1.6.2.9-rc1 release.

Added:
    tags/1.6.2.9-rc1/.lastclean   (with props)
    tags/1.6.2.9-rc1/.version   (with props)
    tags/1.6.2.9-rc1/ChangeLog   (with props)

Added: tags/1.6.2.9-rc1/.lastclean
URL: http://svnview.digium.com/svn/asterisk/tags/1.6.2.9-rc1/.lastclean?view=auto&rev=266656
==============================================================================
--- tags/1.6.2.9-rc1/.lastclean (added)
+++ tags/1.6.2.9-rc1/.lastclean Tue Jun  1 11:16:13 2010
@@ -1,0 +1,1 @@
+36

Propchange: tags/1.6.2.9-rc1/.lastclean
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: tags/1.6.2.9-rc1/.lastclean
------------------------------------------------------------------------------
    svn:keywords = none

Propchange: tags/1.6.2.9-rc1/.lastclean
------------------------------------------------------------------------------
    svn:mime-type = text/plain

Added: tags/1.6.2.9-rc1/.version
URL: http://svnview.digium.com/svn/asterisk/tags/1.6.2.9-rc1/.version?view=auto&rev=266656
==============================================================================
--- tags/1.6.2.9-rc1/.version (added)
+++ tags/1.6.2.9-rc1/.version Tue Jun  1 11:16:13 2010
@@ -1,0 +1,1 @@
+1.6.2.9-rc1

Propchange: tags/1.6.2.9-rc1/.version
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: tags/1.6.2.9-rc1/.version
------------------------------------------------------------------------------
    svn:keywords = none

Propchange: tags/1.6.2.9-rc1/.version
------------------------------------------------------------------------------
    svn:mime-type = text/plain

Added: tags/1.6.2.9-rc1/ChangeLog
URL: http://svnview.digium.com/svn/asterisk/tags/1.6.2.9-rc1/ChangeLog?view=auto&rev=266656
==============================================================================
--- tags/1.6.2.9-rc1/ChangeLog (added)
+++ tags/1.6.2.9-rc1/ChangeLog Tue Jun  1 11:16:13 2010
@@ -1,0 +1,25022 @@
+2010-06-01  Leif Madsen <lmadsen at digium.com>
+
+	* Asterisk 1.6.2.9-rc1 Released.
+
+2010-06-01 15:20 +0000 [r266598]  Tilghman Lesher <tlesher at digium.com>
+
+	* main/asterisk.c, /: 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 ........ ................
+
+2010-05-31 16:07 +0000 [r266570]  Paul Belanger <paul.belanger at polybeacon.com>
+
+	* res/res_agi.c: Fix typo in documentation (closes issue #17395)
+	  Reported by: pabelanger Patches: res_agi.c.patch uploaded by
+	  pabelanger (license 224)
+
+2010-05-30 04:45 +0000 [r266439]  Tilghman Lesher <tlesher at digium.com>
+
+	* contrib/init.d/rc.debian.asterisk, /: 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. ........ ................
+
+2010-05-28 20:55 +0000 [r266338]  Tilghman Lesher <tlesher at digium.com>
+
+	* main/asterisk.c, /: 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
+	  ........
+
+2010-05-28 17:57 +0000 [r266293]  David Vossel <dvossel at digium.com>
+
+	* /, channels/chan_sip.c: 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 ........
+
+2010-05-26 21:19 +0000 [r266154]  Tilghman Lesher <tlesher at digium.com>
+
+	* utils/extconf.c, main/asterisk.c, /, main/logger.c: 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 ........ ................
+
+2010-05-26 18:37 +0000 [r266007]  David Vossel <dvossel at digium.com>
+
+	* /, channels/chan_sip.c: 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 ........
+
+2010-05-26 16:31 +0000 [r265895-265959]  Tilghman Lesher <tlesher at digium.com>
+
+	* res/res_config_pgsql.c, /: 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)
+	  ........ ................
+
+	* configs/res_pgsql.conf.sample, res/res_config_pgsql.c, /: 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 ........
+
+2010-05-26 15:52 +0000 [r265890]  Mark Michelson <mmichelson at digium.com>
+
+	* /, channels/chan_sip.c: 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 ........
+
+2010-05-26 00:33 +0000 [r265748]  Tilghman Lesher <tlesher at digium.com>
+
+	* /, configure, include/asterisk/autoconfig.h.in, configure.ac,
+	  pbx/pbx_lua.c: 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 ........
+
+2010-05-25 21:05 +0000 [r265699]  Mark Michelson <mmichelson at digium.com>
+
+	* /, channels/chan_sip.c: 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. ........
+
+2010-05-25 17:15 +0000 [r265615]  David Vossel <dvossel at digium.com>
+
+	* channels/chan_dahdi.c: fixes build issue with zaptel (closes
+	  issue 0017394) Reported by: aragon Patches: half_buffer_fix.diff
+	  uploaded by dvossel (license 671) Tested by: aragon
+
+2010-05-25 17:06 +0000 [r265612]  Matthew Nicholson <mnicholson at digium.com>
+
+	* apps/app_queue.c, /: 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 ........ ................
+
+2010-05-24 23:52 +0000 [r265521]  Terry Wilson <twilson at digium.com>
+
+	* include/asterisk/options.h, main/asterisk.c, Makefile,
+	  doc/manager_1_1.txt, doc/tex/manager.tex, main/manager.c: 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 ........
+
+2010-05-24 22:07 +0000 [r265450-265452]  Mark Michelson <mmichelson at digium.com>
+
+	* /, channels/h323/ast_h323.cxx: 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) ........
+
+	* /, channels/chan_sip.c: 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
+	  ........
+
+2010-05-24 19:30 +0000 [r265364]  David Vossel <dvossel at digium.com>
+
+	* main/channel.c, /: 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 ........
+
+2010-05-24 18:30 +0000 [r265318]  Tilghman Lesher <tlesher at digium.com>
+
+	* main/asterisk.c, /: 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)
+	  ........
+
+2010-05-21 21:57 +0000 [r264998-265172]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_queue.c: 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/
+
+	* apps/app_queue.c, include/asterisk/file.h, /: 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 ........ ................
+
+	* /, channels/chan_sip.c: 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 ........
+
+	* /, include/asterisk/channel.h: 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. ........
+	  ................
+
+	* main/channel.c, main/autoservice.c, /,
+	  include/asterisk/channel.h: 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/
+	  ........ ................
+
+2010-05-20 23:34 +0000 [r264829]  Richard Mudgett <rmudgett at digium.com>
+
+	* /, main/callerid.c: 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. ........
+	  ................
+
+2010-05-20 22:24 +0000 [r264753-264783]  Tilghman Lesher <tlesher at digium.com>
+
+	* main/pbx.c, /: 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 ........
+
+	* apps/app_stack.c, /: 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)
+	  ........
+
+2010-05-19 22:10 +0000 [r264453]  Mark Michelson <mmichelson at digium.com>
+
+	* include/asterisk/_private.h, include/asterisk/options.h,
+	  main/asterisk.c, main/loader.c, main/channel.c, /,
+	  channels/chan_sip.c: 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/ ........
+
+2010-05-19 20:31 +0000 [r264405]  David Vossel <dvossel at digium.com>
+
+	* main/udptl.c, /: 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) ........
+
+2010-05-19 20:27 +0000 [r264336-264388]  Matthew Nicholson <mnicholson at digium.com>
+
+	* main/udptl.c, /: 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) ........
+
+	* apps/app_speech_utils.c, /: 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 ........ ................
+
+2010-05-19 19:25 +0000 [r264332]  David Vossel <dvossel at digium.com>
+
+	* /, channels/chan_sip.c: 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 ........
+
+2010-05-19 17:49 +0000 [r264205-264250]  Tilghman Lesher <tlesher at digium.com>
+
+	* include/asterisk/options.h, /, configure,
+	  include/asterisk/autoconfig.h.in, configure.ac: 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 ........ ................
+
+	* main/dsp.c, /: 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 ........
+
+2010-05-19 14:47 +0000 [r264115]  David Vossel <dvossel at digium.com>
+
+	* main/rtp.c, /: 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) ........
+
+2010-05-19 08:15 +0000 [r264032]  Alec L Davis <sivad.a at paradise.net.nz>
+
+	* /, configs/indications.conf.sample: 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) ........
+
+2010-05-19 06:41 +0000 [r263951]  Tilghman Lesher <tlesher at digium.com>
+
+	* main/dsp.c, /: 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 ........ ................
+
+2010-05-18 22:49 +0000 [r263906]  David Vossel <dvossel at digium.com>
+
+	* main/strings.c, /: 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 ........
+
+2010-05-18 19:41 +0000 [r263809]  Jeff Peeler <jpeeler at digium.com>
+
+	* apps/app_directory.c, /: 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 ........ ................
+
+2010-05-17 22:10 +0000 [r263642]  Mark Michelson <mmichelson at digium.com>
+
+	* /, main/devicestate.c: 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. ........ ................
+
+2010-05-17 19:37 +0000 [r263587-263590]  Tilghman Lesher <tlesher at digium.com>
+
+	* apps/app_voicemail.c, /: 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 ........
+
+	* main/app.c: Don't close 'n', just close 'above_n'. (closes issue
+	  #17345) Reported by: wdoekes
+
+2010-05-17 14:41 +0000 [r263376-263458]  Leif Madsen <lmadsen at digium.com>
+
+	* main/manager.c, /: 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 ........
+	  ................
+
+	* sounds/Makefile, /: 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
+	  ........ ................
+
+2010-05-17 13:03 +0000 [r263293]  David Vossel <dvossel at digium.com>
+
+	* CHANGES, channels/chan_dahdi.c: backport of DAHDI dynamic buffer
+	  policy dialstring option
+
+2010-05-15 23:41 +0000 [r263202]  Paul Belanger <paul.belanger at polybeacon.com>
+
+	* /, codecs/gsm/Makefile: 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 ........
+
+2010-05-13 22:13 +0000 [r263070]  Richard Mudgett <rmudgett at digium.com>
+
+	* channels/chan_dahdi.c, /: 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
+	  ........
+
+2010-05-13 15:36 +0000 [r262898]  Russell Bryant <russell at digium.com>
+
+	* channels/chan_console.c, /: 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)

[... 24360 lines stripped ...]



More information about the svn-commits mailing list