[asterisk-commits] russell: tag 1.4.19-rc2 r107528 - /tags/1.4.19-rc2/
SVN commits to the Asterisk project
asterisk-commits at lists.digium.com
Tue Mar 11 10:58:23 CDT 2008
Author: russell
Date: Tue Mar 11 10:58:23 2008
New Revision: 107528
URL: http://svn.digium.com/view/asterisk?view=rev&rev=107528
Log:
Importing files for 1.4.19-rc2 release
Added:
tags/1.4.19-rc2/.lastclean (with props)
tags/1.4.19-rc2/.version (with props)
tags/1.4.19-rc2/ChangeLog (with props)
Added: tags/1.4.19-rc2/.lastclean
URL: http://svn.digium.com/view/asterisk/tags/1.4.19-rc2/.lastclean?view=auto&rev=107528
==============================================================================
--- tags/1.4.19-rc2/.lastclean (added)
+++ tags/1.4.19-rc2/.lastclean Tue Mar 11 10:58:23 2008
@@ -1,0 +1,1 @@
+32
Propchange: tags/1.4.19-rc2/.lastclean
------------------------------------------------------------------------------
svn:eol-style = native
Propchange: tags/1.4.19-rc2/.lastclean
------------------------------------------------------------------------------
svn:keywords = none
Propchange: tags/1.4.19-rc2/.lastclean
------------------------------------------------------------------------------
svn:mime-type = text/plain
Added: tags/1.4.19-rc2/.version
URL: http://svn.digium.com/view/asterisk/tags/1.4.19-rc2/.version?view=auto&rev=107528
==============================================================================
--- tags/1.4.19-rc2/.version (added)
+++ tags/1.4.19-rc2/.version Tue Mar 11 10:58:23 2008
@@ -1,0 +1,1 @@
+1.4.19-rc2
Propchange: tags/1.4.19-rc2/.version
------------------------------------------------------------------------------
svn:eol-style = native
Propchange: tags/1.4.19-rc2/.version
------------------------------------------------------------------------------
svn:keywords = none
Propchange: tags/1.4.19-rc2/.version
------------------------------------------------------------------------------
svn:mime-type = text/plain
Added: tags/1.4.19-rc2/ChangeLog
URL: http://svn.digium.com/view/asterisk/tags/1.4.19-rc2/ChangeLog?view=auto&rev=107528
==============================================================================
--- tags/1.4.19-rc2/ChangeLog (added)
+++ tags/1.4.19-rc2/ChangeLog Tue Mar 11 10:58:23 2008
@@ -1,0 +1,16411 @@
+2008-03-11 Russell Bryant <russell at digium.com>
+
+ * Asterisk 1.4.19-rc2 released.
+
+2008-03-11 15:18 +0000 [r107352-107472] Kevin P. Fleming <kpfleming at digium.com>
+
+ * apps/app_rpt.c: backport a fix from trunk
+
+ * channels/misdn/isdn_lib.c, codecs/Makefile,
+ channels/chan_misdn.c: fix various other problems found by gcc
+ 4.3
+
+ * configure, include/asterisk/autoconfig.h.in, configure.ac,
+ apps/app_sms.c: stop checking for mktime() in the configure
+ script... we don't use it, and the test is buggy under gcc 4.3
+
+ * configure, main/Makefile, configure.ac, makeopts.in: check for
+ compiler support for -fno-strict-overflow before using it (tested
+ with Debian's gcc 4.3, 4.1 and 3.4) (closes issue #12179)
+ Reported by: Netview
+
+ * configure, configure.ac: fix small bug in IMAP toolkit testing
+
+ * main/udptl.c, utils/Makefile, main/Makefile,
+ main/editline/readline.c, pbx/Makefile: fix up various compiler
+ warnings found with gcc-4.3: - the output of flex includes a
+ static function called 'input' that is not used, so for the
+ moment we'll stop having the compiler tell us about unused
+ variables in the flex source files (a better fix would be to
+ improve our flex post-processing to remove the unused function) -
+ main/stdtime/localtime.c makes assumptions about signed integer
+ overflow, and gcc-4.3's improved optimizer tries to take
+ advantage of handling potential overflow conditions at compile
+ time; for now, suppress these optimizations until we can fiure
+ out if the code needs improvement - main/udptl.c has some
+ references to uninitialized variables; in one case there was no
+ bug, but in the other it was certainly possibly for unexpected
+ behavior to occur - main/editline/readline.c had an unused
+ variable
+
+2008-03-11 00:59 +0000 [r107290] Terry Wilson <twilson at digium.com>
+
+ * channels/chan_sip.c: If we fail to alloc a channel, we should
+ re-lock the pvt structure before returning.
+
+2008-03-10 21:32 +0000 [r107230] Tilghman Lesher <tlesher at digium.com>
+
+ * main/pbx.c: Use non-global storage for eswitch
+
+2008-03-10 20:27 +0000 [r107173] Jason Parker <jparker at digium.com>
+
+ * channels/chan_zap.c: Make sure to reenable echo can after a
+ "failed" (canceled, etc) three-way call. (closes issue #11335)
+ Reported by: rebuild
+
+2008-03-10 20:17 +0000 [r107099-107161] Russell Bryant <russell at digium.com>
+
+ * main/pbx.c: Fix another bug specifically related to asynchronous
+ call origination. Once the PBX is started on the channel using
+ ast_pbx_start(), then the ownership of the channel has been
+ passed on to another thread. We can no longer access it in this
+ code. If the channel gets hung up very quickly, it is possible
+ that we could access a channel that has been free'd. (inspired by
+ BE-386)
+
+ * main/pbx.c: Fix some bugs related to originating calls. If the
+ code failed to start a PBX on the channel (such as if you set a
+ call limit based on the system's load average), then there were
+ cases where a channel that has already been free'd using
+ ast_hangup() got accessed. This caused weird memory corruption
+ and crashes to occur. (fixes issue BE-386) (much debugging credit
+ goes to twilson, final patch written by me)
+
+ * main/channel.c: Resolve a compiler warning.
+
+ * main/channel.c: Fix a race condition where the generator can go
+ away (closes issue #12175, reported by edantie, patched by me)
+
+2008-03-10 14:33 +0000 [r107016] Joshua Colp <jcolp at digium.com>
+
+ * apps/app_dial.c, main/cdr.c, include/asterisk/cdr.h: Move where
+ unanswered CDRs are dropped to the CDR core, not everything uses
+ app_dial. (closes issue #11516) Reported by: ys Patches:
+ branch_1.4_cdr.diff uploaded by ys (license 281) Tested by:
+ anest, jcapp, dartvader
+
+2008-03-08 15:59 +0000 [r106945] Kevin P. Fleming <kpfleming at digium.com>
+
+ * channels/chan_zap.c: don't generate D-Channel "up" and "down"
+ messages unless the channel state is actually changing; also,
+ generate the "up" message when an implicit "up" occurs due to
+ reception of a normal event when we thought the channel was
+ "down"
+
+2008-03-07 22:51 +0000 [r106895] Russell Bryant <russell at digium.com>
+
+ * apps/app_meetme.c: Only start the SLA thread if SLA has actually
+ been configured.
+
+2008-03-07 22:14 +0000 [r106842] Jason Parker <jparker at digium.com>
+
+ * main/editline/Makefile.in: Fix hardcoded grep in editline, were
+ GNU grep is required. (closes issue #12124) Reported by: dmartin
+
+2008-03-07 19:32 +0000 [r106788] Joshua Colp <jcolp at digium.com>
+
+ * main/channel.c: Ignore source update control frame. (closes issue
+ #12168) Reported by: plack
+
+2008-03-07 17:16 +0000 [r106704] Russell Bryant <russell at digium.com>
+
+ * include/asterisk/sched.h: Change a warning message to a debug
+ message. This is happening quite frequently, and it is not worth
+ spamming users with these messages unless we are pretty confident
+ that it should never happen. As it stands today, it _will_ and
+ _does_ happen and until that gets cleaned up a reasonable amount
+ on the development side, let's not spam the logs of everyone
+ else. (closes issue #12154)
+
+2008-03-07 16:22 +0000 [r106552-106635] Tilghman Lesher <tlesher at digium.com>
+
+ * apps/app_voicemail.c: Warn the user when a temporary greeting
+ exists (Closes issue #11409)
+
+ * main/rtp.c: Properly initialize rtp->schedid (Closes issue
+ #12154)
+
+ * apps/app_chanspy.c, apps/app_rpt.c, main/asterisk.c,
+ apps/app_speech_utils.c, apps/app_voicemail.c, main/channel.c,
+ funcs/func_enum.c, channels/chan_misdn.c, main/frame.c,
+ main/manager.c: Safely use the strncat() function. (closes issue
+ #11958) Reported by: norman Patches: 20080209__bug11958.diff.txt
+ uploaded by Corydon76 (license 14)
+
+2008-03-06 22:10 +0000 [r106437] Mark Michelson <mmichelson at digium.com>
+
+ * main/pbx.c: Quell an annoying message that is likely to print
+ every single time that ast_pbx_outgoing_app is called. The reason
+ is that __ast_request_and_dial allocates the cdr for the channel,
+ so it should be expected that the channel will have a cdr on it.
+ Thanks to joetester on IRC for pointing this out
+
+2008-03-06 04:40 +0000 [r106328] Tilghman Lesher <tlesher at digium.com>
+
+ * sounds/Makefile: Upgrade to the next release of sounds
+
+2008-03-05 22:37 +0000 [r106237] Russell Bryant <russell at digium.com>
+
+ * channels/chan_iax2.c: Fix a potential deadlock and a few
+ different potential crashes. (closes issue #12145, reported by
+ thiagarcia, patched by me)
+
+2008-03-05 22:32 +0000 [r106235] Joshua Colp <jcolp at digium.com>
+
+ * channels/chan_oss.c, main/rtp.c, channels/chan_mgcp.c,
+ apps/app_dial.c, main/channel.c, channels/chan_phone.c,
+ main/dial.c, channels/chan_zap.c, channels/chan_sip.c,
+ channels/chan_skinny.c, channels/chan_h323.c, main/file.c,
+ channels/chan_alsa.c, apps/app_followme.c,
+ include/asterisk/frame.h: Add a control frame to indicate the
+ source of media has changed. Depending on the underlying
+ technology it may need to change some things. (closes issue
+ #12148) Reported by: jcomellas
+
+2008-03-05 21:12 +0000 [r106178] Michiel van Baak <michiel at vanbaak.info>
+
+ * doc/realtime.txt: document var_metric so no bugreports will come
+ in when it's actually a configuration issue. (issue #12151)
+ Reported and patched by: caio1982 1.4 patch by me
+
+2008-03-05 15:32 +0000 [r106038] Kevin P. Fleming <kpfleming at digium.com>
+
+ * channels/chan_zap.c: when a PRI call must be moved to a different
+ B channel at the request of the other endpoint, ensure that any
+ DSP active on the original channel is moved to the new one
+ (closes issue #11917) Reported by: mavetju Tested by: mavetju
+
+2008-03-05 15:17 +0000 [r106015] Tilghman Lesher <tlesher at digium.com>
+
+ * channels/chan_sip.c, include/asterisk/sched.h: Correctly
+ initialize retransid in SIP, and ensure that the warning when
+ failing to delete a schedule entry can actually hit the log.
+ (closes issue #12140) Reported by: slavon Patches: sch2.patch
+ uploaded by slavon (license 288) (Patch slightly modified by me)
+
+2008-03-05 01:52 +0000 [r105932] Russell Bryant <russell at digium.com>
+
+ * main/rtp.c, main/translate.c, include/asterisk/frame.h: Fix a bug
+ that I just noticed in the RTP code. The calculation for setting
+ the len field in an ast_frame of audio was wrong when G.722 is in
+ use. The len field represents the number of ms of audio that the
+ frame contains. It would have set the value to be twice what it
+ should be.
+
+2008-03-04 18:10 +0000 [r105674-105676] Joshua Colp <jcolp at digium.com>
+
+ * main/rtp.c: In addition to setting the marker bit let's change
+ our ssrc so they know for sure it is a different source.
+
+ * main/rtp.c, channels/chan_sip.c, include/asterisk/rtp.h: When a
+ new source of audio comes in (such as music on hold) make sure
+ the marker bit gets set. (closes issue #10355) Reported by:
+ wdecarne Patches: 10355.diff uploaded by file (license 11)
+ (closes issue #11491) Reported by: kanderson
+
+2008-03-04 Russell Bryant <russell at digium.com>
+
+ * Asterisk 1.4.19-rc1 released.
+
+2008-03-04 04:31 +0000 [r105591] Russell Bryant <russell at digium.com>
+
+ * main/pbx.c: Backport a minor bug fix from trunk that I found
+ while doing random code cleanup. Properly break out of the loop
+ when a context isn't found when verify that includes are valid.
+
+2008-03-03 18:06 +0000 [r105572] Jason Parker <jparker at digium.com>
+
+ * res/snmp/agent.c: Fix type for astNumChannels. (closes issue
+ #12114) Reported by: jeffg Patches: 12114.patch uploaded by jeffg
+ (license 192)
+
+2008-03-03 17:16 +0000 [r105563-105570] Russell Bryant <russell at digium.com>
+
+ * channels/chan_local.c: In the case of an ast_channel allocation
+ failure, take the local_pvt out of the pvt list before destroying
+ it.
+
+ * channels/chan_local.c: Fix a potential memory leak of the
+ local_pvt struct when ast_channel allocation fails. Also, in
+ passing, centralize the code necessary to destroy a local_pvt.
+
+ * main/autoservice.c: Update the copyright information for
+ autoservice. Most of the code in this file now is stuff that I
+ have written recently ...
+
+ * main/asterisk.c, main/channel.c, include/asterisk.h,
+ main/autoservice.c: Merge in some changes from
+ team/russell/autoservice-nochans-1.4 These changes fix up some
+ dubious code that I came across while auditing what happens in
+ the autoservice thread when there are no channels currently in
+ autoservice. 1) Change it so that autoservice thread doesn't keep
+ looping around calling ast_waitfor_n() on 0 channels twice a
+ second. Instead, use a thread condition so that the thread
+ properly goes to sleep and does not wake up until a channel is
+ put into autoservice. This actually fixes an interesting bug, as
+ well. If the autoservice thread is already running (almost always
+ is the case), then when the thread goes from having 0 channels to
+ have 1 channel to autoservice, that channel would have to wait
+ for up to 1/2 of a second to have the first frame read from it.
+ 2) Fix up the code in ast_waitfor_nandfds() for when it gets
+ called with no channels and no fds to poll() on, such as was the
+ case with the previous code for the autoservice thread. In this
+ case, the code would call alloca(0), and pass the result as the
+ first argument to poll(). In this case, the 2nd argument to
+ poll() specified that there were no fds, so this invalid pointer
+ shouldn't actually get dereferenced, but, this code makes it
+ explicit and ensures the pointers are NULL unless we have valid
+ data to put there. (related to issue #12116)
+
+2008-03-03 15:28 +0000 [r105557-105560] Joshua Colp <jcolp at digium.com>
+
+ * main/channel.c: It is possible for no audio to pass between the
+ current digit and next digit so expand logic that clears
+ emulation to AST_FRAME_NULL. (closes issue #11911) Reported by:
+ edgreenberg Patches: v1-11911.patch uploaded by dimas (license
+ 88) Tested by: tbsky
+
+ * channels/chan_sip.c: Add a comment to describe some logic.
+ (closes issue #12120) Reported by: flefoll Patches:
+ chan_sip.c.br14.patch-just-a-comment uploaded by flefoll (license
+ 244)
+
+2008-02-29 23:34 +0000 [r105409] Russell Bryant <russell at digium.com>
+
+ * main/autoservice.c: Fix a major bug in autoservice. There was a
+ race condition in the handling of the list of channels in
+ autoservice. The problem was that it was possible for a channel
+ to get removed from autoservice and destroyed, while the
+ autoservice thread was still messing with the channel. This led
+ to memory corruption, and caused crashes. This explains multiple
+ backtraces I have seen that have references to autoservice, but
+ do to the nature of the issue (memory corruption), could cause
+ crashes in a number of areas. (fixes the crash in BE-386) (closes
+ issue #11694) (closes issue #11940) The following issues could be
+ related. If you are the reporter of one of these, please update
+ to include this fix and try again. (potentially fixes issue
+ #11189) (potentially fixes issue #12107) (potentially fixes issue
+ #11573) (potentially fixes issue #12008) (potentially fixes issue
+ #11189) (potentially fixes issue #11993) (potentially fixes issue
+ #11791)
+
+2008-02-29 14:47 +0000 [r105326] Philippe Sultan <philippe.sultan at gmail.com>
+
+ * res/res_jabber.c: Fix a potential memory leak
+
+2008-02-29 14:34 +0000 [r105296] Tilghman Lesher <tlesher at digium.com>
+
+ * apps/app_voicemail.c: If the message file does not exist, just
+ return harmlessly, instead of crashing. (Closes issue #12108)
+
+2008-02-29 13:48 +0000 [r105261] Joshua Colp <jcolp at digium.com>
+
+ * apps/app_voicemail.c: Bump up the size of the uniqueid variable.
+ (closes issue #12107) Reported by: asgaroth
+
+2008-02-29 13:05 +0000 [r105209] Philippe Sultan <philippe.sultan at gmail.com>
+
+ * res/res_jabber.c: Automatically create new buddy upon reception
+ of a presence stanza of type subscribed. (closes issue #12066)
+ Reported by: ffadaie Patches: branch-1.4-12066-1.diff uploaded by
+ phsultan (license 73) trunk-12066-1.diff uploaded by phsultan
+ (license 73) Tested by: ffadaie, phsultan
+
+2008-02-28 22:23 +0000 [r105116] Russell Bryant <russell at digium.com>
+
+ * main/utils.c, include/asterisk/lock.h: Fix a bug in the lock
+ tracking code that was discovered by mmichelson. The issue is
+ that if the lock history array was full, then the functions to
+ mark a lock as acquired or not would adjust the stats for
+ whatever lock is at the end of the array, which may not be
+ itself. So, do a sanity check to make sure that we're updating
+ lock info for the proper lock. (This explains the bizarre stats
+ on lock #63 in BE-396, thanks Mark!)
+
+2008-02-28 21:56 +0000 [r105113] Tilghman Lesher <tlesher at digium.com>
+
+ * contrib/init.d/rc.debian.asterisk: Update init script for LSB
+ compat (closes issue #9843) Reported by: ibc Patches:
+ rc.debian.asterisk.patch uploaded by ibc (license 211) Tested by:
+ paravoid
+
+2008-02-28 20:11 +0000 [r105059] Mark Michelson <mmichelson at digium.com>
+
+ * apps/app_queue.c: When using autofill, members who are in use
+ should be counted towards the number of available members to call
+ if ringinuse is set to yes. Thanks to jmls who brought this issue
+ up on IRC
+
+2008-02-28 19:20 +0000 [r104920-105005] Jason Parker <jparker at digium.com>
+
+ * main/cdr.c, main/pbx.c: Make pbx_exec pass an empty string into
+ applications, if we get NULL. This protects against possible
+ segfaults in applications that may try to use data before
+ checking length (ast_strdupa'ing it, for example) (closes issue
+ #12100) Reported by: foxfire Patches: 12100-nullappargs.diff
+ uploaded by qwell (license 4)
+
+ * channels/chan_skinny.c: According to a video at www.cisco.com,
+ the 7921G supports 6 line appearances.
+
+2008-02-28 00:05 +0000 [r104868] Tilghman Lesher <tlesher at digium.com>
+
+ * main/Makefile, build_tools/strip_nonapi: Compatibility fix for
+ PPC64 (closes issue #12081) Reported by: jcollie Patches:
+ asterisk-1.4.18-funcdesc.patch uploaded by jcollie (license 412)
+ Tested by: jcollie, Corydon76
+
+2008-02-27 21:49 +0000 [r104841] Mark Michelson <mmichelson at digium.com>
+
+ * main/dial.c: Two fixes: 1. Make the list of ast_dial_channels a
+ lockable list. This is because in some cases, the ast_dial may
+ exist in multiple threads due to asynchronous execution of its
+ application, and I found some cases where race conditions could
+ exist. 2. Check in ast_dial_join to be sure that the channel
+ still exists before attempting to lock it, since it could have
+ gotten hung up but the is_running_app flag on the
+ ast_dial_channel may not have been cleared yet. (closes issue
+ #12038) Reported by: jvandal Patches: 12038v2.patch uploaded by
+ putnopvut (license 60) Tested by: jvandal
+
+2008-02-27 20:56 +0000 [r104787] Joshua Colp <jcolp at digium.com>
+
+ * apps/app_chanspy.c: Don't loop around infinitely trying to spy on
+ our own channel, and don't forget to free/detach the datastore
+ upon hangup of the spy.
+
+2008-02-27 20:36 +0000 [r104783] Mark Michelson <mmichelson at digium.com>
+
+ * main/file.c: Bump a couple of more buffers up by 2 so that
+ annoying warnings aren't generated like crazy on every
+ fileexists_core call.
+
+2008-02-27 18:15 +0000 [r104704] Tilghman Lesher <tlesher at digium.com>
+
+ * main/manager.c: Ensure the session ID can't be 0.
+
+2008-02-27 17:41 +0000 [r104665] Joshua Colp <jcolp at digium.com>
+
+ * main/file.c: Bump up the buffer by 2.
+
+2008-02-27 17:33 +0000 [r104625] Russell Bryant <russell at digium.com>
+
+ * apps/app_chanspy.c: Fix a problem in ChanSpy where it could get
+ stuck in an infinite loop without being able to detect that the
+ calling channel hung up. (closes issue #12076, reported by junky,
+ patched by me)
+
+2008-02-27 17:26 +0000 [r104598] Jason Parker <jparker at digium.com>
+
+ * res/res_features.c: Inherit language from the transfering channel
+ on a blind transfer. (closes issue #11682) Reported by: caio1982
+ Patches: local_atxfer_lang3-1.4.diff uploaded by caio1982
+ (license 22) Tested by: caio1982, victoryure
+
+2008-02-27 17:07 +0000 [r104596] Joshua Colp <jcolp at digium.com>
+
+ * main/loader.c: Use the lock (which already existed, it just
+ wasn't used) on the updaters list to protect the contents instead
+ of the overall module list lock. (closes issue #12080) Reported
+ by: ChaseVenters
+
+2008-02-27 16:53 +0000 [r104593] Kevin P. Fleming <kpfleming at digium.com>
+
+ * main/file.c: fallback to standard English prompts properly when
+ using new prompt directory layout (closes issue #11831) Reported
+ by: IgorG Patches: fallbacken.v1.diff uploaded by IgorG (license
+ 20) (modified by me to improve code and conform rest of function
+ to coding guidelines)
+
+2008-02-27 16:45 +0000 [r104591] Russell Bryant <russell at digium.com>
+
+ * channels/chan_zap.c: When we receive a known alarm, make sure
+ that the unknown alarm flag is not still set to make sure that
+ when we come back out of alarm, it gets reported in the log and
+ manager interface (after discussion with tzafrir on the -dev
+ list)
+
+2008-02-27 15:52 +0000 [r104536] Joshua Colp <jcolp at digium.com>
+
+ * res/res_smdi.c: Only stop the MWI monitor thread if it was
+ actually started. (closes issue #12086) Reported by: francesco_r
+
+2008-02-27 01:15 +0000 [r104332-104334] Russell Bryant <russell at digium.com>
+
+ * apps/app_chanspy.c: Avoid some recursion in the cleanup code for
+ the chanspy datastore (closes issue #12076, reported by junky,
+ patched by me)
+
+ * channels/chan_zap.c: Zaptel 1.4 now exposes FXO battery state as
+ an alarm. However, Asterisk 1.4 does not know what to do with
+ these alarms. Only Asterisk 1.6 cares about it. So, if we get an
+ unknown alarm in chan_zap, don't generate confusing log messages
+ about it.
+
+2008-02-26 18:26 +0000 [r104132-104141] Jason Parker <jparker at digium.com>
+
+ * Makefile: Add badshell to .PHONY target (thanks Kevin)
+
+ * Makefile: Since all shells aren't as awesome as bash, we have to
+ fail if somebody tries to use a literal "~" in DESTDIR.
+
+ * sounds/Makefile: Revert previous abspath change. ...abspath is
+ new in GNU make 3.81. I feel so...defeated. Must find new fix!
+
+ * sounds/Makefile: Fix a very bizarre issue we were seeing with our
+ buildbot when using a DESTDIR that wasn't an absolute path (such
+ as DESTDIR=~/asterisk-1.4). Apparently what was happening, was
+ that some of the targets were being expanded to the full path, so
+ $@ ended up being /root/asterisk-1.4/[...]/ rather than
+ ~/asterisk-1.4/[...]/ It appears that this may be a new "feature"
+ in GNU make. (*cough*
+ http://en.wikipedia.org/wiki/Principle_of_least_surprise *cough*)
+
+2008-02-26 00:25 +0000 [r104119] Russell Bryant <russell at digium.com>
+
+ * include/asterisk/smdi.h, apps/app_voicemail.c,
+ channels/chan_zap.c, res/res_smdi.c, configs/smdi.conf.sample:
+ Merge changes from team/russell/smdi-1.4 This commit brings in a
+ significant set of changes to the SMDI support in Asterisk. There
+ were a number of bugs in the current implementation, most notably
+ being that it was very likely on busy systems to pop off the
+ wrong message from the SMDI message queue. So, this set of
+ changes fixes the issues discovered as well as introducing some
+ new ways to use the SMDI support which are required to avoid the
+ bugs with grabbing the wrong message off of the queue. This code
+ introduces a new interface to SMDI, with two dialplan functions.
+ First, you get an SMDI message in the dialplan using
+ SMDI_MSG_RETRIEVE() and then you access details in the message
+ using the SMDI_MSG() function. A side benefit of this is that it
+ now supports more than just chan_zap. For example, with this
+ implementation, you can have some FXO lines being terminated on a
+ SIP gateway, but the SMDI link in Asterisk. Another issue with
+ the current implementation is that it is quite common that the
+ station ID that comes in on the SMDI link is not necessarily the
+ same as the Asterisk voicemail box. There are now additional
+ directives in the smdi.conf configuration file which let you map
+ SMDI station IDs to Asterisk voicemail boxes. Yet another issue
+ with the current SMDI support was related to MWI reporting over
+ the SMDI link. The current code could only report a MWI change
+ when the change was made by someone calling into voicemail. If
+ the change was made by some other entity (such as with IMAP
+ storage, or with a web interface of some kind), then the MWI
+ change would never be sent. The SMDI module can now poll for MWI
+ changes if configured to do so. This work was inspired by and
+ primarily done for the University of Pennsylvania. (also related
+ to issue #9260)
+
+2008-02-26 00:03 +0000 [r104111] Jason Parker <jparker at digium.com>
+
+ * channels/chan_h323.c: IPTOS_MINCOST is not defined on Solaris.
+ (closes issue #12050) Reported by: asgaroth Patches: 12050.patch
+ uploaded by putnopvut (license 60)
+
+2008-02-25 23:42 +0000 [r104102-104106] Russell Bryant <russell at digium.com>
+
+ * apps/app_chanspy.c: This patch fixes some pretty significant
+ problems with how app_chanspy handles pointers to channels that
+ are being spied upon. It was very likely that a crash would occur
+ if the channel being spied upon hung up. This was because the
+ current ast_channel handling _requires_ that the object is locked
+ or else it could disappear at any time (except in the owning
+ channel thread). So, this patch uses some channel datastore magic
+ on the spied upon channel to be able to detect if and when the
+ channel goes away. (closes issue #11877) (patch written by me,
+ but thanks to kpfleming for the idea, and to file for review)
+
+ * main/utils.c: Improve the lock tracking code a bit so that a
+ bunch of old locks that threads failed to lock don't sit around
+ in the history. When a lock is first locked, this checks to see
+ if the last lock in the list was one that was failed to be
+ locked. If it is, then that was a lock that we're no longer
+ sitting in a trylock loop trying to lock, so just remove it.
+ (inspired by issue #11712)
+
+2008-02-25 21:37 +0000 [r104095] Joshua Colp <jcolp at digium.com>
+
+ * channels/chan_sip.c: Make it so a users.conf user creates both a
+ SIP peer and a SIP user. The user will be used for inbound
+ authentication for the device, and peer will be used for placing
+ calls to the device. (closes issue #9044) Reported by: queuetue
+ Patches: sip-gui-friend.diff uploaded by qwell (license 4)
+
+2008-02-25 21:31 +0000 [r104094] Tilghman Lesher <tlesher at digium.com>
+
+ * apps/app_voicemail.c: If the destination folder is full, don't
+ delete a message when exiting. (closes issue #12065) Reported by:
+ selsky Patch by: (myself)
+
+2008-02-25 20:49 +0000 [r104092] Jason Parker <jparker at digium.com>
+
+ * main/config.c: Allow the use of #include and #exec in situations
+ where the max include depth was only 1. Specifically, this fixes
+ using #include and #exec in extconfig.conf. This was basically
+ caused because the config file itself raises the include level to
+ 1. I opted not to raise the include limit, because recursion here
+ could cause very bizarre behavior. Pointed out, and tested by
+ jmls (closes issue #12064)
+
+2008-02-25 18:38 +0000 [r104086] Russell Bryant <russell at digium.com>
+
+ * channels/chan_agent.c: Ensure that the channel doesn't disappear
+ in agent_logoff(). If it does, it could cause a crash. (fixes the
+ crash reported in BE-396)
+
+2008-02-25 16:16 +0000 [r104082-104084] Joshua Colp <jcolp at digium.com>
+
+ * channels/chan_sip.c: If a resubscription comes in for a dialog we
+ no longer know about tell the remote side that the dialog does
+ not exist so they subscribe again using a new dialog. (closes
+ issue #10727) Reported by: s0l4rb03 Patches: 10727-2.diff
+ uploaded by file (license 11)
+
+ * channels/chan_sip.c: Due to recent changes tag will no longer be
+ NULL if not present so we have to use ast_strlen_zero to see if
+ it's actually blank. (closes issue #12061) Reported by: flefoll
+ Patches: chan_sip.c.br14.patch_pedantic_no_totag uploaded by
+ flefoll (license 244)
+
+2008-02-22 22:45 +0000 [r104037] Tilghman Lesher <tlesher at digium.com>
+
+ * channels/chan_sip.c: Backwards debug message. (closes issue
+ #12052) Reported by: flefoll Patches:
+ chan_sip.c.br14.patch_found-notfound uploaded by flefoll (license
+ 244)
+
+2008-02-21 21:05 +0000 [r104026-104027] Mark Michelson <mmichelson at digium.com>
+
+ * channels/chan_zap.c: And as a followup to revision 104026,
+ completely remove event-related calls from a section of code
+ where we know there was no event to handle or get.
+
+ * channels/chan_zap.c: Remove an incorrect debug message. It
+ reported that it had received a specific event and tried to
+ report which event was received. What actually was happening was
+ that it was reporting the number of bytes returned from a call to
+ read(). Thanks to Jared Smith for bringing the issue up on IRC
+
+2008-02-21 14:33 +0000 [r104015] Kevin P. Fleming <kpfleming at digium.com>
+
+ * main/manager.c: reduce the likelihood that HTTP Manager session
+ ids will consist of primarily '1' bits
+
+2008-02-20 22:32 +0000 [r103956] Mark Michelson <mmichelson at digium.com>
+
+ * apps/app_queue.c: Clear up confusion when viewing the
+ QUEUE_WAITING_COUNT of a "dead" realtime queue. Since from the
+ user's perspective, the queue does exist, we shouldn't tell them
+ we couldn't find the queue. Instead since it is a dead queue,
+ report a 0 waiting count This issue was brought up on IRC by jmls
+
+2008-02-20 22:06 +0000 [r103953] Joshua Colp <jcolp at digium.com>
+
+ * channels/chan_zap.c: Don't wait for additional digits when
+ overlap dialing is enabled if the setup message contains the
+ sending_complete information element. (closes issue #11785)
+ Reported by: klaus3000 Patches:
+ sending_complete_overlap_asterisk-1.4.17.patch.txt uploaded by
+ klaus3000 (license 65)
+
+2008-02-20 21:40 +0000 [r103904] Mark Michelson <mmichelson at digium.com>
+
+ * channels/chan_local.c: Fix a crash if the channel becomes NULL
+ while attempting to lock it. (closes issue #12039) Reported by:
+ danpwi
+
+2008-02-20 17:53 +0000 [r103845] Tilghman Lesher <tlesher at digium.com>
+
+ * main/stdtime/localtime.c: Compat fix for Solaris (closes issue
+ #12022) Reported by: asgaroth Patches:
+ 20080219__bug12022.diff.txt uploaded by Corydon76 (license 14)
+ Tested by: asgaroth
+
+2008-02-19 20:28 +0000 [r103823] Joshua Colp <jcolp at digium.com>
+
+ * channels/h323/ast_h323.cxx: Send CallerID Name in setup message.
+ (closes issue #11241) Reported by: tusar Patches:
+ h323id_as_callerid_name.patch uploaded by tusar (license 344)
+
+2008-02-19 20:02 +0000 [r103821] Russell Bryant <russell at digium.com>
+
+ * channels/chan_local.c: Account for the fact that the "other"
+ channel can disappear while the local pvt is not locked. (fixes a
+ problem introduced in rev 100581) (closes issue #12012) Reported
+ by: stevedavies Patch by me
+
+2008-02-19 17:31 +0000 [r103807-103812] Joshua Colp <jcolp at digium.com>
+
+ * configure, configure.ac: Don't look for launchd when cross
+ compiling. (closes issue #12029) Reported by: ovi
+
+ * channels/chan_sip.c: Fix building of chan_sip.
+
+2008-02-19 10:27 +0000 [r103806] Olle Johansson <oej at edvina.net>
+
+ * channels/chan_sip.c: Make sure we send error replies correctly by
+ checking the via header.
+
+2008-02-18 23:56 +0000 [r103801] Joshua Colp <jcolp at digium.com>
+
+ * main/channel.c: Ensure that emulated DTMFs do not get interrupted
+ by another begin frame. (closes issue #11740) Reported by: gserra
+ Patches: v1-11740.patch uploaded by dimas (license 88) (closes
+ issue #11955) Reported by: tsearle (closes issue #10530) Reported
+ by: xmarksthespot
+
+2008-02-18 22:28 +0000 [r103790-103795] Jason Parker <jparker at digium.com>
+
+ * channels/chan_zap.c: Fix previous commit so that we actually
+ disable echocanbridged if echocancel is off.
+
+ * channels/chan_zap.c: Correct a message when echocancelwhenbridged
+ is on, but echocancel is not. Issue #12019
+
+2008-02-18 20:52 +0000 [r103786] Mark Michelson <mmichelson at digium.com>
+
+ * main/app.c: There was an invalid assumption when calculating the
+ duration of a file that the filestream in question was created
+ properly. Unfortunately this led to a segfault in the situation
+ where an unknown format was specified in voicemail.conf and a
+ voicemail was recorded. Now, we first check to be sure that the
+ stream was written correctly or else assume a zero duration.
+ (closes issue #12021) Reported by: jakep Tested by: putnopvut
+
+2008-02-18 17:31 +0000 [r103780] Tilghman Lesher <tlesher at digium.com>
+
+ * main/rtp.c, channels/chan_sip.c: When a SIP channel is being
+ auto-destroyed, it's possible for it to still be in bridge code.
+ When that happens, we crash. Delay the RTP destruction until the
+ bridge is ended. (closes issue #11960) Reported by: norman
+ Patches: 20080215__bug11960__2.diff.txt uploaded by Corydon76
+ (license 14) Tested by: norman
+
+2008-02-18 16:37 +0000 [r103770] Mark Michelson <mmichelson at digium.com>
+
+ * channels/chan_zap.c: Fix a linked list corruption that under the
+ right circumstances could lead to a looped list, meaning it will
+ traverse forever. (closes issue #11818) Reported by: michael-fig
+ Patches: 11818.patch uploaded by putnopvut (license 60) Tested
+ by: michael-fig
+
+2008-02-18 16:11 +0000 [r103763-103768] Joshua Colp <jcolp at digium.com>
+
+ * main/asterisk.c: Backport fix from issue #9325. (closes issue
+ #11980) Reported by: rbrunka
+
+ * channels/chan_sip.c: Don't care if the extension given doesn't
+ exist for subscription based MWI.
+
+2008-02-15 23:31 +0000 [r103726-103741] Russell Bryant <russell at digium.com>
+
+ * channels/chan_iax2.c: Fix a crash in chan_iax2 due to a race
+ condition (closes issue #11780) Reported by: guillecabeza
+ Patches: bug_iax2_jb_1.4.patch uploaded by guillecabeza (license
+ 380) bug_iax2_jb_trunk.patch uploaded by guillecabeza (license
+ 380)
+
+ * main/loader.c: In the case that you try to directly reload a
+ module has returned AST_MODULE_LOAD_DECLINE, log a message
+ indicating that the module is not fully initialized and must be
+ initialized using "module load".
+
+ * main/loader.c: Don't attempt to execute the reload callback for a
+ module that returned AST_MODULE_LOAD_DECLINE. This fixes a crash
+ that was reported against chan_console in trunk. (closes issue
+ #11953, reported by junky, fixed by me)
+
+2008-02-15 17:26 +0000 [r103688-103722] Mark Michelson <mmichelson at digium.com>
+
+ * doc/imapstorage.txt, configure, configure.ac: Final round of
+ changes for configure script logic for IMAP Now if a directory is
+ specified, then we will search that directory for a source
+ installation of the IMAP toolkit. If none is found, then we will
+ use that directory as the basis for detecting a package
+ installation of the IMAP c-client. If that check fails, then
+ configure will fail.
+
+ * configure, configure.ac: Fix a bit of wrong logic in the
+ configure script that caused problems when trying to configure
+ without IMAP. Patch suggestion from phsultan, but I modified it
+ slightly. (closes issue #12003) Reported by: pj Tested by:
+ putnopvut
+
+ * doc/imapstorage.txt, configure, configure.ac: I apparently
+ misunderstood one of the requirements of this configure change.
+ Now, if a source directory is specified with the --with-imap
+ option, and a valid source installation is not detected there,
+ then configure will fail and will not check for a package
+ installation.
+
+ * doc/imapstorage.txt: Make a small clarification in the
+ documentation
+
+ * doc/imapstorage.txt: Update documentation regarding configuration
+ of IMAP
+
+ * apps/app_voicemail.c, configure,
+ include/asterisk/autoconfig.h.in, configure.ac: Change to the
+ configure logic regarding IMAP. Prior to this commit, if you
+ wished to configure Asterisk with IMAP support, you would use the
+ --with-imap configure switch in one of the following two ways:
+ --with-imap=/some/directory would look in the directory specified
+ for a UW IMAP source installation --with-imap would assume that
+ you had imap-2004g installed in .. relative to the Asterisk
+ source With this set of changes the two above options still work
+ the same, but there are two new behaviors, too.
+ --with-imap=system will assume that you have -libc-client.so
+ where you store your shared objects and will attempt to find
+ c-client headers in your include path either in the imap or
+ c-client directory. If either of the two original methods of
+ specifying the imap option should fail, then the check for
+ --with-imap =system will be performed in addition. It is only
+ after this "system" check that failure can happen.
+
+ * apps/app_voicemail.c: Fix build for non-IMAP builds
+
+ * apps/app_voicemail.c: Fix the new message count if delete=yes
+ when using IMAP storage. (closes issue #11406) Reported by:
+ jaroth Patches: deleteflag_v2.patch uploaded by jaroth (license
+ 50) Tested by: jaroth
+
+2008-02-14 19:51 +0000 [r103683-103684] Jason Parker <jparker at digium.com>
+
+ * funcs/func_cdr.c: swap location for this..
+
+ * funcs/func_cdr.c: Document the 'l' option to the CDR() function.
+ (Thanks voipgate for pointing out the option, and Leif for
+ providing text for it.) Closes issue #11695.
+
+2008-02-13 06:25 +0000 [r103556-103607] Tilghman Lesher <tlesher at digium.com>
+
+ * channels/chan_agent.c: We aren't talking to ourselves; we're
+ talking to someone else. (closes issue #11771) Reported by:
+ msetim Patches: ami_agent_talkingto-1.4.diff uploaded by caio1982
+ (license 22) Tested by: caio1982, msetim
+
+ * apps/app_voicemail.c: Refuse to load app_voicemail if res_adsi is
+ not loaded (which is a symbol dependency) (closes issue #11760)
+ Reported by: non-poster Patches: 20080114__bug11760.diff.txt
+ uploaded by Corydon76 (license 14) Tested by: Corydon76,
+ non-poster, jamesgolovich
+
+2008-02-12 22:24 +0000 [r103503-103504] Jason Parker <jparker at digium.com>
+
+ * main/asterisk.c: revert accidental change from last commit. oops
+
+ * contrib/scripts/safe_asterisk, main/asterisk.c: Remove condition
+ that was impossible.
+
+2008-02-12 15:09 +0000 [r103324-103385] Joshua Colp <jcolp at digium.com>
+
+ * channels/chan_sip.c: Even if no CallerID name or number has been
+ provided by the remote party still use the configured sip.conf
+ ones. (closes issue #11977) Reported by: pj
+
+ * apps/app_meetme.c: If entering a conference with the 'w' option
+ ensure that we can't listen or speak until the marked user
+ appears. (closes issue #11835) Reported by: alanmcmillan
+
+2008-02-11 17:05 +0000 [r103315] Kevin P. Fleming <kpfleming at digium.com>
+
+ * configs/zapata.conf.sample: improve 2BCT documentation a bit
+ (thanks Jared)
+
+2008-02-09 06:23 +0000 [r103197] Tilghman Lesher <tlesher at digium.com>
+
[... 15609 lines stripped ...]
More information about the asterisk-commits
mailing list