[asterisk-commits] mjordan: branch mjordan/tags r370768 - /team/mjordan/tags/1.8.15.0/ChangeLog
SVN commits to the Asterisk project
asterisk-commits at lists.digium.com
Fri Aug 3 15:20:50 CDT 2012
Author: mjordan
Date: Fri Aug 3 15:17:40 2012
New Revision: 370768
URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=370768
Log:
Add ChangeLog
Added:
team/mjordan/tags/1.8.15.0/ChangeLog (with props)
Added: team/mjordan/tags/1.8.15.0/ChangeLog
URL: http://svnview.digium.com/svn/asterisk/team/mjordan/tags/1.8.15.0/ChangeLog?view=auto&rev=370768
==============================================================================
--- team/mjordan/tags/1.8.15.0/ChangeLog (added)
+++ team/mjordan/tags/1.8.15.0/ChangeLog Fri Aug 3 15:17:40 2012
@@ -1,0 +1,40776 @@
+2012-07-30 Asterisk Development Team <asteriskteam at digium.com>
+
+ * Asterisk 1.8.15.0 Released.
+
+2012-07-11 Asterisk Development Team <asteriskteam at digium.com>
+
+ * Asterisk 1.8.15.0-rc1 Released.
+
+2012-07-10 13:33 +0000 [r369869] Kinsey Moore <kmoore at digium.com>
+
+ * apps/app_stack.c, main/pbx.c: Improve Goto and GotoIf related
+ documentation Correct documentation on labeliftrue and
+ labeliffalse parameters of GotoIf() and update several other
+ locations that use the same syntax. (closes issue ASTERISK-20007)
+ Patch-by: Leif Madsen Reported-by: WIMPy
+
+2012-07-09 17:05 +0000 [r369818] Jason Parker <jparker at digium.com>
+
+ * configs/sip_notify.conf.sample: Add Digium phones context to
+ sip_notify sample config. This makes it so that they can be
+ reconfigured remotely. (closes issue ASTERISK-19910)
+
+2012-07-09 14:38 +0000 [r369792] Jonathan Rose <jrose at digium.com>
+
+ * channels/chan_sip.c: chan_sip: Fix small behavioral change
+ accidentally introduced in r369750 When removing the warning for
+ AST_CONTROL_FLASH from sip_indicate, I also inadvertently changed
+ the return value, which would likely make the indication not be
+ sent in audio. This fixes that while still removing the warning
+ message.
+
+2012-07-06 20:54 +0000 [r369750] Jonathan Rose <jrose at digium.com>
+
+ * channels/chan_sip.c: chan_sip: Add case for FLASH control frames
+ so that we don't display a warning. chan_sip channels can receive
+ flash control frames when connected to analog phones and possibly
+ for other reasons. There really isn't a reason to warn when these
+ frames are received, we can safely ignore them. Patches:
+ dahdi_sip_flash.diff uploaded by Jonathan Rose (license 6182)
+
+2012-07-06 18:40 +0000 [r369708-369731] Mark Michelson <mmichelson at digium.com>
+
+ * main/tcptls.c: Remove a superfluous and dangerous freeing of an
+ SSL_CTX. The problem here is that multiple server sessions share
+ a SSL_CTX. When one session ended, the SSL_CTX would be freed and
+ set NULL, leaving the other sessions unable to function. The code
+ being removed is superfluous because the SSL_CTX structures for
+ servers will be properly freed when ast_ssl_teardown is called.
+ (closes issue ASTERISK-20074) Reported by Trevor Helmsley
+ Patches: ASTERISK-20074.diff uploaded by Mark Michelson (license
+ #5049) Testers: Trevor Helmsley
+
+ * main/bridging.c: Fix bridging thread leak. The bridge thread was
+ exiting but was never being reaped using pthread_join(). This has
+ been fixed now by calling pthread_join() in ast_bridge_destroy().
+ (closes issue ASTERISK-19834) Reported by Marcus Hunger Review:
+ https://reviewboard.asterisk.org/r/2012
+
+2012-07-05 19:01 +0000 [r369652] Kinsey Moore <kmoore at digium.com>
+
+ * apps/app_voicemail.c: AST-2012-011: Resolve heap corruption issue
+ with voicemail The heard and deleted arrays in the voicemail
+ state structure were not handled properly following the memory
+ leak fix in r354890 and a fix for an invalid free in r356797.
+ This could result in accessing and writing into freed memory. The
+ allocation for these arrays has been reworked to avoid the
+ possibility of invalid frees, access of freed memory, and crashes
+ that were occurring as a result of this. Locking around accesses
+ and modifications of the voicemail state structure members
+ dh_arraysize, heard, and deleted has been added to prevent
+ simultaneous modification and access when IMAP storage is in use.
+ If IMAP storage is not in use, this locking is not compiled in.
+ Review: https://reviewboard.asterisk.org/r/1994/ (closes issue
+ ASTERISK-19923) Reported by: Dan Delaney Tested by: Dan Delaney,
+ Julian Yap Patches: vm_alloc_fix.diff uploaded by kmoore (license
+ 6273)
+
+2012-07-05 17:01 +0000 [r369626] Matthew Jordan <mjordan at digium.com>
+
+ * channels/chan_sip.c: Do not send a BYE when a provisional
+ response arrives during a re-INVITE Commits r369557 and r369579
+ were done to improve handling of re-INVITEs when the UA that was
+ supposed to receive the re-INVITE fails to respond. A limitation
+ of those patches occurred when a UA sent a provisional response
+ to the re-INVITE. This triggered a sending of a BYE in
+ check_pending. This patch tweaks the handling of the re-INVITE
+ such that a BYE is not sent in response to those messages. (issue
+ ASTERISK-19992) Reported by: Steve Davies Tested by: Steve Davies
+ patches: (reinvite_tweak.diff license #5012 by Steve Davies)
+
+2012-07-03 16:58 +0000 [r369557-369579] Terry Wilson <twilson at digium.com>
+
+ * channels/chan_sip.c: More improvements to re-INVITEs timing out
+ after a provisional response There is no need to call
+ check_pendings() on a final response to an INVITE when destroying
+ the scheduler entry as it will be done later during normal
+ processing. (issue ASTERISK-19992)
+
+ * channels/sip/include/sip.h, channels/chan_sip.c: Better handle
+ re-INVITEs with provisional but no final repsonses A previous
+ attempt at fixing this issue had negative side effects related to
+ attended transfers which this patch should resolve. Many thanks
+ to Steve Davies for all of the good suggestions and testing.
+ (closes issue ASTERISK-19992) Reported by: Steve Davies Tested
+ by: Steve Davies, Terry Wilson Review:
+ https://reviewboard.asterisk.org/r/2009/
+
+2012-06-29 16:52 +0000 [r369471-369490] Joshua Colp <jcolp at digium.com>
+
+ * channels/chan_sip.c: With some configurations a transport is not
+ actually specified so assume UDP in these cases.
+
+ * channels/chan_sip.c: Make the address family filter specific to
+ the transport. (closes issue ASTERISK-16618) Reported by: Leif
+ Madsen Review: https://reviewboard.asterisk.org/r/1667/
+
+2012-06-27 20:58 +0000 [r369436] Terry Wilson <twilson at digium.com>
+
+ * channels/sip/include/sip.h, channels/chan_sip.c: AST-2012-010:
+ Clean up after a reinvite that never gets a final response The
+ basic problem is that if a re-INVITE is sent by Asterisk and it
+ receives a provisional response, but no final response, then the
+ dialog is never torn down. In addition to leaking memory, this
+ also leaks file descriptors and will eventually lead to Asterisk
+ no longer being able to process calls. This patch just keeps
+ track of whether there is an outstanding re-INVITE, and if there
+ is goes ahead and cleans up everything as though there was no
+ outstanding reinvite. Review:
+ https://reviewboard.asterisk.org/r/2009/ (closes issue
+ ASTERISK-19992) Reported by: Steve Davies Tested by: Steve
+ Davies, Terry Wilson
+
+2012-06-26 13:21 +0000 [r369366-369390] Matthew Jordan <mjordan at digium.com>
+
+ * main/adsi.c: Fix crash in unloading of res_adsi module When
+ res_adsi is unloaded, it removes the ADSI functions that it
+ previously installed by passing a NULL adsi_funcs pointer to
+ ast_adsi_install_funcs. This function was not checking whether or
+ not the adsi_funcs pointer passed in was NULL before
+ dereferencing it to check whether or not the version of the
+ functions matches what the core was expecting it. This patch
+ makes it so that the version is only checked if a potentially
+ valid adsi_funcs pointer was passed in. Passing in NULL removes
+ the installed functions, bypassing the version check.
+
+ * main/cdr.c: Tweak CDR change in r369351 As Tilghman pointed out
+ on review 1996, the check to see if a CDR end time has been set
+ is sufficient to know whether or not the duration value can be
+ used. The check-in done for r369351 forgot to include this
+ change.
+
+2012-06-25 19:13 +0000 [r369352] Mark Michelson <mmichelson at digium.com>
+
+ * channels/sip/include/sip.h, channels/chan_sip.c: Re-fix how local
+ tag is generated when sending a 481 to an INVITE. Match our local
+ tag to whatever to-tag was sent in the initial INVITE. Because
+ the size of the to-tag may not fit in the buffer in the sip_pvt,
+ it has been changed to a string field. (closes issue
+ ASTERISK-19892) reported by Walter Doekes Review:
+ https://reviewboard.asterisk.org/r/1977
+
+2012-06-25 19:12 +0000 [r369351] Matthew Jordan <mjordan at digium.com>
+
+ * main/cdr.c: Fix incorrect duration reporting in CDRs created in
+ batch mode Certain places in core/cdr.c would, if the duration
+ value were 0, calculate the duration as being the delta between
+ the current time and the time at which the CDR record was
+ started. While this does not typically cause a problem in
+ non-batch mode, this can cause an issue in batch mode where CDR
+ records are gathered and written long after those calls have
+ ended. In particular, this affects calls that were never
+ answered, as those are expected to have a duration of 0. Often,
+ this would result in CDR logs with a significant number of calls
+ with lengthy durations, but dispositions of "BUSY". Note that
+ this does not affect cdr_csv, as that backend does not use
+ ast_cdr_getvar and instead directly reports the duration value.
+ The affected core backends include cdr_apative_odbc and
+ cdr_custom; other extended or deprecated CDR backends may
+ potentially still directly manipulate the duration values. (issue
+ ASTERISK-19860) Reported by: Thomas Arimont (issue AST-883)
+ Reported by: Thomas Arimont Tested by: Matt Jordan Review:
+ https://reviewboard.asterisk.org/r/1996/
+
+2012-06-25 15:57 +0000 [r369327] Richard Mudgett <rmudgett at digium.com>
+
+ * main/features.c: Fix Bridge application occasionally returning to
+ the wrong location. * Fix do_bridge_masquerade() getting the
+ resume location from the zombie channel. The code must not touch
+ a clone channel after it has masqueraded it. The clone channel
+ has become a zombie and is starting to hangup. (closes issue
+ ASTERISK-19985) Reported by: jamicque Patches:
+ jira_asterisk_19985_v1.8.patch (license #5621) patch uploaded by
+ rmudgett Tested by: jamicque
+
+2012-06-25 15:50 +0000 [r369302-369324] Mark Michelson <mmichelson at digium.com>
+
+ * main/adsi.c (added): Forgot to svn add this file in my last
+ commit.
+
+ * res/res_adsi.exports.in (removed), include/asterisk/adsi.h,
+ main/Makefile, res/res_adsi.c: Eliminate embedding of res_adsi.so
+ module. The way this is done is to stop using the optional API.
+ Instead, res_adsi.so, when loaded fills in a table of function
+ pointers. Review: https://reviewboard.asterisk.org/r/1991
+
+ * channels/chan_sip.c: Be more consistent with the return code for
+ requests received from invalid domain. When Asterisk receives an
+ INVITE from an external domain when allowexternaldomains=no send
+ a 403 instead of a 404. This is consistent with Asterisk's
+ behavior when receiving a REGISTER in this situation. (Closes
+ issue ASTERISK-19601) Reported by Matthew Jordan Patches:
+ ASTERISK-19601-no401.patch uploaded by Mark Michelson (License
+ #5049)
+
+2012-06-23 00:04 +0000 [r369235-369282] Richard Mudgett <rmudgett at digium.com>
+
+ * main/features.c: Fix Bridge application and AMI Bridge action
+ error handling. * Fix AMI Bridge action disconnecting the AMI
+ link on error. * Fix AMI Bridge action and Bridge application not
+ checking if their masquerades were successful. * Fix Bridge
+ application running the h-exten when it should not. * Made
+ do_bridge_masquerade() return if the masquerade was successful so
+ the Bridge application and AMI Bridge action could deal with it
+ correctly. * Made bridge_call_thread_launch() hangup the passed
+ in channels if the bridge_call_thread fails to start. Those
+ channels would have been orphaned. * Made builtin_atxfer() check
+ the success of the transfer masquerade setup.
+
+ * apps/app_queue.c: Explicitly check caller hangup in app Queue
+ rather than a polluted res2 value.
+
+ * apps/app_dial.c: Check if PBX was started and fix F and F(x)
+ action logic in Dial application.
+
+ * main/ccss.c: Check if PBX was started for generic CCSS recall.
+
+ * channels/chan_sip.c: Change incorrect chan_sip zombie hangup
+ debug message. They are all zombies now.
+
+2012-06-22 19:28 +0000 [r369214] Terry Wilson <twilson at digium.com>
+
+ * channels/chan_sip.c: Don't crash on a guest directmedia call A
+ sip_pvt may not have relatedpeer set if a call doesn't match up
+ with a peer. If there is no relatedpeer, there is no direct media
+ ACL to apply, so just return that it is allowed. (closes issue
+ ASTERISK-20040) Reported by: Terry Wilson
+
+2012-06-22 17:14 +0000 [r369195] Kinsey Moore <kmoore at digium.com>
+
+ * channels/chan_sip.c: Don't parse media stream state for SIP video
+ streams The sendonly/recvonly/sendrecv/inactive media stream
+ attributes were parsed for video, but nothing was ever done with
+ them. With this code removed, an UNSUPPORTED message is produced
+ when these attributes are used in conjunction with a video stream
+ which is the better behavior since they were never really
+ supported in the first place.
+
+2012-06-20 17:33 +0000 [r369130-369146] Alexandr Anikin <may at telecom-service.ru>
+
+ * addons/ooh323c/src/ooq931.c, addons/ooh323c/src/ooCalls.c: fix
+ locking issue on empty callList (issue ASTERISK-19298) Reported
+ by: Dmitry Melekhov Patches: ASTERISK-18322-2.patch
+
+ * addons/chan_ooh323.c: fix compile error (1.8 don't have
+ ast_channel_name macro)
+
+2012-06-20 02:03 +0000 [r369108] Michael L. Young <elgueromexicano at gmail.com>
+
+ * include/asterisk/netsock2.h, main/netsock2.c: Fix NULL pointer
+ segfault in ast_sockaddr_parse() While working with
+ ast_parse_arg() to perform a validity check, a segfault occurred.
+ The segfault occurred due to passing a NULL pointer to
+ ast_sockaddr_parse() from ast_parse_arg(). According to the
+ documentation in config.h, "result pointer to the result. NULL is
+ valid here, and can be used to perform only the validity checks."
+ This patch fixes the segfault by checking for a NULL pointer.
+ This patch also adds documentation to netsock2.h about why it is
+ necessary to check for a NULL pointer. (Closes issue
+ ASTERISK-20006) Reported by: Michael L. Young Tested by: Michael
+ L. Young Patches: asterisk-20006-netsock-null-ptr.diff uploaded
+ by Michael L. Young (license 5026) Review:
+ https://reviewboard.asterisk.org/r/1990/
+
+2012-06-19 23:28 +0000 [r369090] Alexandr Anikin <may at telecom-service.ru>
+
+ * addons/chan_ooh323.c: check rtptimeouts in ooh323 channels as per
+ config file (rtp voice, video, udptl except rtcp) (closes issue
+ ASTERISK-19179) Reported by: TSAREGORODTSEV Yury Patches:
+ 19179-ooh323-2.patch
+
+2012-06-19 15:30 +0000 [r369066] Mark Michelson <mmichelson at digium.com>
+
+ * channels/chan_sip.c: Fix request routing issue when outboundproxy
+ is used. Asterisk was incorrectly setting the destination of
+ CANCELs and ACKs for error responses to the URI of the initial
+ INVITE. This resulted in further requests, such as INVITEs with
+ authentication credentials, to be routed incorrectly. Instead,
+ when these CANCEL or ACKs are to be sent, we should simply keep
+ the destination the same as what it previously was. There is no
+ need to alter it any. (closes issue ASTERISK-20008) Reported by
+ Marcus Hunger Patches: ASTERISK-20008.patch uploaded by Mark
+ Michelson (license #5049)
+
+2012-06-18 18:07 +0000 [r369043] Richard Mudgett <rmudgett at digium.com>
+
+ * main/features.c: Fix monitoring calls put in a parking lot. * Fix
+ a regression that was introduced by -r366167 which effectively
+ disabled monitoring parked calls. (closes issue ASTERISK-20012)
+ Reported by: sdolloff Tested by: rmudgett
+
+2012-06-15 15:57 +0000 [r369001-369002] Kevin P. Fleming <kpfleming at digium.com>
+
+ * build_tools/find_missing_support_level (added): Add a script to
+ enable finding source files without support-levels defined.
+
+ * main/devicestate.c, main/astfd.c, main/ssl.c,
+ main/taskprocessor.c, main/astobj2.c, main/indications.c,
+ main/config.c, main/loader.c, main/term.c, main/cli.c,
+ channels/sig_analog.c, main/framehook.c, main/strcompat.c,
+ main/plc.c, res/ais/evt.c, main/fskmodem_int.c, main/syslog.c,
+ main/stdtime/localtime.c, main/db.c, main/bridging.c,
+ channels/sig_ss7.c, main/datastore.c, main/sched.c,
+ channels/sip/sdp_crypto.c, main/pbx.c, main/strings.c,
+ channels/vcodecs.c, channels/iax2-provision.c, main/aoc.c,
+ pbx/dundi-parser.c, main/cel.c, channels/iax2-parser.c,
+ main/chanvars.c, main/netsock.c, main/data.c, main/srv.c,
+ channels/chan_misdn.c, main/privacy.c,
+ channels/sip/dialplan_functions.c, main/fixedjitterbuf.c,
+ main/test.c, main/audiohook.c, main/alaw.c, main/asterisk.c,
+ main/timing.c, main/global_datastores.c, main/fskmodem_float.c,
+ main/ccss.c, channels/sip/reqresp_parser.c,
+ channels/misdn/isdn_msg_parser.c, main/utils.c, main/xml.c,
+ main/autochan.c, main/enum.c, channels/misdn/isdn_lib.c,
+ main/fskmodem.c, channels/misdn_config.c, main/io.c,
+ res/ael/pval.c, main/channel.c, main/cdr.c, main/ulaw.c,
+ main/dial.c, main/tdd.c, main/heap.c, channels/console_gui.c,
+ channels/misdn/ie.c, main/logger.c, channels/console_board.c,
+ main/app.c, main/image.c, main/dns.c, main/lock.c, main/stun.c,
+ main/dnsmgr.c, channels/sip/srtp.c, main/translate.c,
+ main/slinfactory.c, main/jitterbuf.c, main/acl.c,
+ channels/sig_pri.c, main/tcptls.c, main/hashtab.c,
+ main/abstract_jb.c, main/callerid.c, main/file.c,
+ res/snmp/agent.c, main/astmm.c, channels/misdn/portinfo.c,
+ main/event.c, channels/sip/config_parser.c, channels/vgrabbers.c,
+ main/xmldoc.c, main/dsp.c, main/udptl.c, main/netsock2.c,
+ main/autoservice.c, main/rtp_engine.c, main/frame.c,
+ main/security_events.c, res/ais/clm.c, main/threadstorage.c,
+ main/say.c, channels/console_video.c: Add support-level
+ indications to many more source files. Since we now have tools
+ that scan through the source tree looking for files with specific
+ support levels, we need to ensure that every file that is a
+ component of a 'core' or 'extended' module (or the main Asterisk
+ binary) is explicitly marked with its support level. This patch
+ adds support-level indications to many more source files in tree,
+ but avoids adding them to third-party libraries that are included
+ in the tree and to source files that don't end up involved in
+ Asterisk itself.
+
+2012-06-14 15:23 +0000 [r368898-368927] Mark Michelson <mmichelson at digium.com>
+
+ * main/Makefile: Revert Makefile change to remove embedding
+ res_adsi.so The change has resulted in a linking error for
+ certain versions of GCC. This is much worse than the original
+ issue, so for now, temporarily revert the change. A more thorough
+ change will be sought out.
+
+ * funcs/func_volume.c: Fix a deadlock that occurs when func_volume
+ is used on a local channel. This was discovered by trying to
+ perform a call forward to an extension that makes use of
+ func_volume. When the local channel is optimized away, the
+ datastore on the local;2 channel would have its audiohook
+ destroyed rather than detaching the audiohook from the channel
+ and then destroying it. With this patch, func_volume's datastore
+ destructor takes the proper route of detaching the audiohook and
+ then destroying it. (closes issue ASTERISK-19611) reported by
+ Volker Sauer Patches: ASTERISK-19611.patch uploaded by Mark
+ Michelson (license #5049)
+
+2012-06-13 20:26 +0000 [r368894] Matthew Jordan <mjordan at digium.com>
+
+ * res/res_smdi.c, res/res_adsi.c: Mark res_smdi/res_adsi as 'core'
+ supported modules Recently, various issues surrounding weak
+ symbols have caused problems with modules that rely on that
+ feature to be enabled in menuselect. This includes app_voicemail
+ and chan_dahdi, as they both rely upon res_smdi and res_adsi,
+ which, in certain circumstances, may not be enabled by default in
+ menuselect. Because res_smdi/res_adsi are dependencies for
+ chan_dahdi/app_voicemail, this patch marks both as 'core'
+ supported modules. This will allow both app_voicemail and
+ chan_dahdi to be enabled as well, regardless of whether or not
+ that system supports weak symbols. (issue AST-900) Reported by:
+ Thomas Arimont (issue AST-885) Reported by: Denis Alberto
+ Martinez
+
+2012-06-13 19:00 +0000 [r368873] Mark Michelson <mmichelson at digium.com>
+
+ * main/Makefile: Remove forced linking of res_adsi.o In GCC 4.5+
+ the result is that Asterisk has a phantom module loaded at
+ startup, claiming to be res_adsi. (closes issue ASTERISK-19920)
+ reported by Leif Madsen
+
+2012-06-13 14:27 +0000 [r368830-368852] Matthew Jordan <mjordan at digium.com>
+
+ * Makefile: Do not install empty directories; add ASTLIBDIR r368830
+ modified the installation script to only create a directory if
+ that directory does not exist. If some directory variable was
+ empty, it would attempt to create the empty location. It also
+ failed to create the ASTLIBDIR directory. This patch fixes it
+ such that the correct directories are made and only created if a
+ value specifying them actually exists.
+
+ * Makefile: Do not perform install on existing directories If a
+ directory already exists, performing a 'make install' will remove
+ the permissions associated with the current directory and replace
+ them with the permissions of the user executing the install. This
+ patch changes this behavior to only perform an install on the
+ directory if the directory does not exist. Thus, if a user later
+ changes the permissions on that directory, those permissions will
+ be preserved in subsequent installs. Review:
+ https://reviewboard.asterisk.org/r/1986 Review:
+ https://reviewboard.asterisk.org/r/1864 (closes issue
+ ASTERISK-19492) Reported by: Karl Fife Tested by: Paul Belanger,
+ Tilghman Lesher patches: ASTERISK-19492 by pabelanger (uploaded
+ by mjordan)
+
+2012-06-12 15:36 +0000 [r368807] Mark Michelson <mmichelson at digium.com>
+
+ * channels/chan_sip.c: Set the Caller ID "tag" on peers even if
+ remote party information is present. On incoming calls, we were
+ setting the cid_tag on the dialog only if there was no remote
+ party information (Remote-Party-ID or P-Asserted-Identity)
+ present. The Caller ID tag is an invented parameter, though, and
+ should be set no matter the circumstance. (closes issue
+ ASTERISK-19859) Reported by Thomas Arimont (closes issue AST-884)
+ Reported by Trey Blancher
+
+2012-06-11 17:03 +0000 [r368759] Richard Mudgett <rmudgett at digium.com>
+
+ * include/asterisk/channel.h, channels/chan_iax2.c, main/channel.c,
+ channels/chan_dahdi.c, channels/sig_analog.c,
+ channels/chan_sip.c: Fix deadlock potential with
+ ast_set_hangupsource() calls. Calling ast_set_hangupsource() with
+ the channel lock held can result in a deadlock because the
+ function also locks the bridged channel. (issue ASTERISK-19537)
+ (closes issue AST-891) Reported by: Guenther Kelleter Tested by:
+ Guenther Kelleter (closes issue ASTERISK-19801) Reported by: Alec
+ Davis
+
+2012-06-11 15:13 +0000 [r368719-368738] Kinsey Moore <kmoore at digium.com>
+
+ * apps/app_queue.c, main/loader.c, channels/chan_dahdi.c,
+ res/res_config_odbc.c, channels/sip/dialplan_functions.c,
+ pbx/pbx_config.c, apps/app_directory.c, res/res_odbc.c,
+ res/res_speech.c, apps/app_voicemail.c, main/udptl.c,
+ channels/sip/sdp_crypto.c, channels/chan_sip.c, res/res_fax.c,
+ main/say.c, funcs/func_strings.c, channels/sip/reqresp_parser.c:
+ Fix coverity UNUSED_VALUE findings in core support level files
+ Most of these were just saving returned values without using them
+ and in some cases the variable being saved to could be removed as
+ well. (issue ASTERISK-19672)
+
+ * main/md5.c: Fix compilation in dev-mode Backport a compilation
+ fix in md5.c from trunk that only showed up in dev-mode under
+ certain compiler versions.
+
+2012-07-10 Asterisk Development Team <asteriskteam at digium.com>
+
+ * Asterisk 1.8.14.0 Released.
+
+2012-07-06 Asterisk Development Team <asteriskteam at digium.com>
+
+ * Asterisk 1.8.14.0-rc2 Released.
+
+ * AST-2012-010: Possible Resource Leak on Uncompleted Re-INVITE
+ transactions
+
+ * AST-2012-011: Remote Crash Vulnerability in VoiceMail Application
+
+ * Fix crash on a guest directmedia call
+
+ A sip_pvt may not have relatedpeer set if a call doesn't match up
+ with a peer. If there is no relatedpeer, there is no direct media
+ ACL to apply, so just return that is is allowed.
+
+ (closes issue ASTERISK-20040)
+
+ * Fix request routing issue when outboundproxy is used
+
+ Asterisk was incorrectly setting the destination of CANCELs and ACKs
+ for error responses to the URI of the initial INVITE. This resulted
+ in further requests, such as INVITEs with authentication
+ credentials, to be routed incorrectly. Instead when these CANCEL or
+ ACKs are to be esnt, we should simply keep the destination the same
+ as what it previously was. There is no need to alter it any.
+
+ (closes issue ASTERISK-20008)
+
+ * Fix monitoring calls put in a parking lot
+
+ Fix a regression that was introduced by r366167 which effectively
+ disabled monitoring parked calls.
+
+ (closes issue ASTERISK-20012)
+
+2012-06-08 Asterisk Development Team <asteriskteam at digium.com>
+
+ * Asterisk 1.8.14.0-rc1 Released.
+
+2012-06-06 21:27 +0000 [r368644] Richard Mudgett <rmudgett at digium.com>
+
+ * channels/chan_dahdi.c, channels/sig_analog.c: Fix POTS flash hook
+ to orignate a second call deadlock. A deadlock can occur when a
+ POTS phone tries to flash hook to originate a second call for
+ 3-way or transfer. If another process is scanning the channels
+ container when the POTS line flash hooks then a deadlock will
+ occur. * Release the channel and private locks when creating a
+ new channel as a result of a flash hook. (closes issue
+ ASTERISK-19842) Reported by: rmudgett Tested by: rmudgett
+
+2012-06-06 19:13 +0000 [r368625] Mark Michelson <mmichelson at digium.com>
+
+ * channels/chan_sip.c: Fix a specific scenario where ACKs are not
+ matched. If a dialog-starting INVITE contains a to-tag, then
+ Asterisk will respond with a 481. In this case, the resulting
+ incoming ACK would not be matched, so Asterisk would continue
+ retransmitting the 481 until the transaction times out. There
+ were two issues. Asterisk, upon creating a sip_pvt would generate
+ a local tag. However, when the time came to transmit the 481,
+ since there was a to-tag in the INVITE, Asterisk would place this
+ original to-tag in the 481 response. When the ACK came in,
+ Asterisk would attempt to match the to-tag in the ACK to the
+ generated local tag. Unfortunately, Asterisk never actually
+ transmitted a response with the generated local tag, so the
+ to-tag in the ACK would not match. The other problem was that
+ when the 481 was sent, nothing was set on the sip_pvt to indicate
+ what CSeq is expected in the ACK. To fix the first problem, we
+ zero out the to-tag seen in the incoming INVITE. This way,
+ Asterisk, when time to send a response, will send its generated
+ local tag instead. To fix the second problem, we set the
+ sip_pvt's pendinginvite to the CSeq of the INVITE when we send a
+ 481. (closes issue ASTERISK-19892) Reported by Mark Michelson
+
+2012-06-06 17:20 +0000 [r368604] Matthew Jordan <mjordan at digium.com>
+
+ * build_tools/make_version: Add feature modifier to versions
+ produced from branches Certain branches, such as Certified
+ Asterisk, may have a modifier added to them that specifies the
+ features available in that branch. For branches, this modifier is
+ expected to be reflected in the location of the branch in
+ subversion. For example, a subversion of URL of
+ /certified/branches/1.8.11 would have a feature modifier of
+ 'certified'. This is slightly different then how features are
+ determined for tags, where the feature is part of the actual tag
+ name, e.g., "10.5.0-digiumphones". In keeping with the
+ nomenclature used for tags, the feature specifier for branches is
+ translated and placed after the revision numbers. For the example
+ given previously, this would result in a branch version of
+ "Asterisk SVN-branch-1.8.11-cert-rXXXXXX".
+
+2012-06-06 16:07 +0000 [r368586] Kinsey Moore <kmoore at digium.com>
+
+ * channels/chan_sip.c: Ensure overlapping hold flags do not
+ conflict When changing between different modes of hold, the flags
+ were not being cleared out properly causing a failure to change
+ hold states. (closes issue ASTERISK-19919) Patch-by: Morten
+ Tryfoss Reported-by: Morten Tryfoss
+
+2012-06-06 01:08 +0000 [r368567] Richard Mudgett <rmudgett at digium.com>
+
+ * main/features.c: Fix parked call performing a DTMF blind transfer
+ after being retrieved. When a parked call was retrieved from the
+ parking lot, it could not do a blind transfer because it caused
+ the involved calls to be hung up unconditionally. * Made the
+ ParkedCall application return the ast_bridge_call() return value.
+ (closes issue ABE-2862) Reported by: Vlad Povorozniuc
+
+2012-06-05 15:26 +0000 [r368520-368533] Kinsey Moore <kmoore at digium.com>
+
+ * apps/app_minivm.c: Resolve some build warnings My newly upgraded
+ compiler caught these usages of uninitialized values. They
+ weren't actually used.
+
+ * apps/app_voicemail.c: Ensure that pages and emails are sent using
+ RFC822-compliant date format When localization was added to
+ app_voicemail, these headers were altered when they should have
+ remained in en_US format for RFC compliance. This reverts the
+ changes to those two lines. (closes issue ASTERISK-19876)
+
+2012-06-04 21:56 +0000 [r368498] Mark Michelson <mmichelson at digium.com>
+
+ * channels/chan_sip.c: Relay proper SIP responses on calling side.
+ Revision 351130 broke corect HANGUPCAUSE setting for the 404 case
+ in chan_sip. Other cases were also potentially broken. This patch
+ fixes the relaying of causes to be what they used to be. (closes
+ issue ASTERISK-19914) Reported by Pavel Troller Tested by Walter
+ Doekes (via a reviewboard test to be committed later) Patches:
+ chan_sip.diff uploaded by Pavel Troller (license #6302)
+
+2012-06-04 21:10 +0000 [r368405-368469] Richard Mudgett <rmudgett at digium.com>
+
+ * UPGRADE.txt: Document BLINDTRANSFER behavior change. (issue
+ ASTERISK-19322) (closes issue ASTERISK-19875) Reported by: call
+
+ * main/channel.c: Fix potential deadlock between masquerade and
+ chan_local. * Restructure ast_do_masquerade() to not hold channel
+ locks while it calls ast_indicate(). * Simplify many calls to
+ ast_do_masquerade() since it will never return a failure now. If
+ it does fail internally because a channel driver callback
+ operation failed, the only thing ast_do_masquerade() can do is
+ generate a warning message about strange things may happen and
+ press on. * Fixed the call to ast_bridged_channel() in
+ ast_do_masquerade(). This change fixes half of the deadlock
+ reported in ASTERISK-19801 between masquerades and chan_iax.
+ (closes issue ASTERISK-19537) Reported by: rmudgett Tested by:
+ rmudgett Review: https://reviewboard.asterisk.org/r/1915/
+
+2012-06-01 23:21 +0000 [r368308] Richard Mudgett <rmudgett at digium.com>
+
+ * apps/app_stack.c: Fix deadlock when Gosub used with alternate
+ dialplan switches. Attempting to remove a channel from
+ autoservice with the channel lock held will result in deadlock. *
+ Restructured gosub_exec() to not call ast_parseable_goto() and
+ ast_exists_extension() with the channel lock held. (closes issue
+ ASTERISK-19764) Reported by: rmudgett Tested by: rmudgett
+
+2012-06-01 18:18 +0000 [r368218] Kevin P. Fleming <kpfleming at digium.com>
+
+ * channels/chan_sip.c: Improve SDP parsing warning messages *
+ 'Unsupported media type' is only reported when that is in fact
+ the case, not when a supported media type is included in an 'm'
+ line that has an invalid format. * All warning messages related
+ to parsing 'm' lines now include the 'm' line contents. * (minor
+ bugfix) newline added to port-number-zero warning messages. *
+ Warning messages improved to use RFC-specified terminology for
+ various items. * Warnings for offers that include more than one
+ port for a single media type now include the media type. Review:
+ https://reviewboard.asterisk.org/r/1811/
+
+2012-06-01 03:25 +0000 [r368092] Michael L. Young <elgueromexicano at gmail.com>
+
+ * funcs/func_channel.c: Add documentation to function CHANNEL for
+ options echocan_mode and buffers The ability to set
+ "echocan_mode" and "buffers" through the dialplan was added to
+ chan_dahdi some time ago. This patch adds some documentation to
+ func_channel. (Closes issue ASTERISK-19911) Reported by: Dale
+ Noll Tested by: Michael L. Young Patches:
+ asterisk-19911-branch18.diff uploaded by Michael L. Young
+ (license 5026) Review: https://reviewboard.asterisk.org/r/1949/
+
+2012-05-31 18:00 +0000 [r367906-368039] Richard Mudgett <rmudgett at digium.com>
+
+ * main/db1-ast/btree/bt_open.c, apps/app_queue.c,
+ channels/chan_iax2.c, pbx/pbx_config.c, res/ael/pval.c,
+ main/tcptls.c, main/manager.c, res/res_config_odbc.c,
+ channels/chan_sip.c, channels/chan_agent.c, funcs/func_math.c,
+ main/features.c: Coverity Report: Fix issues for error type
+ REVERSE_INULL (core modules) * Fixes findings:
+ 0-2,5,7-15,24-26,28-31 (issue ASTERISK-19648) Reported by: Matt
+ Jordan
+
+ * channels/sig_pri.c, channels/sig_ss7.c: Use the
+ DEADLOCK_AVOIDANCE() macro instead. (issue ASTERISK-19854)
+
+ * channels/sig_pri.c, channels/sig_ss7.c: Fix deadlock when
+ executing CLI "pri show channels" and "ss7 show channels"
+ commands. * Fix sig_pri_lock_owner() to avoid deadlock properly.
+ * Code pri_grab() better. * Fix sig_ss7_lock_owner() to avoid
+ deadlock properly. * Code ss7_grab() better. (closes issue
+ ASTERISK-19854) Reported by: Jaxon Patches:
+ jira_asterisk_19854_v1.8.6.patch (license #5621) patch uploaded
+ by rmudgett (Modified to do the same thing to sig_ss7) Tested by:
+ Jaxon
+
+ * apps/app_meetme.c: Coverity Report: Fix issues for error type
+ REVERSE_INULL (deprecated modules) * Fix only issue pointed out
+ by deprecated_REVERSE_INULL.txt for app_meetme.c in find_user().
+ * Change use of %i to %d in sscanf() in find_user(). The use of
+ %i gives unexpected parsing because it can accept hex, octal, and
+ decimal integer formats. * Changed other uses of %i in
+ app_meetme() to use %d for consistency. (issue ASTERISK-19648)
+ Reported by: Matt Jordan
+
+2012-05-29 18:30 +0000 [r367843] Matthew Jordan <mjordan at digium.com>
+
+ * channels/chan_skinny.c: AST-2012-008: Fix remote crash
+ vulnerability in chan_skinny When a skinny session is
+ unregistered, the corresponding device pointer is set to NULL in
+ the channel private data. If the client was not in the on-hook
+ state at the time the connection was closed, the device pointer
+ can later be dereferenced if a message or channel event attempts
+ to use a line's pointer to said device. The patches prevent this
+ from occurring by checking the line's pointer in message handlers
+ and channel callbacks that can fire after an unregistration
+ attempt. (closes issue ASTERISK-19905) Reported by: Christoph
+ Hebeisen Tested by: mjordan, Damien Wedhorn Patches:
+ AST-2012-008-1.8.diff uploaded by mjordan (license 6283)
+ AST-2012-008-10.diff uploaded by mjordan (license 6283)
+
+2012-05-25 16:28 +0000 [r367781] Richard Mudgett <rmudgett at digium.com>
+
+ * channels/chan_iax2.c: AST-2012-007: Fix IAX receiving HOLD
+ without suggested MOH class crash. * Made schedule_delivery() set
+ the received frame f->data.ptr to NULL if the datalen is zero. *
+ Fix queue_signalling() memcpy() size error. * Made
+ queue_signalling() not use C++ keyword variable names. (closes
+ issue ASTERISK-19597) Reported by: mgrobecker Patches:
+ jira_asterisk_19597_v1.8.patch (license #5621) patch uploaded by
+ rmudgett Tested by: rmudgett, Michael L. Young
+
+2012-05-25 02:27 +0000 [r367730] Michael L. Young <elgueromexicano at gmail.com>
+
+ * channels/chan_sip.c: Fix pvt_sip for inbound call to use peer's
+ allowtransfer setting The pvt_sip allowtransfer was not being set
+ to that of the peer's setting. Therefore, the global
+ allowtransfer setting was being used instead which would lead to
+ calls not being transfered if the global setting was set to 'no'
+ despite the setting on the peer being 'yes' and vice versa, calls
+ would be allowed to transfer even if the peer's setting was 'no'
+ but the global setting was 'yes'. (Closes issue ASTERISK-19856)
+ Reported by: Jacek Tested by: Michael L. Young, Jacek Patches:
+ issue-asterisk-19856-branch10-v3.diff uploaded by Michael L.
+ Young (license 5026) Review:
+ https://reviewboard.asterisk.org/r/1923/
+
+2012-05-24 22:21 +0000 [r367469-367678] Richard Mudgett <rmudgett at digium.com>
+
+ * apps/app_queue.c, apps/app_dial.c: Fix Dial I option ignored if
+ dial forked and one fork redirects. The Dial and Queue I option
+ is intended to block connected line updates and redirecting
+ updates. However, it is a feature that when a call is locally
+ redirected, the I option is disabled if the redirected call runs
+ as a local channel so the administrator can have an opportunity
+ to setup new connected line information. Unfortunately, the Dial
+ and Queue I option is disabled for *all* forked calls if one of
+ those calls is redirected. * Make the Dial and Queue I option
+ apply to each outgoing call leg independently. Now if one
+ outgoing call leg is locally redirected, the other outgoing calls
+ are not affected. * Made Dial not pass any redirecting updates
+ when forking calls. Redirecting updates do not make sense for
+ this scenario. * Made Queue not pass any redirecting updates when
+ using the ringall strategy. Redirecting updates do not make sense
+ for this scenario. * Fixed deadlock potential with chan_local
+ when Dial and Queue send redirecting updates for a local
+ redirect. * Converted the Queue stillgoing flag to a boolean
+ bitfield. (closes issue ASTERISK-19511) Reported by: rmudgett
+ Tested by: rmudgett Review:
+ https://reviewboard.asterisk.org/r/1920/
+
+ * main/pbx.c: Fix WaitExten(x,m(musicclass)) string termination.
+ The AST_CONTROL_HOLD MOH class from the WaitExten application can
+ now be queued onto a channel, passed over local channels with the
+ /m option, and passed over IAX channels.
+
+2012-05-23 20:27 +0000 [r367416] Mark Michelson <mmichelson at digium.com>
+
+ * main/tcptls.c: Only call SSL_CTX_free if DO_SSL is defined.
+ Thanks to Paul Belanger for pointing out this error.
+
+2012-05-23 13:06 +0000 [r367362] Matthew Jordan <mjordan at digium.com>
+
+ * channels/chan_sip.c: Update a peer's LastMsgsSent when the peer
[... 40028 lines stripped ...]
More information about the asterisk-commits
mailing list