[asterisk-commits] tilghman: tag 1.4.23-rc2 r160145 - /tags/1.4.23-rc2/

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Mon Dec 1 16:22:05 CST 2008


Author: tilghman
Date: Mon Dec  1 16:22:05 2008
New Revision: 160145

URL: http://svn.digium.com/view/asterisk?view=rev&rev=160145
Log:
Importing files for 1.4.23-rc2 release

Added:
    tags/1.4.23-rc2/.lastclean   (with props)
    tags/1.4.23-rc2/.version   (with props)
    tags/1.4.23-rc2/ChangeLog   (with props)

Added: tags/1.4.23-rc2/.lastclean
URL: http://svn.digium.com/view/asterisk/tags/1.4.23-rc2/.lastclean?view=auto&rev=160145
==============================================================================
--- tags/1.4.23-rc2/.lastclean (added)
+++ tags/1.4.23-rc2/.lastclean Mon Dec  1 16:22:05 2008
@@ -1,0 +1,1 @@
+33

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

Propchange: tags/1.4.23-rc2/.lastclean
------------------------------------------------------------------------------
    svn:keywords = none

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

Added: tags/1.4.23-rc2/.version
URL: http://svn.digium.com/view/asterisk/tags/1.4.23-rc2/.version?view=auto&rev=160145
==============================================================================
--- tags/1.4.23-rc2/.version (added)
+++ tags/1.4.23-rc2/.version Mon Dec  1 16:22:05 2008
@@ -1,0 +1,1 @@
+1.4.23-rc2

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

Propchange: tags/1.4.23-rc2/.version
------------------------------------------------------------------------------
    svn:keywords = none

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

Added: tags/1.4.23-rc2/ChangeLog
URL: http://svn.digium.com/view/asterisk/tags/1.4.23-rc2/ChangeLog?view=auto&rev=160145
==============================================================================
--- tags/1.4.23-rc2/ChangeLog (added)
+++ tags/1.4.23-rc2/ChangeLog Mon Dec  1 16:22:05 2008
@@ -1,0 +1,21078 @@
+2008-12-01  Tilghman Lesher <tilghman at digium.com>
+
+	* Released 1.4.23-rc2
+
+2008-12-01 17:27 +0000 [r160003]  Russell Bryant <russell at digium.com>
+
+	* channels/chan_iax2.c: Apply some logic used in iax2_indicate() to
+	  iax2_setoption(), as well, since they both have the potential to
+	  send control frames in the middle of call setup. We have to wait
+	  until we have received a message back from the remote end before
+	  we try to send any more frames. Otherwise, the remote end will
+	  consider it invalid, and we'll get stuck in an INVAL/VNAK storm.
+
+2008-12-01 16:08 +0000 [r159976]  Michiel van Baak <michiel at vanbaak.info>
+
+	* main/manager.c: Get rid of the useless format string and argument
+	  in the Bogus/ manager channelname. Noted by kpfleming and name
+	  Bogus/manager suggested by eliel
+
+2008-12-01 14:52 +0000 [r159900]  Russell Bryant <russell at digium.com>
+
+	* .cleancount: Force a "make clean" to avoid a bizarre build issue
+	  ...
+
+2008-12-01 14:05 +0000 [r159897]  Michiel van Baak <michiel at vanbaak.info>
+
+	* main/manager.c: make manager compile on OpenBSD. The last (10th)
+	  argument to ast_channel_alloc here should be a pointer and NULL
+	  is not really a pointer.
+
+2008-11-29 16:58 +0000 [r159808]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* main/enum.c, utils/frame.c, configure, res/res_agi.c,
+	  include/asterisk/module.h, main/logger.c, main/dns.c,
+	  include/asterisk/threadstorage.h, include/asterisk/utils.h,
+	  include/asterisk/devicestate.h, channels/chan_sip.c,
+	  include/asterisk/dundi.h, main/jitterbuf.c,
+	  channels/chan_agent.c, configure.ac, utils/astman.c,
+	  include/asterisk/cli.h, include/asterisk/channel.h,
+	  include/jitterbuf.h, include/asterisk/manager.h,
+	  main/ast_expr2.c, Makefile, include/asterisk/logger.h,
+	  include/asterisk/res_odbc.h, main/srv.c, channels/chan_misdn.c,
+	  include/asterisk/linkedlists.h, include/asterisk/lock.h,
+	  include/asterisk/strings.h, makeopts.in,
+	  include/asterisk/stringfields.h, utils/check_expr.c,
+	  channels/chan_vpb.cc, res/res_features.c, channels/chan_iax2.c:
+	  update dev-mode compiler flags to match the ones used by default
+	  on Ubuntu Intrepid, so all developers will see the same warnings
+	  and errors since this branch already had some printf format
+	  attributes, enable checking for them and tag functions that
+	  didn't have them format attributes in a consistent way
+
+2008-11-26 20:21 +0000 [r159476-159571]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* channels/chan_oss.c, channels/busy.h (removed),
+	  channels/ring_tone.h (added), channels/chan_alsa.c,
+	  channels/ringtone.h (removed), channels/busy_tone.h (added),
+	  channels/Makefile: rename these files so as to avoid conflicts
+	  when users update their working copies and have unversioned files
+	  already in place
+
+	* channels, agi/Makefile, utils/Makefile, channels/busy.h (added),
+	  Makefile.moddir_rules, Makefile.rules, channels/ringtone.h
+	  (added), channels/Makefile: simplify (and slightly bug-fix) the
+	  recent developer-oriented COMPILE_DOUBLE mode add channels/busy.h
+	  and channels/ringtone.h to the repository instead of generating
+	  them repeatedtly; most users do not change the settings to build
+	  them, but the Makefile rules are still there if they wish to do
+	  so ensure that 'make clean' removes dependency files for .i files
+	  that are created in COMPILE_DOUBLE mode
+
+2008-11-25 22:41 +0000 [r159316]  Steve Murphy <murf at digium.com>
+
+	* main/cdr.c, channels/chan_iax2.c: (closes issue #12694) Reported
+	  by: yraber Patches: 12694.2nd.diff uploaded by murf (license 17)
+	  Tested by: murf, laurav Thanks to file (Joshua Colp) for his IAX
+	  fix. the change to cdr.c allows no-answer to percolate up into
+	  CDR's, and feels like the right place to locate this fix; if BUSY
+	  is done here, no-answer should be, too.
+
+2008-11-25 21:56 +0000 [r159246-159269]  Tilghman Lesher <tlesher at digium.com>
+
+	* channels/chan_iax2.c: Don't try to send a response on a NULL pvt.
+	  (closes issue #13919) Reported by: barthpbx Patches:
+	  chan_iax2.c.patch uploaded by eliel (license 64) Tested by:
+	  barthpbx
+
+	* /, channels/chan_iax2.c: Merged revisions 159245 via svnmerge
+	  from https://origsvn.digium.com/svn/asterisk/branches/1.2
+	  ........ r159245 | tilghman | 2008-11-25 15:37:06 -0600 (Tue, 25
+	  Nov 2008) | 7 lines Regression fix for last security fix. Set the
+	  iseqno correctly. (closes issue #13918) Reported by: ffloimair
+	  Patches: 20081119__bug13918.diff.txt uploaded by Corydon76
+	  (license 14) Tested by: ffloimair ........
+
+2008-11-25 17:34 +0000 [r159158]  Russell Bryant <russell at digium.com>
+
+	* main/astobj2.c, include/asterisk/astobj2.h: Add ao2_trylock() to
+	  go along with ao2_lock() and ao2_unlock()
+
+2008-11-25 16:23 +0000 [r159096]  Terry Wilson <twilson at digium.com>
+
+	* apps/app_festival.c: Add missing variable declaration in the PPC
+	  code
+
+2008-11-25 04:50 +0000 [r159025]  Tilghman Lesher <tlesher at digium.com>
+
+	* apps/app_rpt.c, configure, include/asterisk/autoconfig.h.in,
+	  configure.ac: System call ioperm is non-portable, so check for
+	  its existence in autoconf. (Closes issue #13863)
+
+2008-11-22 00:04 +0000 [r158629]  Jeff Peeler <jpeeler at digium.com>
+
+	* include/asterisk/dahdi_compat.h, channels/chan_dahdi.c: (closes
+	  issue #13786) Reported by: tzafrir When compiling against Zaptel
+	  dahdi_compat will now only define all the DAHDI defines if the
+	  Zaptel define is present. Also, there is no such thing as
+	  DAHDI_PRI.
+
+2008-11-21 23:14 +0000 [r158603]  Steve Murphy <murf at digium.com>
+
+	* res/res_features.c: In reference to the fix made for 13871, I was
+	  merging the fix into 1.6.0 and realized I missed the code in the
+	  h-exten block, and didn't catch it because my test case had the
+	  h-exten commented out. So, this corrects the code I missed, as a
+	  preventative against another crash report. Tested with the
+	  h-exten defined, all is well.
+
+2008-11-21 23:07 +0000 [r158600]  Tilghman Lesher <tlesher at digium.com>
+
+	* main/pbx.c: The passed extension may not be the same in the list
+	  as the current entry, because we strip spaces when copying the
+	  extension into the structure. Therefore, use the copied item to
+	  place the item into the list. (found by lmadsen on -dev, fixed by
+	  me)
+
+2008-11-21 22:05 +0000 [r158539]  Russell Bryant <russell at digium.com>
+
+	* main/astobj2.c, include/asterisk/astobj2.h: When compiling with
+	  DEBUG_THREADS, report the real file/func/line for
+	  ao2_lock/ao2_unlock
+
+2008-11-21 21:19 +0000 [r158483]  Steve Murphy <murf at digium.com>
+
+	* res/res_features.c: (closes issue #13871) Reported by: mdu113
+	  This one is totally my fault. The code doesn't even create a
+	  bridge if the channel CDR has POST_DISABLED. I didn't check for
+	  that at the end of the bridge. Fixed with a few small insertions.
+	  Tested. Looks good. No cdr generated, no crash, no unnecc. data
+	  objects created either.
+
+2008-11-21 15:24 +0000 [r158053-158306]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_queue.c: This change had somehow gotten reverted due to
+	  a completely unrelated commit. Thanks to Theo Belder on the
+	  Asterisk-dev list for pointing this out.
+
+	* include/asterisk/file.h, main/frame.c, main/file.c,
+	  include/asterisk/frame.h: There was an issue when attempting to
+	  reference an embedded frame in a freed ast_filestream. This patch
+	  makes use of the ao2 functions to make sure that we do not free
+	  an ast_filestream structure until the embedded ast_frame has been
+	  "freed" as well. (closes issue #13496) Reported by: fst-onge
+	  Patches: filestream_frame_1_4.diff uploaded by putnopvut (license
+	  60) Tested by: putnopvut Closes AST-89
+
+	* channels/chan_sip.c: We don't handle 4XX responses to BYE well.
+	  According to section 15 of RFC 3261, we should terminate a dialog
+	  if we receive a 481 or 408 in response to our BYE. Since I am
+	  aware of at least one phone manufacturer who may sometimes send a
+	  404 as well, I am being liberal and saying that any 4XX response
+	  to a BYE should result in a terminated dialog. (closes issue
+	  #12994) Reported by: pabelanger Patches: 12994.patch uploaded by
+	  putnopvut (license 60) Closes AST-129
+
+	* apps/app_dial.c, channels/chan_sip.c: Make sure to set the hangup
+	  cause on the calling channel in the case that ast_call() fails.
+	  For incoming SIP channels, this was causing us to send a 603
+	  instead of a 486 when the call-limit was reached on the
+	  destination channel. (closes issue #13867) Reported by: still_nsk
+	  Patches: 13867.diff uploaded by putnopvut (license 60) Tested by:
+	  blitzrage
+
+2008-11-20 01:46 +0000 [r158010]  Richard Mudgett <rmudgett at digium.com>
+
+	* channels/chan_misdn.c: Merged revision 157977 from
+	  https://origsvn.digium.com/svn/asterisk/team/group/issue8824
+	  ........ Fixes JIRA ABE-1726 The dial extension could be empty if
+	  you are using MISDN_KEYPAD to control ISDN provider features.
+
+2008-11-19 21:34 +0000 [r157859]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* main/stdtime/Makefile, codecs/gsm/src, main/db1-ast/btree,
+	  channels/misdn/Makefile, main/db1-ast/recno, pbx/ael, channels,
+	  main/db1-ast/Makefile, main/stdtime, main/db1-ast/hash,
+	  codecs/gsm/Makefile, main/db1-ast/db, Makefile.moddir_rules,
+	  channels/misdn, main/db1-ast/mpool, pbx/Makefile, Makefile.rules,
+	  res/snmp, res/Makefile: the gcc optimizer frequently finds broken
+	  code (use of uninitalized variables, unreachable code, etc.),
+	  which is good. however, developers usually compile with the
+	  optimizer turned off, because if they need to debug the resulting
+	  code, optimized code makes that process very difficult. this
+	  means that we get code changes committed that weren't adequately
+	  checked over for these sorts of problems. with this build system
+	  change, if (and only if) --enable-dev-mode was used and
+	  DONT_OPTIMIZE is turned on, when a source file is compiled it
+	  will actually be preprocessed (into a .i or .ii file), then
+	  compiled once with optimization (with the result sent to
+	  /dev/null) and again without optimization (but only if the first
+	  compile succeeded, of course). while making these changes, i did
+	  some cleanup work in Makefile.rules to move commonly-used
+	  combinations of flag variables into their own variables, to make
+	  the file easier to read and maintain
+
+2008-11-18 22:47 +0000 [r157503]  Mark Michelson <mmichelson at digium.com>
+
+	* channels/chan_sip.c: Add some missing invite state changes
+	  necessary in the sip_write function. Not setting the invite state
+	  correctly on the call was resulting in the Record application
+	  leaving empty files. I also have updated the doxygen comment next
+	  to the declaration of the INV_EARLY_MEDIA constant to reflect
+	  that we also use this state when we *send* a 18X response to an
+	  INVITE. (closes issue #13878) Reported by: nahuelgreco Patches:
+	  sip-early-media-recording-1.4.22.patch uploaded by nahuelgreco
+	  (license 162) Tested by: putnopvut
+
+2008-11-18 19:13 +0000 [r157365]  Jeff Peeler <jpeeler at digium.com>
+
+	* apps/app_meetme.c: (closes issue #13899) Reported by: akkornel
+	  This fix is the result of a bug fix in ast_app_separate_args
+	  r124395. If an argument does not exist it should always be set to
+	  a null string rather than a null pointer.
+
+2008-11-18 18:25 +0000 [r157305]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_dial.c, channels/chan_local.c, res/res_features.c,
+	  include/asterisk/channel.h, apps/app_followme.c: Fix a crash in
+	  the end_bridge_callback of app_dial and app_followme which would
+	  occur at the end of an attended transfer. The error occurred
+	  because we initially stored a pointer to an ast_channel which
+	  then was hung up due to a masquerade. This commit adds a "fixup"
+	  callback to the bridge_config structure to allow for
+	  end_bridge_callback_data to be changed in the case that a new
+	  channel pointer is needed for the end_bridge_callback.
+
+2008-11-15 19:31 +0000 [r157104-157163]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* Makefile, Makefile.rules: when an individual directory dist-clean
+	  is run, run clean in that directory first, and when running
+	  top-level dist-clean, do not run subdirectory clean operations
+	  twice
+
+	* Makefile.moddir_rules: dist-clean should remove dependency
+	  information files as well
+
+	* contrib/asterisk-ng-doxygen: major update to doxygen
+	  configuration file: 1) update to doxygen 1.5.x style file, as
+	  used in trunk 2) tell doxygen where are header files are, so
+	  include-file processing can be done 3) make all macros that are
+	  used to define variables/functions be expanded, so that doxygen
+	  will properly document the resulting variable/function 4) make
+	  all macros that are used to provide the contents of a variable
+	  (structure) be expanded, so that doxygen will be able to document
+	  the resulting fields 5) suppress compiler attributes
+	  (__attribute__(xxx)) from being seen by doxygen, so it will
+	  properly match up function definition and usage (for an example
+	  of th effect of this, look at the doxygen docs for ast_log() from
+	  before and afte this commit)
+
+2008-11-14 15:18 +0000 [r156816]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_voicemail.c: If the prompt to reenter a voicemail
+	  password timed out, it resulted in the password not being saved,
+	  even if the input matched what you gave when first prompted to
+	  enter a new password. This is because the return value of
+	  ast_readstring was checked, but not checked properly. This bug
+	  was discovered by Jared Smith during an Asterisk training course.
+	  Thanks for reporting it!
+
+2008-11-14 00:41 +0000 [r156688-156755]  Tilghman Lesher <tlesher at digium.com>
+
+	* apps/app_while.c: ast_waitfordigit() requires that the channel be
+	  up, for no good logical reason. This prevents While/EndWhile from
+	  working within the "h" extension. Reported by: jgalarneau (for
+	  ABE C.2) Fixed by: me
+
+	* main/manager.c: Provide more space for all the data which can
+	  appear in an originating channel name. (closes issue #13398)
+	  Reported by: bamby Patches: manager.c.diff uploaded by bamby
+	  (license 430)
+
+2008-11-13 11:58 +0000 [r156485-156510]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* configure, autoconf/ast_gcc_attribute.m4: revert this change...
+	  non-functional changes don't belong here
+
+	* configure, autoconf/ast_gcc_attribute.m4: correct minor syntax
+	  error... no functional change
+
+2008-11-12 21:18 +0000 [r156386]  Tilghman Lesher <tlesher at digium.com>
+
+	* apps/app_dial.c: When using call limits under 1 second, infinite
+	  call lengths are allowed, instead. (closes issue #13851) Reported
+	  by: ruddy
+
+2008-11-12 19:36 +0000 [r156297]  Steve Murphy <murf at digium.com>
+
+	* main/pbx.c: It turns out that the 0x0XX00 codes being returned
+	  for N, X, and Z are off by one, as per conversation with jsmith
+	  on #asterisk-dev; he was teaching a class and disconcerted that
+	  this published rule was not being followed, with patterns _NXX,
+	  _[1-8]22 and _[2-9]22... and NXX was winning, but [1-8] should
+	  have been. This change, tested on these 3 patterns now picks the
+	  proper one. However, this change may surprise users who set up
+	  dialplans based on previous behavior, which has been there for
+	  what, 2 and half years or so now.
+
+2008-11-12 19:26 +0000 [r156294]  Tilghman Lesher <tlesher at digium.com>
+
+	* apps/app_meetme.c: If the SLA thread is not started, then reload
+	  causes a memory leak. (closes issue #13889) Reported by: eliel
+	  Patches: app_meetme.c.patch uploaded by eliel (license 64)
+
+2008-11-12 19:10 +0000 [r156289]  Jeff Peeler <jpeeler at digium.com>
+
+	* apps/app_meetme.c: For whatever reason, gcc only warned me about
+	  the possible use of an uninitialized variable when compiling
+	  1.6.1.
+
+2008-11-12 18:39 +0000 [r156229]  Tilghman Lesher <tlesher at digium.com>
+
+	* channels/chan_iax2.c: Revert revision 132506, since it
+	  occasionally caused IAX2 HANGUP packets not to be sent, and
+	  instead, schedule a task to destroy the iax2 pvt structure 10
+	  seconds later. This allows the IAX2 HANGUP packet to be queued,
+	  transmitted, and ACKed before the pvt is destroyed. (closes issue
+	  #13645) Reported by: dzajro Patches:
+	  20081111__bug13645__3.diff.txt uploaded by Corydon76 (license 14)
+	  Tested by: vazir Reviewed: http://reviewboard.digium.com/r/51/
+
+2008-11-12 17:53 +0000 [r156178]  Jeff Peeler <jpeeler at digium.com>
+
+	* apps/app_meetme.c: (closes issue #13173) Reported by: pep This
+	  change adds an announce_thread responsible for playing
+	  announcements to an existing conference. This allows all
+	  announcing to be immediately stopped if necessary but more
+	  importantly allows other threads that need to play something to
+	  not block. There are multiple benefits to this, but the actual
+	  bug is for solving the scenario for a channel to be unusable
+	  after hang up for the entire duration of the parting
+	  announcement. The parting announcement can be extremely long
+	  depending on what the user recorded upon joining the conference.
+	  Reviewed by Russell on Review Board:
+	  http://reviewboard.digium.com/r/25/
+
+2008-11-12 17:38 +0000 [r156167]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_dial.c: When doing some tests, I was having a crash at
+	  the end of every call if an attended transfer occurred during the
+	  call. I traced the cause to the CDR on one of the channels being
+	  NULL. murf suggested a check in the end bridge callback to be
+	  sure the CDR is non-NULL before proceeding, so that's what I'm
+	  adding.
+
+2008-11-12 17:29 +0000 [r156164]  Russell Bryant <russell at digium.com>
+
+	* main/asterisk.c: Move the sanity check that makes sure "always
+	  fork" is not set along with the console option to be after the
+	  code that reads options from asterisk.conf. This resolves a
+	  situation where Asterisk can start taking up 100% when
+	  misconfigured. (Thanks to Bryce Porter (x86 on IRC) for letting
+	  me log in to his system to figure out what was causing the 100%
+	  CPU problem.)
+
+2008-11-10 21:07 +0000 [r155861]  Mark Michelson <mmichelson at digium.com>
+
+	* channels/chan_agent.c: Channel drivers assume that when their
+	  indicate callback is invoked, that the channel on which the
+	  callback was called is locked. This patch corrects an instance in
+	  chan_agent where a channel's indicate callback is called directly
+	  without first locking the channel. This was leading to some
+	  observed locking issues in chan_local, but considering that all
+	  channel drivers operate under the same expectations, the generic
+	  fix in chan_agent is the right way to go. AST-126
+
+2008-11-10 20:49 +0000 [r155803]  Tilghman Lesher <tlesher at digium.com>
+
+	* doc/valgrind.txt: I got tired of saying this in every single
+	  bugnote referring to this file.
+
+2008-11-09 01:08 +0000 [r155553]  Sean Bright <sean.bright at gmail.com>
+
+	* apps/app_dial.c, res/res_features.c, include/asterisk/channel.h,
+	  apps/app_followme.c: Use static functions here instead of nested
+	  ones. This requires a small change to the ast_bridge_config
+	  struct as well. To understand the reason for this change, see the
+	  following post:
+	  http://gcc.gnu.org/ml/gcc-help/2008-11/msg00049.html
+
+2008-11-07 22:27 +0000 [r155398]  Tilghman Lesher <tlesher at digium.com>
+
+	* channels/chan_sip.c: Clarify error message. (closes issue #13809)
+	  Reported by: denke Patches: 20081104__bug13809.diff.txt uploaded
+	  by Corydon76 (license 14) Tested by: denke
+
+2008-11-06 19:45 +0000 [r155011]  Mark Michelson <mmichelson at digium.com>
+
+	* configs/voicemail.conf.sample: The documentation listed the
+	  ability to set 'maxmsg' per context. The truth is that you can
+	  only set this in the general section or per mailbox. Thus I am
+	  updating the sample config file to be more accurate. Thanks to
+	  sasargen on IRC for bringing up this issue.
+
+2008-11-05 16:44 +0000 [r154724]  Mark Michelson <mmichelson at digium.com>
+
+	* channels/chan_agent.c: The logic of a strcasecmp call was
+	  reversed (closes issue #13841) Reported by: clegall_proformatique
+
+2008-11-05 16:06 +0000 [r154685]  Steve Murphy <murf at digium.com>
+
+	* main/channel.c: This fix was prompted by communication from user,
+	  who was seeing thousands of error logs... looks like EAGAIN. Made
+	  such uninteresting.
+
+2008-11-04 20:49 +0000 [r154365]  Tilghman Lesher <tlesher at digium.com>
+
+	* channels/chan_iax2.c: On busy systems, it's possible for the
+	  values checked within a single line of code to change, unless the
+	  structure is locked to ensure a consistent state. (closes issue
+	  #13717) Reported by: kowalma Patches: 20081102__bug13717.diff.txt
+	  uploaded by Corydon76 (license 14) Tested by: kowalma
+
+2008-11-04 19:01 +0000 [r154266]  Richard Mudgett <rmudgett at digium.com>
+
+	* channels/chan_misdn.c: JIRA ABE-1703 mISDN sets the channel to
+	  the wrong state when it receives the indication
+	  AST_CONTROL_RINGING.
+
+2008-11-04 18:58 +0000 [r154060-154263]  Tilghman Lesher <tlesher at digium.com>
+
+	* channels/chan_h323.c: Make the monitor thread non-detached, so it
+	  can be joined (suggested by Russell on -dev list).
+
+	* apps/app_voicemail.c: Attempting to expunge a mailbox when the
+	  mailstream is NULL will crash Asterisk. (Closes issue #13829)
+	  Reported by: jaroth Patch by: me (modified jaroth's patch)
+
+	* main/rtp.c: Remove the potential for a division by zero error.
+	  (Closes issue #13810)
+
+2008-11-03 13:01 +0000 [r153823]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* channels/chan_oss.c, channels/chan_dahdi.c, funcs/func_odbc.c,
+	  main/file.c, main/http.c, main/utils.c, pbx/pbx_config.c,
+	  res/res_jabber.c: somehow missed a bunch of gcc 4.3.x warnings in
+	  this branch on the first pass
+
+2008-11-02 19:51 +0000 [r153651]  Russell Bryant <russell at digium.com>
+
+	* include/asterisk/features.h: features.h depends on linkedlists.h,
+	  so include it
+
+2008-11-01 18:22 +0000 [r153337]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* utils/frame.c, main/cli.c, utils/stereorize.c, main/channel.c,
+	  funcs/func_enum.c, channels/chan_dahdi.c, main/manager.c,
+	  channels/chan_skinny.c, main/ast_expr2f.c, res/res_agi.c,
+	  pbx/ael/ael_lex.c, main/http.c, channels/chan_alsa.c,
+	  pbx/ael/ael.flex, formats/format_gsm.c, apps/app_adsiprog.c,
+	  formats/format_wav.c, apps/app_festival.c,
+	  main/db1-ast/hash/hash_page.c, main/translate.c,
+	  res/res_crypto.c, agi/eagi-test.c, formats/format_ogg_vorbis.c,
+	  utils/astman.c, channels/chan_oss.c, agi/eagi-sphinx-test.c,
+	  pbx/ael/ael.tab.c, main/file.c, pbx/ael/ael.tab.h,
+	  apps/app_sms.c, pbx/pbx_dundi.c, res/res_indications.c,
+	  utils/streamplayer.c, apps/app_chanspy.c, main/asterisk.c,
+	  apps/app_voicemail.c, utils/muted.c, pbx/ael/ael.y,
+	  apps/app_authenticate.c, formats/format_wav_gsm.c,
+	  res/res_musiconhold.c, channels/chan_iax2.c: fix a bunch of
+	  potential problems found by gcc 4.3.x, primarily bare strings
+	  being passed to printf()-like functions and ignored results from
+	  read()/write() and friends
+
+2008-10-31 22:36 +0000 [r153270]  Terry Wilson <twilson at digium.com>
+
+	* res/res_features.c, apps/app_followme.c: Add end_bridge_callback
+	  for app_follome and add AUTOLOOP flag to res_features
+
+2008-10-31  Tilghman Lesher <tlesher at digium.com>
+
+	* Asterisk 1.4.23-rc1 released.
+
+2008-10-31 16:30 +0000 [r153114]  Tilghman Lesher <tlesher at digium.com>
+
+	* channels/chan_sip.c: Turn off qualify on uncached realtime peers.
+	  (Closes issue #13383)
+
+2008-10-31 15:45 +0000 [r153095]  Terry Wilson <twilson at digium.com>
+
+	* apps/app_dial.c, res/res_features.c, include/asterisk/channel.h:
+	  Recent CDR fixes moved execution of the 'h' exten into the
+	  bridging code, so variables that were set after ast_bridge_call
+	  was called would not show up in the 'h' exten. Added a callback
+	  function to handle setting variables, etc. from w/in the bridging
+	  code. Calls back into a nested function within the function
+	  calling ast_bridge_call (closes issue #13793) Reported by:
+	  greenfieldtech
+
+2008-10-30 20:58 +0000 [r152992]  Sean Bright <sean.bright at gmail.com>
+
+	* bootstrap.sh: The -I argument to aclocal needs a space before the
+	  include directory name.
+
+2008-10-30 20:33 +0000 [r152922-152958]  Tilghman Lesher <tlesher at digium.com>
+
+	* channels/chan_h323.c: Cannot join detached threads. See
+	  http://www.opengroup.org/onlinepubs/000095399/functions/pthread_join.html
+	  (Closes issue #13400)
+
+	* channels/chan_local.c: Unlock before returning, when extension
+	  doesn't exist. (closes issue #13807) Reported by: eliel Patches:
+	  chan_local.c.patch uploaded by eliel (license 64)
+
+2008-10-30 16:53 +0000 [r152811]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* main/cdr.c: instead of comparing the string pointer to 0, let's
+	  compare the value that was actually parsed out of the string
+	  (found by sparse)
+
+2008-10-29 05:23 +0000 [r152539]  Russell Bryant <russell at digium.com>
+
+	* channels/chan_sip.c: Fix an incorrect usage of sizeof() (closes
+	  issue #13795) Reported by: andrew53 Patches:
+	  chan_sip_sizeof.patch uploaded by andrew53 (license 519)
+
+2008-10-29 05:19 +0000 [r152535-152538]  Steve Murphy <murf at digium.com>
+
+	* configs/features.conf.sample, apps/app_dial.c, apps/app_queue.c:
+	  A little documentation cross-ref between features and dial and
+	  queue... I wasted some time (stupidly) trying to get the
+	  one-touch parking stuff working, because it didn't occur to me
+	  that I had to also have the corresponding options in the dial
+	  command! Duh! (In all this time, I never set this up before!) So,
+	  to keep some poor fool from suffering the same fate, I made the
+	  features.conf.sample file mention the corresponding opts in
+	  dial/queue; and the docs for dial/app specifically mention the
+	  corresponding decls in the feature.conf file. I hope this doesn't
+	  spoil some vast, eternal plan...
+
+	* apps/app_dial.c, res/res_features.c, funcs/func_channel.c,
+	  include/asterisk/pbx.h, apps/app_queue.c: The magic trick to
+	  avoid this crash is not to try to find the channel by name in the
+	  list, which is slow and resource consuming, but rather to pay
+	  attention to the result codes from the ast_bridge_call, to which
+	  I added the AST_PBX_NO_HANGUP_PEER_PARKED value, which now are
+	  returned when a channel is parked. If you get AST_PBX_KEEPALIVE,
+	  then don't touch the channel pointer. If you get
+	  AST_PBX_NO_HANGUP_PEER, or AST_PBX_NO_HANGUP_PEER_PARKED, then
+	  don't touch the peer pointer. Updated the several places where
+	  the results from a bridge were not being properly obeyed, and
+	  fixed some code I had introduced so that the results of the
+	  bridge were not overridden (in trunk). All the places that
+	  previously tested for AST_PBX_NO_HANGUP_PEER now have to check
+	  for both AST_PBX_NO_HANGUP_PEER and
+	  AST_PBX_NO_HANGUP_PEER_PARKED. I tested this against the 4 common
+	  parking scenarios: 1. A calls B; B answers; A parks B; B hangs up
+	  while A is getting the parking slot announcement, immediately
+	  after being put on hold. 2. A calls B; B answers; A parks B; B
+	  hangs up after A has been hung up, but before the park times out.
+	  3. A calls B; B answers; B parks A; A hangs up while B is getting
+	  the parking slot announcement, immediately after being put on
+	  hold. 4. A calls B; B answers; B parks A; A hangs up after B has
+	  been hung up, but before the park times out. No crash. I also ran
+	  the scenarios above against valgrind, and accesses looked good.
+
+2008-10-28 22:32 +0000 [r152368-152463]  Tilghman Lesher <tlesher at digium.com>
+
+	* apps/app_voicemail.c: Quoting in the wrong direction (Fixes
+	  AST-107)
+
+	* apps/app_dial.c: Reset all DIAL variables back to blank, in case
+	  Dial is called multiple times per call (which could otherwise
+	  lead to inconsistent status reports). (closes issue #13216)
+	  Reported by: ruddy Patches: 20081014__bug13216.diff.txt uploaded
+	  by Corydon76 (license 14) Tested by: ruddy
+
+2008-10-27 23:28 +0000 [r152286]  Jeff Peeler <jpeeler at digium.com>
+
+	* channels/chan_dahdi.c: Buffer policy setting for half is not
+	  needed.
+
+2008-10-27 21:32 +0000 [r152215]  Tilghman Lesher <tlesher at digium.com>
+
+	* channels/chan_local.c: Inherit ALL elements of CallerID across a
+	  local channel. (closes issue #13368) Reported by: Peter Schlaile
+	  Patches: 20080826__bug13368.diff.txt uploaded by Corydon76
+	  (license 14)
+
+2008-10-26 20:23 +0000 [r152059]  Sean Bright <sean.bright at gmail.com>
+
+	* funcs/func_strings.c: Since passing \0 as the second argument to
+	  strchr is valid (and will match the trailing \0 of a string) we
+	  need to check that first, otherwise we end up with incorrect
+	  results. Fix suggested by reporter. (closes issue #13787)
+	  Reported by: meitinger
+
+2008-10-25 10:59 +0000 [r151905]  Russell Bryant <russell at digium.com>
+
+	* main/asterisk.c: Move AMI initialization to occur after loading
+	  modules. This prevents a deadlock when someone tries to initiate
+	  a module reload from the AMI just as Asterisk is starting.
+	  (closes issue #13778) Reported by: hotsblanc Fix suggested by
+	  hotsblanc
+
+2008-10-23 16:04 +0000 [r151763]  Terry Wilson <twilson at digium.com>
+
+	* configs/features.conf.sample, res/res_features.c, CHANGES:
+	  Backport fix from 1.6.0 that allows you to set
+	  parkedcalltransfers=no|caller|callee|both, but default to both
+	  which would be the equivalent of the existing behaviour. The
+	  problem was that if someone parked a call, the callee and caller
+	  would both get assigned the builtin transfer feature, which would
+	  not only be potentially giving someone the ability to transfer
+	  themselves when they shouldn't have it, but would also dissallow
+	  reinviting the media off of the call. (closes issue #12854)
+	  Reported by: davidw Patches: parkingfix4.diff.txt uploaded by
+	  otherwiseguy Tested by: davidw, otherwiseguy
+
+2008-10-20 04:57 +0000 [r151240-151241]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* autoconf/ast_check_pwlib.m4, autoconf/ast_check_openh323.m4,
+	  configure.ac: rename this macro to properly reflect what it does
+
+	* autoconf/ast_check_pwlib.m4 (added), autoconf (added),
+	  autoconf/acx_pthread.m4 (added), autoconf/ast_func_fork.m4
+	  (added), configure, autoconf/ast_gcc_attribute.m4 (added),
+	  bootstrap.sh, autoconf/ast_check_gnu_make.m4 (added),
+	  autoconf/ast_ext_lib.m4 (added), autoconf/ast_prog_ld.m4 (added),
+	  autoconf/ast_c_compile_check.m4 (added),
+	  autoconf/ast_c_define_check.m4 (added),
+	  autoconf/ast_prog_egrep.m4 (added),
+	  autoconf/ast_check_openh323.m4 (added),
+	  autoconf/ast_prog_ld_gnu.m4 (added), autoconf/ast_prog_sed.m4
+	  (added), acinclude.m4 (removed): break up acinclude.m4 into
+	  individual files, which will make it easier to maintain, easier
+	  to add new macros (less patching) and will ease maintenance of
+	  these macros across Asterisk branches
+
+2008-10-19 19:51 +0000 [r151100-151167]  BJ Weschke <bweschke at btwtech.com>
+
+	* main/asterisk.c: As per kpfleming's comments to the prior commit,
+	  I'm reverting some of the changes here. A comment was made in bug
+	  #13726 "3. The same mistake as in (2) is done in a few other
+	  places in the code that check for: #if defined(HAVE_ZAPTEL) ||
+	  defined(HAVE_DAHDI) Harmless, but still incorrect." In the case
+	  of main/asterisk.c, this is not incorrect because without
+	  HAVE_ZAPTEL defined, we're missing the include for ioctl and the
+	  namespace that defines DAHDI_TIMERCONFIG which is still required
+	  when using Zaptel with the 1.4 branch.
+
+	* main/asterisk.c: Fix the 1.4 branch compile again broken with
+	  r150557 when using with Zaptel and not DAHDI (closes issue
+	  #13740) reported by: jmls patch by: bweschke
+
+2008-10-18 01:42 +0000 [r150816]  BJ Weschke <bweschke at btwtech.com>
+
+	* main/manager.c: Using the GetVar handler in AMI is potentially
+	  dangerous (insta-crash [tm]) when you use a dialplan function
+	  that requires a channel and then you don't provide one or provide
+	  an invalid one in the Channel: parameter. We'll handle this
+	  situation exactly the same way it was handled in pbx.c back on
+	  r61766. We'll create a bogus channel for the function call and
+	  destroy it when we're done. If we have trouble allocating the
+	  bogus channel then we're not going to try executing the function
+	  call at all and run the risk of crashing. (closes issue #13715)
+	  reported by: makoto patch by: bweschke
+
+2008-10-17 17:18 +0000 [r150637]  Steve Murphy <murf at digium.com>
+
+	* res/res_features.c: Interesting crash. In this case, you exit the
+	  bridge with peer completely GONE. I moved the channel find call
+	  up to cover the whole peer CDR reset code segment. This appears
+	  to solve the crash without changing the logic at all.
+
+2008-10-17 15:31 +0000 [r150557]  Jason Parker <jparker at digium.com>
+
+	* main/asterisk.c, main/channel.c, channels/chan_dahdi.c,
+	  configure, configure.ac: Correctly allow chan_dahdi to compile
+	  against older versions of Zaptel. Don't always define
+	  HAVE_ZAPTEL_CHANALARMS (since we check if it's defined..) Minor
+	  cleanup to make things clear. (closes issue #13726) Reported by:
+	  tzafrir Patches: dahdi_def.diff uploaded by tzafrir (license 46)
+
+2008-10-16 23:40 +0000 [r150298-150304]  Mark Michelson <mmichelson at digium.com>
+
+	* main/manager.c: Reverting changes from commits 150298 and 150301
+	  since I was mistakenly under the assumption that dialplan
+	  functions *always* required that a channel be present. I need to
+	  go home earlier, I think :)
+
+	* main/manager.c: And don't forget to return on the error condition
+
+	* main/manager.c: Don't try to call a dialplan function's read
+	  callback from the manager's GetVar handler if an invalid channel
+	  has been specified. Several dialplan functions, including CHANNEL
+	  and SIP_HEADER, do not check for NULL-ness of the channel being
+	  passed in. (closes issue #13715) Reported by: makoto
+
+2008-10-16 15:56 +0000 [r150124]  Richard Mudgett <rmudgett at digium.com>
+
+	* channels/chan_misdn.c: Fix memory leak found by customer
+
+2008-10-16 15:26 +0000 [r150056]  Steve Murphy <murf at digium.com>
+
+	* cdr/cdr_odbc.c: This patch is relevant to: ABE-1628 and
+	  RYM-150398 and AST-103 in internal Digium bug trackers. These
+	  fixes address a really subtle memory corruption problem that
+	  would happen in machines heavily loaded in production
+	  environments. The corruption would always take the form of the
+	  STMT object getting nulled out and one of the unixODBC calls
+	  would crash trying to access statement->connection. It isn't
+	  fully proven yet, but the server has now been running 2.5 days
+	  without appreciable memory growth, or any gain of %cpu, and no
+	  crashes. Whether this is the problem or not on that server, these
+	  fixes are still warranted. As it turns out, **I** introduced
+	  these errors unwittingly, when I corrected another crash earlier.
+	  I had formed the build_query routine, and failed to remove
+	  mutex_unlock calls in 3 places in the transplanted code. These
+	  unlocks would only happen in error situations, but unlocking the
+	  mutex early set the code up for a catastrophic failure, it
+	  appears. It would happen only once every 100K-200K or more calls,
+	  under heavy load... but that is enough. If another crash occurs,
+	  with the same MO, I'll come back and remove my confession from
+	  the log, and we'll keep searching, but the fact that we have
+	  Asterisk dying from an asynchronous wiping of the STMT object,
+	  only on some connection error, and that the server has lived for
+	  2.5 days on this code without a crash, sure make it look like
+	  this was the problem! Also, in several points, Statement handles
+	  are set to NULL after SQLFreeHandle. This was mainly for
+	  insurance, to guarantee a crash. As it turns out, the code does
+	  not appear to be attempting to use these freed pointers. Asterisk
+	  owes a debt of gratitude to Federico Alves and Frediano Ziglio
+	  for their untiring efforts in finding this bug, among others.
+
+2008-10-15 21:34 +0000 [r149683-149840]  BJ Weschke <bweschke at btwtech.com>
+
+	* CHANGES: Another documentation fix. (closes issue #13708)
+
+	* configs/agents.conf.sample: An update to the
+	  documentation/example of agents.conf.sample with the correct
+	  parameter for this feature as defined in chan_agent.c (closes
+	  issue #13709)
+
+2008-10-15 10:30 +0000 [r149452]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* channels/chan_sip.c: fix some problems when parsing SIP messages
+	  that have the maximum number of headers or body lines that we
+	  support
+
+2008-10-14 23:43 +0000 [r149130-149266]  Mark Michelson <mmichelson at digium.com>
+
+	* channels/chan_sip.c: Change this warning to an error message.
+	  Suggestion comes from Sean Bright. Thanks Sean!
+
+	* channels/chan_sip.c: Call register_peer_exten even in the case
+	  that the peer's IP/port does not change. (closes issue #13309)
+	  Reported by: dimas Patches: v2-13309.patch uploaded by dimas
+	  (license 88)
+
+	* include/asterisk/audiohook.h, main/audiohook.c: Add a tolerance
+	  period for sync-triggered audiohooks so that if packetization of
+	  audio is close (but not equal) we don't end up flushing the

[... 20319 lines stripped ...]



More information about the asterisk-commits mailing list