[asterisk-commits] russell: tag 1.4.13 r85319 - in /tags/1.4.13: .lastclean .version ChangeLog

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Wed Oct 10 11:04:35 CDT 2007


Author: russell
Date: Wed Oct 10 11:04:35 2007
New Revision: 85319

URL: http://svn.digium.com/view/asterisk?view=rev&rev=85319
Log:
importing files for 1.4.13 release

Added:
    tags/1.4.13/.lastclean   (with props)
    tags/1.4.13/.version   (with props)
    tags/1.4.13/ChangeLog   (with props)

Added: tags/1.4.13/.lastclean
URL: http://svn.digium.com/view/asterisk/tags/1.4.13/.lastclean?view=auto&rev=85319
==============================================================================
--- tags/1.4.13/.lastclean (added)
+++ tags/1.4.13/.lastclean Wed Oct 10 11:04:35 2007
@@ -1,0 +1,1 @@
+29

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

Propchange: tags/1.4.13/.lastclean
------------------------------------------------------------------------------
    svn:keywords = none

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

Added: tags/1.4.13/.version
URL: http://svn.digium.com/view/asterisk/tags/1.4.13/.version?view=auto&rev=85319
==============================================================================
--- tags/1.4.13/.version (added)
+++ tags/1.4.13/.version Wed Oct 10 11:04:35 2007
@@ -1,0 +1,1 @@
+1.4.13

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

Propchange: tags/1.4.13/.version
------------------------------------------------------------------------------
    svn:keywords = none

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

Added: tags/1.4.13/ChangeLog
URL: http://svn.digium.com/view/asterisk/tags/1.4.13/ChangeLog?view=auto&rev=85319
==============================================================================
--- tags/1.4.13/ChangeLog (added)
+++ tags/1.4.13/ChangeLog Wed Oct 10 11:04:35 2007
@@ -1,0 +1,12320 @@
+2007-10-10  Russell Bryant <russell at digium.com>
+
+	* Asterisk 1.4.13 released.
+
+2007-10-10 15:56 +0000 [r85316]  Russell Bryant <russell at digium.com>
+
+	* include/asterisk/file.h: I introduced a new member to the
+	  ast_filestream struct in 1.4.12, but put it in the middle of the
+	  struct, instead of at the end. One of the Debian folks, paravoid,
+	  pointed out that this breaks binary compatability with modules
+	  compiled against older headers. So, I'm moving the new member to
+	  the end of the struct to resolve the situation.
+
+2007-10-10 15:51 +0000 [r85315]  Mark Michelson <mmichelson at digium.com>
+
+	* main/utils.c: The thread ID should be unsigned.
+
+2007-10-10 14:42 +0000 [r85277-85280]  Joshua Colp <jcolp at digium.com>
+
+	* channels/chan_sip.c: If devicestate is passed a port number strip
+	  it out. (closes issue #10930) Reported by: ibc
+
+	* channels/chan_sip.c: Add support for handling a 182 Queued
+	  response. (closes issue #10924) Reported by: ramonpeek Patches:
+	  queued-182.diff uploaded by ramonpeek (license 266)
+
+2007-10-10 14:26 +0000 [r85276]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_voicemail.c: A bunch of changes from sprintf to
+	  snprintf. See security advisory AST-2002-022
+
+2007-10-10 14:14 +0000 [r85242]  Joshua Colp <jcolp at digium.com>
+
+	* apps/app_voicemail.c: Close voicemail message description file if
+	  duration did not meet the minimum, or else we will eventually run
+	  out of file descriptors. (closes issue #10918) Reported by:
+	  brak2718 Patches: vm1.4.12.1.patch uploaded by brak2718 (license
+	  279)
+
+2007-10-10 06:24 +0000 [r85195]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* include/asterisk/frame.h: use a macro instead of an inline
+	  function, so that backtraces will report the caller of
+	  ast_frame_free() properly
+
+2007-10-09 21:55 +0000 [r85158]  Tilghman Lesher <tlesher at digium.com>
+
+	* main/channel.c, main/utils.c, include/asterisk/lock.h: This
+	  commit fixes the following issues: - Deadlock in ast_write (issue
+	  #10406) - Deadlock in ast_read (issue #10406) - Possible mutex
+	  initialization error in lock.h (issue #10571)
+
+2007-10-09 14:30 +0000 [r84990-85093]  Joshua Colp <jcolp at digium.com>
+
+	* channels/chan_sip.c: Don't perform a reinvite if a transfer is in
+	  progress. (issue #10915) Reported by: ramonpeek
+
+	* main/rtp.c: Only update codec information if the channel has a
+	  technology private structure. (issue #10915) Reported by:
+	  ramonpeek
+
+	* main/rtp.c: Update codec information as well as address when
+	  doing hold reinvites. (issue #10868) Reported by: mavince
+
+	* main/channel.c: Don't keep trying to native bridge if either of
+	  the channels are involved in a masquerade operation to be done.
+	  (closes issue #10696) Reported by: tbelder
+
+2007-10-08 03:28 +0000 [r84957]  Russell Bryant <russell at digium.com>
+
+	* Makefile.rules: Enable file dependency tracking for _all_ builds,
+	  and not just for builds with dev-mode enabled. I have seen enough
+	  problems caused by this that I don't think it's worth keeping. I
+	  want to continue to encourage anybody that is interested to
+	  continue to run Asterisk from svn. Furthermore, I do not want
+	  their systems to break when we change a structure definition in a
+	  header file. :)
+
+2007-10-07 16:15 +0000 [r84890-84902]  Philippe Sultan <philippe.sultan at gmail.com>
+
+	* res/res_jabber.c: Presence packets from a client who's connected
+	  with our Jabber ID are valid, therefore, those clients must be
+	  considered as buddies. The resource string helps us make the
+	  distinction between clients. Closes issue #10707, reported by
+	  yusufmotiwala.
+
+	* res/res_jabber.c: Prevent Asterisk from crashing when receiving a
+	  presence packet without resource from a buddy that is known to
+	  have a resource list. Revert a change I previously made, where
+	  Asterisk could point to a freed memory location.
+
+2007-10-05 19:42 +0000 [r84851]  Tilghman Lesher <tlesher at digium.com>
+
+	* main/db.c: Log exactly why we can't open the database, if we fail
+	  (closes issue #10887)
+
+2007-10-05 18:55 +0000 [r84818]  Joshua Colp <jcolp at digium.com>
+
+	* main/rtp.c: Update the remembered RTP peer information when
+	  putting an endpoint on hold or taking it off hold so that the RTP
+	  stack does not initiate a needless reinvite. (closes issue
+	  #10868) Reported by: mavince
+
+2007-10-05 16:44 +0000 [r84783]  Russell Bryant <russell at digium.com>
+
+	* channels/chan_zap.c: Do deadlock avoidance in a couple more
+	  places. You can't lock two channels at the same time without
+	  doing extra work to make sure it succeeds. (closes issue #10895,
+	  patch by me)
+
+2007-10-05  Russell Bryant <russell at digium.com>
+
+	* Asterisk 1.4.12.1 released. (This is mainly to include the 
+	  app_queue fix for a memory leak on reload, but includes a couple
+	  of other bug fixes, as well.)
+
+2007-10-05 01:39 +0000 [r84742]  Russell Bryant <russell at digium.com>
+
+	* main/manager.c: Fix a copy/paste error in the description of
+	  UpdateConfig that was pointed out by JerJer on #asterisk-dev
+
+2007-10-04 21:57 +0000 [r84692]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_queue.c: Don't allocate space for queue members unless
+	  it's needed. You end up deleting dynamic members on a reload. Not
+	  good. closes issue (#10879, reported by dazza76, patched by me)
+
+2007-10-04 21:36 +0000 [r84690]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* channels/chan_zap.c: callers of sig2str already add the word
+	  'signalling' in the appropriate place, so don't duplicate it
+
+2007-10-04 14:51 +0000 [r84637]  Joshua Colp <jcolp at digium.com>
+
+	* apps/app_queue.c: Create a duplicate of the channel's member name
+	  as the tab completion stuff will free it. (closes issue #10884)
+	  Reported by: adamg
+
+2007-10-03 22:59 +0000 [r84581]  Tilghman Lesher <tlesher at digium.com>
+
+	* main/rtp.c: When an RFC 2833 event is sent that we don't
+	  recognize, ignore it, don't queue a NULL digit (closes issue
+	  #10877)
+
+2007-10-03 18:20 +0000 [r84511-84544]  Steve Murphy <murf at digium.com>
+
+	* pbx/pbx_ael.c: closes issue #10870 ; where a CUT() function call
+	  in a switch expr doesn't execute correctly, because the commas in
+	  the function args are not converted to vertbars before the func
+	  is called. I modified just the switch code to convert the commas
+	  to vertbars if there, but if more of these sort of probs are
+	  found, I may have to resort to something a little more
+	  fundamental. We'll see, I guess.
+
+	* pbx/ael/ael-test/ref.ael-test8, pbx/ael/ael-test/ref.ael-test18,
+	  pbx/ael/ael-test/ref.ael-vtest13,
+	  pbx/ael/ael-test/ref.ael-vtest17,
+	  pbx/ael/ael-test/ref.ael-ntest10, pbx/ael/ael-test/ref.ael-test1,
+	  pbx/ael/ael-test/ref.ael-test3, pbx/pbx_ael.c,
+	  pbx/ael/ael-test/ref.ael-test5: closes issue #10834 ; where a
+	  null input to a switch statement results in a hangup; since
+	  switch is implemented with extensions, and the default case is
+	  implemented with a '.', and the '.' matches 1 or more remaining
+	  characters, the case where 0 characters exist isn't matched, and
+	  the extension isn't matched, and the goto fails, and a hangup
+	  occurs. Now, when a default case is generated, it also generates
+	  a single fixed extension that will match a null input. That
+	  extension just does a goto to the default extension for that
+	  switch. I played with an alternate solution, where I just tack an
+	  extra char onto all the patterns and the goto, but not the
+	  default case's pattern. Then even a null input will still have at
+	  least one char in it. But it made me nervous, having that extra
+	  char in , even if that's a pretty secret and low-level issue.
+
+2007-10-02  Russell Bryant <russell at digium.com>
+
+	* Asterisk 1.4.12 released.
+
+2007-10-02 20:06 +0000 [r84474]  Russell Bryant <russell at digium.com>
+
+	* Makefile, build_tools/prep_tarball: * Don't build the
+	  menuselect-tree for the tarball, as it requires running the
+	  configure script first * Change the Makefile to note that
+	  menuselect-tree depends on the configure script.
+
+2007-10-02 19:01 +0000 [r84410-84437]  Jason Parker <jparker at digium.com>
+
+	* res/res_features.c: Fix some odd formatting I missed..
+
+	* res/res_features.c: Finish up on transferee channel before return
+	  on failure. Issue 10821, patch by Ivan
+
+2007-10-02 14:12 +0000 [r84370]  Russell Bryant <russell at digium.com>
+
+	* channels/chan_sip.c: Use snprintf instead of sprintf in one
+	  place. There is no vulnerability here due to various buffer sizes
+	  around the code, but I still didn't like seeing a non
+	  length-limited copy of data coming off of the wire into a stack
+	  buffer, as this would be a problem in the future if buffer sizes
+	  elsewhere got changed or size limitations removed ...
+
+2007-10-02 09:48 +0000 [r84345]  Christian Richter <christian.richter at beronet.com>
+
+	* channels/chan_misdn.c: terminate USERUSER String with 0
+
+2007-10-01 21:52 +0000 [r84291]  Jason Parker <jparker at digium.com>
+
+	* Makefile, Makefile.rules, channels/Makefile: Add dist-clean
+	  support for subdirs. Change h323 to only remove the Makefile on a
+	  dist-clean, rather than a clean. This fixes a bug I found with
+	  trying to run make after a make clean
+
+2007-10-01 21:25 +0000 [r84274]  Dwayne M. Hubbard <dhubbard at digium.com>
+
+	* main/channel.c, main/manager.c, channels/chan_agent.c: moved
+	  get_base_channel() code from action_redirect to
+	  ast_channel_masquerade() for issue 7706 and BE-160
+
+2007-10-01 21:18 +0000 [r84273]  Steve Murphy <murf at digium.com>
+
+	* pbx/pbx_ael.c: Anything to keep gcc 4.2 happy...
+
+2007-10-01 21:07 +0000 [r84271]  Russell Bryant <russell at digium.com>
+
+	* main/utils.c, include/asterisk/lock.h: Fulfull a feature request
+	  from Qwell on the "core show locks" output. It will now note the
+	  lock type for each lock that a thread holds. (mutex, rdlock, or
+	  wrlock)
+
+2007-10-01 20:27 +0000 [r84239]  Steve Murphy <murf at digium.com>
+
+	* pbx/ael/ael.tab.c, pbx/ael/ael.y, pbx/pbx_ael.c: closes issue
+	  #10777 -- by returning a null for the parse tree when there's
+	  really nothing there, and making sure we don't try to do checking
+	  on a null tree.
+
+2007-10-01 19:56 +0000 [r84166-84236]  Russell Bryant <russell at digium.com>
+
+	* res/res_agi.c: Add another sanity check in the AGI read loop. We
+	  really don't care about EAGAIN unless we didn't read an entire
+	  line. If there is a newline at the end if the read buffer, break,
+	  because we got the whole thing. (reported and patched by bmd)
+
+	* include/asterisk/lock.h: Show rwlocks in the "core show locks"
+	  output. Before, it only showed mutexes.
+
+	* channels/Makefile: Remove another file in "make clean". (closes
+	  issue #10814, paravoid)
+
+	* apps/app_dial.c: Simplify the CAN_EARLY_BRIDGE macro a bit.
+
+2007-10-01 14:10 +0000 [r84158-84163]  Joshua Colp <jcolp at digium.com>
+
+	* configs/usbradio.conf.sample (removed): Remove chan_usbradio
+	  config file from tree, it is not present in here. (closes issue
+	  #10839) Reported by: casper
+
+	* res/res_musiconhold.c: Fix randomness. save_pos was being set to
+	  0 initially instead of -1, causing it to jump to position 0 when
+	  moh started. (closes issue #10859) Reported by: jamesgolovich
+	  Patches: asterisk-mohpos2.diff.txt uploaded by jamesgolovich
+	  (license 176)
+
+	* apps/app_dial.c: Only attempt early bridging if the options given
+	  to Dial() permit it. (closes issue #10861) Reported by: peekyb
+
+2007-09-30 20:02 +0000 [r84146]  Russell Bryant <russell at digium.com>
+
+	* include/asterisk/module.h: Fix the AST_MODULE_INFO macro for C++
+	  modules. The load and reload parameters were in the wrong place.
+	  (closes issue #10846, alebm)
+
+2007-09-29 23:00 +0000 [r84133-84135]  Steve Murphy <murf at digium.com>
+
+	* pbx/ael/ael-test/ael-ntest22/t1/a.ael (added),
+	  pbx/ael/ael-test/ael-ntest22/t1/b.ael (added),
+	  pbx/ael/ael-test/ael-ntest22/t1/c.ael (added),
+	  pbx/ael/ael-test/ael-ntest22/t2/d.ael (added),
+	  pbx/ael/ael-test/ael-ntest22/t2/e.ael (added),
+	  pbx/ael/ael-test/ael-ntest22/t2/f.ael (added),
+	  pbx/ael/ael-test/ref.ael-test2, pbx/ael/ael-test/ref.ael-ntest22
+	  (added), pbx/ael/ael-test/ael-ntest22/t3/g.ael (added),
+	  pbx/ael/ael-test/ref.ael-test3,
+	  pbx/ael/ael-test/ael-ntest22/t3/h.ael (added),
+	  pbx/ael/ael-test/ref.ael-test4,
+	  pbx/ael/ael-test/ael-ntest22/t3/i.ael (added),
+	  pbx/ael/ael-test/ael-ntest22/t3/j.ael (added),
+	  pbx/ael/ael-test/ael-ntest22/qq.ael (added),
+	  pbx/ael/ael-test/ael-ntest22/t1 (added),
+	  pbx/ael/ael-test/ael-ntest22/t2 (added),
+	  pbx/ael/ael-test/ael-ntest22/t3 (added),
+	  pbx/ael/ael-test/ael-ntest22/extensions.ael (added),
+	  pbx/ael/ael-test/ael-ntest22 (added): This is a regression update
+	  that matches what I did in 84134 for AEL regressions.
+
+	* pbx/ael/ael_lex.c, pbx/ael/ael.flex: This issue sort of closes
+	  10786; All config files support #include with globbing (you know,
+	  *,[chars],?,{list,list},etc), so I've updated the AEL system to
+	  support this also.
+
+2007-09-28 14:13 +0000 [r84049-84078]  Tilghman Lesher <tlesher at digium.com>
+
+	* main/say.c: Correct pronunciations of numbers for .nl (Closes
+	  issue #10837)
+
+	* main/channel.c: Avoid a deadlock with ALL of the locks in the
+	  masquerade function, not just the pairs of channels. (Closes
+	  issue #10406)
+
+2007-09-27 23:12 +0000 [r84018]  Dwayne M. Hubbard <dhubbard at digium.com>
+
+	* main/manager.c, channels/chan_agent.c,
+	  include/asterisk/channel.h: if an Agent is redirected, the base
+	  channel should actually be redirected. This was causing multiple
+	  issues, especially issue 7706 and BE-160
+
+2007-09-27 00:01 +0000 [r83976]  Russell Bryant <russell at digium.com>
+
+	* pbx/pbx_dundi.c: remove a todo item that has been completed
+
+2007-09-26 23:53 +0000 [r83974]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* channels/chan_alsa.c: avoid the weird usage of assert() in the
+	  ALSA header files that gcc 4.2 wants to complain about
+
+2007-09-26 21:35 +0000 [r83910-83943]  Russell Bryant <russell at digium.com>
+
+	* channels/chan_sip.c: I changed my mind ... I think this should be
+	  a LOG_NOTICE.
+
+	* channels/chan_sip.c: Add a log message that was requested by the
+	  masses in the developer tutorial session at Astricon. chan_sip
+	  did not output any message when a call was rejected because the
+	  extension was not found. This adds a verbose message (at verbose
+	  level 3) to note when this happens.
+
+	* channels/chan_misdn.c: Fix building chan_misdn under dev-mode.
+	  (please run the configure script with --enable-dev-mode so this
+	  doesn't happen again ...)
+
+2007-09-26 18:35 +0000 [r83879]  Tilghman Lesher <tlesher at digium.com>
+
+	* channels/chan_zap.c: Remove unused 4k of memory on the program
+	  stack (closes issue #10827)
+
+2007-09-25 14:13 +0000 [r83637-83773]  Tilghman Lesher <tlesher at digium.com>
+
+	* main/app.c: jmls pointed out that unsetting the group and setting
+	  the group to the blank string aren't quite the same.
+
+	* build_tools/make_defaults_h: In the source, keys are relative to
+	  the datadir, not varlib (which is the same in most cases, but
+	  it's good to be accurate). Closes issue #10811
+
+	* doc/realtime.txt: Oops. Removed the unworkable workaround. This
+	  note should never have been in the release.
+
+	* main/app.c: Making change to group splitting, as discussed on the
+	  -dev list. The main effect of this will be to permit
+	  Set(GROUP([cat])=), i.e. unsetting a group.
+
+2007-09-24 07:54 +0000 [r83620]  Christian Richter <christian.richter at beronet.com>
+
+	* channels/chan_misdn.c: fixed round_robin group dial method, this
+	  never worked well on BRI Ports (2 channels)
+
+2007-09-22 19:39 +0000 [r83558-83589]  Steve Murphy <murf at digium.com>
+
+	* pbx/pbx_ael.c: This closes issue #10788 -- The exact same fixes
+	  are made here for the first arg in the for(arg1; arg2; arg3) {}
+	  statement, as were done for the 3rd arg. It can now be an
+	  assignment that will embedded in a Set() app, or a macro call, or
+	  an app call.
+
+	* pbx/pbx_ael.c: This closes issue #10788 -- the 3rd arg in the for
+	  statement is now wrapped in Set() only if there's an '=' in that
+	  string. Otherwise, if it begins with '&', then a Macro call is
+	  generated; otherwise it is made into an app call. A bit more
+	  accomodating, keeps the new guys happy, and the guys with ael-1
+	  code should be happy, too
+
+2007-09-21 14:37 +0000 [r83432]  Russell Bryant <russell at digium.com>
+
+	* main/rtp.c, channels/misdn_config.c, main/cdr.c, main/channel.c,
+	  channels/chan_misdn.c, pbx/ael/ael.tab.c, main/ast_expr2f.c,
+	  main/file.c, include/asterisk/sched.h, channels/chan_h323.c,
+	  pbx/pbx_dundi.c, utils/ael_main.c, main/ast_expr2.fl,
+	  channels/chan_mgcp.c, main/sched.c, res/res_config_pgsql.c,
+	  main/dnsmgr.c, channels/chan_sip.c, pbx/ael/ael.y,
+	  main/db1-ast/hash/hash.c, include/asterisk/channel.h,
+	  channels/chan_iax2.c: gcc 4.2 has a new set of warnings dealing
+	  with cosnt pointers. This set of changes gets all of Asterisk
+	  (minus chan_alsa for now) to compile with gcc 4.2. (closes issue
+	  #10774, patch from qwell)
+
+2007-09-21 13:34 +0000 [r83400]  Joshua Colp <jcolp at digium.com>
+
+	* channels/chan_sip.c: Fix video under certain circumstances. It
+	  would have been possible for the formats on the channel to not
+	  contain the video format. (closes issue #10782) Reported by:
+	  cwhuang
+
+2007-09-20 21:16 +0000 [r83316-83348]  Russell Bryant <russell at digium.com>
+
+	* main/asterisk.c: When daemonizing, don't change working directory
+	  to "/". It makes it not be able to do a core dump when not
+	  running as uid=root. (closes issue #10766, xrg)
+
+	* contrib/scripts/safe_asterisk: Change safe_asterisk to explicitly
+	  ask for /bin/bash, as it uses bashisms. (closes issue #10772,
+	  reported by culrich)
+
+2007-09-20 17:09 +0000 [r83246]  Jason Parker <jparker at digium.com>
+
+	* apps/app_disa.c: If # is pressed after dialing an extension in
+	  DISA, stop trying to collect more digits. (issue #10754) Reported
+	  by: atis Patches: app_disa.c.branch.patch uploaded by atis
+	  (license 242) app_disa.c.trunk.patch uploaded by atis (license
+	  242)
+
+2007-09-20 16:25 +0000 [r83230-83232]  Joshua Colp <jcolp at digium.com>
+
+	* channels/chan_sip.c: Make sure the minimum T1 timer value is
+	  obeyed in all cases. (closes issue #10768) Reported by: flefoll
+	  Patches: chan_sip.c.trunk.83071.retrans-patch uploaded by flefoll
+	  (license 244) chan_sip.c.br14.83070.retrans-patch uploaded by
+	  flefoll (license 244)
+
+	* channels/chan_sip.c: Fix a minor spelling error. (closes issue
+	  #10769) Reported by: flefoll Patches:
+	  chan_sip.c.trunk.83071.inita-patch uploaded by flefoll (license
+	  244) chan_sip.c.br14.83070.inita-patch uploaded by flefoll
+	  (license 244)
+
+2007-09-19 19:50 +0000 [r83121-83179]  Russell Bryant <russell at digium.com>
+
+	* apps/app_system.c: The System() and TrySystem() applications can
+	  take a substantial amount of time to execute while not servicing
+	  the channel. So, put the channel in autoservice while the command
+	  is being executed. (closes issue #10726, reported by mnicholson)
+
+	* funcs/func_curl.c: Using curl can take a substantial amount of
+	  time, so the channel should be autoserviced while waiting for it
+	  to complete. (closes issue #10725, reported by mnicholson)
+
+	* channels/chan_iax2.c: When handling a reload of chan_iax2, don't
+	  use an ao2_callback() to POKE all peers. Instead, use an
+	  iterator. By using an iterator, the peers container is not locked
+	  while the POKE is being done. It can cause a deadlock if the
+	  peers container is locked because poking a peer will try to lock
+	  pvt structs, while there is a lot of other code that will hold a
+	  pvt lock when trying to go lock the peers container. (reported to
+	  me directly by Loic Didelot. Thank you for the debug info!)
+
+	* main/manager.c: Fix up another potential race condition. Do the
+	  loop decrementing use count on events with the eventq protected
+	  from being changed. (reported on IRC by Ivan)
+
+2007-09-19 13:47 +0000 [r83070-83074]  Joshua Colp <jcolp at digium.com>
+
+	* apps/app_queue.c: Protect the CDR record from modification by
+	  pbx_exec so that the application data contains the Queue data.
+	  (closes issue #10761) Reported by: snar Patches:
+	  app-queue-mixmonitor.patch uploaded by snar (license 245)
+
+	* channels/chan_sip.c: (closes issue #10760) Reported by: dimas
+	  Patches: chan_sip.patch uploaded by dimas (license 88) Read in
+	  subscribecontext option in general to be the default.
+
+2007-09-19 09:32 +0000 [r83023-83024]  Christian Richter <christian.richter at beronet.com>
+
+	* channels/chan_misdn.c: removed comment which violates the coding
+	  guidelines.
+
+	* channels/misdn_config.c, channels/chan_misdn.c,
+	  channels/misdn/chan_misdn_config.h: added 'astdtmf' option to
+	  allow configuring the asterisk dtmf detector instead of the
+	  mISDN_dsp ones. also added the patch from irroot #10190, so that
+	  dtmf tones detected by the asterisk detector are passed outofband
+	  to asterisk, to make any use of dtmf tones at all.
+
+2007-09-19 00:19 +0000 [r82992]  Russell Bryant <russell at digium.com>
+
+	* apps/app_flash.c: Change the description of app_flash to note how
+	  it can be a useful tool instead of just saying that it is
+	  generally a worthless feature. (Thanks to Jim Van Meggelen for
+	  pointing it out and providing the proposed text)
+
+2007-09-18 23:41 +0000 [r82961]  Joshua Colp <jcolp at digium.com>
+
+	* apps/app_queue.c: Initialize a variable to NULL to make the world
+	  happy.
+
+2007-09-18 22:42 +0000 [r82929]  Russell Bryant <russell at digium.com>
+
+	* include/asterisk/agi.h, res/res_agi.c: Add a new patch to handle
+	  interrupting the fgets() call when using FastAGI. This version of
+	  the patch maintains the original behavior of the code when not
+	  using FastAGI. (closes issue #10553) Reported by: juggie Patches:
+	  res_agi_fgets-4.patch uploaded by juggie (license 24)
+	  res_agi_fgets_1.4svn.patch uploaded by juggie (license 24) Slight
+	  mods by me Tested by: juggie, festr
+
+2007-09-18 21:49 +0000 [r82887-82913]  Doug Bailey <dbailey at digium.com>
+
+	* main/manager.c: Corrected patch applied in revision r82887.
+
+	* main/manager.c: Fixed a bug where http manager sessions prevented
+	  the eventq from being cleaned out because http manager sessions
+	  do not have a valid file descriptor.
+
+2007-09-18 20:56 +0000 [r82867]  Russell Bryant <russell at digium.com>
+
+	* main/manager.c: Fix a memory leak that can occur on systems under
+	  higher load. The issue is that when events are appended to the
+	  master event queue, they use the number of active sessions as a
+	  use count so it will know when all active sessions at the time
+	  the event happened have consumed it. However, the handling of the
+	  number of sessions was not properly synchronized, so the use
+	  count was not always correct, causing an event to disappear
+	  early, or get stuck in the event queue for forever. (closes issue
+	  #9238, reported by bweschke, patch from Ivan, modified by me)
+
+2007-09-18 20:09 +0000 [r82865]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_queue.c: Moving the logic for handling an empty
+	  membername to the create_member function so that there is a
+	  common place where this occurs instead of being spread out to
+	  several different places.
+
+2007-09-18 18:59 +0000 [r82834]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* apps/app_queue.c: there is no need for conditional logic to
+	  select ->interface or ->membername, snince ->membername will
+	  always be populated
+
+2007-09-18 16:31 +0000 [r82802]  Russell Bryant <russell at digium.com>
+
+	* pbx/pbx_dundi.c: When copying the contents from the wildcard
+	  peer, do a deep copy instead of shallow copy so that it doesn't
+	  crash when beging destroyed. (closes issue #10546, patch by me)
+
+2007-09-18 15:28 +0000 [r82751]  Jason Parker <jparker at digium.com>
+
+	* configs/sip.conf.sample: Correct the allowexternaldomains option
+	  in SIP sample config. Issue 10753
+
+2007-09-17 20:16 +0000 [r82594-82676]  Russell Bryant <russell at digium.com>
+
+	* apps/app_voicemail.c, main/stdtime/localtime.c: Put a memset in
+	  ast_localtime() instead of a couple places in app_voicemail to
+	  prevent the problem everywhere instead of just a couple of
+	  places. (related to issue #10746)
+
+	* apps/app_voicemail.c: Initialize some memory to fix crashes when
+	  leaving voicemail. This problem was fixed by running Asterisk
+	  under valgrind. (closes issue #10746, reported by arcivanov,
+	  patched by me) *** IMPORTANT NOTE: We need to check to see if
+	  this same bug exists elsewhere.
+
+	* res/res_features.c: Handle the case where there are multiple
+	  dynamic features with the same digit mapping, but won't always
+	  match the activated on/by access controls. In that case, the code
+	  needs to keep trying features for a match. (reported by Atis on
+	  the asterisk-dev list, patched by me)
+
+2007-09-17 16:40 +0000 [r82590-82592]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* channels/chan_iax2.c: revert a change that wasn't supposed to be
+	  committed... doh!
+
+	* apps/app_queue.c, channels/chan_iax2.c: fix a couple of places
+	  where a logical member name (if specified) was not used, but
+	  instead the direct interface was listed
+
+2007-09-17 02:00 +0000 [r82514]  Joshua Colp <jcolp at digium.com>
+
+	* main/pbx.c: (closes issue #10734) Reported by: asgaroth Instead
+	  of passing a NULL pointer into snprintf pass "". It makes Solaris
+	  much happier.
+
+2007-09-14 21:19 +0000 [r82444]  Steve Murphy <murf at digium.com>
+
+	* main/cdr.c: closes issue #10668; thanks to arkadia for his patch;
+	  had to leave out the bit about ending the previous cdr in the
+	  fork; it would destroy current implementations.
+
+2007-09-14 21:17 +0000 [r82435]  Russell Bryant <russell at digium.com>
+
+	* configs/zapata.conf.sample: Add a note to help clarify the value
+	  set with the echocancel option. (inspired by Malcolm's blog post
+	  on blogs.digium.com about HPEC)
+
+2007-09-14 18:35 +0000 [r82396-82398]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_queue.c: Crap, I broke the build. Fixed.
+
+	* apps/app_queue.c: Adding member name field to manager events
+	  where they were missing before (closes issue #10721, reported by
+	  snar)
+
+2007-09-14 17:48 +0000 [r82394]  Jason Parker <jparker at digium.com>
+
+	* channels/chan_zap.c: If a channel does not have an owner, do not
+	  try to set a channel variable. This will end up making the
+	  channel variable global, which is not right. Closes issue #10720,
+	  patch by flefoll.
+
+2007-09-14 15:50 +0000 [r82382-82385]  Russell Bryant <russell at digium.com>
+
+	* build_tools/menuselect-deps.in, configure,
+	  include/asterisk/autoconfig.h.in, configure.ac, makeopts.in: Add
+	  checking for libusb here, so nobody has to deal with conflicts in
+	  the chan_usbradio-1.4 branch every time the configure script gets
+	  changed
+
+	* channels/chan_usbradio.c (removed), channels/xpmr (removed),
+	  channels/Makefile: Remove chan_usbradio from the main 1.4 branch.
+	  It can't live here because we have a strict policy to not include
+	  new features in release branches. However, I'm going to merge it
+	  into trunk, and I also have a special 1.4 based branch that
+	  includes this module. svn co
+	  http://svn.digium.com/svn/asterisk/team/jdixon/chan_usbradio-1.4
+
+2007-09-14 14:42 +0000 [r82376]  Mark Michelson <mmichelson at digium.com>
+
+	* doc/CODING-GUIDELINES: Fixing a typo in the coding guidelines
+	  (closes issue #10717, reported and patched by leedm777)
+
+2007-09-14 01:24 +0000 [r82368]  Jim Dixon <telesistant at hotmail.com>
+
+	* apps/app_rpt.c: Fixed problem with changes made to cdr
+	  functionality
+
+2007-09-14 00:52 +0000 [r82367]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* channels/chan_usbradio.c: this new driver may not live in this
+	  branch for long (since it is a new feature), but it definitely
+	  should not be built by default
+
+2007-09-14 00:34 +0000 [r82366]  Jim Dixon <telesistant at hotmail.com>
+
+	* apps/app_rpt.c, channels/xpmr/xpmr_coef.h (added),
+	  channels/chan_usbradio.c (added), channels/xpmr/xpmr.h (added),
+	  channels/xpmr (added), channels/xpmr/LICENSE (added),
+	  channels/xpmr/sinetabx.h (added), configs/usbradio.conf.sample
+	  (added), channels/Makefile, channels/xpmr/xpmr.c (added): Added
+	  channel driver for USB Radio device and support thereof.
+
+2007-09-13 23:11 +0000 [r82358]  Jason Parker <jparker at digium.com>
+
+	* pbx/pbx_spool.c: Fix a small typo. retrytime > waittime
+
+2007-09-13 20:16 +0000 [r82346]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_queue.c: Preemptively fixing a possible segfault. It is
+	  possible that queuename is NULL (meaning pause ALL queues), so
+	  use q->name instead.
+
+2007-09-13 20:11 +0000 [r82344]  Jason Parker <jparker at digium.com>
+
+	* cdr/cdr_csv.c: Fix a crash that could occur in cdr_csv when
+	  mutliple threads tried to close the same file. Do we actually
+	  need the locking here? What happens if you open the same file
+	  twice, and two threads try to write to it at the same time? Is
+	  fputs() going to write out the entire line at once? I suspect
+	  that it could be possible for the second fopen to run during the
+	  first fputs, so the position could be in the middle of the
+	  previously written line... Issue 10347, initial patch by
+	  explidous (but I removed all of the paranoia stuff..)
+
+2007-09-13 18:57 +0000 [r82337-82339]  Russell Bryant <russell at digium.com>
+
+	* main/astobj2.c: resolve a warning when not building under dev
+	  mode
+
+	* main/astobj2.c, main/asterisk.c, include/asterisk.h: Only compile
+	  in tracking astobj2 statistics if dev-mode is enabled. Also, when
+	  dev mode is enabled, register the CLI command that can be used to
+	  run the astobj2 test and print out statistics.
+
+2007-09-13 18:12 +0000 [r82335]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* /, LICENSE: Merged revisions 82334 via svnmerge from
+	  https://origsvn.digium.com/svn/asterisk/branches/1.2 ........
+	  r82334 | kpfleming | 2007-09-13 11:10:12 -0700 (Thu, 13 Sep 2007)
+	  | 2 lines clarify the OpenSSL and OpenH323 license exceptions
+	  ........
+
+2007-09-13 16:25 +0000 [r82326]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_queue.c: Added logic to handle the unlikely case that
+	  someone has two queues with the same name. Asterisk will log a
+	  warning message letting the user know that one was already
+	  defined with that name and is it skipping all further instances.
+	  This also will work for realtime queues but in order for that to
+	  happen, the user would have to trigger a perfectly timed reload
+	  as a realtime queue is being looked up, which is highly unlikely
+	  (but taken care of nonetheless).
+
+2007-09-13 11:47 +0000 [r82309]  Philippe Sultan <philippe.sultan at gmail.com>
+
+	* channels/chan_gtalk.c: Closes issue #9401, reported and patched
+	  by irrot, with slight modifications by me. Handle DTMF sent by
+	  Asterisk properly.
+
+2007-09-12 21:56 +0000 [r82296]  Russell Bryant <russell at digium.com>
+
+	* res/res_agi.c: Fix a check of the wrong pointer, as pointed out
+	  by an XXX comment left in the code. The problem was harmless,
+	  however.
+
+2007-09-12 21:28 +0000 [r82291]  Tilghman Lesher <tlesher at digium.com>
+
+	* main/stdtime/tzfile.h: Oops, wrong location for FreeBSD zone
+	  files
+
+2007-09-12 20:24 +0000 [r82286]  Dwayne M. Hubbard <dhubbard at digium.com>
+
+	* apps/app_meetme.c: remove a race condition for the creation of
+	  recordthread's, and fix a small memory leak. This closes issue#
+	  10636
+
+2007-09-12 20:12 +0000 [r82285]  Tilghman Lesher <tlesher at digium.com>
+
+	* main/stdtime/private.h, main/stdtime/tzfile.h,
+	  include/asterisk/localtime.h, main/stdtime/localtime.c: Working
+	  on issue #10531 exposed a rather nasty 64-bit issue on
+	  ast_mktime, so we updated the localtime.c file from source. Next
+	  we'll have to write ast_strptime to match.
+
+2007-09-12 15:16 +0000 [r82278-82280]  Russell Bryant <russell at digium.com>
+
+	* main/asterisk.c: Clean up the output of "asterisk -h". This
+	  tweaks the wording and wraps lines at 80 characters. (closes
+	  issue #10699, seanbright)
+
+	* res/res_agi.c: revert patch from issue #10553, as someone not
+	  using fastagi reported that this broke their system.
+
+2007-09-12 14:30 +0000 [r82274-82276]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_voicemail.c: Accidentally committed changes to
+	  app_voicemail which do NOT need to be in the 1.4 branch yet.
+	  reverting...
+
+	* apps/app_voicemail.c, apps/app_queue.c: We should only initialize
+	  a realtime queue when it is allocated, not every time we access
+	  it. This prevents the members ao2_container from being
+	  reallocated every time the queue is accessed. I also removed a
+	  debug message I had accidentally left in on a previous commit.
+
+2007-09-11 22:37 +0000 [r82267]  Russell Bryant <russell at digium.com>
+
+	* apps/app_queue.c: Fix incorrect uses of ao2_find(). Every one of
+	  these calls was reading bogus memory ...
+
+2007-09-11 21:41 +0000 [r82265]  Joshua Colp <jcolp at digium.com>
+
+	* codecs/gsm/src/lpc.c, codecs/gsm/src/long_term.c: (closes issue
+	  #10679) Reported by: andrew Build under dev mode when K6OPTS is
+	  enabled.
+
+2007-09-11 20:49 +0000 [r82263]  Russell Bryant <russell at digium.com>
+
+	* apps/app_queue.c: Fix another missing unref of member objects.
+	  This one was pointed out by Marta. When building the outgoing
+	  list in try_calling(), a member reference is stored in each
+	  outgoing entry. However, when this list got destroyed, the
+	  reference was not released.
+
+2007-09-11 20:36 +0000 [r82261]  Steve Murphy <murf at digium.com>
+
+	* main/cdr.c: this change should fix issue # 10659 -- what I worry
+	  about is how many other bug reports it may generate. Hopefully,
+	  we can please the/a majority. Hopefully. We shall see. Calls not
+	  marked ANSWERED and with only one channel name will not be
+	  posted. This should eliminate the double CDR's.
+
+2007-09-11 16:05 +0000 [r82252]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_queue.c: All instances of ao2_iterators which were just
+	  named 'i' have been renamed to 'mem_iter' so that when refcounted
+	  queues are merged into trunk, there will be little confusion
+	  regarding iterator names, especially when a queue and member
+	  iterator are used in the same function.
+
+2007-09-11 16:03 +0000 [r82250]  Russell Bryant <russell at digium.com>
+
+	* pbx/pbx_dundi.c: The sample dundi.conf claims support for a
+	  wildcard peer entry - [*], but the code did not support it. This
+	  patch makes it work. (closes issue #10546, patch by dds, with
+	  some changes by me)
+
+2007-09-11 16:01 +0000 [r82249]  Christian Richter <christian.richter at beronet.com>
+
+	* channels/misdn/isdn_lib.c, channels/chan_misdn.c: fixed a
+	  hold/retrieve issue.
+
+2007-09-11 15:26 +0000 [r82245]  Russell Bryant <russell at digium.com>
+
+	* res/res_agi.c: (closes issue #10553) Reported by: juggie Patches:
+	  res_agi_fgets-2.patch uploaded by juggie (license 24) Tested by:
+	  juggie When using fastagi, fgets() can return before a full line
+	  is read. Add explicit handling for the case where it gets
+	  interrupted.
+
+2007-09-11 14:56 +0000 [r82243]  Joshua Colp <jcolp at digium.com>
+
+	* pbx/pbx_dundi.c: (closes issue #10577) Reported by: jamesgolovich
+	  Patches: asterisk-dundifree.diff.txt uploaded by jamesgolovich
+	  (license 176) Don't leak memory when unloading DUNDi.
+
+2007-09-11 14:34 +0000 [r82198-82240]  Russell Bryant <russell at digium.com>
+
+	* apps/app_queue.c: Add a couple more missing unrefs of queue
+	  member objects
+
+	* apps/app_queue.c: Add a missing unref of a queue member in an
+	  error handling block
+
+	* apps/app_queue.c: Document why membercount can not simply be
+	  replaced by ao2_container_count()
+
+	* main/astobj2.c: backport astobj2 race condition fix. This
+	  function is the exact same as trunk so it applies here as well.
+
+2007-09-10 18:02 +0000 [r82155]  Tilghman Lesher <tlesher at digium.com>
+
+	* apps/app_queue.c: Convert struct member to use refcounts (closes
+	  issue #10199)
+
+2007-09-10 15:02 +0000 [r82091]  Mark Michelson <mmichelson at digium.com>
+
+	* configs/misdn.conf.sample: Removing non-existent options from
+	  misdn configuration sample. (closes issue #10678, reported and
+	  patched by IgorG)
+
+2007-09-09 02:35 +0000 [r82028]  Tilghman Lesher <tlesher at digium.com>
+
+	* include/asterisk/lock.h: Fix inline compiles on really old
+	  compilers (who uses gcc 2.7 anymore, really?)
+
+2007-09-08 18:41 +0000 [r81952-81997]  Russell Bryant <russell at digium.com>
+
+	* main/asterisk.c: Fix a small memory leak. ast_unregister_atexit()
+	  did not free the entry it removed.
+
+	* .cleancount: (closes issue #10672) Bump the cleancount so that a
+	  "make clean" will be forced. This is needed because my fix in
+	  revision 81599 made a change to a data structure in file.h, and
+	  since file dependency tracking is only on with dev-mode enabled,
+	  file format modules that don't get rebuilt may crash, as is the
+	  case with this issue. This makes me wonder - how much faster does
+	  the code build without the file dependency tracking enabled? If
+	  it doesn't make much of a difference, then it may be worth just
+	  keeping it on all of the time, or perhaps just not in release
+	  tarballs, so that this type of issue is avoided.
+
+2007-09-07 19:48 +0000 [r81923]  Jason Parker <jparker at digium.com>
+

[... 11472 lines stripped ...]



More information about the asterisk-commits mailing list