[asterisk-commits] tilghman: tag 1.6.1-beta2 r153262 - /tags/1.6.1-beta2/

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Fri Oct 31 16:02:50 CDT 2008


Author: tilghman
Date: Fri Oct 31 16:02:49 2008
New Revision: 153262

URL: http://svn.digium.com/view/asterisk?view=rev&rev=153262
Log:
Importing files for 1.6.1-beta2 release

Added:
    tags/1.6.1-beta2/.lastclean   (with props)
    tags/1.6.1-beta2/.version   (with props)
    tags/1.6.1-beta2/ChangeLog   (with props)

Added: tags/1.6.1-beta2/.lastclean
URL: http://svn.digium.com/view/asterisk/tags/1.6.1-beta2/.lastclean?view=auto&rev=153262
==============================================================================
--- tags/1.6.1-beta2/.lastclean (added)
+++ tags/1.6.1-beta2/.lastclean Fri Oct 31 16:02:49 2008
@@ -1,0 +1,1 @@
+35

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

Propchange: tags/1.6.1-beta2/.lastclean
------------------------------------------------------------------------------
    svn:keywords = none

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

Added: tags/1.6.1-beta2/.version
URL: http://svn.digium.com/view/asterisk/tags/1.6.1-beta2/.version?view=auto&rev=153262
==============================================================================
--- tags/1.6.1-beta2/.version (added)
+++ tags/1.6.1-beta2/.version Fri Oct 31 16:02:49 2008
@@ -1,0 +1,1 @@
+1.6.1-beta2

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

Propchange: tags/1.6.1-beta2/.version
------------------------------------------------------------------------------
    svn:keywords = none

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

Added: tags/1.6.1-beta2/ChangeLog
URL: http://svn.digium.com/view/asterisk/tags/1.6.1-beta2/ChangeLog?view=auto&rev=153262
==============================================================================
--- tags/1.6.1-beta2/ChangeLog (added)
+++ tags/1.6.1-beta2/ChangeLog Fri Oct 31 16:02:49 2008
@@ -1,0 +1,47654 @@
+2008-10-31 16:36 +0000 [r153123]  Tilghman Lesher <tlesher at digium.com>
+
+	* channels/chan_sip.c: Turn off qualify on uncached realtime
+	  peers. (Closes issue #13383)
+
+2008-10-30 21:01 +0000 [r152995]  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:36 +0000 [r152924-152974]  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 19:41 +0000 [r152878-152921]  Russell Bryant <russell at digium.com>
+
+	* channels/chan_sip.c: Fix the sip_peer reference count with
+	  respect to scheduler entries for scheduling peer pokes, and
+	  scheduling peer poke expirations.
+
+	* channels/chan_sip.c: Fix the sip_peer reference count with
+	  respect to scheduler entries for registration expirations.
+
+	* include/asterisk/sched.h: Fix a bug in AST_SCHED_REPLACE_UNREF().
+	  The reference count of the object _must_ be increased before
+	  creating the scheduler entry. Otherwise, you create a race
+	  condition where the reference count may hit zero and the object
+	  can disappear out from under you. This could also would have
+	  incorrectly decreased the reference count in the case that the
+	  scheduler add failed.
+
+	* channels/chan_sip.c: Modify the documentation of the sip_registry
+	  struct - Remove a comment that says that the monitor thread is the
+	  only one that ever touches these objects. This is no longer the
+	  case with TCP.  Also, I would eventually like to get the scheduler
+	  in its own thread, so this is just a poor assumption to make. -
+	  Note that reference counting of these objects with respect to
+	  scheduler entries is not complete. There are some leaked
+	  references when deleting scheduler entries.
+
+2008-10-30 16:55 +0000 [r152814]  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-30 04:29 +0000 [r152690-152777]  Tilghman Lesher <tlesher at digium.com>
+
+	* configs/extensions.conf.sample: Set up an example stdexten that
+	  preserves the original context and extension in the CDR. (Related
+	  to issue #13799) Reported by: davidw
+
+	* main/pbx.c: Track down and fix annoying lock errors.  These would
+	  occur when merging hints that resulted from a pattern matched hint
+	  during a 'dialplan reload'.
+
+2008-10-29 20:55 +0000 [r152648]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_directory.c: If there was no named defined in a
+	  voicemail.conf mailbox entry, then app_directory would crash when
+	  attempting to read that entry from the file. We now check for the
+	  NULL or empty string properly so that there will be no crash.
+	  (closes issue #13804) Reported by: bluecrow76
+
+2008-10-29 20:16 +0000 [r152645]  Terry Wilson <twilson at digium.com>
+
+	* apps/app_queue.c: Small modification to putnopvut's patch to fix
+	  this issue. Thanks for all the help, putnopvut! (closes issue
+	  #12884) Reported by: bcnit Patches: 12884v4-1.6.0-branch.patch
+	  uploaded by otherwiseguy (license 396) Tested by: otherwiseguy
+
+2008-10-29 05:52 +0000 [r152606]  Steve Murphy <murf at digium.com>
+
+	* apps/app_queue.c, configs/features.conf.sample, apps/app_dial.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...
+
+2008-10-29 05:35 +0000 [r152573]  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:09 +0000 [r152537]  Steve Murphy <murf at digium.com>
+
+	* apps/app_queue.c, include/asterisk/features.h, apps/app_dial.c,
+	  main/features.c, include/asterisk/pbx.h: 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. Why? because CDR's aren't generated via
+	  parking, so nothing is needed, but if a transfer occurred, there
+	  are critical things I need. 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:35 +0000 [r152370-152471]  Tilghman Lesher <tlesher at digium.com>
+
+	* apps/app_voicemail.c: Quoting in the wrong direction (Fixes
+	  AST-107)
+
+	* channels/chan_mgcp.c: Only re-add the io port if it was closed,
+	  otherwise reload causes a memory leak. (closes issue #13785)
+	  Reported by: eliel Patches: chan_mgcp.c.patch uploaded by eliel
+	  (license 64)
+
+	* 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:32 +0000 [r152288]  Jeff Peeler <jpeeler at digium.com>
+
+	* channels/chan_dahdi.c: Buffer policy setting for half is not
+	  needed.
+
+2008-10-27 21:53 +0000 [r152173-152217]  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)
+
+	* apps/app_stack.c: Oops, only delete the ARG variables once upon
+	  release. The following section would have removed them again
+	  (removing variables from 2 stack frames, instead of just one).
+
+2008-10-27 16:06 +0000 [r152133]  Jason Parker <jparker at digium.com>
+
+	* apps/app_transfer.c: Remove options argument parsing/syntax (it
+	  isn't used any longer) (closes issue #13789) Reported by: IgorG
+	  Patches: app_transfer.c.diff uploaded by IgorG (license 20)
+
+2008-10-26 20:27 +0000 [r152068]  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 11:11 +0000 [r151907]  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-22 20:08 +0000 [r151603]  Tilghman Lesher <tlesher at digium.com>
+
+	* contrib/scripts/live_ast: Add a contributed script for running
+	  Asterisk without installing it, first. (closes issue #11680)
+	  Reported by: tzafrir Patches: live_ast_6 uploaded by tzafrir
+	  (license 46)
+
+2008-10-22 20:05 +0000 [r151421-151602]  Mark Michelson <mmichelson at digium.com>
+
+	* channels/chan_dahdi.c: Change some logical ands to bitwise ands
+	  and add messages alerting that a channel is being ignored if the
+	  PROC_DAHDI_NOCHAN option is set in process_dahdi. (closes issue
+	  #13759) Reported by: smurfix Patches: dahdi.patch uploaded by
+	  smurfix (license 547)
+
+	* channels/chan_sip.c: The logic of a strncasecmp call was reversed.
+	  (closes issue #13706) Reported by: andrew53 Patches:
+	  sip_notify_from_rfc3265.patch uploaded by andrew53 (license 519)
+
+	* channels/chan_sip.c: Make the sip_standard_port function more
+	  granular by allowing separate type and port arguments. This is
+	  necessary because when building our From and Contact headers, we
+	  need to be absolutely sure that we are placing our source port
+	  there and not the peer's source port. (closes issue #12761)
+	  Reported by: asbestoshead Patches: patch-chan-sip-contact-port.txt
+	  uploaded by asbestoshead (license 455)
+
+	* channels/chan_sip.c: Get this compiling in dev-mode
+
+	* channels/chan_sip.c: If a peer uses any transport other than UDP,
+	  then MWI will fail for that peer since sip_alloc will allocate a
+	  sip_pvt with a default transport of UDP. This change resets the
+	  socket type immediately after allocating the sip_pvt in
+	  sip_send_mwi_from_peer, so that the proceeding call to
+	  create_addr_from_peer does not fail right away. The socket data
+	  from the peer is properly copied to the sip_pvt in
+	  create_addr_from_peer. (closes issue #13710) Reported by:
+	  andrew53 Patches: sip_notify_use_tcp.patch uploaded by andrew53
+	  (license 519)
+
+	* channels/chan_sip.c: When attempting to resolve hostnames, we
+	  need to be sure to remove any parameters from the string so that
+	  name resolution succeeds. (closes issue #13727) Reported by:
+	  fnordian Patches: resolvewithouturiparameter.patch uploaded by
+	  fnordian (license 110)
+
+2008-10-21 15:21 +0000 [r151372]  Tilghman Lesher <tlesher at digium.com>
+
+	* apps/app_mixmonitor.c: Default file modes should always be full
+	  read and write, to allow the system administrator to make the
+	  decision of what permissions will actually be given, through the
+	  use of the process umask. (Closes issue# 13751)
+
+2008-10-21 11:03 +0000 [r151328]  BJ Weschke <bweschke at btwtech.com>
+
+	* channels/chan_sip.c: Fix configuration parsing so type=friend
+	  still identifies "friend" as a peer even though it is now a
+	  legacy configuration verb. (closes issue #13705) reported by:
+	  blitzrage patched by: bweschke
+
+2008-10-20 05:06 +0000 [r151135-151245]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* autoconf (added), autoconf/ast_check_pwlib.m4,
+	  autoconf/acx_pthread.m4, autoconf/ast_func_fork.m4, configure,
+	  autoconf/ast_gcc_attribute.m4, bootstrap.sh,
+	  autoconf/ast_check_gnu_make.m4, autoconf/ast_ext_lib.m4,
+	  autoconf/ast_prog_ld.m4, autoconf/ast_c_compile_check.m4,
+	  autoconf/ast_c_define_check.m4, autoconf/ast_prog_egrep.m4,
+	  autoconf/ast_ext_tool_check.m4, autoconf/ast_check_mandatory.m4,
+	  autoconf/ast_check_openh323.m4, autoconf/ast_prog_ld_gnu.m4,
+	  configure.ac, acinclude.m4 (removed), autoconf/ast_prog_sed.m4:
+	  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.
+	  Rename this macro to properly reflect what it does
+
+	* main/tcptls.c, main/manager.c, channels/chan_sip.c, main/http.c,
+	  apps/app_externalivr.c, include/asterisk/tcptls.h: cleaup of the
+	  TCP/TLS socket API: 1) rename 'struct server_args' to 'struct
+	  ast_tcptls_session_args', to follow coding guidelines 2) make
+	  ast_make_file_from_fd() static and rename it to something that
+	  indicates what it really is for (again coding guidelines) 3)
+	  rename address variables inside 'struct ast_tcptls_session_args'
+	  to be more descriptive (dare i say it... coding guidelines) 4)
+	  change ast_tcptls_client_start() to use the new 'remote_address'
+	  field of the session args for the destination of the connection,
+	  and use the 'local_address' field to bind() the socket to the
+	  proper source address, if one is supplied 5) in chan_sip, ensure
+	  that we pass in the PP address we are bound to when creating
+	  outbound (client) connections, so that our connections will
+	  appear from the correct address
+
+2008-10-18 02:29 +0000 [r150829]  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:10 +0000 [r150606-150636]  Tilghman Lesher <tlesher at digium.com>
+
+	* channels/chan_iax2.c: Make helper call a little safer (suggested
+	  by Russell on IRC)
+
+	* channels/chan_iax2.c, include/asterisk/sched.h: Fix the FRACK!
+	  warnings in chan_iax2 when POKE/LAGRQ packets are not answered.
+
+2008-10-16 23:41 +0000 [r150208-150306]  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: 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
+	  And don't forget to return on the error condition
+
+	* apps/app_sms.c: Answer the channel prior to checking for the 'a'
+	  option in app_sms. (closes issue #13675) Reported by: alecdavis
+	  Patches: app_sms.bug13675.148985.diff.txt uploaded by alecdavis
+	  (license 585)
+
+	* configure, configure.ac: Change configure script to search for
+	  openais in both /usr/lib and /usr/lib64 since some distros place
+	  64-bit libraries only in the /usr/lib64 directory. (closes issue
+	  #13721) Reported by: jcollie Patches:
+	  0007-Look-in-64bit-dirs-for-openais.patch uploaded by jcollie
+	  (license 412)
+
+	* channels/chan_sip.c: INVITES with proxy auth were sent with a
+	  different branch than what was in the invite_branch of a sip_pvt,
+	  meaning that if a CANCEL were sent later, the branch in the
+	  CANCEL would not match the branch in the latest INVITE sent out,
+	  leading to some endpoints responding to the CANCEL with a 481.
+	  (closes issue #13714) Reported by: fnordian Patches:
+	  invite_branch.patch uploaded by fnordian (license 110)
+
+2008-10-16 16:17 +0000 [r150127]  Richard Mudgett <rmudgett at digium.com>
+
+	* channels/chan_misdn.c: Fix memory leak found by customer
+
+2008-10-16 13:32 +0000 [r149919-149995]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* channels/chan_sip.c: return this logic to where it used to be,
+	  *after* the dialog->needdestroy flag has been determined to be set;
+	  otherwise, we generate these debug messages every time we inspect
+	  every active dialog
+
+	* apps/app_stack.c: building this module depends on res_agi being
+	  built as well
+
+	* res/res_phoneprov.c: inter-module dependencies should be included
+	  in the source code, not just in sample config files
+
+	* res/res_phoneprov.c: correct file name in message
+
+2008-10-15 21:00 +0000 [r149803]  Mark Michelson <mmichelson at digium.com>
+
+	* channels/chan_sip.c: Make the sip_proxy struct reference counted.
+	  This is necessary to allow for a sip_pvt to maintain a reference
+	  to a sip_peer's outboundproxy even after the peer has been freed.
+	  (closes issue #13700) Reported by: fnordian Patches: 13700.patch
+	  uploaded by putnopvut (license 60) Tested by: fnordian
+
+2008-10-15 20:22 +0000 [r149758]  BJ Weschke <bweschke at btwtech.com>
+
+	* 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 19:09 +0000 [r149589-149688]  Tilghman Lesher <tlesher at digium.com>
+
+	* funcs/func_odbc.c: Permit data fields to contain more than 255
+	  characters. (closes issue #13631) Reported by: seanbright Patches:
+	  20081015__bug13631.diff.txt uploaded by Corydon76 (license 14)
+	  Tested by: blitzrage
+
+	* funcs/func_odbc.c: Only set buf to blank before the goto.
+
+	* codecs/lpc10/lpcini.c: When using MALLOC_DEBUG, codec_lpc10 leaks
+	  memory, because it matches a library malloc() with an ast_free
+	  (which, of course, doesn't match up with known allocated memory,
+	  so the free fails). (closes issue #13702) Reported by: eliel
+	  Patches: codec_lpc10_lpcini.c uploaded by eliel (license 64)
+
+	* apps/app_echo.c: Minor spacing change (closes issue #13697)
+	  Reported by: alecdavis Patches: app_echo.bug13697.103249.diff.txt
+	  uploaded by alecdavis (license 585)
+
+2008-10-15 11:32 +0000 [r149512]  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:58 +0000 [r149203-149280]  Mark Michelson <mmichelson at digium.com>
+
+	* CHANGES, apps/app_dial.c: When specifying an invalid timeout to
+	  Dial, take it to mean that no timeout is desired. (closes issue
+	  #13625) Reported by: atis
+
+	* 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
+	  audiohooks over small inconsistencies in synchronization. Related
+	  to issue #13005, and solves the issue for most people who were
+	  experiencing the problem. However, a small number of people are
+	  still experiencing the problem on long calls, so I am not
+	  closing the issue yet
+
+	* apps/app_queue.c: Update the queue with the correct number of
+	  calls and whether the call was completed within the service level
+	  when a transfer takes place. This way, we do not "break" the
+	  leastrecent and fewestcalls strategies by not logging a call
+	  until after the transferred call has ended. (closes issue #13395)
+	  Reported by: Marquis Patches: app_queue.c.transfer.patch uploaded
+	  by Marquis (license 32)
+
+2008-10-14 22:42 +0000 [r149202]  Tilghman Lesher <tlesher at digium.com>
+
+	* include/asterisk/hashtab.h, main/chanvars.c, main/config.c,
+	  main/hashtab.c, pbx/pbx_spool.c, channels/chan_sip.c,
+	  include/asterisk/chanvars.h, include/asterisk/config.h,
+	  include/asterisk/strings.h, res/res_indications.c: Add additional
+	  memory debugging to several core APIs, and fix several memory
+	  leaks found with these changes. (Closes issue #13505, closes
+	  issue #13543) Reported by: mav3rick, triccyx Patches:
+	  20081001__bug13505.diff.txt uploaded by Corydon76 (license 14)
+	  Tested by: mav3rick, triccyx
+
+2008-10-14 21:09 +0000 [r149132]  Mark Michelson <mmichelson at digium.com>
+
+	* channels/chan_sip.c: Don't allow reserved characters to be used in
+	  register lines in sip.conf. (closes issue #13570) Reported by:
+	  putnopvut
+
+2008-10-14 20:17 +0000 [r149063]  Tilghman Lesher <tlesher at digium.com>
+
+	* apps/app_waitforsilence.c: Check correct values in the return of
+	  ast_waitfor(); also, get rid of a possible memory leak. (closes
+	  issue #13658) Reported by: explidous Patch by: me
+
+2008-10-14 19:42 +0000 [r149060]  Leif Madsen <lmadsen at digium.com>
+
+	* doc/manager_1_1.txt: Add missing documentation for SipShowRegistry
+	  action and RegistryEntry event. (closes issue #13342) Reported and
+	  patch by: Laureano
+
+2008-10-14 18:59 +0000 [r148918-148986]  Tilghman Lesher <tlesher at digium.com>
+
+	* apps/app_sms.c: App is ignoring 'p' parameter -- initial pause.
+	  (closes issue #13617) Reported by: alecdavis Patches:
+	  app_sms.13oct.diff.txt uploaded by alecdavis (license 585)
+
+	* apps/app_voicemail.c: Ensure that mail headers are 7-bit clean,
+	  even when UTF-8 characters are used in headers like 'Subject' and
+	  'To'.  Closes AST-107.
+
+2008-10-14 17:39 +0000 [r148915]  Mark Michelson <mmichelson at digium.com>
+
+	* channels/chan_local.c: Deadlock prevention in chan_local. (closes
+	  issue #13676) Reported by: tacvbo Patches: 13676.patch uploaded by
+	  putnopvut (license 60) Tested by: tacvbo
+
+2008-10-14 15:18 +0000 [r148869]  Tilghman Lesher <tlesher at digium.com>
+
+	* apps/app_fax.c: API differences in spandsp 0.0.6pre1 and higher
+	  (closes issue #13688) Reported by: irroot Patches:
+	  app_fax-span6.patch uploaded by irroot (license 52) with minor
+	  modifications by me
+
+2008-10-14 11:35 +0000 [r148614-148763]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* channels/chan_sip.c: fix some references to the owner of a
+	  private structure that may not be present
+
+	* Makefile: on Ubuntu (at least), recent versions of ld in binutils
+	  delete all debugging symbols when -x is supplied; since the
+	  reasons why -x is being passed are lost in the mists of time,
+	  remove it so debugging will work properly
+
+	* channels/chan_sip.c: ensure that *all* fields in the req
+	  structure are cleared out before reusing it; has_to_tag was not
+	  cleared, which caused the second incoming call over a TCP socket
+	  to fail if pedantic checking was enabled
+
+	* main/translate.c: it would be nice if this message printing code
+	  had actually been tested before it was committed...
+
+2008-10-13 17:56 +0000 [r148562]  Steve Murphy <murf at digium.com>
+
+	* main/pbx.c: Hmmm. Nobody (but me) is interested in seeing the
+	  trie info when they do 'dialplan show ...' (even with debug set
+	  to non-zero); so I set up a 'dialplan debug [context]' cli
+	  command instead, to explicitly show just the trie info. I even
+	  added an extension_exists() call to make sure the trie info is
+	  built. I moved the explanatory header to above the extension loop
+	  to ensure it only prints once. And it will do this now, whether
+	  debug is set or not. I removed the trie printing from the
+	  'dialplan show' command entirely.
+
+2008-10-13 15:36 +0000 [r148472]  Olle Johansson <oej at edvina.net>
+
+	* channels/chan_sip.c: Sending a 403 after a 200 is considered very
+	  bad. (found at SIPit)
+
+2008-10-10 21:22 +0000 [r148375-148377]  Mark Michelson <mmichelson at digium.com>
+
+	* channels/chan_sip.c: The logic used when checking a peer got
+	  changed subtly in the "kill the user" commit and caused calls
+	  relying on the insecure setting to not work properly. I changed
+	  for finding a peer back to how it was prior to that commit.
+	  (closes issue #13644) Reported by: pj Patches: 13644_trunkv2.patch
+	  uploaded by putnopvut (license 60) Tested by: pj
+
+	* channels/chan_sip.c: Make sure that the inUse and inRinging
+	  fields for a sip peer cannot go below zero. This is a
+	  regression from 1.4 and so it will be applied to 1.6.0 as
+	  well. (closes issue #13668) Reported by: mjc
+
+2008-10-10 16:37 +0000 [r148269]  Tilghman Lesher <tlesher at digium.com>
+
+	* apps/app_voicemail.c: User not notified of temporary greeting, if
+	  ODBC storage is in use. (closes issue #13659) Reported by:
+	  moliveras Patches: 20081009__bug13659.diff.txt uploaded by
+	  Corydon76 (license 14) Tested by: moliveras
+
+2008-10-10 01:33 +0000 [r148240]  Sean Bright <sean.bright at gmail.com>
+
+	* res/res_config_sqlite.c, apps/app_voicemail.c,
+	  include/asterisk.h, main/tdd.c, main/cryptostub.c: Don't include
+	  logger.h in asterisk.h by default as it is causing problems
+	  building app_voicemail. Instead, include it where it is needed.
+	  This turned out to be a relatively minor issue because other
+	  headers include logger.h as well. Need to test -addons before
+	  merging this back to 1.6.0. (closes issue #13605) Reported by:
+	  tomo1657 Patches: 13605_seanbright.diff uploaded by seanbright
+	  (license 71) Tested by: mmichelson
+
+2008-10-09 23:55 +0000 [r148151-148161]  Mark Michelson <mmichelson at digium.com>
+
+	* main/manager.c: The priority was unnecessary for the manager
+	  atxfer, so it has been removed. Furthermore, now we actually use
+	  the Context argument passed to set the transfer context and don't
+	  error out if no context is specified. This addresses the actual
+	  problems outlined in issue 12158. Regarding the other points
+	  brought up, regarding the inability to not transfer to extensions
+	  which cannot be represented by DTMF, it is not enough of a
+	  constraint that it is worth attempting to rework the feature.
+	  (closes issue #12158) Reported by: davidw
+
+	* apps/app_voicemail.c: Read the callerid in the correct order and
+	  make sure to read the Urgent flag value from the IMAP headers.
+	  (closes issue #13652) Reported by: jaroth Patches:
+	  imapheaders.patch uploaded by jaroth (license 50)
+
+2008-10-09 23:27 +0000 [r148128]  Tilghman Lesher <tlesher at digium.com>
+
+	* configs/res_ldap.conf.sample: Fix example schema (closes issue
+	  #12860) Reported by: flyn Patches: res_ldap.conf.patch uploaded
+	  by flyn (license 503)
+
+2008-10-09 23:20 +0000 [r148115]  Mark Michelson <mmichelson at digium.com>
+
+	* main/features.c: (closes issue #13579) Reported by: dwagner
+	  (closes issue #13584) Reported by: dwagner Tested by: murf,
+	  putnopvut The thought occurred to me that the res= from the
+	  extension spawn was ending up being returned from the bridge.
+	  "Thou shalt not poison the return value". Made the change and it
+	  appears to allow blind xfers to work as normal. If I'm wrong,
+	  reopen the bugs. But it looks good to me! Many thanks to
+	  putnopvut for helping me reproduce this!
+
+2008-10-09 20:01 +0000 [r148006-148011]  Tilghman Lesher <tlesher at digium.com>
+
+	* sounds/Makefile, sounds/sounds.xml: Publish MOH files in sln16
+	  format
+
+	* apps/app_voicemail.c: When blank, callerid name and number
+	  should display "unknown caller" in voicemail emails. (Closes
+	  issue #13643)
+
+2008-10-09 19:28 +0000 [r147957]  Jeff Peeler <jpeeler at digium.com>
+
+	* main/features.c: (closes issue #13139) Reported by: krisk84
+	  Tested by: krisk84 This change prevents a call that is placed in
+	  the parkinglot to be picked up before the PBX is finished. If
+	  another extension dials the parking extension before the PBX
+	  thread has completed at minimum warnings will occur about the PBX
+	  not properly being terminated. At worst, a crash could occur.
+
+2008-10-09 17:54 +0000 [r147901]  Michiel van Baak <michiel at vanbaak.info>
+
+	* include/asterisk/endian.h: only include this for OpenBSD. At least
+	  FreeBSD is borked when including it (closes issue #13649)
+	  Reported by: ys
+
+2008-10-09 17:47 +0000 [r147898]  Tilghman Lesher <tlesher at digium.com>
+
+	* configs/extensions.conf.sample: Remove "second form" of
+	  extensions, as it no longer applies. Also, cleanup the grammar,
+	  formatting, and introduce several clarifications to the text.
+	  (Closes issue #13654)
+
+2008-10-09 15:06 +0000 [r147811]  Steve Murphy <murf at digium.com>
+
+	* channels/chan_iax2.c, main/astobj2.c, channels/chan_oss.c,
+	  main/config.c, main/rtp.c, main/cli.c, channels/chan_usbradio.c,
+	  configure, channels/console_gui.c, utils/extconf.c, main/pbx.c,
+	  include/asterisk.h, doc/CODING-GUIDELINES,
+	  include/asterisk/autoconfig.h.in, main/translate.c,
+	  channels/vcodecs.c, configure.ac, channels/console_video.c:
+	  (closes issue #13557) Reported by: nickpeirson Patches:
+	  pbx.c.patch uploaded by nickpeirson (license 579)
+	  replace_bzero+bcopy.patch uploaded by nickpeirson (license 579)
+	  Tested by: nickpeirson, murf 1. replaced all refs to bzero and
+	  bcopy to memset and memmove instead. 2. added a note to the
+	  CODING-GUIDELINES 3. add two macros to asterisk.h to prevent
+	  bzero, bcopy from creeping back into the source 4. removed bzero
+	  from configure, configure.ac, autoconfig.h.in
+
+2008-10-08 22:33 +0000 [r147719]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_meetme.c: Some small tweaks regarding realtime conference
+	  announcements. (closes issue #13522) Reported by: DEA Patches:
+	  meetme-rt-fixes.txt uploaded by DEA (license 3)
+
+2008-10-08 22:27 +0000 [r147692]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* channels/chan_dahdi.c: when parsing a text configuration option,
+	  ensure that the buffer on the stack is actually large enough to
+	  hold the legal values of that option, and also ensure that
+	  sscanf() knows to stop parsing if it would overrun the buffer
+	  (without these changes, specifying "buffers=...,immediate" would
+	  overflow the buffer on the stack, and could not have worked as
+	  expected)
+
+2008-10-08 19:09 +0000 [r147593]  Tilghman Lesher <tlesher at digium.com>
+
+	* apps/app_sms.c: Correct a typo in the help; also, ensure that the
+	  date and time are correctly set, if not specified in the message.
+	  (Closes issue #13594, closes issue #13595) Reported by: alecdavis
+	  Patches: 20081001__bug13595.diff.txt uploaded by Corydon76
+	  (license 14) Tested by: alecdavis
+
+2008-10-08 15:10 +0000 [r147519]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_speech_utils.c: If we receive DTMF make sure that the
+	  state of the speech structure goes back to being not ready.
+	  (issue #LUMENVOX-8)
+
+2008-10-07 16:54 +0000 [r147196]  Sean Bright <sean.bright at gmail.com>
+
+	* apps/app_voicemail.c: Make 'imapsecret' an alias to
+	  'imappassword' in voicemail.conf.
+
+2008-10-07 16:05 +0000 [r147147]  Jeff Peeler <jpeeler at digium.com>
+
+	* main/features.c: Explicitly setting these fields to NULL was done
+	  because I wasn't sure if they would be NULL otherwise. Since they
+	  will be set automatically, removing.
+
+2008-10-07 15:06 +0000 [r147100]  Richard Mudgett <rmudgett at digium.com>
+
+	* funcs/func_callerid.c: Independent change from branch issue8824
+	  that is not part of COLP. (-r142574 rmudgett)
+
+2008-10-07 12:03 +0000 [r147052]  Sean Bright <sean.bright at gmail.com>
+
+	* apps/app_dial.c: Make sure to compare the correct number of
+	  characters when special-casing our DAHDI operator mode stuff.
+	  Technically, it would work fine, as 'DAH' is currently unique
+	  amongst our channel technologies, but as Jared points out:
+	  <@jsmith> Sure... as long as the technology starts whith DAH....
+	  but it could be DAHDOO!
+
+2008-10-07 00:13 +0000 [r146972]  Terry Wilson <twilson at digium.com>
+
+	* channels/chan_sip.c: A blind transfer to the parking thread would
+	  cause a segfault because copy_request accesses dst->data w/o
+	  being able to tell whether it is proerly initialized
+
+2008-10-06 23:22 +0000 [r146930]  Tilghman Lesher <tlesher at digium.com>
+
+	* include/asterisk/threadstorage.h: Update documentation;
+	  AST_THREADSTORAGE() in trunk only takes a single argument.
+
+2008-10-06 23:08 +0000 [r146876-146924]  Jeff Peeler <jpeeler at digium.com>
+
+	* include/asterisk/features.h, main/features.c, res/res_agi.c:
+	  Similar to r143204, masquerade the channel in the case of Park
+	  being called from AGI. ........
+
+	* include/asterisk/endian.h: Mvanbaak said this was needed to
+	  compile on OpenBSD, so put it in the OpenBSD section.
+
+	* main/features.c: This commit squashes together three commits
+	  because the wrong approach was originally used. (One of the
+	  commits was only one line.) 1) r143204: The main change here was
+	  to masquerade the channel if the channel that was to be parked
+	  was running a PBX on it. The PBX thread can then maintain full
+	  control of the channel (the zombie) as it expects to while
+	  allowing the parking thread full control of the real (parked)
+	  channel. 2) r143270: Changed park_call_full to hold the
+	  parkinglot lock a little longer, which protects the parkeduser
+	  struct from being freed out from underneath. Made sure that the
+	  parking extension is added to the parking context while holding
+	  the lock thereby ensuring that there are no spurious warnings
+	  from removal attempts when a hangup occurs while the parking lot
+	  is being announced. 3) r143475: (the one liner) compare peer and
+	  chan instead of looking at the parked user (pu), which could have
+	  possibly already have been freed by the parking thread
+
+	* main/features.c: fix some comment placement
+
+	* main/features.c: Explicitly set args in park_call_exec NULL so in
+	  the case of no options being passed in, there is no garbage
+	  attempted to be used. Also, do not set args to unknown value
+	  again if there are no options passed in.
+
+2008-10-06 21:53 +0000 [r146874]  Michiel van Baak <michiel at vanbaak.info>
+
+	* include/asterisk/endian.h: make aescrypt.c compile on OpenBSD again
+
+2008-10-06 21:32 +0000 [r146715-146838]  Tilghman Lesher <tlesher at digium.com>
+
+	* channels/chan_iax2.c, funcs/func_callerid.c,
+	  apps/app_speech_utils.c, funcs/func_curl.c,
+	  funcs/func_groupcount.c, res/res_smdi.c, channels/chan_sip.c,
+	  funcs/func_timeout.c, funcs/func_odbc.c, funcs/func_cdr.c,
+	  funcs/func_math.c: Dialplan functions should not actually return
+	  0, unless they have modified the workspace. To signal an error
+	  (and no change to the workspace), -1 should be returned instead.
+	  (closes issue #13340) Reported by: kryptolus Patches:
+	  20080827__bug13340__2.diff.txt uploaded by Corydon76 (license 14)
+
+	* channels/chan_local.c: Check whether an extension exists in the
+	  _call method, rather than the _alloc method, because we need to
+	  evaluate the callerid (since that data affects whether an
+	  extension exists).  (closes issue #13343) Reported by: efutch
+	  Patches: 20080915__bug13343.diff.txt uploaded by Corydon76
+	  (license 14) Tested by: efutch
+
+2008-10-06 16:39 +0000 [r146698]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* channels/chan_dahdi.c: ensure that the private structure for
+	  pseudo channels is created without 'leaking' configuration data
+	  from other configured channels (closes issue #13555) Reported by:
+	  jeffg Patches: issue_13555.patch uploaded by kpfleming (license
+	  421) Tested by: jeffg
+
+2008-10-06 00:23 +0000 [r146557]  Sean Bright <sean.bright at gmail.com>
+
+	* utils/Makefile: Quote arguments to cp so we can handle spaces in
+	  our paths.
+
+2008-10-05 21:24 +0000 [r146451]  Jason Parker <jparker at digium.com>
+
+	* channels/chan_sip.c: Fix silly formatting.
+
+2008-10-04 01:57 +0000 [r146314]  Sean Bright <sean.bright at gmail.com>
+
+	* configs/sip_notify.conf.sample: Add ability to remotely reboot
+	  snom phones. Also cleaned up and reorganized
+	  sip_notify.conf.sample a bit as well. Tested snom reboot on snom
+	  360 and verified snom-check-cfg worked as well. (closes issue
+	  #13601) Reported by: mjc Tested by: seanbright
+
+2008-10-03 22:42 +0000 [r146243]  Jeff Peeler <jpeeler at digium.com>
+
+	* main/features.c: remove superfluous reference counting operations
+	  in manage_parkinglot since ao2_interator_next increments the ref
+	  count automatically
+
+2008-10-03 22:13 +0000 [r146200]  Sean Bright <sean.bright at gmail.com>
+
+	* main/cli.c: Resolve a subtle bug where we would never
+	  successfully be able to get the first item in the CLI entry list.
+	  This was preventing '!' from showing up in either 'help' or in tab
+	  completion. (closes issue #13578) Reported by: mvanbaak
+
+2008-10-02 19:31 +0000 [r145960-145964]  Russell Bryant <russell at digium.com>
+
+	* CHANGES: The 'P' command for ExternalIVR was also added in 1.6.0
+
+	* CHANGES: TCP support for ExternalIVR went in to 1.6.1, not 1.6.0
+
+2008-10-02 15:30 +0000 [r145781]  Sean Bright <sean.bright at gmail.com>
+
+	* configure, configure.ac: This is much cleaner, methinks.
+
+2008-10-02 15:19 +0000 [r145754]  Tilghman Lesher <tlesher at digium.com>
+
+	* res/res_odbc.c: Some sanity checks that may have led to prior

[... 46874 lines stripped ...]



More information about the asterisk-commits mailing list