[svn-commits] bebuild: tag 1.8.25.0-rc1 r402102 - /tags/1.8.25.0-rc1/

SVN commits to the Digium repositories svn-commits at lists.digium.com
Mon Oct 28 08:56:10 CDT 2013


Author: bebuild
Date: Mon Oct 28 08:56:08 2013
New Revision: 402102

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

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

Added: tags/1.8.25.0-rc1/.lastclean
URL: http://svnview.digium.com/svn/asterisk/tags/1.8.25.0-rc1/.lastclean?view=auto&rev=402102
==============================================================================
--- tags/1.8.25.0-rc1/.lastclean (added)
+++ tags/1.8.25.0-rc1/.lastclean Mon Oct 28 08:56:08 2013
@@ -1,0 +1,1 @@
+40

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

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

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

Added: tags/1.8.25.0-rc1/.version
URL: http://svnview.digium.com/svn/asterisk/tags/1.8.25.0-rc1/.version?view=auto&rev=402102
==============================================================================
--- tags/1.8.25.0-rc1/.version (added)
+++ tags/1.8.25.0-rc1/.version Mon Oct 28 08:56:08 2013
@@ -1,0 +1,1 @@
+1.8.25.0-rc1

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

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

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

Added: tags/1.8.25.0-rc1/ChangeLog
URL: http://svnview.digium.com/svn/asterisk/tags/1.8.25.0-rc1/ChangeLog?view=auto&rev=402102
==============================================================================
--- tags/1.8.25.0-rc1/ChangeLog (added)
+++ tags/1.8.25.0-rc1/ChangeLog Mon Oct 28 08:56:08 2013
@@ -1,0 +1,46755 @@
+2013-10-28  Asterisk Development Team <asteriskteam at digium.com>
+
+	* Asterisk 1.8.25.0-rc1 Released.
+
+2013-10-25 21:51 +0000 [r401959-402000]  Scott Griepentrog <sgriepentrog at digium.com>
+
+	* include/asterisk/rtp_engine.h, main/rtp_engine.c: rtp_engine: fix
+	  rtp payloads copy and improve argument names In function
+	  ast_rtp_instance_early _bridge_make_compatible the use of
+	  instance 0/1 as arguments doesn't clearly communicate a direction
+	  that the copying of payloads from the source channel to the
+	  destination channel will occur, making it more probable to have
+	  the arguments to ast_rtp_codecs_payloads_copy() put in the
+	  reverse order. This patch renames the arguments with _dst and
+	  _src suffixes and corrects the copy direction.
+
+	* include/asterisk/pbx.h, main/pbx.c: pbx.c: fix confused match
+	  caller id that deleted exten still in hash This fixes a bug where
+	  a zero length callerid match adjacent to a no match callerid
+	  extension entry would be deleted together, which then resulted in
+	  hashtable references to free'd memory. A third state of the
+	  matchcid value has been added to indicate match to any extension
+	  which allows enforcing comparison of matchcid on/off without
+	  errors. (closes issue AST-1235) Reported by: Guenther Kelleter
+	  Review: https://reviewboard.asterisk.org/r/2930/
+
+2013-10-25 17:21 +0000 [r401619-401914]  Jonathan Rose <jrose at digium.com>
+
+	* utils/clicompat.c: Put clicompat-r2.patch back in We've figured
+	  out how to resolve the problems this was causing in 12/trunk, so
+	  this can go back in now. (issue ASTERISK-22467) Reported by:
+	  Corey Farrell Patches: clicompat-r2.patch uploaded by
+	  coreyfarrell (license 5909)
+
+	* utils/clicompat.c: revert clicompat-r2.patch from r401704 Patch
+	  caused the following build errors against testsuite
+	  https://bamboo.asterisk.org/bamboo/browse/AST-ATRUNKBUILD4-244
+	  (issue ASTERISK-22467) Reported by: Corey Farrell
+
+	* main/utils.c: utils: Fix memory leaks and missed unregistration
+	  of CLI commands on shutdown Final set of patches in a series of
+	  memory leak/cleanup patches by Corey Farrell (closes issue
+	  ASTERISK-22467) Reported by: Corey Farrell Patches:
+	  main-utils-1.8.patch uploaded by coreyfarrell (license 5909)
+	  main-utils-11.patch uploaded by coreyfarrell (license 5909)
+	  main-utils-12up.patch uploaded by coreyfarrell (license 5909)
+
+	* tests/test_linkedlists.c: test_linkedlists: Fix memory leak
+	  (issue ASTERISK-22467) Reported by: Corey Farrell Patches:
+	  test_linkedlists-1.8.patch uploaded by coreyfarrell (license
+	  5909) test_linkedlists-11up.patch uploaded by coreyfarrell
+	  (license 5909)
+
+	* main/jitterbuf.c: jitterbuf: Fix memory leak on jitter buffer
+	  reset (issue ASTERISK-22467) Reported by: Corey Farrell Patches:
+	  jitterbuf-jb_reset-leak-1.8.patch
+	  jitterbuf-jb_reset-leak-11up.patch
+
+	* main/astobj2.c: astobj2: Unregister debug CLI commands at exit
+	  (issue ASTERISK-22467) Reported by: Corey Farrell Patches:
+	  astobj2-clean-debug-cli-1.8-11.patch uploaded by coreyfarrell
+	  (license 5909) astobj2-clean-debug-cli-12up.patch uploaded by
+	  coreyfarrell (license 5909)
+
+	* apps/app_voicemail.c: app_voicemail: Memory Leaks against tests
+	  (issue ASTERISK-22467) Reported by: Corey Farrell Patches:
+	  app_voicemail-1.8.patch uploaded by coreyfarrell (license 5909)
+	  app_voicemail-11up.patch uploaded by coreyfarrell (license 5909)
+
+	* main/asterisk.c, utils/clicompat.c, channels/chan_dahdi.c,
+	  codecs/ilbc/doCPLC.c, main/data.c, main/app.c: memory leaks:
+	  Memory leak cleanup patch by Corey Farrell (second set) Also
+	  covers ast_app_parse_timelen-fail-zero-length.patch, but the
+	  patch was replaced with one of my own. (issue ASTERISK-22467)
+	  Reported by: Corey Farrell Patches: chan_dahdi-cleanup_push.patch
+	  uploaded by coreyfarrell (license 5909) clicompat-r2.patch
+	  uploaded by coreyfarrell (license 5909) codecs-ilbc-doCPLC.patch
+	  uploaded by coreyfarrell (license 5909)
+	  data-cleanup-test-registration.patch uploaded by coreyfarrell
+	  (license 5909) main-asterisk-kill-listener.patch uploaded by
+	  coreyfarrell (license 5909)
+
+	* tests/test_dlinklists.c, funcs/func_math.c,
+	  channels/sip/reqresp_parser.c, main/test.c,
+	  main/editline/readline.c: memory leaks: Memory leak cleanup patch
+	  by Corey Farrell (first set) (issue ASTERSIK-22467) Reported by:
+	  Corey Farrell Patches:
+	  chan_sip-parse_contact_header_test-free-contacts.patch uploaded
+	  by coreyfarrell (license 5909) cli-filename-completion-leak.patch
+	  uploaded by coreyfarrell (license 5909) func_math.patch uploaded
+	  by corefarrell (license 5909) main-test-cleanup.patch uploaded by
+	  coreyfarrell (license 5909) test_dlinklists.patch uploaded by
+	  coreyfarrell (license 5909)
+
+	* main/translate.c, res/res_rtp_asterisk.c: res_rtp_asterisk:
+	  Address jittery DTMF events in RTP streams (closes issue
+	  ASTERISK-21170) Reported by: NITESH BANSAL Patches:
+	  dtmf-timestamp.patch uploaded by NITESH BANSAL (license 6418)
+	  Review: https://reviewboard.asterisk.org/r/2938/
+
+2013-10-23 16:34 +0000 [r401577]  Richard Mudgett <rmudgett at digium.com>
+
+	* cdr/cdr_adaptive_odbc.c: cdr_adaptive_odbc: Also apply a filter
+	  when the CDR value is empty. Extra CDR records are written if a
+	  filtered CDR value is empty because the filter is not checked.
+	  (closes issue ASTERISK-22272) Reported by: Jordi Llull Chavarria
+
+2013-10-23 15:19 +0000 [r401537]  Kinsey Moore <kmoore at digium.com>
+
+	* channels/chan_mgcp.c: chan_mgcp: Properly handle malformed media
+	  lines This corrects a situation in which a media line was not
+	  parsed properly and resulted in a crash. (closes issue
+	  ASTERISK-21190) Reported by: adomjan Patches:
+	  chan_mgcp.c-sscnaf_fix uploaded by adomjan (License 5448)
+
+2013-10-23 11:10 +0000 [r401497]  Joshua Colp <jcolp at digium.com>
+
+	* channels/chan_sip.c: Fix an issue where an incompatible audio
+	  format may be added to SDP. If preferred codecs included any
+	  non-audio format the code would mistakenly add the audio format,
+	  even if it was not a joint capability with the remote side.
+	  (closes issue ASTERISK-21131) Reported by: nbougues Patches:
+	  patch_unsupported_codec_1.8.patch uploaded by nbougues (license
+	  6470)
+
+2013-10-22 22:36 +0000 [r401445]  Matthew Jordan <mjordan at digium.com>
+
+	* res/res_rtp_asterisk.c: res_rtp_asterisk: Fix crash when RTCP is
+	  not available during SSRC change In r400089, a patch was put in
+	  to correct erroneous RTCP statistic resets. Unfortunately,
+	  ast_rtp_read can be called on an RTP instance that does not have
+	  RTCP information. This patch prevents that crash by only
+	  resetting the statistics if we do actually have an RTCP instance.
+	  (issue AST-1174) (closes issue ASTERISK-22667) Reported by: John
+	  Bigelow
+
+2013-10-22 00:13 +0000 [r401378]  Richard Mudgett <rmudgett at digium.com>
+
+	* channels/sig_analog.c: chan_dahdi: Fix unable to get index
+	  warning when transferring an analog call. Transferring an analog
+	  call using flashhooks generated an unable to get index WARNING
+	  message when the transfer is completed. * Removed unnecessary
+	  analog subchannel shell games when transferring a call using
+	  flashhooks. Thanks to Tzafrir Cohen for mentioning this in a
+	  comment on issue ASTERISK-22720.
+
+2013-10-21 19:45 +0000 [r401325]  Kevin Harwell <kharwell at digium.com>
+
+	* main/editline/term.c: Segfault in LIBEDIT_INTERNAL after
+	  tgetstr(), when libncurses5-dev isn't installed Include the
+	  appropriate declarations when not using termcap, but term+curses
+	  and [n]curses do not exist. (closes issue ASTERISK-22351)
+	  Reported by: A. Iglesias Patches:
+	  issueA22351_libedit_internal_without_ncurses_dev.patch uploaded
+	  by wdoekes (license 5674)
+
+2013-10-18 14:40 +0000 [r401178]  Walter Doekes <walter+asterisk at wjd.nu>
+
+	* main/channel.c: Properly copy/remove the device state cache flag
+	  over a masquerade. In r378303 the AST_FLAG_DISABLE_DEVSTATE_CACHE
+	  flag was added that tells the devstate system to not cache states
+	  for non-real devices. However, when optimizing away channels
+	  (ast_do_masquerade), that flag wasn't copied. In my case, using
+	  Local devices as queue members created a situation where the
+	  endpoint was considered in use, but the state change of the
+	  device being available again was ignored (not cached). The
+	  endpoint channel was optimized into the (previously) Local
+	  channel, but kept the do-not-cache flag. The end result being
+	  that the queue member apparently stayed in use forever. (closes
+	  issue ASTERISK-22718) Reported by: Walter Doekes Review:
+	  https://reviewboard.asterisk.org/r/2925/
+
+2013-10-17 15:22 +0000 [r401119]  Kinsey Moore <kmoore at digium.com>
+
+	* res/res_jabber.c: Reduce log level of a non-pubsub error message
+	  Drop an error log message to debug level 1 since distributed
+	  device state functions correctly when receiving this message and
+	  it spams the logs. (closes issue ASTERISK-22410) Reported by:
+	  abelbeck Patches:
+	  asterisk-1.8-res_jabber-log-nonpubsub-error-to-debug.patch
+	  uploaded by abelbeck (License 5903)
+
+2013-10-16 11:04 +0000 [r401049]  Walter Doekes <walter+asterisk at wjd.nu>
+
+	* apps/app_queue.c: Don't check all realtime queues when doing
+	  "queue show some_queue". When using realtime queues, queues have
+	  to be fetched from the database every now and then to see if any
+	  info has been changed or to see if the queue has been removed.
+	  When fetching info for an individual queue, the pruning of other
+	  queues is unnecessarily costly. Review:
+	  https://reviewboard.asterisk.org/r/2907/
+
+2013-10-15 14:52 +0000 [r400970]  Mark Michelson <mmichelson at digium.com>
+
+	* channels/chan_sip.c: Prevent chan_sip from sending duplicate
+	  BYEs. When a 200 OK for an initial INVITE is received, we were
+	  doing the right thing by ACKing and sending an immediate BYE.
+	  However, we also were doing the wrong thing and queuing an answer
+	  frame, thus causing the call to be answered. This would cause the
+	  call to be hung up by the channel thread, thus resulting in a
+	  second BYE being sent out. In this fix, I also have set the
+	  hangupcause to be correct since the initial BYE being sent by
+	  Asterisk had an unknown hangup cause. I have changed to using
+	  "Bearer capabilty not available" since the call was hung up due
+	  to an SDP offer/answer error. (closes issue ASTERISK-22621)
+	  reported by Kinsey Moore
+
+2013-10-14 21:40 +0000 [r400907]  Richard Mudgett <rmudgett at digium.com>
+
+	* channels/chan_dahdi.c: chan_dahdi: Reflect the set software gain
+	  in the CLI "dahdi show channel" output. * Remember the swgain
+	  setting from CLI "dahdi set swgain" command so the CLI "dahdi
+	  show channel" output will reflect the current setting. * Updated
+	  CLI "dahdi set hwgain" and "dahdi set swgain" documentation.
+	  (issue ASTERISK-22429) Reported by: Jaco Kroon Patches:
+	  jira_asterisk_22429_v1.8_v2.patch (license #5621) patch uploaded
+	  by rmudgett
+
+2013-10-14 21:32 +0000 [r400906]  Mark Michelson <mmichelson at digium.com>
+
+	* channels/chan_sip.c: Do not increment the SDP version between 183
+	  and 200 responses. Bumping the SDP version number can cause
+	  interoperability problems since receivers of the responses will
+	  expect that a 200 SDP will be identical to a previous 183 SDP.
+	  (closes issue ASTERISK-21204) reported by NITESH BANSAL Patches:
+	  dont-increment-session-version-in-2xx-after-183.patch uploaded by
+	  NITESH BANSAL (License #6418)
+
+2013-10-08 22:26 +0000 [r400694-400767]  Kinsey Moore <kmoore at digium.com>
+
+	* configure, configure.ac: Add warning when compiling with iODBC
+	  support When running configure, libiodbc2 development headers
+	  will fulfill the requirement for ODBC development headers, but
+	  will not function properly. This adds a warning when libiodbc2
+	  development headers are detected instead of unixodbc development
+	  headers. (closes issue ASTERISK-22459) Reported by: Patrick
+	  Maille Tested by: Walter Doekes Patches:
+	  issueA22459_warn_when_using_iodbc.patch uploaded by Walter Doekes
+	  (License 5674)
+
+	* funcs/func_config.c: Fix func_config list entry allocation The
+	  AST_CONFIG dialplan function defined in func_config.c allocates
+	  its config file list entries using ast_malloc. List entry
+	  allocations destined for use with Asterisk's linked list API must
+	  be ast_calloc()d or otherwise initialized so that list pointers
+	  are set to NULL. These uses of ast_malloc have been replaced by
+	  ast_calloc to prevent dereferencing of uninitialized pointer
+	  values when traversing the list. (closes issue ASTERISK-22483)
+	  Reported by: Brian Scott
+
+2013-10-06 17:07 +0000 [r400622]  Michael L. Young <elgueromexicano at gmail.com>
+
+	* apps/app_queue.c: Fix Regression With Queuelog EXITWITHKEY Only
+	  Logging Two Out Of Four Fields Commit r62462 added two extra
+	  fields for logging "the original position the caller entered the
+	  queue at, and the amount of time the caller was waiting in the
+	  queue." But when r75969 was merged from 1.4 into trunk (r75977),
+	  these two fields disappeared. Those two extra fields were not
+	  logged in 1.4 and when the patch was merged, those fields went
+	  away. Therefore, this is a regression and was caught by the
+	  reporter because he was reading the awesome "Asterisk: The
+	  Definitive Guide" book. (closes issue ASTERISK-22197) Reported
+	  by: Dalius M. Tested by: Dalius M. Patches:
+	  asterisk-22197-q-log-exitwithkey.diff uploaded by Michael L.
+	  Young (license 5026) Review:
+	  https://reviewboard.asterisk.org/r/2901/
+
+2013-10-03 22:51 +0000 [r400469]  Jonathan Rose <jrose at digium.com>
+
+	* channels/chan_sip.c: chan_sip: Don't ignore expires value in
+	  contact header if it lacks semicolon (closes issue
+	  ASTERISK-22574) Reported by: Filip Jenicek Patches:
+	  chan_sip_expires.patch uploaded by Filip Jenicek (license 6277)
+
+2013-10-03 18:25 +0000 [r400393]  Kinsey Moore <kmoore at digium.com>
+
+	* res/res_rtp_multicast.c: Ensure res_rtp_mutlicast sets SSRC
+	  properly This fixes a bug where the SSRC field on multicast RTP
+	  can be stuck at 0 which can cause problems for endpoints trying
+	  to make sense of incoming streams. (closes issue ASTERISK-22567)
+	  Reported by: Simone Camporeale Patches:
+	  22567_res_mulitcast_ssrc.patch uploaded by Simone Camporeale
+	  (License 6536)
+
+2013-10-02 21:30 +0000 [r400314]  Michael L. Young <elgueromexicano at gmail.com>
+
+	* channels/chan_iax2.c: Cast Integer Argument To Unsigned Char The
+	  member reg in the peercnt structure is an unsigned char and
+	  peercnt_modify() is expecting an unsigned char argument which
+	  gets assigned to peercnt->reg. This patch fixes that by casting
+	  the integer argument being passed to peercnt_modify to unsigned
+	  char.
+
+2013-09-30 15:19 +0000 [r400137]  Kinsey Moore <kmoore at digium.com>
+
+	* channels/chan_sip.c, UPGRADE.txt, configs/sip.conf.sample: Allow
+	  Asterisk to retry after 403 on register This adds a global option
+	  in chan_sip to allow it to continue attempting registration if a
+	  403 is received, clearing the cached nonce and treating it as a
+	  non-fatal response. Normally, this would cause registration
+	  attempts to that endpoint to stop. (closes issue ASTERISK-17138)
+	  Review: https://reviewboard.asterisk.org/r/2874/ Reported by:
+	  Rudi
+
+2013-09-28 22:20 +0000 [r400073-400089]  Matthew Jordan <mjordan at digium.com>
+
+	* res/res_rtp_asterisk.c: res_rtp_asterisk: Correct erroneous lost
+	  packet information in RTCP reports RTCP's calculation of the
+	  number of lost packets in an RTP stream is based on that stream's
+	  sequence number count, the number of received packets, and how
+	  many packets we expect to receive. When the SSRC for an RTP
+	  stream changes, there can - and almost always will be - a large
+	  jump in the next packet's timestamp and sequence number. If we
+	  don't reset the number of received packets, sequence number
+	  count, and other metrics used by RTCP, the next RR/SR report will
+	  use the previous SSRC's values to calculate the lost packet count
+	  for the new SSRC - resulting in a very large number of lost
+	  packets. This patch modifies res_rtp_asterisk such that, if it
+	  detects a SSRC change, it will reset the various values used by
+	  the RTCP calculations. From the perspective of RTCP, this appears
+	  as a new media stream - which is what it is. Review:
+	  https://reviewboard.asterisk.org/r/2886/ (closes issue AST-1174)
+	  Reported by: Thomas Arimont
+
+	* configure.ac, configure: Add check for openSUSE when detecting
+	  bfd library In ASTERISK-17842, some additional library checks
+	  were added to the configure script so that the bfd library could
+	  be found on CentOS and Fedora systems. As it turns out, openSUSE
+	  requires an additional library. This patch adds another check to
+	  the configure script for openSUSE that will add that library.
+	  Review: https://reviewboard.asterisk.org/r/2885/ (closes issue
+	  AST-1169) Reported by: Guenther Kelleter
+
+2013-09-27 21:31 +0000 [r400013]  Richard Mudgett <rmudgett at digium.com>
+
+	* channels/chan_sip.c, channels/sip/reqresp_parser.c: chan_sip:
+	  Increase some scratch buffer sizes dealing with caller id. *
+	  Eliminated an unnecessary initialization in check_user_full().
+	  (closes issue ASTERISK-22477) Reported by: Michael Shepelev
+
+2013-09-27 17:13 +0000 [r399939]  Jonathan Rose <jrose at digium.com>
+
+	* channels/sip/include/sip.h, channels/chan_sip.c: chan_sip: Reject
+	  calls on 200 OKs if no SDP has been received When Asterisk
+	  receives a 200 OK in response to an invite, that peer should have
+	  sent an SDP at some point by then. If the channel has never
+	  received an SDP, media won't have been set and the remote address
+	  won't be known. Endpoints in general should not be doing this.
+	  This patch makes it so that Asterisk will simply hang up a call
+	  if it sends a 200 OK at this point. So far this odd behavior for
+	  endpoints has only been observed in tests which involved manually
+	  created SIP transactions in SIPp. (closes issue ASTERISK-22424)
+	  Reported by: Jonathan Rose Review:
+	  https://reviewboard.asterisk.org/r/2827/
+
+2013-09-25 20:23 +0000 [r399818]  Richard Mudgett <rmudgett at digium.com>
+
+	* channels/chan_dahdi.c, channels/sig_ss7.c: chan_dahdi: CLI "core
+	  stop gracefully" has needless delay for PRI and SS7. The PRI and
+	  SS7 link control threads are not stopped correctly when the
+	  chan_dahdi.so module is unloaded. The link control threads
+	  pri_dchannel() and ss7_linkset() are not awakened from a poll()
+	  to cancel the thread. * Added a SIGURG signal after requesting
+	  the thread cancel to break the link control thread poll()
+	  immediately. For SS7 it was slightly worse, the link poll()
+	  timeout would always be whatever was the last libss7 scheduled
+	  event time used. If no libss7 scheduled event was pending, the
+	  thread could run more often than necessary. * Set nextms to 60
+	  seconds for the ss7_linkset() poll() if there is no other libss7
+	  scheduled event.
+
+2013-09-25 19:25 +0000 [r399794]  Michael L. Young <elgueromexicano at gmail.com>
+
+	* channels/chan_sip.c: Fix Realtime Peer Update Problem When
+	  Un-registering And Expires Header In 200ok 1st Issue When a
+	  realtime peer sends an un-REGISTER request, Asterisk un-registers
+	  the peer but the database table record still has regseconds and
+	  fullcontact for the peer. This results in calls attempting to be
+	  routed to the peer which is no longer registered. The expected
+	  behavior is to get busy/congested when attempting to call an
+	  un-registered peer through the dialplan. What was discovered is
+	  that we are clearing out the peer's registration in the database
+	  in parse_register_contact() when calling expire_register() but
+	  then upon returning from parse_register_contact(), update_peer()
+	  is run which stores back in the database table regseconds and
+	  fullcontact. 2nd Issue The reporter pointed out that the 200 ok
+	  being returned by Asterisk after un-registering a peer contains a
+	  Contact header with ;expires= and the Expires header is not set
+	  to 0. This is actually a regression. Tests were created for this
+	  second issue (ASTERISK-22548). The tests have been reviewed and a
+	  Ship It! was received on those tests. This patch does the
+	  following: * Do not ignore the Expires header value even when it
+	  is set to 0. The patch sets the pvt->expiry earlier on in the
+	  function so that it is set properly and used. * If pvt->expiry is
+	  0, do not call update_peer since that means the peer has already
+	  been un-registered and there is no need to update the database
+	  record again since nothing has changed. (closes issue
+	  ASTERISK-22428) Reported by: Ben Smithurst Tested by: Ben
+	  Smithurst, Michael L. Young Patches:
+	  asterisk-22428-rt-peer-update-and-expires-header.diff by Michael
+	  L. Young (license 5026) Review:
+	  https://reviewboard.asterisk.org/r/2869/
+
+2013-09-24 20:03 +0000 [r399697]  Richard Mudgett <rmudgett at digium.com>
+
+	* channels/chan_iax2.c: chan_iax2: Prevent some needless breaking
+	  of the native IAX2 bridge. * Clean up some twisted code in the
+	  iax2_bridge() loop. * Add AST_CONTROL_VIDUPDATE and
+	  AST_CONTROL_SRCCHANGE to a list of frames to prevent the native
+	  bridge loop from breaking. * Passing the
+	  AST_CONTROL_T38_PARAMETERS frame should also allow FAX over a
+	  native IAX2 bridge. (issue ABE-2912) Review:
+	  https://reviewboard.asterisk.org/r/2870/
+
+2013-09-19 16:34 +0000 [r399456]  Jonathan Rose <jrose at digium.com>
+
+	* channels/chan_sip.c: chan_sip: Make direct media reinvites for
+	  T38 put Asterisk in the media path Prior to this patch, Asterisk
+	  would incorrectly use the previous endpoint addresses in SDP in
+	  spite of providing its own port. T38 is never meant to be done
+	  through directmedia and Asterisk should always be in the media
+	  path for these streams. (closes issue ASTERISK-17273) Reported
+	  by: Kevin Stewart (closes issue ASTERISK-18706) Reported by:
+	  Jeremy Kister Review: https://reviewboard.asterisk.org/r/2853/
+
+2013-10-21  Asterisk Development Team <asteriskteam at digium.com>
+
+	* Asterisk 1.8.24.0 Released.
+
+2013-10-18  Asterisk Development Team <asteriskteam at digium.com>
+
+	* Asterisk 1.8.24.0-rc2 Released.
+
+	* Properly copy/remove the device state cache flag over a masquerade.
+
+	  In r378303 the AST_FLAG_DISABLE_DEVSTATE_CACHE flag was added that
+	  tells the devstate system to not cache states for non-real devices.
+	  However, when optimizing away channels (ast_do_masquerade), that flag
+	  wasn't copied.
+
+	  In my case, using Local devices as queue members created a situation
+	  where the endpoint was considered in use, but the state change of the
+	  device being available again was ignored (not cached). The endpoint
+	  channel was optimized into the (previously) Local channel, but kept
+	  the do-not-cache flag. The end result being that the queue member
+	  apparently stayed in use forever.
+
+2013-09-19  Asterisk Development Team <asteriskteam at digium.com>
+
+	* Asterisk 1.8.24.0-rc1 Released.
+
+2013-09-18 19:54 +0000 [r399402]  Kinsey Moore <kmoore at digium.com>
+
+	* main/abstract_jb.c: Fix jitter buffer log file creation This
+	  adjusts '/'-to-'#' replacement to replace all instances of '/'
+	  instead of just the first to ensure that the jitter buffer log
+	  file gets the correct name as per Richard Kenner's suggestion.
+	  (closes issue ASTERISK-21036) Reported by: Richard Kenner
+
+2013-09-18 17:15 +0000 [r399351]  Matthew Jordan <mjordan at digium.com>
+
+	* build_tools/prep_tarball: Update prep_tarball with new
+	  documentation files on the Asterisk wiki This will now pull both
+	  a command reference for the version being prepared, as well as an
+	  Admin Guide that applies to all versions of Asterisk. (issue
+	  ASTERISK-22439) Reported by: Olle Johansson
+
+2013-09-18 01:32 +0000 [r399304]  Michael L. Young <elgueromexicano at gmail.com>
+
+	* main/features.c: Fix Segfault When Syntax Of A Line Under
+	  [applicationmap] Is Invalid When processing the lines under the
+	  [applicationmap] context in features.conf, a segfault occurs from
+	  attempting to process a line with an invalid syntax (basically
+	  missing most of the arguments). Example: [applicationmap]
+	  automon=*6 * This patch moves the checking for empty arguments to
+	  before they are accessed. * Also, checked the "todo" comment and
+	  removed it. Some applications do not require arguments. (closes
+	  issue ASTERISK-22416) Reported by: CGI.NET Tested by: CGI.NET
+	  Patches: asterisk-22416-check-syntax-first_v2.diff by Michael L.
+	  Young (license 5026) Review:
+	  https://reviewboard.asterisk.org/r/2803
+
+2013-09-16 16:37 +0000 [r399158]  Richard Mudgett <rmudgett at digium.com>
+
+	* channels/chan_iax2.c: chan_iax2: Fix saving the wrong expiry time
+	  in astdb. When a new IAX2 client registers, the astdb database is
+	  updated with the value of minregexpire defined in iax.conf
+	  instead of using the expiry time that is provided by the client.
+	  The provided expiry time of the client is updated after inserting
+	  the astdb entry. As a consequence, restarting or reloading
+	  asterisk creates clients whose registration may expire before
+	  they reregister. The clients are therefore unavailable after
+	  minregexpire seconds until they reregister. * Move updating of
+	  the expiry time to before inserting into the astdb. (closes issue
+	  ASTERISK-22504) Reported by: Stefan Wachtler Patches:
+	  chan_iax2.c.patch (license #6533) patch uploaded by Stefan
+	  Wachtler
+
+2013-09-13 20:47 +0000 [r399098]  David M. Lee <dlee at digium.com>
+
+	* main/astobj2.c: Don't write to /tmp/refs when REF_DEBUG is not
+	  defined. If MALLOC_DEBUG is enabled, then the debug destructor
+	  for the container is used, which would erroneously write to
+	  /tmp/refs. This patch only uses the debug destructor if ref_debug
+	  is used. (closes issue ASTERISK-22536)
+
+2013-09-13 13:31 +0000 [r399033]  Kinsey Moore <kmoore at digium.com>
+
+	* apps/app_meetme.c: Fix several crashes in MeetMeAdmin This change
+	  ensures that MeetMeAdmin commands requiring a user actually get a
+	  user and fixes another issue where an extra dereference could
+	  occur for a last-entered user being ejected if a user identifier
+	  was also provided. (closes issue ASTERISK-21907) Reported by:
+	  Alex Epshteyn Review: https://reviewboard.asterisk.org/r/2844/
+
+2013-09-12 20:09 +0000 [r398937-398977]  Jonathan Rose <jrose at digium.com>
+
+	* channels/sip/include/sip.h, channels/chan_sip.c: chan_sip: Revert
+	  r398835 due to failing tests involving originate (issue
+	  ASTERISK-22424) Reported by: Jonathan Rose
+
+	* res/res_musiconhold.c: res_musiconhold: Fix reference leaks
+	  caused when reloading with REF_DEBUG set Due to a faulty function
+	  for debugging reference decrementing, it was possible to reduce
+	  the refcount on the wrong object if two moh classes of the same
+	  name were in the moh class container. (closes issue
+	  ASTERISK-22252) Reported by: Walter Doekes Patches:
+	  18_moh_debug_ref_patch.diff Uploaded by Jonathan Rose (license
+	  6182)
+
+2013-09-12 00:00 +0000 [r398880-398884]  Rusty Newton <rnewton at digium.com>
+
+	* apps/app_queue.c: 'queue add member' help text correction You are
+	  adding dial strings to the queue, not channels. An aribitrary
+	  string could be used, but you are typically referencing a
+	  channel. Correcting the command help text. (issue ASTERISK-22263)
+	  (closes issue ASTERISK-22263) Reported By: Rusty Newton
+
+	* configs/chan_dahdi.conf.sample: Documentation fix -
+	  waitfordialtone is not boolean, it's time in milliseconds
+	  Changing text in chan_dahdi.conf sample to be accurate. (issue
+	  ASTERISK-22308) (closes issue ASTERISK-22308) Reported By:
+	  Malcolm Davenport
+
+2013-09-11 19:39 +0000 [r398835]  Jonathan Rose <jrose at digium.com>
+
+	* channels/sip/include/sip.h, channels/chan_sip.c: chan_sip: Reject
+	  calls without prior SDP on 200 OK If we receive a 200 OK without
+	  SDP, we will now check to see if the remote address has been
+	  established for that channel's RTP session and if the to tag for
+	  that channel has changed from the most recent to tag in a
+	  response less than 200. If either a change has been made since
+	  the last to-tag was received or the remote address is unset, then
+	  we will drop the call. (closes issue ASTERISK-22424) Reported by:
+	  Jonathan Rose Review:
+	  https://reviewboard.asterisk.org/r/2827/diff/#index_header
+
+2013-09-10 17:53 +0000 [r398757]  Richard Mudgett <rmudgett at digium.com>
+
+	* main/xmldoc.c, main/cli.c, funcs/func_dialgroup.c, main/heap.c,
+	  main/event.c, res/res_musiconhold.c, main/indications.c,
+	  main/asterisk.c: Fix incorrect usages of ast_realloc(). There are
+	  several locations in the code base where this is done: buf =
+	  ast_realloc(buf, new_size); This is going to leak the original
+	  buf contents if the realloc fails. Review:
+	  https://reviewboard.asterisk.org/r/2832/
+
+2013-09-10 17:47 +0000 [r398748-398752]  David M. Lee <dlee at digium.com>
+
+	* utils/check_expr.c: Fixed utils directory breakage from r398748,
+	  this time with extra hate.
+
+	* utils/check_expr.c, utils/ael_main.c, utils/conf2ael.c: Fixed
+	  utils directory breakage from r398648
+
+2013-09-09 23:15 +0000 [r398703]  Richard Mudgett <rmudgett at digium.com>
+
+	* main/astmm.c: MALLOC_DEBUG: Change fence magic number to be
+	  completely different from the freed magic number. Race conditions
+	  between freeing a nul terminated string and ast_strdup()'ing it
+	  are more likely to be detected if the fence and freed magic
+	  numbers are completely different.
+
+2013-09-09 19:56 +0000 [r398648]  David M. Lee <dlee at digium.com>
+
+	* main/utils.c, include/asterisk/lock.h, main/lock.c: Fix
+	  DEBUG_THREADS when lock is acquired in __constructor__ This patch
+	  fixes some long-standing bugs in debug threads that were
+	  exacerbated with recent Optional API work in Asterisk 12. With
+	  debug threads enabled, on some systems, there's a lock ordering
+	  problem between our mutex and glibc's mutex protecting its module
+	  list (Ubuntu Lucid, glibc 2.11.1 in this instance). In one
+	  thread, the module list will be locked before acquiring our
+	  mutex. In another thread, our mutex will be locked before locking
+	  the module list (which happens in the depths of calling
+	  backtrace()). This patch fixes this issue by moving backtrace()
+	  calls outside of critical sections that have the mutex acquired.
+	  The bigger change was to reentrancy tracking for
+	  ast_cond_{timed,}wait, which wrongly assumed that waiting on the
+	  mutex was equivalent to a single unlock (it actually suspends all
+	  recursive locks on the mutex). (closes issue ASTERISK-22455)
+	  Review: https://reviewboard.asterisk.org/r/2824/
+
+2013-09-06 20:56 +0000 [r398523-398576]  Kinsey Moore <kmoore at digium.com>
+
+	* res/res_jabber.c: Commit the remainder of r398523 This is a
+	  missing part of the commit in revision 398523 that corrects the
+	  name of a variable. (issue ASTERISK-22435)
+
+	* res/res_jabber.c: Fix Jabber/XMPP distributed MWI The mailbox and
+	  context are swapped on the receiving end for all users of Jabber
+	  and XMPP distributed MWI in Asterisk 1.8 and all more recent
+	  versions. This swaps those values to be correct when publishing
+	  to the internal event system from Jabber/XMPP distributed MWI
+	  state. (closes issue ASTERISK-22435) Reported by: abelbeck Tested
+	  by: Michael Keuter Patches:
+	  asterisk-1.8-res_jabber-aji_handle_pubsub_event.patch uploaded by
+	  abelbeck asterisk-11-res_xmpp-xmpp_pubsub_handle_event.patch
+	  uploaded by abelbeck
+
+2013-09-05 19:00 +0000 [r398235-398456]  Richard Mudgett <rmudgett at digium.com>
+
+	* channels/chan_iax2.c: chan_iax2: Reduce indentation in
+	  __attempt_transmit(). * Reduce indentation in
+	  __attempt_transmit(). * Don't update the static last error time
+	  variable every time in __schedule_action() and socket_read().
+
+	* channels/chan_iax2.c: chan_iax2: Fix stray reference to worker
+	  thread idle_list. * Fix stray reference to idle_list in
+	  cleanup_thread_list(). This may be the reason for the note in
+	  iax2_process_thread() about threads not being removed from the
+	  task lists. * Move cleanup_thread_list(&idle_list) to after the
+	  other lists are cleaned up.
+
+	* channels/chan_iax2.c: chan_iax2: Fix bridgecallno deadlock
+	  avoidance. * Fix bridgecallno deadlock avoidance. When doing
+	  deadlock avoidance, you need to retest the status of values for
+	  each loop to see if you still need the lock for bridgecallno. *
+	  As a safety check, after acquiring the bridgecallno lock you
+	  should check if iaxs[bridgecallno] is NULL just like the current
+	  callno checks. * Move setting thread->iostate to IAX_IOSTATE_IDLE
+	  to after processing any deferred frames to ensure that the
+	  iostate is IDLE when it is placed back into the idle list.
+	  defer_full_frame() tries to ensure iax2_process_thread() wakes up
+	  to process the frame.
+
+	* channels/iax2-parser.c: chan_iax2: Add missing control frame
+	  names to debug frame decode output. (Part 2)
+
+	* channels/iax2-parser.c: chan_iax2: Add missing control frame
+	  names to debug frame decode output.
+
+	* channels/chan_misdn.c: chan_misdn: Fix misdn debug output printed
+	  with arbitrary verbose levels. Fix the misdn debug output to
+	  remote consoles. chan_misdn uses ast_console_puts() which doesn't
+	  know about verbose levels. Better to use ast_verbose() instead.
+	  Without this patch the misdn debug messages are appended to the
+	  verbose level which ever was set by the message sent to the
+	  console before, i.e. any undefined level. (closes issue AST-1218)
+	  Reported by: Guenther Kelleter Patches: misdnlog.patch (license
+	  #6372) patch uploaded by Guenther Kelleter
+
+2013-09-02 07:24 +0000 [r398167]  Walter Doekes <walter+asterisk at wjd.nu>
+
+	* cel/cel_custom.c: Be a little more verbose when loading
+	  cel_custom.conf. Review: https://reviewboard.asterisk.org/r/2805/
+
+2013-08-30 18:55 +0000 [r398021-398102]  Kevin Harwell <kharwell at digium.com>
+
+	* channels/chan_sip.c, main/config.c, res/res_security_log.c: Fix
+	  various memory leaks main/config.c - cleanup cache fie includes
+	  res/res_security_log.c - unregister logger level
+	  channesl/chan_sip.c - cleanup io context and notify_types (closes
+	  issues ASTERISK-22378) Reported by: Corey Farrell Patches:
+	  config_shutdown.patch uploaded by coreyfarrell (license 5909)
+	  res_security_log.patch uploaded by coreyfarrell (license 5909)
+	  chan_sip-1.8.patch uploaded by coreyfarrell (license 5909)
+
+	* main/manager.c, res/res_agi.c: Memory leak fix
+	  ast_xmldoc_printable returns an allocated block that must be
+	  freed by the caller. Fixed manager.c and res_agi.c to stop
+	  leaking these results. (closes issue ASTERISK-22395) Reported by:
+	  Corey Farrell Patches: manager-leaks-1.8.patch uploaded by
+	  coreyfarrell (license 5909) res_agi-xmldoc-leaks.patch uploaded
+	  by coreyfarrell (license 5909)
+
+	* main/features.c: Fix memory leak Fixed a features.c test that
+	  leaked a reference to a parked call. This caused chancount to
+	  never reach 0, so graceful shutdown stops. Also added an
+	  unregister test. (closes issue ASTERISK-22413) Reported by: Corey
+	  Farrell Patches: features-TEST_FRAMEWORK.patch uploaded by
+	  coreyfarrell (license 5909)
+
+2013-08-30 16:46 +0000 [r398018]  Richard Mudgett <rmudgett at digium.com>
+
+	* tests/test_substitution.c: test_substituition: Fix failed test
+	  reporting to actually report failure. You cannot put the "Testing
+	  <blah> pass/fail" on a single line before actually performing the
+	  test. Now any additional failure information is logged before the
+	  test pass/fail announcement. * Added an additional CDR(answer,u)
+	  test.
+
+2013-08-27 17:55 +0000 [r397710-397756]  Matthew Jordan <mjordan at digium.com>
+
+	* channels/chan_sip.c: AST-2013-005: Fix crash caused by invalid
+	  SDP If the SIP channel driver processes an invalid SDP that
+	  defines media descriptions before connection information, it may
+	  attempt to reference the socket address information even though
+	  that information has not yet been set. This will cause a crash.
+	  This patch adds checks when handling the various media
+	  descriptions that ensures the media descriptions are handled only
+	  if we have connection information suitable for that media. Thanks
+	  to Walter Doekes, OSSO B.V., for reporting, testing, and
+	  providing the solution to this problem. (closes issue
+	  ASTERISK-22007) Reported by: wdoekes Tested by: wdoekes patches:
+	  issueA22007_sdp_without_c_death.patch uploaded by wdoekes
+	  (License 5674)
+
+	* channels/chan_sip.c: AST-2013-004: Fix crash when handling ACK on
+	  dialog that has no channel A remote exploitable crash
+	  vulnerability exists in the SIP channel driver if an ACK with SDP
+	  is received after the channel has been terminated. The handling
+	  code incorrectly assumed that the channel would always be
+	  present. This patch adds a check such that the SDP will only be
+	  parsed and applied if Asterisk has a channel present that is
+	  associated with the dialog. Note that the patch being applied was

[... 46042 lines stripped ...]



More information about the svn-commits mailing list