[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