[asterisk-commits] russell: tag 1.4.12 r84479 - in /tags/1.4.12: .lastclean .version ChangeLog

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Tue Oct 2 15:11:17 CDT 2007


Author: russell
Date: Tue Oct  2 15:11:17 2007
New Revision: 84479

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

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

Added: tags/1.4.12/.lastclean
URL: http://svn.digium.com/view/asterisk/tags/1.4.12/.lastclean?view=auto&rev=84479
==============================================================================
--- tags/1.4.12/.lastclean (added)
+++ tags/1.4.12/.lastclean Tue Oct  2 15:11:17 2007
@@ -1,0 +1,1 @@
+29

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

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

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

Added: tags/1.4.12/.version
URL: http://svn.digium.com/view/asterisk/tags/1.4.12/.version?view=auto&rev=84479
==============================================================================
--- tags/1.4.12/.version (added)
+++ tags/1.4.12/.version Tue Oct  2 15:11:17 2007
@@ -1,0 +1,1 @@
+1.4.12

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

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

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

Added: tags/1.4.12/ChangeLog
URL: http://svn.digium.com/view/asterisk/tags/1.4.12/ChangeLog?view=auto&rev=84479
==============================================================================
--- tags/1.4.12/ChangeLog (added)
+++ tags/1.4.12/ChangeLog Tue Oct  2 15:11:17 2007
@@ -1,0 +1,12146 @@
+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>
+
+	* apps/app_queue.c: Allow the MEMBERINTERFACE variable to be used
+	  as the mixmonitor filename. This moves the setting of the
+	  MEMBERINTERFACE variable to before mixmonitor. Issue 10671, patch
+	  by sim.
+
+2007-09-07 15:25 +0000 [r81886]  Mark Michelson <mmichelson at digium.com>
+
+	* configs/queues.conf.sample: Moving the explanation for joinempty
+	  to a more appropriate place
+
+2007-09-06 22:28 +0000 [r81832]  Russell Bryant <russell at digium.com>
+
+	* channels/chan_sip.c: (closes issue #9724, closes issue #10374)
+	  Reported by: kenw Patches: 9724.txt uploaded by russell (license
+	  2) Tested by: kenw, russell Resolve a deadlock that occurs when
+	  doing a SIP transfer to parking. I come across this type of
+	  deadlock fairly often it seems. It is very important to mind the
+	  boundary between the channel driver and the core in respect to
+	  the channel lock and the channel-pvt lock. Channel drivers lock
+	  to lock the pvt and then the channel once it calls into the core,
+	  while the core will do it in the opposite order. The way this is
+	  avoided is by having channel drivers either release their pvt
+	  lock while calling into the core, or such as in this case,
+	  unlocking the pvt just long enough to acquire the channel lock.
+
+2007-09-06 22:05 +0000 [r81778-81826]  Jason Parker <jparker at digium.com>
+
+	* Makefile: We added COPTS for ASTCFLAGS additions, but not LDOPTS
+	  for ASTLDFLAGS. This adds LDOPTS
+
+	* include/asterisk/astobj2.h: This should fix a build issue that
+	  people building against uClibc were seeing with the addition of
+	  astobj2
+
+2007-09-06 19:40 +0000 [r81776]  Joshua Colp <jcolp at digium.com>
+
+	* apps/app_meetme.c: (closes issue #10122) Reported by:
+	  stevefeinstein Patches: meetme-unmute-manager.diff uploaded by
+	  qwell (license 4) Tested by: stevefeinstein After looking over
+	  the code I agree with Qwell. Setting the file descriptor to
+	  conference each time just causes a fight back and forth.
+
+2007-09-06 16:56 +0000 [r81743]  Philippe Sultan <philippe.sultan at gmail.com>
+
+	* include/asterisk/jabber.h, channels/chan_gtalk.c: Various string
+	  length fixes. Removed an unused variable in aji_client structure
+	  (context)
+
+2007-09-06 16:25 +0000 [r81682-81713]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_queue.c: Fixes an issue where valid DTMF had to be
+	  pressed twice to exit a queue if a member's phone was ringing.
+	  (closes issue #10655, reported by strider2k, patched by me)
+
+	* res/res_features.c: Fixes a memory leak (closes issue #10658,
+	  reported and patched by Ivan)
+
+2007-09-06 14:20 +0000 [r81650]  Philippe Sultan <philippe.sultan at gmail.com>
+
+	* res/res_jabber.c: According to both RFC 3920 - section 9.1.2 -
+	  and Google's XMPP server complaint, if set, the 'from' attribute
+	  must be set to the user's full JID.
+
+2007-09-05 20:53 +0000 [r81599]  Russell Bryant <russell at digium.com>
+
+	* include/asterisk/file.h, main/say.c, res/res_features.c,
+	  main/file.c, include/asterisk/channel.h: Fix an issue that can
+	  occur when you do an attended transfer to parking. If you
+	  complete the transfer before the announcement of the parking spot
+	  finishes, then the channel being parked will hear the remainder
+	  of the announcement. These changes make it so that will not
+	  happen anymore. Basically, res_features sets a flag on the
+	  channel is playing the announcement to so that the file streaming
+	  core knows that it needs to watch out for a channel masquerade,
+	  and if it occurs, to abort the announcement. (closes BE-182)
+
+2007-09-05 17:18 +0000 [r81569]  Tilghman Lesher <tlesher at digium.com>
+
+	* include/asterisk/lock.h: Solaris x86 compatibility fix
+
+2007-09-05 15:19 +0000 [r81525]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_queue.c: Fixing the build...
+
+2007-09-05 15:14 +0000 [r81523]  Jason Parker <jparker at digium.com>
+
+	* channels/chan_phone.c: Do not try to unregister a NULL channel
+	  tech. Also changed load_module function to use defines rather
+	  than numbers for return values. Issue 10651, patch by
+	  rbraun_proformatique, with additions by me.
+
+2007-09-05 15:03 +0000 [r81520]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_queue.c: Reverting behavior of QUEUE_MEMBER_COUNT to
+	  only count members who are logged in and available. (related to
+	  issue #10652, reported by wuwu)
+
+2007-09-05 13:11 +0000 [r81492]  Joshua Colp <jcolp at digium.com>
+
+	* main/channel.c: (closes issue #10650) Reported by: tacvbo Only
+	  print out that the spy was removed while holding the spy lock.
+
+2007-09-04 20:54 +0000 [r81453-81455]  Jason Parker <jparker at digium.com>
+
+	* apps/app_followme.c: Rather than attempt to play a file, we can
+	  just check whether it exists. Issue 10634, patch by me, testing
+	  by pabelanger, sanity checked by bweschke
+
+	* configs/followme.conf.sample: Change default followme config file
+	  to point to the correct files. Issue 10644, patch by pabelanger
+
+2007-09-04 18:37 +0000 [r81448]  Russell Bryant <russell at digium.com>
+
+	* main/astobj2.c, include/asterisk/astobj2.h, channels/chan_iax2.c:
+	  Remove the typedefs on ao2_container and ao2_iterator. This is
+	  simply because we don't typedef objects anywhere else in
+	  Asterisk, so we might as well make this follow the same
+	  convention.
+
+2007-09-04 16:40 +0000 [r81442]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* channels/chan_sip.c: there is no point in sending 401
+	  Unauthorized to a UAS that sent us a properly-formatted
+	  Authentication header with the expected username and nonce but an
+	  incorrect response (which indicates the shared secret does not
+	  match)... instead, let's send 403 Forbidden so that the UAS
+	  doesn't retry with the same authentication credentials repeatedly
+
+2007-09-04 14:23 +0000 [r81435-81439]  Joshua Colp <jcolp at digium.com>
+
+	* channels/chan_iax2.c: (closes issue #10632) Reported by:
+	  jamesgolovich Patches: asterisk-iaxfirmwareleak.diff.txt uploaded
+	  by jamesgolovich (license 176) Fix memory leak when unloading
+	  chan_iax2. The firmware files were not being freed.
+
+	* main/channel.c: (closes issue #10476) Reported by: mdu113 Only
+	  look for the end of a digit when waiting for a digit. This in
+	  turn disables emulation in the core.
+
+	* main/dns.c: (closes issue #10610) Reported by: john Patches:
+	  dns.c.patch uploaded by john (license 218) Tested by: mvanbaak
+	  Don't return a match if no SRV record actually exists.
+
+2007-09-03 18:57 +0000 [r81433]  Russell Bryant <russell at digium.com>
+
+	* channels/chan_iax2.c: Remove a couple of calls to
+	  ast_string_field_free_pools() on peers in error handling blocks
+	  in the code for building peers. The peer object destructor does
+	  this and doing it twice will cause a crash. (closes issue #10625,
+	  reported by and patched by pnlarsson)
+
+2007-09-01 15:57 +0000 [r81426-81428]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_queue.c: Changed a comment to be more accurate. (really
+	  this is just a test to make sure I can commit properly from home)
+
+	* main/astobj2.c, include/asterisk/astobj2.h: Making match_by_addr
+	  into ao2_match_by_addr and making it available everywhere since
+	  it could be a handy callback to have
+
+2007-08-31 21:27 +0000 [r81418]  Russell Bryant <russell at digium.com>
+
+	* include/asterisk/astobj2.h: Remove references to a debugging
+	  parameter that does not exist
+
+2007-08-31 19:48 +0000 [r81416]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_queue.c: Fixed broken behavior of a reload on realtime
+	  queues. Prior to this patch, if a reload was issued and a
+	  realtime queue had callers waiting in it, then the queue would be

[... 11302 lines stripped ...]



More information about the asterisk-commits mailing list