[asterisk-commits] russell: tag 1.4.10.1 r78987 - in /tags/1.4.10.1: .lastclean .version ChangeLog
SVN commits to the Asterisk project
asterisk-commits at lists.digium.com
Fri Aug 10 09:37:06 CDT 2007
Author: russell
Date: Fri Aug 10 09:37:05 2007
New Revision: 78987
URL: http://svn.digium.com/view/asterisk?view=rev&rev=78987
Log:
importing files for 1.4.10.1 release
Added:
tags/1.4.10.1/.lastclean (with props)
tags/1.4.10.1/.version (with props)
tags/1.4.10.1/ChangeLog (with props)
Added: tags/1.4.10.1/.lastclean
URL: http://svn.digium.com/view/asterisk/tags/1.4.10.1/.lastclean?view=auto&rev=78987
==============================================================================
--- tags/1.4.10.1/.lastclean (added)
+++ tags/1.4.10.1/.lastclean Fri Aug 10 09:37:05 2007
@@ -1,0 +1,1 @@
+28
Propchange: tags/1.4.10.1/.lastclean
------------------------------------------------------------------------------
svn:eol-style = native
Propchange: tags/1.4.10.1/.lastclean
------------------------------------------------------------------------------
svn:keywords = none
Propchange: tags/1.4.10.1/.lastclean
------------------------------------------------------------------------------
svn:mime-type = text/plain
Added: tags/1.4.10.1/.version
URL: http://svn.digium.com/view/asterisk/tags/1.4.10.1/.version?view=auto&rev=78987
==============================================================================
--- tags/1.4.10.1/.version (added)
+++ tags/1.4.10.1/.version Fri Aug 10 09:37:05 2007
@@ -1,0 +1,1 @@
+1.4.10.1
Propchange: tags/1.4.10.1/.version
------------------------------------------------------------------------------
svn:eol-style = native
Propchange: tags/1.4.10.1/.version
------------------------------------------------------------------------------
svn:keywords = none
Propchange: tags/1.4.10.1/.version
------------------------------------------------------------------------------
svn:mime-type = text/plain
Added: tags/1.4.10.1/ChangeLog
URL: http://svn.digium.com/view/asterisk/tags/1.4.10.1/ChangeLog?view=auto&rev=78987
==============================================================================
--- tags/1.4.10.1/ChangeLog (added)
+++ tags/1.4.10.1/ChangeLog Fri Aug 10 09:37:05 2007
@@ -1,0 +1,10534 @@
+2007-08-10 Russell Bryant <russell at digium.com>
+
+ * Asterisk 1.4.10.1 released.
+
+2007-08-10 14:15 +0000 [r78951-78955] Joshua Colp <jcolp at digium.com>
+
+ * main/file.c: Don't bother having the core pass through or emulate
+ begin DTMF frames when in an ast_waitstream. It only cares about
+ the end of DTMF.
+
+ * configs/queues.conf.sample: (closes issue #10422) Reported by:
+ bhowell Add note to sample configuration about module load order
+ and how it can cause perfectly good queue members to be marked as
+ invalid.
+
+2007-08-10 13:24 +0000 [r78936] Christian Richter <christian.richter at beronet.com>
+
+ * channels/chan_misdn.c, channels/misdn/ie.c,
+ channels/misdn/isdn_msg_parser.c: fixed a bug with the useruser
+ information element. We send them now also in the disconnect
+ message.
+
+2007-08-09 23:47 +0000 [r78907] Mark Michelson <mmichelson at digium.com>
+
+ * apps/app_voicemail.c: Improved a bit of logic regarding
+ comma-separated mailboxes in has_voicemail. Also added some
+ braces to some compound if statements since unbraced if
+ statements scare me in general.
+
+2007-08-09 23:10 +0000 [r78891] Steve Murphy <murf at digium.com>
+
+ * Makefile: This fixes bug 10416; thanks to mvanbaak for the pretty
+ output
+
+2007-08-09 22:03 +0000 [r78826-78860] Mark Michelson <mmichelson at digium.com>
+
+ * apps/app_voicemail.c: Removing some extra debug code I left in my
+ last commit
+
+ * apps/app_voicemail.c: Quite a few changes regarding IMAP storage.
+ 1. instead of using inboxcount as the core message counting
+ function, we use messagecount instead. This makes it possible to
+ count messages in folders besides just INBOX and Old. 2.
+ inboxcount and hasvoicemail now use messagecount as their means
+ of determining return values. 3. Added a copy_message function
+ for IMAP storage. Unfortunately I don't have the means to test
+ it, but it seems like a pretty straightforward function. 4.
+ Removed a #ifndef IMAP_STORAGE and matching #endif from
+ leave_voicemail for a couple of reasons. One, we want to support
+ copying mail to multiple IMAP boxes, and two, IMAP was broken
+ because a STORE macro had been moved into this section of code.
+
+ * channels/chan_sip.c: I broke canreinvite...Now I'm fixing it. I
+ put some new code in the wrong place and so I've reverted the
+ canreinvite section to how it was and put my new code where it
+ should be.
+
+2007-08-09 17:58 +0000 [r78717-78778] Russell Bryant <russell at digium.com>
+
+ * apps/app_voicemail.c: add a comment to indicate that inboxcount
+ for ODBC_STORAGE needs to be fixed to support multiple mailboxes
+
+ * apps/app_voicemail.c: Fix subscriptions to multiple mailboxes for
+ ODBC_STORAGE. Also, leave a comment for this to be fixed for
+ IMAP_STORAGE, as well. I left IMAP alone since I know MarkM was
+ working on this code right now for another reason. This is broken
+ even worse in trunk, but for a different reason. The fact that
+ the mailbox option supported multiple mailboxes is completely not
+ obvious from the code in the channel drivers. Anyway, I will fix
+ that in another commit ...
+
+ * apps/app_meetme.c: Fix a problem with the combination of the 'F'
+ option to pass DTMF through a conference and options that use
+ DTMF to activate various features. The problem was that the BEGIN
+ frame would be passed through, but the END frame would get
+ intercepted to activate a feature. Then, the other conference
+ members would hear DTMF for forever, which they didn't seem to
+ like very much. (closes issue #10400, reported by stevefeinstein,
+ fixed by me)
+
+2007-08-08 19:29 +0000 [r78646] Jason Parker <jparker at digium.com>
+
+ * doc/jabber.txt: Fix mogs email address.
+
+2007-08-08 18:16 +0000 [r78575-78620] Mark Michelson <mmichelson at digium.com>
+
+ * apps/app_voicemail.c: Fixed some compiler warnings so that
+ compiling with dev-mode and IMAP storage would not have any
+ errors. This section of code may get changed again shortly since
+ my change uncovers a rather silly bit of logic.
+
+ * apps/app_queue.c: Changing a bit of logic so that someone will
+ NEVER exit the queue on timeout unless they have enabled the 'n'
+ option. This commit relates to issue #10320. Thanks to
+ jfitzgibbon for detailing the idea behind this code change.
+
+2007-08-08 13:51 +0000 [r78569] Joshua Colp <jcolp at digium.com>
+
+ * configs/sip.conf.sample: (closes issue #10335) Reported by:
+ adamgundy Update sip.conf to include another scenario where
+ directrtpsetup will fail.
+
+2007-08-07 Russell Bryant <russell at digium.com>
+
+ * Asterisk 1.4.10 released.
+
+2007-08-07 20:57 +0000 [r78488] Russell Bryant <russell at digium.com>
+
+ * res/res_config_odbc.c: Fix the build of this module on 64-bit
+ platforms
+
+2007-08-07 19:43 +0000 [r78450] Mark Michelson <mmichelson at digium.com>
+
+ * apps/app_voicemail.c: The logic behind inboxcount's return value
+ was reversed in has_voicemail and message_count. (closes issue
+ #10401, reported by st1710, patched by me)
+
+2007-08-07 19:34 +0000 [r78437] Tilghman Lesher <tilghman at mail.jeffandtilghman.com>
+
+ * res/res_odbc.c: Don't free the environment handle when the
+ connection fails, because other connections might be depending
+ upon it
+
+2007-08-07 19:11 +0000 [r78416] Jason Parker <jparker at digium.com>
+
+ * channels/chan_sip.c: Allow chan_sip to build in devmode
+
+2007-08-07 19:09 +0000 [r78415] Tilghman Lesher <tilghman at mail.jeffandtilghman.com>
+
+ * apps/app_voicemail.c, res/res_config_odbc.c,
+ apps/app_directory.c: Reconnection doesn't happen automatically
+ when a DB goes down (fixes issue #9389)
+
+2007-08-07 18:25 +0000 [r78375] Jason Parker <jparker at digium.com>
+
+ * channels/chan_skinny.c: Properly check the capabilities count to
+ avoid a segfault. (ASA-2007-019)
+
+2007-08-07 17:45 +0000 [r78371] Russell Bryant <russell at digium.com>
+
+ * channels/chan_zap.c, /: Merged revisions 78370 via svnmerge from
+ https://origsvn.digium.com/svn/asterisk/branches/1.2 ........
+ r78370 | russell | 2007-08-07 12:44:04 -0500 (Tue, 07 Aug 2007) |
+ 4 lines Revert patch committed for issue #9660. It broke E&M
+ trunks. (closes issue #10360) (closes issue #10364) ........
+
+2007-08-06 21:41 +0000 [r78275] Joshua Colp <jcolp at digium.com>
+
+ * main/channel.c: Add additional DTMF log messages to help when
+ debugging issues.
+
+2007-08-06 20:44 +0000 [r78184-78242] Russell Bryant <russell at digium.com>
+
+ * channels/chan_iax2.c: Fix an issue where dynamic threads can get
+ free'd, but still exist in the dynamic thread list. (closes issue
+ #10392, patch from Mihai, with credit to his colleague, Pete)
+
+ * include/asterisk/linkedlists.h: Fix the return value of
+ AST_LIST_REMOVE(). This shouldn't be causing any problems,
+ though, because the only code that uses the return value only
+ checks to see if it is NULL. (closes issue #10390, pointed out by
+ mihai)
+
+2007-08-06 16:32 +0000 [r78182] Joshua Colp <jcolp at digium.com>
+
+ * channels/chan_sip.c: It is possible for a transfer to occur
+ before the remote device has our tag in which case they send none
+ in the transfer. In this case we need to not fail the transfer
+ dialog lookup.
+
+2007-08-06 16:30 +0000 [r78180] Jason Parker <jparker at digium.com>
+
+ * main/config.c: Fix an issue with using UpdateConfig (manager
+ action) where escaped semicolons in a config would be converted
+ to just semicolons (\; to ;) Issue 9938
+
+2007-08-06 15:27 +0000 [r78166-78172] Joshua Colp <jcolp at digium.com>
+
+ * main/rtp.c: (closes issue #10355) Reported by: wdecarne Now that
+ we pass through RTP timestamp information we need to make the
+ allowed timestamp skew considerably less. There are situations
+ where a source may change and due to the timestamp difference the
+ receiver will experience an audio gap since we did not indicate
+ by setting the marker bit that the source changed.
+
+ * configure, configure.ac: (closes issue #10383) Reported by: rizzo
+ Include stdlib.h so NULL gets defined for gethostbyname_r checks.
+
+2007-08-06 13:33 +0000 [r78164] Mark Michelson <mmichelson at digium.com>
+
+ * channels/chan_sip.c: Fixed a mistake I made in realtime_peer
+ which caused it to return NULL every time. Thanks to Jon Fealy
+ for emailing me the correction.
+
+2007-08-05 14:18 +0000 [r78146] Tilghman Lesher <tilghman at mail.jeffandtilghman.com>
+
+ * cdr/cdr_pgsql.c: Portability fix for devmode compiling (closes
+ bug #10382)
+
+2007-08-05 04:15 +0000 [r78143] Russell Bryant <russell at digium.com>
+
+ * include/asterisk/lock.h: Fix compilation failure when
+ MALLOC_DEBUG is enabled, but DEBUG_THREADS is not
+
+2007-08-05 03:29 +0000 [r78139] Tilghman Lesher <tilghman at mail.jeffandtilghman.com>
+
+ * channels/chan_sip.c: If peer is not found, the error message is
+ misleading (should be peer not found, not ACL failure)
+
+2007-08-03 20:25 +0000 [r78103] Mark Michelson <mmichelson at digium.com>
+
+ * main/config.c, channels/chan_sip.c, include/asterisk/config.h:
+ Changed the behavior of sip's realtime_peer function to match the
+ corresponding way of matching for non-realtime peers. Now matches
+ are made on both the IP address and port number, or if the
+ insecure setting is set to "port" then just match on the IP
+ address. In order to accomplish this, I also added a new API
+ call, ast_category_root, which returns the first variable of an
+ ast_category struct
+
+2007-08-03 20:14 +0000 [r78028-78101] Russell Bryant <russell at digium.com>
+
+ * apps/app_voicemail.c: (closes issue #10194) Reported by:
+ blitzrage Patches: bug0010194 uploaded by vovochka Tested by:
+ blitzrage Fix a problem when you call Voicemail() with multiple
+ mailboxes specified and ODBC_STORAGE is in use. The audio part of
+ the message was only given to the first mailbox specified.
+
+ * main/utils.c, include/asterisk/lock.h, main/astmm.c: Add some
+ improvements to lock debugging. These changes take effect with
+ DEBUG_THREADS enabled and provide the following: * This will keep
+ track of which locks are held by which thread as well as which
+ lock a thread is waiting for in a thread-local data structure. A
+ reference to this structure is available on the stack in the
+ dummy_start() function, which is the common entry point for all
+ threads. This information can be easily retrieved using gdb if
+ you switch to the dummy_start() stack frame of any thread and
+ print the contents of the lock_info variable. * All of the
+ thread-local structures for keeping track of this lock
+ information are also stored in a list so that the information can
+ be dumped to the CLI using the "core show locks" CLI command.
+ This introduces a little bit of a performance hit as it requires
+ additional underlying locking operations inside of every
+ lock/unlock on an ast_mutex. However, the benefits of having this
+ information available at the CLI is huge, especially considering
+ this is only done in DEBUG_THREADS mode. It means that in most
+ cases where we debug deadlocks, we no longer have to request
+ access to the machine to analyze the contents of ast_mutex_t
+ structures. We can now just ask them to get the output of "core
+ show locks", which gives us all of the information we needed in
+ most cases. I also had to make some additional changes to astmm.c
+ to make this work when both MALLOC_DEBUG and DEBUG_THREADS are
+ enabled. I disabled tracking of one of the locks in astmm.c
+ because it gets used inside the replacement memory allocation
+ routines, and the lock tracking code allocates memory. This
+ caused infinite recursion.
+
+ * channels/chan_iax2.c: Only pass through HOLD and UNHOLD control
+ frames when the mohinterpret option is set to "passthrough". This
+ was pointed out by Kevin in the middle of a training session.
+
+ * channels/chan_iax2.c: Don't reuse the timespec that was set to 0
+ in the previous timedwait as it will just return immediately.
+ Also, fix some logic so the thread's lock isn't unlocked twice in
+ the weird case of dynamic threads getting acquired right after a
+ timeout. (pointed out by SteveK)
+
+2007-08-02 21:53 +0000 [r77993-77996] Jason Parker <jparker at digium.com>
+
+ * channels/chan_skinny.c, configs/skinny.conf.sample: Make sure we
+ actually allow 6 chars to be sent. Also make note of the "A"
+ option of date format. Issue 9779, modifications by DEA, wedhorn,
+ and myself.
+
+ * channels/chan_skinny.c: If a device disconnects, the session will
+ go away. If this happens during call setup, we need to give up.
+ Issue 10325.
+
+2007-08-02 19:25 +0000 [r77949] Russell Bryant <russell at digium.com>
+
+ * channels/chan_iax2.c: Fix the case where a dynamic thread times
+ out waiting for something to do during the first time it runs.
+ This shouldn't ever happen, but we should account for it anyway.
+ (pointed out by pete, who works with mihai)
+
+2007-08-02 18:42 +0000 [r77947] Jason Parker <jparker at digium.com>
+
+ * channels/chan_skinny.c: Make sure we clear the prompt status
+ message on a hangup. Also rearrange messages to better fit with
+ what a wireshark trace shows it should be. Issue 10299, initial
+ patch and solution by sbisker, modified by me to fit with
+ wireshark trace.
+
+2007-08-02 18:21 +0000 [r77945] Steve Murphy <murf at digium.com>
+
+ * main/fskmodem.c, /: Merged revisions 77942 via svnmerge from
+ https://origsvn.digium.com/svn/asterisk/branches/1.2 ........
+ r77942 | murf | 2007-08-02 11:56:37 -0600 (Thu, 02 Aug 2007) | 1
+ line This patch hopefully solves 10141; The user is running with
+ it, and it doesn't appear to harm asterisk's operation, and may
+ prevent a crash. I'll store it in 1.2, as we have shut down
+ support on 1.2, but since I developed the patch before support
+ finished, and it might affect 1.4 and trunk, I'm going ahead with
+ it. ........
+
+2007-08-02 18:04 +0000 [r77939-77943] Russell Bryant <russell at digium.com>
+
+ * channels/chan_iax2.c: Fix another race condition in the handling
+ of dynamic threads. If the dynamic thread timed out waiting for
+ something to do, but was acquired to perform an action
+ immediately afterwords, then wait on the condition again to give
+ the other thread a chance to finish setting up the data for what
+ action this thread should perform. Otherwise, if it immediately
+ continues, it will perform the wrong action. (reported on IRC by
+ mihai, patch by me) (related to issue #10289)
+
+ * channels/chan_iax2.c: Add another sanity check to
+ vnak_retransmit(). This check ensures that frames that have
+ already been marked for deletion don't get retransmitted. (closes
+ issue #10361, patch from mihai)
+
+2007-08-02 15:15 +0000 [r77890-77894] Jason Parker <jparker at digium.com>
+
+ * channels/chan_skinny.c: Make sure that we show the correct
+ extension if dialed from a macro "From: 5555" rather than "From:
+ s" Issue 10358, initial patch by DEA, reworked by me to use S_OR,
+ tested by sbisker
+
+ * channels/chan_skinny.c: Put in some additional debug information
+ for softkey/stimulus messages. Issue 10291, patch by DEA.
+
+2007-08-01 22:16 +0000 [r77887] Russell Bryant <russell at digium.com>
+
+ * channels/chan_iax2.c: Fix some race conditions which have been
+ causing weird problems in chan_iax2. The most notable problem is
+ that people have been seeing storms of VNAK frames being sent due
+ to really old frames mysteriously being in the retransmission
+ queue and never getting removed. It was possible that a dynamic
+ thread got created, but did not acquire its lock before the
+ thread that created it signals it to perform an action. When this
+ happens, the thread will sleep until it hits a timeout, and then
+ get destroyed. So, the action never gets performed and in some
+ cases, means a frame doesn't get transmitted and never gets freed
+ since the scheduler never gets a chance to reschedule
+ transmission. Another less severe race condition is in the
+ handling of a timeout for a dynamic thread. It was possible for
+ it to be acquired to perform at action at the same time that it
+ hit a timeout. When this occurs, whatever action it was acquired
+ for would never get performed. (patch contributed by Mihai and
+ SteveK) (closes issue #10289) (closes issue #10248) (closes issue
+ #10232) (possibly related to issue #10359)
+
+2007-08-01 22:14 +0000 [r77886] Tilghman Lesher <tilghman at mail.jeffandtilghman.com>
+
+ * apps/app_voicemail.c: Voicemail with ODBC_STORAGE defined does
+ not compile cleanly (missing def)
+
+2007-08-01 21:08 +0000 [r77883] Jason Parker <jparker at digium.com>
+
+ * channels/chan_skinny.c: Fix an issue that caused one-way audio on
+ some newer devices (specifically the 7921), due to sending
+ packets in the wrong order during hangup. Also make sure we clear
+ tones/messages on the correct line/instance. Issue 10291, patch
+ by DEA, tested by sbisker and myself.
+
+2007-08-01 18:08 +0000 [r77863-77871] Joshua Colp <jcolp at digium.com>
+
+ * main/cli.c: (closes issue #10351) Reported by: ftarz Some
+ platforms don't like it when you pass NULL to vsnprintf so pass
+ "" instead.
+
+ * include/asterisk/threadstorage.h, channels/chan_mgcp.c,
+ apps/app_voicemail.c, main/acl.c, utils/smsq.c,
+ channels/chan_iax2.c: Add some fixes for building on Solaris.
+
+ * main/utils.c: Whoops, I meant R_5 not R5.
+
+ * configure, configure.ac: And for my last trick... make sure that
+ if gethostbyname_r is exported by a library that it is used.
+
+ * configure, include/asterisk/autoconfig.h.in, configure.ac,
+ main/utils.c: Extend autoconf logic to determine which version of
+ gethostbyname_r is on the system.
+
+2007-08-01 14:08 +0000 [r77852-77854] Mark Michelson <mmichelson at digium.com>
+
+ * apps/app_queue.c: Fixes an issue I introduced to queues wherein a
+ queue with joinempty=yes would kick people out of the queue
+ because of erroneously thinking the 'n' option was in use.
+ (closes issue #10320, reported by jfitzgibbon, patched by me,
+ tested by blitzrage and me) Thank you blitzrage for all the
+ testing you've done lately with queues! It's much appreciated!
+
+ * apps/app_queue.c: If a queue uses dynamic realtime members, then
+ the member list should be updated after each attempt to call the
+ queue. This fixes an issue where if a caller calls into a queue
+ where no one is logged in, they would wait forever even if a
+ member logged in at some point. (closes issue #10346, reported by
+ and tested by blitzrage, patched by me)
+
+2007-07-31 21:09 +0000 [r77845-77846] Jim Dixon <telesistant at hotmail.com>
+
+ * apps/app_rpt.c: Much newer version, 0.70 with much additions
+
+ * main/dsp.c, channels/chan_zap.c: Made VAST improvements in DTMF
+ receiver in RADIO_RELAX mode (thanx Steve W9SH), and oversight in
+ logic in TONE_VERIFY/RELAX mode in chan_zap.
+
+2007-07-31 20:59 +0000 [r77844] Steve Murphy <murf at digium.com>
+
+ * /, contrib/scripts/ast_grab_core: Merged revisions 77842 via
+ svnmerge from
+ https://origsvn.digium.com/svn/asterisk/branches/1.2 ........
+ r77842 | murf | 2007-07-31 13:19:35 -0600 (Tue, 31 Jul 2007) | 1
+ line This probably isn't super-general, but it's a first stab at
+ using kill -11 to generate a core file instead of gcore. ........
+
+2007-07-31 16:17 +0000 [r77831] Joshua Colp <jcolp at digium.com>
+
+ * res/res_speech.c, include/asterisk/speech.h: Add a flag to the
+ speech API that allows an engine to set whether it received
+ results or not.
+
+2007-07-31 15:53 +0000 [r77827] Kevin P. Fleming <kpfleming at digium.com>
+
+ * build_tools/cflags.xml: DETECT_DEADLOCKS can't be enabled without
+ DEBUG_THREADS or it does nothing
+
+2007-07-31 15:21 +0000 [r77824] Mark Michelson <mmichelson at digium.com>
+
+ * channels/chan_sip.c: This patch makes Asterisk send 100 Trying
+ provisional responses upon receipt of re-invites. This makes it
+ so that if there are two or more Asterisk servers between
+ endpoints, the Asterisk servers will not keep retransmitting the
+ re-invites. (closes issue #10274, reported by cstadlmann, patched
+ by me with approval from file)
+
+2007-07-30 20:17 +0000 [r77795] Jason Parker <jparker at digium.com>
+
+ * main/say.c: Applications like SayAlpha() should not hang up the
+ channel if you request an "unknown" character such as a comma.
+ Instead, skip the character and move on. Issue 10083, initial
+ patch by jsmith, modified by me.
+
+2007-07-30 20:16 +0000 [r77785-77794] Russell Bryant <russell at digium.com>
+
+ * channels/chan_iax2.c: Fix an issue that could potentially cause
+ corruption of the global iax frame queue. In the network_thread()
+ loop, it traverses the list using the AST_LIST_TRAVERSE_SAFE
+ macro. However, to remove an element of the list within this
+ loop, it used AST_LIST_REMOVE, instead of
+ AST_LIST_REMOVE_CURRENT, which I believe could leave some of the
+ internal variables of the SAFE macro invalid. Mihai says that he
+ already made this change in his local copy and it didn't help his
+ VNAK storm issues, but I still think it's wrong. :)
+
+ * res/res_agi.c: (closes issue #10279) Reported by: seanbright
+ Patches: res_agi.carefulwrite.1.4.07252007.patch uploaded by
+ seanbright (license 71) res_agi.carefulwrite.trunk.07252007.patch
+ uploaded by seanbright (license 71) Allow the "agi_network: yes"
+ line to be printed out in the AGI debug output. Also, allow
+ partial writes to be handled when writing out this line just like
+ it is for all of the others.
+
+ * main/channel.c: file and I both committed changes for issue
+ #10301. Remove a duplicated assignment to restore the original
+ value of the previous channel.
+
+2007-07-30 18:43 +0000 [r77783] Tilghman Lesher <tilghman at mail.jeffandtilghman.com>
+
+ * /, res/res_agi.c: Merged revisions 77782 via svnmerge from
+ https://origsvn.digium.com/svn/asterisk/branches/1.2 ........
+ r77782 | tilghman | 2007-07-30 13:40:54 -0500 (Mon, 30 Jul 2007)
+ | 2 lines Revert change in revision 71656, even though it fixed a
+ bug, because many people were depending upon the (broken)
+ behavior. ........
+
+2007-07-30 17:29 +0000 [r77780] Russell Bryant <russell at digium.com>
+
+ * main/channel.c: (closes issue #10301) Reported by: fnordian
+ Patches: asterisk-1.4.9-channel.c.patch uploaded by fnordian
+ (license 110) Additional changes by me Fix some problems in
+ channel_find_locked() which can cause an infinite loop. The
+ reference to the previous channel is set to NULL in some cases.
+ These changes ensure that the reference to the previous channel
+ gets restored before needing it again. I'm not convinced that the
+ code that is setting it to NULL is really the right thing to do.
+ However, I am making these changes to fix the obvious problem and
+ just leaving an XXX comment that it needs a better explanation
+ that what is there now.
+
+2007-07-30 17:11 +0000 [r77768-77778] Joshua Colp <jcolp at digium.com>
+
+ * res/res_features.c: (closes issue #10327) Reported by: kkiely
+ Instead of directly mucking with the extension/context/priority
+ of the channel we are transferring when it has a PBX simply call
+ ast_async_goto on it. This will ensure that the channel gets
+ handled properly and sent to the right place.
+
+ * main/channel.c: (closes issue #10301) Reported by: fnordian
+ Patches: asterisk-1.4.9-channel.c.patch uploaded by fnordian
+ (license 110) Restore previous behavior where if we failed to
+ lock the channel we wanted we would return to exactly the same
+ point as if we had just reentered the function.
+
+ * /, apps/app_macro.c: Merged revisions 77767 via svnmerge from
+ https://origsvn.digium.com/svn/asterisk/branches/1.2 ........
+ r77767 | file | 2007-07-30 11:50:02 -0300 (Mon, 30 Jul 2007) | 4
+ lines (closes issue #10334) Reported by: ramonpeek Pass through
+ the return value from macro_exec through the MacroIf application.
+ ........
+
+2007-07-27 18:15 +0000 [r77571] Tilghman Lesher <tilghman at mail.jeffandtilghman.com>
+
+ * res/res_odbc.c: Missing newline
+
+2007-07-27 17:04 +0000 [r77536-77540] Joshua Colp <jcolp at digium.com>
+
+ * cdr/cdr_pgsql.c: (closes issue #10310) Reported by: prashant_jois
+ Patches: cdr_pgsql.patch uploaded by prashant (license 114)
+ Finish the Postgresql connection after the log messages are
+ printed so we don't access invalid memory.
+
+ * channels/chan_sip.c: (closes issue #10323) Reported by: julianjm
+ Patches: chan_sip_device_state_hold_fix.v1.diff.txt uploaded by
+ julianjm (license 99) Clear ONHOLD flag when decrementing the
+ onHold peer count. If we did not do this the count may keep
+ decreasing.
+
+2007-07-27 14:30 +0000 [r77490] Mark Michelson <mmichelson at digium.com>
+
+ * channels/chan_sip.c: "re-invite" was misspelled
+
+2007-07-26 23:19 +0000 [r77460] Joshua Colp <jcolp at digium.com>
+
+ * main/channel.c: (closes issue #10302) Reported by: litnialex If a
+ DTMF end frame comes from a channel without a begin and it is
+ going to a technology that only accepts end frames (aka INFO)
+ then use the minimum DTMF duration if one is not in the frame
+ already.
+
+2007-07-26 22:16 +0000 [r77424-77429] Kevin P. Fleming <kpfleming at digium.com>
+
+ * doc/mp3.txt: change protocol for downloads as well
+
+ * doc/mp3.txt, sounds/Makefile: use new canonical name for download
+ server
+
+2007-07-26 21:23 +0000 [r77410] Russell Bryant <russell at digium.com>
+
+ * Makefile, build_tools/make_buildopts_h: AST_DEVMODE was defined
+ in trunk, but not in 1.4. When Asterisk is compiled under dev
+ mode, AST_DEVMODE will get defined in buildopts.h. Change 1.4 to
+ define it in the same way that trunk does. Also, revert the
+ change that added this define in the Makefile The advantage to
+ doing it this way is that buildopts.h gets installed when you
+ install Asterisk. Then, when building any out of tree modules, or
+ building asterisk-addons, these modules know which options the
+ rest of Asterisk was built with.
+
+2007-07-26 20:35 +0000 [r77380] Mark Michelson <mmichelson at digium.com>
+
+ * Makefile, main/logger.c: Fixes to get ast_backtrace working
+ properly. The AST_DEVMODE macro was never defined so the majority
+ of ast_backtrace never attempted compilation. The makefile now
+ defines AST_DEVMODE if configure was run with --enable-dev-mode.
+ Also, changes were made to acccomodate 64 bit systems in
+ ast_backtrace. Thanks to qwell, kpfleming, and Corydon76 for
+ their roles in allowing me to get this committed
+
+2007-07-26 19:32 +0000 [r77348-77350] Tilghman Lesher <tilghman at mail.jeffandtilghman.com>
+
+ * main/logger.c: Missed one
+
+ * main/logger.c: Oops, that builtin define should be all-lowercase.
+
+2007-07-26 18:30 +0000 [r77318] Mark Michelson <mmichelson at digium.com>
+
+ * cdr/cdr_pgsql.c: Two consecutive calls to PQfinish could occur,
+ meaning free gets called on the same variable twice. This patch
+ sets the connection to NULL after calls to PQfinish so that the
+ problem does not occur. Also in this patch, prashant_jois
+ informed me that it is safe to pass a null pointer to PQfinish,
+ so I have removed the check for conn's existence from
+ my_unload_module. (closes issue 10295, reported by junky, patched
+ by me with input from prashant_jois)
+
+2007-07-25 22:39 +0000 [r77191] Steve Murphy <murf at digium.com>
+
+ * apps/app_meetme.c: This fix solves problem with intense squelch
+ noise when someone joins conf in bug 9430; We repro'd the problem
+ with meetme opts of 'CciMo'; Josh Colp supplied this patch, and
+ I'm applying it. It looks like playing the recorded username will
+ louse up the next thing played into the channel. Josh rearranged
+ the code so as to start things over before playing data directly
+ into the conference.
+
+2007-07-25 22:16 +0000 [r77176] Joshua Colp <jcolp at digium.com>
+
+ * apps/app_speech_utils.c: (closes issue #10303) Reported by: jtodd
+ Add SPEECH_DTMF_TERMINATOR variable so the user can specify the
+ digit to terminate a DTMF string with. If none is specified then
+ no terminator will be used.
+
+2007-07-25 21:52 +0000 [r77154] Mark Michelson <mmichelson at digium.com>
+
+ * main/channel.c: chan->emulate_dtmf_duration is an unsigned int,
+ not a signed int, so use %u instead of %d in the format string
+
+2007-07-25 20:23 +0000 [r77116-77136] Jason Parker <jparker at digium.com>
+
+ * /: so are my fingers...
+
+ * /: autotagexternals script is still obviously misbehaving...
+
+ * /: use autotagged externals
+
+2007-07-25 17:14 +0000 [r77071] Joshua Colp <jcolp at digium.com>
+
+ * configure, acinclude.m4: Fix autoconf logic for finding OpenH323
+ when it is not in the first place searched (/usr/share/openh323).
+
+2007-07-25 09:34 +0000 [r77022] Luigi Rizzo <rizzo at icir.org>
+
+ * main/rtp.c: set the sequence number in a frame for all frame
+ types
+
+2007-07-25 00:18 +0000 [r76983] Steve Murphy <murf at digium.com>
+
+ * channels/chan_zap.c, /: Merged revisions 76978 via svnmerge from
+ https://origsvn.digium.com/svn/asterisk/branches/1.2 ........
+ r76978 | murf | 2007-07-24 18:07:24 -0600 (Tue, 24 Jul 2007) | 1
+ line this fixes bug 10293, where the error message because
+ defaultzone or loadzone was not defined was confusing ........
+
+2007-07-24 22:12 +0000 [r76891-76937] Tilghman Lesher <tilghman at mail.jeffandtilghman.com>
+
+ * /, include/asterisk/lock.h: Merged revisions 76934 via svnmerge
+ from https://origsvn.digium.com/svn/asterisk/branches/1.2
+ ........ r76934 | tilghman | 2007-07-24 17:11:33 -0500 (Tue, 24
+ Jul 2007) | 2 lines Oops, res contains the error code, not errno.
+ I was wondering why a mutex was reporting "No such file or
+ directory"... ........
+
+ * main/app.c: Found another place where we should be using the
+ umask (thanks jcmoore)
+
+2007-07-24 Jason Parker <jparker at digium.com>
+
+ * Asterisk 1.4.9 released.
+
+2007-07-24 16:42 +0000 [r76803-76805] Jason Parker <jparker at digium.com>
+
+ * /: Blocked revisions 76802 via svnmerge ........ r76802 | qwell |
+ 2007-07-24 11:32:04 -0500 (Tue, 24 Jul 2007) | 3 lines Don't
+ create the Asterisk channel until we are starting the PBX on it.
+ (ASA-2007-018) ........
+
+ * channels/chan_iax2.c: Don't create the Asterisk channel until we
+ are starting the PBX on it. (ASA-2007-018)
+
+2007-07-24 16:26 +0000 [r76801] Mark Michelson <mmichelson at digium.com>
+
+ * apps/app_queue.c: Added a membercount variable to call_queue
+ struct which keeps track of the number of logged in members in a
+ particular queue. This makes it so that the 'n' option for
+ Queue() can act properly depending on which strategy is used. If
+ the strategy is roundrobin, rrmemory, or ringall, we want to ring
+ each phone once before moving on in the dialplan. However, if any
+ other strategy is used, we will only ring one phone since it
+ cannot be guaranteed that a different phone will ring on
+ subsequent attempts to ring a phone. As a side effect of this,
+ the QUEUE_MEMBER_COUNT dialplan function now just reads the
+ membercount variable instead of traversing through the member
+ list to figure out how many members there are. Special thanks to
+ blitzrage for helping to test this out. (closes issue #10127,
+ reported by bcnit, patched by me, tested by blitzrage)
+
+2007-07-23 22:38 +0000 [r76708] Tilghman Lesher <tilghman at mail.jeffandtilghman.com>
+
+ * apps/app_voicemail.c: It was our stated intention for 1.4 that
+ files created in app_voicemail should depend upon the umask.
+ Unfortunately, mkstemp() creates files with mode 0600, regardless
+ of the umask. This corrects that deficiency.
+
+2007-07-23 18:59 +0000 [r76656] Jason Parker <jparker at digium.com>
+
+ * channels/chan_skinny.c: Fix some incorrect softkey labels in
+ messages. Don't try to play dialtone in some unimplemented
+ features.
+
+2007-07-23 18:29 +0000 [r76654] Joshua Colp <jcolp at digium.com>
+
+ * /, channels/chan_agent.c: Merged revisions 76653 via svnmerge
+ from https://origsvn.digium.com/svn/asterisk/branches/1.2
+ ........ r76653 | file | 2007-07-23 15:28:13 -0300 (Mon, 23 Jul
+ 2007) | 4 lines (closes issue #5866) Reported by: tyler Do not
+ force channel format changes when a generator is present. The
+ generator may have changed the formats itself and changing them
+ back would cause issues. ........
+
+2007-07-23 17:57 +0000 [r76620] Jason Parker <jparker at digium.com>
+
+ * channels/chan_skinny.c: Don't try to queue up hold/unhold frames
+ on a non-existent channel. Issue 10276.
+
+2007-07-23 17:48 +0000 [r76519-76618] Joshua Colp <jcolp at digium.com>
+
+ * apps/app_morsecode.c: Allow app_morsecode to build on PPC Linux
+ by putting the value of the digit char in an int.
+
+ * /, channels/chan_sip.c: Merged revisions 76560 via svnmerge from
+ https://origsvn.digium.com/svn/asterisk/branches/1.2 ........
+ r76560 | file | 2007-07-23 11:32:07 -0300 (Mon, 23 Jul 2007) | 6
+ lines (closes issue #10236) Reported by: homesick Patches:
+ rpid_1.4_75840.patch uploaded by homesick (license 91) Accept
+ Remote Party ID on guest calls. ........
+
+ * channels/chan_skinny.c: (closes issue #10268) Reported by:
+ mvanbaak Patches: chan_skinny_openbsd.diff uploaded by mvanbaak
+ (license 7) Add another OS that has to use the Macros for byte
+ ordering.
+
+2007-07-23 12:25 +0000 [r76485] Russell Bryant <russell at digium.com>
+
+ * channels/chan_iax2.c: Use a signed integer for storing the number
+ of bytes in the packet read from the network. Using an unsigned
+ value here made it impossible to handle an error returned from
+ recvfrom(). Furthermore, in the case that recvfrom() did return
+ an error, this would cause a crash due to a heap overflow.
+ (closes issue #10265, reported by and fix suggested by
+ timrobbins)
+
+2007-07-22 21:42 +0000 [r76410] Tilghman Lesher <tilghman at mail.jeffandtilghman.com>
+
+ * /: Blocked revisions 76409 via svnmerge ........ r76409 |
+ tilghman | 2007-07-22 16:39:55 -0500 (Sun, 22 Jul 2007) | 2 lines
+ We should not use C++ reserved words in API headers (closes issue
+ #10266) ........
+
+2007-07-21 02:02 +0000 [r76227] Russell Bryant <russell at digium.com>
+
+ * /, channels/chan_sip.c: Merged revisions 76226 via svnmerge from
+ https://origsvn.digium.com/svn/asterisk/branches/1.2 ........
+ r76226 | russell | 2007-07-20 21:01:46 -0500 (Fri, 20 Jul 2007) |
+ 4 lines Backport a fix for a memory leak that was fixed in trunk
+ in reivision 76221 by rizzo. The memory used for the localaddr
+ list was not freed during a configuration reload. ........
+
+2007-07-20 21:36 +0000 [r76211] Steve Murphy <murf at digium.com>
+
+ * sounds/Makefile: This patch from 10249 is worth applying! It
+ prevents downloading sound files if they are already downloaded.
+ Darn Practical, if you ask me
+
+2007-07-20 21:03 +0000 [r76174-76178] Jason Parker <jparker at digium.com>
+
+ * channels/chan_skinny.c: Allow getting a call from an existing
+ "sub" channel. Cancel ringing if endpoint hangs up before
+ answering. Fixes were backported from trunk (there was apparently
+ a bit of confusion during merge of a previous patch). (closes
+ issue #10241)
+
+ * main/manager.c: Eliminate a compiler warning with gcc 4.2 by
+ constifying a char *
+
+ * channels/chan_skinny.c: It's possible for sub->owner to be NULL
+ here if you cancel the call immediately after/during sending a
+ digit.
+
+2007-07-20 18:42 +0000 [r76139] Mark Michelson <mmichelson at digium.com>
+
+ * apps/app_directory.c: When using users.conf for the entries in
+ the directory, if multiple users had the same last name, only the
+ first user listed would be available in the directory. (closes
+ issue #10200, reported by mrskippy, patched by me)
+
+2007-07-20 18:22 +0000 [r76132] Russell Bryant <russell at digium.com>
+
+ * main/channel.c: Use the define that specifies the default length
+ of an artificially created DTMF digit in the ast_senddigit()
+ function. The define is set to 100ms by default, which is the
+ same thing that this function was using. But, using the define
+ lets changes take effect in this case, as well as the others
+ where it was already used.
+
+2007-07-20 17:20 +0000 [r76054-76087] Joshua Colp <jcolp at digium.com>
+
+ * /, channels/chan_sip.c: Merged revisions 76080 via svnmerge from
+ https://origsvn.digium.com/svn/asterisk/branches/1.2 ........
+ r76080 | file | 2007-07-20 14:16:48 -0300 (Fri, 20 Jul 2007) | 6
+ lines (closes issue #10247) Reported by: fkasumovic Patches:
+ chan_sip.patch uploaded by fkasumovic (license #101) Drop any
+ peer realm authentication entries when reloading so multiple
+ entries do not get added to the peer. ........
+
+ * res/res_convert.c: (closes issue #10246) Reported by: fkasumovic
+ Patches: res_conver.patch uploaded by fkasumovic (license #101)
+ Use the last occurance of . to find the extension, not the first
+ occurance.
+
+ * apps/app_queue.c: Move makeannouncement variable declaration to
+ proper place.
+
+2007-07-19 20:36 +0000 [r75980] Jason Parker <jparker at digium.com>
+
+ * channels/chan_skinny.c: Remove some duplicate code.
+
+2007-07-19 18:59 +0000 [r75969-75978] Mark Michelson <mmichelson at digium.com>
+
+ * apps/app_queue.c: The diff on this looks pretty big but all I did
+ was remove a pointless if statement (always evaluates true).
+
+ * apps/app_queue.c: Changes in handling return values of several
[... 9733 lines stripped ...]
More information about the asterisk-commits
mailing list