[asterisk-commits] bebuild: tag certified-1.8.11-cert3-rc1 r368036 - /certified/tags/1.8.11-cert...
SVN commits to the Asterisk project
asterisk-commits at lists.digium.com
Thu May 31 12:39:00 CDT 2012
Author: bebuild
Date: Thu May 31 12:38:56 2012
New Revision: 368036
URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=368036
Log:
Importing files for 1.8.11-cert3-rc1 release.
Added:
certified/tags/1.8.11-cert3-rc1/.lastclean (with props)
certified/tags/1.8.11-cert3-rc1/.version (with props)
certified/tags/1.8.11-cert3-rc1/ChangeLog (with props)
Added: certified/tags/1.8.11-cert3-rc1/.lastclean
URL: http://svnview.digium.com/svn/asterisk/certified/tags/1.8.11-cert3-rc1/.lastclean?view=auto&rev=368036
==============================================================================
--- certified/tags/1.8.11-cert3-rc1/.lastclean (added)
+++ certified/tags/1.8.11-cert3-rc1/.lastclean Thu May 31 12:38:56 2012
@@ -1,0 +1,3 @@
+39
+
+
Propchange: certified/tags/1.8.11-cert3-rc1/.lastclean
------------------------------------------------------------------------------
svn:eol-style = native
Propchange: certified/tags/1.8.11-cert3-rc1/.lastclean
------------------------------------------------------------------------------
svn:keywords = none
Propchange: certified/tags/1.8.11-cert3-rc1/.lastclean
------------------------------------------------------------------------------
svn:mime-type = text/plain
Added: certified/tags/1.8.11-cert3-rc1/.version
URL: http://svnview.digium.com/svn/asterisk/certified/tags/1.8.11-cert3-rc1/.version?view=auto&rev=368036
==============================================================================
--- certified/tags/1.8.11-cert3-rc1/.version (added)
+++ certified/tags/1.8.11-cert3-rc1/.version Thu May 31 12:38:56 2012
@@ -1,0 +1,1 @@
+1.8.11-cert3-rc1
Propchange: certified/tags/1.8.11-cert3-rc1/.version
------------------------------------------------------------------------------
svn:eol-style = native
Propchange: certified/tags/1.8.11-cert3-rc1/.version
------------------------------------------------------------------------------
svn:keywords = none
Propchange: certified/tags/1.8.11-cert3-rc1/.version
------------------------------------------------------------------------------
svn:mime-type = text/plain
Added: certified/tags/1.8.11-cert3-rc1/ChangeLog
URL: http://svnview.digium.com/svn/asterisk/certified/tags/1.8.11-cert3-rc1/ChangeLog?view=auto&rev=368036
==============================================================================
--- certified/tags/1.8.11-cert3-rc1/ChangeLog (added)
+++ certified/tags/1.8.11-cert3-rc1/ChangeLog Thu May 31 12:38:56 2012
@@ -1,0 +1,38380 @@
+2012-05-31 Asterisk Development Team <asteriskteam at digium.com>
+
+ * Certified Asterisk 1.8.11-cert3-rc1 Released.
+
+2012-05-29 Asterisk Development Team <asteriskteam at digium.com>
+
+ * Certified Asterisk 1.8.11-cert2 Released.
+
+ * AST-2012-007
+
+ * AST-2012-008
+
+2012-05-29 18:47 +0000 [r367846-367847] Matthew Jordan <mjordan 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. Youngi
+
+ * /, 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 dereferened 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 (licesen 6283)
+
+2012-05-21 19:05 +0000 [r367161] Mark Michelson <mmichelson at digium.com>
+
+ * include/asterisk/callerid.h, channels/chan_sip.c,
+ main/callerid.c: Add "send to voicemail" Digium phone
+ functionality to Asterisk. This change accommodates two methods
+ by which calls can be directed to a user's voicemail. * Incoming
+ calls can be redirected to any user's voicemail. * Established
+ calls can be blind transferred to any user's voicemail. Digium
+ phones indicate the desire to direct a call to voicemail by using
+ a Diversion header with a reason parameter of "send_to_vm". This
+ patch adds the "send_to_vm" reason as a valid redirecting reason.
+ In addition, chan_sip.c has been modified to update redirecting
+ information on the transferred channel by reading a Diversion
+ header on a REFER request. (closes issue AST-871) Reported by
+ Malcolm Davenport Review: https://reviewboard.asterisk.org/r/1925
+
+2012-05-04 21:17 +0000 [r365395] Jason Parker <jparker at digium.com>
+
+ * apps/app_mixmonitor.c, apps/app_voicemail.c: Add support for
+ folders in MixMonitor 'm' option. Backport manager actions. The
+ manager actions are needed, so MixMonitor can be executed on
+ existing channels. (issue DPMA-68)
+
+2012-05-01 17:25 +0000 [r364761] Jason Parker <jparker at digium.com>
+
+ * apps/app_voicemail.c, tests/test_voicemail_api.c,
+ include/asterisk/app_voicemail.h: Remove folder_dir from
+ voicemail snapshots API. It was both unused (except in tests,
+ where it was fudged) and unnecessary. (closes issue AST-842)
+
+2012-04-25 Asterisk Development Team <asteriskteam at digium.com>
+
+ * Certified Asterisk 1.8.11-cert1 Released.
+
+2012-04-25 16:53 +0000 [r363674] Jason Parker <jparker at digium.com>
+
+ * / (added): Asterisk 1.8-digiumphones branch has become Certified
+ Asterisk 1.8.11. For more details about Certified Asterisk, see
+ http://tinyurl.com/7pfp639
+
+2012-04-24 20:57 +0000 [r363374] Jason Parker <jparker at digium.com>
+
+ * /res/res_smdi.c,
+ /apps/app_osplookup.c,
+ /channels/chan_misdn.c,
+ /channels/chan_skinny.c,
+ /funcs/func_frame_trace.c,
+ /cdr/cdr_sqlite.c,
+ /pbx/pbx_realtime.c,
+ /apps/app_amd.c,
+ /pbx/pbx_dundi.c,
+ /apps/app_url.c,
+ /channels/chan_nbs.c,
+ /apps/app_externalivr.c,
+ /apps/app_zapateller.c,
+ /cdr/cdr_odbc.c,
+ /res/res_fax_spandsp.c,
+ /channels/chan_mgcp.c,
+ /cel/cel_pgsql.c,
+ /apps/app_readfile.c,
+ /apps/app_test.c,
+ /apps/app_ices.c,
+ /channels/chan_gtalk.c,
+ /cdr/cdr_csv.c,
+ /channels/chan_phone.c,
+ /funcs/func_pitchshift.c,
+ /apps/app_waitforring.c,
+ /formats/format_vox.c,
+ /res/res_timing_pthread.c,
+ /apps/app_minivm.c,
+ /channels/chan_h323.c,
+ /cel/cel_sqlite3_custom.c,
+ /apps/app_confbridge.c,
+ /res/res_config_ldap.c,
+ /apps/app_nbscat.c,
+ /cdr/cdr_sqlite3_custom.c,
+ /res/res_snmp.c,
+ /apps/app_dictate.c,
+ /apps/app_waitforsilence.c,
+ /apps/app_dahdiras.c,
+ /pbx/pbx_lua.c,
+ /apps/app_alarmreceiver.c,
+ /apps/app_image.c,
+ /res/res_ael_share.c,
+ /cdr/cdr_tds.c,
+ /apps/app_setcallerid.c,
+ /apps/app_mp3.c,
+ /channels/chan_alsa.c,
+ /res/res_timing_kqueue.c,
+ /channels/chan_unistim.c,
+ /apps/app_dahdibarge.c,
+ /res/res_config_pgsql.c,
+ /res/res_adsi.c,
+ /res/res_phoneprov.c,
+ /apps/app_morsecode.c,
+ /cdr/cdr_pgsql.c,
+ /res/res_config_sqlite.c,
+ /channels/chan_jingle.c,
+ /pbx/pbx_ael.c,
+ /apps/app_sms.c,
+ /formats/format_jpeg.c,
+ /apps/app_jack.c,
+ /apps/app_adsiprog.c,
+ /cel/cel_radius.c,
+ /res/res_ais.c,
+ /cel/cel_tds.c,
+ /apps/app_festival.c,
+ /apps/app_chanisavail.c,
+ /channels/chan_console.c,
+ /apps/app_talkdetect.c,
+ /res/res_jabber.c,
+ /cdr/cdr_radius.c,
+ /apps/app_getcpeid.c,
+ /channels/chan_oss.c: Disable extended
+ and deprecated modules by default. Users can still enable any of
+ these using menuselect if they so choose. (closes issue AST-873)
+
+2012-04-23 15:17 +0000 [r363161] Jason Parker <jparker at digium.com>
+
+ * /main/manager.c,
+ ,
+ /channels/chan_sip.c,
+ /channels/chan_skinny.c: Multiple
+ revisions 363102,363106,363141 ........ r363102 | mjordan |
+ 2012-04-23 08:37:55 -0500 (Mon, 23 Apr 2012) | 16 lines
+ AST-2012-005: Fix remotely exploitable heap overflow in keypad
+ button handling When handling a keypad button message event, the
+ received digit is placed into a fixed length buffer that acts as
+ a queue. When a new message event is received, the length of that
+ buffer is not checked before placing the new digit on the end of
+ the queue. The situation exists where sufficient keypad button
+ message events would occur that would cause the buffer to be
+ overrun. This patch explicitly checks that there is sufficient
+ room in the buffer before appending a new digit. (closes issue
+ ASTERISK-19592) Reported by: Russell Bryant ........ Merged
+ revisions 363100 from
+ http://svn.asterisk.org/svn/asterisk/branches/1.6.2 ........
+ r363106 | mjordan | 2012-04-23 09:05:02 -0500 (Mon, 23 Apr 2012)
+ | 17 lines AST-2012-006: Fix crash in UPDATE handling when no
+ channel owner exists If Asterisk receives a SIP UPDATE request
+ after a call has been terminated and the channel has been
+ destroyed but before the SIP dialog has been destroyed, a
+ condition exists where a connected line update would be attempted
+ on a non-existing channel. This would cause Asterisk to crash.
+ The patch resolves this by first ensuring that the SIP dialog has
+ an owning channel before attempting a connected line update. If
+ an UPDATE request is received and no channel is associated with
+ the dialog, a 481 response is sent. (closes issue ASTERISK-19770)
+ Reported by: Thomas Arimont Tested by: Matt Jordan Patches:
+ ASTERISK-19278-2012-04-16.diff uploaded by Matt Jordan (license
+ 6283) ........ r363141 | jrose | 2012-04-23 09:33:16 -0500 (Mon,
+ 23 Apr 2012) | 20 lines AST-2012-004: Fix an error that allows
+ AMI users to run shell commands sans authorization. As detailed
+ in the advisory, AMI users without write authorization for SYSTEM
+ class AMI actions were able to run system commands by going
+ through other AMI commands which did not require that
+ authorization. Specifically, GetVar and Status allowed users to
+ do this by setting their variable/s options to the SHELL or EVAL
+ functions. Also, within 1.8, 10, and trunk there was a similar
+ flaw with the Originate action that allowed users with originate
+ permission to run MixMonitor and supply a shell command in the
+ Data argument. That flaw is fixed in those versions of this
+ patch. (closes issue ASTERISK-17465) Reported By: David Woolley
+ Patches: 162_ami_readfunc_security_r2.diff uploaded by jrose
+ (license 6182) 18_ami_readfunc_security_r2.diff uploaded by jrose
+ (license 6182) 10_ami_readfunc_security_r2.diff uploaded by jrose
+ (license 6182) ........ Merged revisions 363117 from
+ http://svn.asterisk.org/svn/asterisk/branches/1.6.2 ........
+ Merged revisions 363102,363106,363141 from
+ http://svn.asterisk.org/svn/asterisk/branches/1.8
+
+2012-04-19 20:31 +0000 [r362673] Mark Michelson <mmichelson at digium.com>
+
+ * /channels/chan_sip.c: Add a test
+ application for sending custom SIP INFO messages. When
+ TEST_FRAMEWORK is enabled, SIPSendCustomInfo is available to test
+ sending custom INFO requests. Review:
+ https://reviewboard.asterisk.org/r/1866
+
+2012-04-13 17:19 +0000 [r362042-362132] Matthew Jordan <mjordan at digium.com>
+
+ * : Rename property branches-1.8-merged to
+ branch-1.8-merged
+
+ * : Update properties on 1.8-digiumphones
+ Change the merge property tag from svnmerge-integrated to
+ branches-1.8-merged. Added merged revisions from r362042.
+
+ * ,
+ /channels/chan_sip.c,
+ /main/features.c: Merge of several
+ needed fixes for 1.8-digiumphones This merges fixes for the
+ following issues into the 1.8-digiumphones branch: *
+ ASTERISK-19355 - Call transfer with consultation frequently fails
+ in cross- linked Asterisk scenario (directmedia & sendrpid
+ active) * ASTERISK 19365 - Remote SIP Call legs are frequently
+ not released in a cross-linked Asterisk scenario (directmedia &
+ sendrpid) * ASTERISK-19183 - Sporadically missing connectedline
+ event to caller channel in directed pickup app
+
+2012-04-09 20:40 +0000 [r361704] Mark Michelson <mmichelson at digium.com>
+
+ * /apps/app_voicemail.c,
+ /apps/app_voicemail.exports.in,
+ /tests/test_voicemail_api.c (added),
+ /include/asterisk/app_voicemail.h: Fix
+ bugs in voicemail APIs and add unit tests. There were several
+ crashes that could occur due to NULL inputs, invalid inputs, and
+ the like. This fixes all known ones and adds unit tests to
+ exercise the APIs.
+
+2012-04-06 19:08 +0000 [r361502] Richard Mudgett <rmudgett at digium.com>
+
+ * /main/message.c: Update Func MESSAGE()
+ and AMI MessageSend documentation. * Document
+ MESSAGE(custom_data) * Update AMI MessageSend documentation *
+ Eliminate a shadowed variable name in msg_func_write() for
+ custom_data.
+
+2012-04-05 17:24 +0000 [r361283] Mark Michelson <mmichelson at digium.com>
+
+ * /funcs/func_presence_state.c,
+ /tests/test_config.c: Add additional
+ configuration and presence unit tests. These were originally
+ written while merging features into trunk, but these tests apply
+ just as much for the 1.8 version of Digium phones, so might as
+ well have them here, too.
+
+2012-04-03 21:03 +0000 [r361088] Jonathan Rose <jrose at digium.com>
+
+ * /apps/app_mixmonitor.c: Make m option
+ for mixmonitor delete the source file once it is finished copying
+ to vm. Review: https://reviewboard.asterisk.org/r/1842/
+
+2012-03-29 21:49 +0000 [r360826] Jason Parker <jparker at digium.com>
+
+ * /main/manager.c,
+ ,
+ /main/utils.c,
+ /include/asterisk/manager.h,
+ /apps/app_milliwatt.c: Multiple
+ revisions 359656,359706,359979 ........ r359656 | mjordan |
+ 2012-03-15 13:35:59 -0500 (Thu, 15 Mar 2012) | 22 lines Fix
+ remotely exploitable stack overrun in Milliwatt Milliwatt is
+ vulnerable to a remotely exploitable stack overrun when using the
+ 'o' option. This occurs due to the milliwatt_generate function
+ not accounting for AST_FRIENDLY_OFFSET when calculating the
+ maximum number of samples it can put in the output buffer. This
+ patch resolves this issue by taking into account
+ AST_FRIENDLY_OFFSET when determining the maximum number of
+ samples allowed. Note that at no point is remote code execution
+ possible. The data that is written into the buffer is the
+ pre-defined Milliwatt data, and not custom data. (closes issue
+ ASTERISK-19541) Reported by: Russell Bryant Tested by: Matt
+ Jordan Patches: milliwatt_stack_overrun.rev1.txt by Russell
+ Bryant (license 6283) Note that this patch was written by
+ Russell, even though Matt uploaded it ........ Merged revisions
+ 359645 from http://svn.asterisk.org/svn/asterisk/branches/1.6.2
+ ........ r359706 | mjordan | 2012-03-15 14:01:22 -0500 (Thu, 15
+ Mar 2012) | 16 lines Fix remotely exploitable stack overflow in
+ HTTP manager There exists a remotely exploitable stack buffer
+ overflow in HTTP digest authentication handling in Asterisk. The
+ particular method in question is only utilized by HTTP AMI. When
+ parsing the digest information, the length of the string is not
+ checked when it is copied into temporary buffers allocated on the
+ stack. This patch fixes this behavior by parsing out pre-defined
+ key/value pairs and avoiding unnecessary copies to the stack.
+ (closes issue ASTERISK-19542) Reported by: Russell Bryant Tested
+ by: Matt Jordan ........ r359979 | rmudgett | 2012-03-20 12:21:16
+ -0500 (Tue, 20 Mar 2012) | 28 lines Allow AMI action callback to
+ be reentrant. Fix AMI module reload deadlock regression from
+ ASTERISK-18479 when it tried to fix the race between calling an
+ AMI action callback and unregistering that action. Refixes
+ ASTERISK-13784 broken by ASTERISK-17785 change. Locking the ao2
+ object guaranteed that there were no active callbacks that
+ mattered when ast_manager_unregister() was called. Unfortunately,
+ this causes the deadlock situation. The patch stops locking the
+ ao2 object to allow multiple threads to invoke the callback
+ re-entrantly. There is no way to guarantee a module unload will
+ not crash because of an active callback. The code attempts to
+ minimize the chance with the registered flag and the maximum 5
+ second delay before ast_manager_unregister() returns. The trunk
+ version of the patch changes the API to fix the race condition
+ correctly to prevent the module code from unloading from memory
+ while an action callback is active. * Don't hold the lock while
+ calling the AMI action callback. (closes issue ASTERISK-19487)
+ Reported by: Philippe Lindheimer Review:
+ https://reviewboard.asterisk.org/r/1818/ Review:
+ https://reviewboard.asterisk.org/r/1820/ ........ Merged
+ revisions 359656,359706,359979 from
+ http://svn.asterisk.org/svn/asterisk/branches/1.8
+
+2012-03-21 15:44 +0000 [r360031-360188] Mark Michelson <mmichelson at digium.com>
+
+ * /main/pbx.c: Prevent potentially passing
+ a NULL pointer to strcasecmp()
+
+ * /main/pbx.c: Fix one more "(null)"
+ string. If a hint with no presence portion were added, it would
+ result in another "(null)" string warning.
+
+ * /main/pbx.c: Fix another "Possible
+ programming error" bug. Similar to the previous commit, don't
+ pass a printf-generated string to ast_strlen_zero.
+
+ * /main/pbx.c: Get rid of an annoying
+ "Possible programming error" message. If an extension's 'app'
+ field is NULL, then a "(null)" string would be written into an
+ ast_str due to the way that snprintf works. When this is passed
+ to ast_strlen_zero(), it fires up a big warning indicating
+ something is probably wrong. There indeed was a problem, but
+ luckily it wasn't a very big problem. After the failed
+ ast_strlen_zero() check and big warning message, the very next if
+ statement, checking to see if the "(null)" matched a presence
+ provider, would fail, so no harm was done.
+
+2012-03-08 18:40 +0000 [r358725] Jonathan Rose <jrose at digium.com>
+
+ * /apps/app_mixmonitor.c: Fixes
+ unitialized variable use warning introduced by addition of
+ mixmonitor forward to vm
+
+2012-03-08 18:02 +0000 [r358692] Jason Parker <jparker at digium.com>
+
+ * , /main/acl.c:
+ Prevent outbound SIP NOTIFY packets from displaying a port of 0
+ In the change from 1.6.2 to 1.8, ast_sockaddr was introduced
+ which changed the behavior of ast_find_ourip such that port
+ number was wiped out. This caused the port in internip (which is
+ used for Contact and Call-ID on NOTIFYs) to be 0. This change
+ causes ast_find_ourip to be port-preserving again. (closes issue
+ ASTERISK-19430) ........ Merged revisions 357665 from
+ http://svn.asterisk.org/svn/asterisk/branches/1.8
+
+2012-03-02 15:18 +0000 [r357808] Paul Belanger <pabelanger at digium.com>
+
+ * /apps/app_mixmonitor.c: Fixed xmldoc
+ formatting error for 'm' option
+
+2012-02-28 21:52 +0000 [r357456-357459] Jason Parker <jparker at digium.com>
+
+ * /main/channel.c,
+ /funcs/func_presence_state.c (added),
+ /main/manager.c,
+ /channels/chan_skinny.c,
+ /funcs/func_frame_trace.c,
+ /include/asterisk/jabber.h,
+ /main/file.c,
+ /main/app.c,
+ /tests/test_config.c (added),
+ /include/asterisk/frame.h,
+ /main/custom_control_frame.c (added),
+ /main/message.c (added),
+ /apps/app_mixmonitor.c,
+ /channels/sip/include/sip.h,
+ /main/asterisk.c,
+ /tests/test_custom_control.c (added),
+ /main/pbx.c,
+ /include/asterisk/presencestate.h
+ (added),
+ /include/asterisk/app_voicemail.h
+ (added), /include/asterisk/channel.h,
+ /include/asterisk/manager.h,
+ /apps/app_queue.c,
+ /main/config.c,
+ /include/asterisk/file.h,
+ /include/asterisk/app.h,
+ /include/asterisk/event_defs.h,
+ /configs/jabber.conf.sample,
+ /include/asterisk/custom_control_frame.h
+ (added), /include/asterisk/message.h
+ (added), /main/features.c,
+ /apps/app_voicemail.exports.in,
+ /main/event.c,
+ /include/asterisk/pbx.h,
+ /configs/sip.conf.sample,
+ /apps/app_voicemail.c,
+ /channels/chan_sip.c,
+ /include/asterisk/config.h,
+ /configs/manager.conf.sample,
+ /include/asterisk/_private.h,
+ /res/res_jabber.c,
+ /main/presencestate.c (added): Add
+ support for Digium Phones.
+
+
+2012-03-29 Asterisk Development Team <asteriskteam at digium.com>
+
+ * Asterisk 1.8.11.0 Released.
+
+2012-03-26 Asterisk Development Team <asteriskteam at digium.com>
+
+ * Asterisk 1.8.11.0-rc3 Released.
+
+ * AST-2012-003
+
+ * AST-2012-002
+
+ * /main/manager.c, /include/asterisk/manager.h: Fix AMI deadlock
+ regression by allowing AMI action callback to be reentrant
+
+ Fix AMI module reload deadlock from ASTERISK-18479 when it tried
+ to fix the race between calling an AMI action callback and
+ unregistering that action. Refixes ASTERISK-13784 broken by
+ ASTERISK-17785 change.
+
+ Locking the ao2 object guaranteed that there were no active
+ callbacks that mattered when ast_manager_unregister() was called.
+ Unfortunately, this causes the deadlock situation. The patch stops
+ locking the ao2 object to allow multiple threads to invoke the
+ callback re-entrantly. There is no way to guarantee a module unload
+ will not crash because of an active callback. The code attempts to
+ minimize the chance with the registered flag and the maximum 5
+ second delay before ast_manager_unregister() returns.
+
+ The trunk version of the patch changes the API to fix the race
+ condition correctly to prevent the module code from unloading from
+ memory while an action callback is active.
+
+ * Don't hold the lock while calling the AMI action callback.
+
+ (closes issue ASTERISK-19487)
+ Reported by: Philippe Lindheimer
+
+ Review: https://reviewboard.asterisk.org/r/1818/
+
+2012-03-06 Asterisk Development Team <asteriskteam at digium.com>
+
+ * Asterisk 1.8.11.0-rc2 Released.
+
+ * main/acl.c: Prevent outbound SIP NOTIFY packets from displaying
+ a port of 0.
+
+ In the change from 1.6.2 to 1.8, ast_sockaddr was
+ introduced which changed the behavior of ast_find_ourip such
+ that port number was wiped out. This caused the port in
+ internip (which is used for Contact and Call-ID on NOTIFYs) to be
+ 0. This change causes ast_find_ourip to be port-preserving again.
+
+2012-01-30 21:57 +0000 [r353368-353320] Alec L Davis <sivad.a at paradise.net.nz>
+
+ * channels/sip/include/sip.h, channels/sip/include/dialog.h,
+ channels/chan_sip.c: RFC3261 Section 8.1.1.5. The sequence number
+ value MUST be expressible as a 32-bit unsigned integer * fix: use
+ %u instead of %d when dealing with CSeq numbers - to remove
+ possibility of -ve numbers. * fix: change all uses of seqno and
+ friends (ocseq icseq) from 'int' or 'unsigned int' to uint32_t.
+ Summary of CSeq numbers. An initial CSeq number must be less than
+ 2^31 A CSeq number can increase in value up to 2^32-1 An
+ incrementing CSeq number must not wrap around to 0. Tested with
+ Asterisk 1.8.8.2 with Grandstream phones. alecdavis (license 585)
+ Tested by: alecdavis Review:
+ https://reviewboard.asterisk.org/r/1699/
+
+ * channels/chan_sip.c: prevent debug messsges displaying -ve Cseq
+ numbers. Missed in R353320
+
+2012-01-30 23:17 +0000 [r353371] Terry Wilson <twilson at digium.com>
+
+ * include/asterisk/dnsmgr.h, main/dnsmgr.c, channels/chan_sip.c:
+ Re-link peers by IP when dnsmgr changes the IP Asterisk's dnsmgr
+ currently takes a pointer to an ast_sockaddr and updates it
+ anytime an address resolves to something different. There are a
+ couple of issues with this. First, the ast_sockaddr is usually
+ the address of an ast_sockaddr inside a refcounted struct and we
+ never bump the refcount of those structs when using dnsmgr. This
+ makes it possible that a refresh could happen after the
+ destructor for that object is called (despite ast_dnsmgr_release
+ being called in that destructor). Second, the module using dnsmgr
+ cannot be aware of an address changing without polling for it in
+ the code. If an action needs to be taken on address update (like
+ re-linking a SIP peer in the peers_by_ip table), then polling for
+ this change negates many of the benefits of having dnsmgr in the
+ first place. This patch adds a function to the dnsmgr API that
+ calls an update callback instead of blindly updating the address
+ itself. It also moves calls to ast_dnsmgr_release outside of the
+ destructor functions and into cleanup functions that are called
+ when we no longer need the objects and increments the refcount of
+ the objects using dnsmgr since those objects are stored on the
+ ast_dnsmgr_entry struct. A helper function for returning the
+ proper default SIP port (non-tls vs tls) is also added and used.
+ This patch also incorporates changes from a patch posted by Timo
+ Teräs to ASTERISK-19106 for related dnsmgr issues. (closes issue
+ ASTERISK-19106) Review: https://reviewboard.asterisk.org/r/1691/
+
+2012-01-31 16:51 +0000 [r353454] Richard Mudgett <rmudgett at digium.com>
+
+ * include/asterisk/channel.h, main/manager.c: Fix memory leak in
+ error paths for action_originate(). * Fix memory leak of vars in
+ error paths for action_originate(). * Moved struct
+ fast_originate_helper tech and data members to stringfields. *
+ Simplified ActionID header handling for fast_originate(). * Added
+ doxygen note to ast_request() and ast_call() and the associated
+ channel callbacks that the data/addr parameters should be treated
+ as const char *. Review: https://reviewboard.asterisk.org/r/1690/
+
+2012-01-31 23:41 +0000 [r353502] Terry Wilson <twilson at digium.com>
+
+ * res/res_calendar.c: Allow res_calendar to be unloaded The
+ calendaring tech modules depend on res_calendar and initially
+ res_calendar just bumped the use count so that it couldn't be
+ unloaded. res_calendar can potentially create many threads and
+ I've seen issues where the Asterisk shutdown has failed where it
+ looked like these threads could be the culprit. This patch adds
+ unload support for res_calendar. Unloading res_calendar will also
+ unload the dependant tech modules as well. (closes issue
+ ASTERISK-16744) Review: https://reviewboard.asterisk.org/r/1657/
+
+2012-02-01 15:02 +0000 [r353550] Matthew Jordan <mjordan at digium.com>
+
+ * contrib/init.d/etc_default_asterisk: Added clarification for the
+ VERBOSITY setting to etc_default_asterisk Clarified that using
+ the VERBOSITY setting in etc_default_asterisk is the same as
+ using the -v command line switch, which causes Asterisk to launch
+ in console mode. (closes issue ASTERISK-17030) Reported by: Jonas
+
+2012-02-01 15:50 +0000 [r353598] Sean Bright <sean at malleable.com>
+
+ * include/asterisk/audiohook.h: Resolve an overlap in the
+ ast_audiohook_flags values. AST_AUDIOHOOK_TRIGGER_WRITE and
+ AST_AUDIOHOOK_WANTS_DTMF were overlapping which may have caused
+ unintended side effects. This patch moves
+ AST_AUDIOHOOK_TRIGGER_WRITE, and updates
+ AST_AUDIOHOOK_TRIGGER_MODE to reflect the original intention.
+ This will affect existing modules that use these flags, so be
+ sure to recompile as necessary. (closes issue ASTERISK-19246)
+ Reported by: feyfre
+
+2012-02-01 21:05 +0000 [r353769-353720] Jonathan Rose <jrose at digium.com>
+
+ * channels/chan_sip.c: Use ast_sockaddr_stringify_fmt wrappers for
+ various functions in chan_sip There are a number of cleaner
+ looking wrappers for ast_sockaddr_stringify_fmt available which
+ are slightly more readable than using a direct call to
+ ast_sockaddr_stringify_fmt. This patch switches a number of those
+ calls in chan_sip to use those wrappers and is generally
+ harmless. (Closes issue ASTERISK-16930) Reported by: Michael L.
+ Young Patches: chan_sip-broken-registration-1.8.diff uploaded by
+ Michael L. Young (license 5026)
+
+ * channels/chan_sip.c: Fix sip show peers port output, align
+ columns, and fix ami port output. A previous patch I committed
+ from ASTERISK-16930 unexpectedly changed some output for the AMI
+ action "sippeers" which this patch changes back. Also, this
+ aligns the output for the cli command "sip show peers" and fixes
+ another issue that patch introduced by using
+ ast_sockaddr_stringify calls multiple times without immediately
+ using the pointer. I also went ahead and did a little janitorial
+ work to clean up whitespace in _sip_show_peers. (issue
+ ASTERISK-16930) (closes issue ASTERISK-19281) Reported by:
+ Patrick El Youssef Patches: ASTERISK-19281.diff uploaded by
+ Walter Doekes (license 5674)
+
+2012-02-02 16:58 +0000 [r353770] Mark Michelson <mmichelson at digium.com>
+
+ * UPGRADE.txt, configs/manager.conf.sample,
+ include/asterisk/manager.h, configs/http.conf.sample,
+ main/manager.c, main/http.c: Fix TLS port binding behavior as
+ well as reload behavior: * Removes references to tlsbindport from
+ http.conf.sample and manager.conf.sample * Properly bind to port
+ specified in tlsbindaddr, using the default port if specified. *
+ On a reload, properly close socket if the service has been
+ disabled. A note has been added to UPGRADE.txt to indicate how
+ ports must be set for TLS. (closes issue ASTERISK-16959) reported
+ by Olaf Holthausen (closes issue ASTERISK-19201) reported by
+ Chris Mylonas (closes issue ASTERISK-19204) reported by Chris
+ Mylonas Review: https://reviewboard.asterisk.org/r/1709
+
+2012-02-02 18:31 +0000 [r353818] Jonathan Rose <jrose at digium.com>
+
+ * funcs/func_curl.c: Backports some documentation for func_curl
+ from 10 to 1.8 For some reason this function was completely
+ undocumented in 1.8. I copied the 10 docs over to 1.8 and removed
+ references to an enumerator that was added in the Asterisk 10
+ version of func_curl. That was the only change I noted. (closes
+ issue ASTERISK-19186) Reported by: Olivier Krief
+
+2012-02-02 20:01 +0000 [r353867] Richard Mudgett <rmudgett at digium.com>
+
+ * channels/sig_pri.c, channels/sig_pri.h, channels/chan_dahdi.c:
+ Restore the 'w' modifier support for ISDN spans.
+ Dial(DAHDI/g0/1234w888) This feature also causes the sending
+ complete ie to be sent for switch types that do not automatically
+ send the ie. (EuroISDN/ETSI) The main difference between dialing
+ Dial(DAHDI/g0/1234w888) and Dial(DAHDI/g0/1234,,D(888)) is the
+ sending of the sending complete ie. (closes issue ASTERISK-19176)
+ Reported by: rmudgett Tested by: rmudgett
+
+2012-02-02 22:26 +0000 [r353915] Kinsey Moore <kmoore at digium.com>
+
+ * channels/chan_sip.c: Ensure entering T.38 passthrough does not
+ cause an infinite loop After R340970 Asterisk was still polling
+ the RTCP file descriptor after RTCP is shut down and removed. If
+ the descriptor happened to have data ready when the removal
+ occured then Asterisk would go into an infinite loop trying to
+ read data that it can never actually access. This change disables
+ the audio RTCP file descriptor for the duration of the T.38
+ transaction. (closes issue ASTERISK-18951) Reported-by: Kristijan
+ Vrban
+
+2012-02-03 21:24 +0000 [r353999] Jonathan Rose <jrose at digium.com>
+
+ * channels/chan_agent.c: Fixes deadlocks occuring in chan_agent due
+ to r335976 Bad locking order was added to chan_agent to prevent
+ segfaults from having no locking in a patch by irroot. This patch
+ addresses the bad locking order by releasing locks before getting
+ the right locking order to stop deadlocks from occuring when
+ doing multiple interactions with agents. (closes issue
+ ASTERISK-19285) Reported by: Alex Villacis Lasso Review:
+ https://reviewboard.asterisk.org/r/1708/
+
+2012-02-06 17:28 +0000 [r354216-354116] Richard Mudgett <rmudgett at digium.com>
+
+ * main/features.c: Add missing headers to AMI UnParkedCall event to
+ uniquely identify the call. The AMI UnParkedCall event was
+ missing the Parkinglot and Uniqueid headers that the AMI
+ ParkedCall event contains. (closes issue ASTERISK-19240) Reported
+ by: Michael Yara
+
+ * pbx/pbx_config.c: Improved documentation of CLI "dialplan add
+ extension" command. * Documented dialplan add extension
+ <exten>,<priority>,<app(<app-data>)> format. * Allow acceptance
+ of command without the app-data value. There are many
+ applications that do no need any parameters so it is silly to
+ require that field for all commands. * Fixed a couple
+ ast_malloc/ast_free mismatches with ast_add_extension2() calls.
+ (closes issue ASTERISK-19222) Reported by: Andrey Solovyev Tested
+ by: rmudgett
+
+2012-02-07 15:04 +0000 [r354263] Jonathan Rose <jrose at digium.com>
+
+ * cdr/cdr_pgsql.c: Fix column duplication bug in module reload for
+ cdr_pgsql. Prior to this patch, attempts to reload cdr_pgsql.so
+ would cause the column list to keep its current data and then add
+ a second copy during the reload. This would cause attempts to log
+ the CDR to the database to fail. This patch also cleans up some
+ unnecessary null checks for ast_free and deals with a few
+ potential locking problems. (closes issue ASTERISK-19216)
+ Reported by: Jacek Konieczny Review:
+ https://reviewboard.asterisk.org/r/1711/
+
+2012-02-07 20:53 +0000 [r354348] Terry Wilson <twilson at digium.com>
+
+ * contrib/realtime/postgresql/realtime.sql, channels/chan_sip.c:
+ Fix multiple SIP realtime issues 1. Set lastms to 0 when clearing
+ instead of "" 2. Don't set ipaddr or port to the string "(null)"
+ when they are empty 3. Add missing required fields, set default
+ for lastms to 0, and modify the length of the ipaddr field to 45
+ in the Postgresql realtime.sql file. (closes issue
+ ASTERISK-19172) Review: https://reviewboard.asterisk.org/r/1703/
+
+2012-02-09 02:23 +0000 [r354492] Russell Bryant <russell at russellbryant.com>
+
+ * main/channel.c: Remove some unnecessary locking from
+ ast_hangup(). This patch removes some unnecessary locking of the
+ channels container in ast_hangup(). The reason this came up is
+ that this lock can very quickly block the entire system. If any
+ of the channel cleanup code decides to block, it causes a problem
+ for the whole system. For example, when audiohooks get destroyed,
+ if that blocks for a while waiting on the mixmonitor thread to
+ exit because it's busy blocking on some I/O, it causes a problem
+ for many other threads in the meantime. Review:
+ https://reviewboard.asterisk.org/r/1712/
+
+2012-02-09 02:52 +0000 [r354495] Richard Mudgett <rmudgett at digium.com>
+
+ * apps/app_parkandannounce.c: Fix crash in ParkAndAnnounce. Well,
+ thats embarrasing. I forgot to initialize the caller_id storage.
+ (closes issue ASTERISK-19311) Reported by: tootai Tested by:
+ rmudgett
+
+2012-02-09 16:30 +0000 [r354542] Matthew Jordan <mjordan at digium.com>
+
+ * channels/chan_sip.c: Fix SIP INFO DTMF handling for non-numeric
+ codes In ASTERISK-18924, SIP INFO DTMF handlingw as changed to
+ account for both lowercase alphatbetic DTMF events, as well as
+ uppercase alphabetic DTMF events. When this occurred, the
+ comparison of the character buffer containing the event code was
+ changed such that the buffer was first compared again '0' and '9'
+ to determine if it was numeric. Unfortunately, since the first
+ character in the buffer will typically be '1' in the case of
+ non-numeric event codes (10-16), this caused those codes to be
+ converted to a DTMF event of '1'. This patch fixes that, and
+ cleans up handling of both application/dtmf-relay and
+ application/dtmf content types. Review:
+ https://reviewboard.asterisk.org/r/1722/ (closes issue
+ ASTERISK-19290) Reported by: Ira Emus Tested by: mjordan
+
+2012-02-09 16:56 +0000 [r354545] Mark Michelson <mmichelson at digium.com>
+
+ * CHANGES, res/res_fax.c: Adding reload support to res_fax.so
+ (closes issue ASTERISK-16712) reported by Frank DiGennaro Review:
+ https://reviewboard.asterisk.org/r/1713
+
+2012-02-09 17:07 +0000 [r354547] Matthew Jordan <mjordan at digium.com>
+
+ * channels/chan_sip.c: Clean-up of minor formatting issues in
+ r354542/3/4 rmudgett pointed out some formatting issues in the
+ check-in for ASTERISK-19290. This cleans those up. Review:
+ https://reviewboards.asterisk.org/r/1722/
+
+2012-02-09 17:32 +0000 [r354640-354594] Mark Michelson <mmichelson at digium.com>
+
+ * main/translate.c: Fix translation path choices. This change makes
+ it so computational cost is not taken into account when deciding
+ if a multistep path is better than a single-step path. This means
+ that the only time a multistep path will be chosen is if no
+ single-step path exists. This ensures a better quality
+ translation even if it turns out to be slightly slower. (closes
+ issue ASTERISK-16821) reported by Andrew Lindh Review:
+ https://reviewboard.asterisk.org/r/1715
+
+ * main/translate.c: Remove outdated comment.
+
+2012-02-09 19:52 +0000 [r354702-354655] Kinsey Moore <kmoore at digium.com>
+
+ * main/config.c: Make the config parser remove escaping backslashes
+ The config parser in Asterisk does not currently remove a
+ backslash that is used to escape a semicolon which would
+ otherwise be interpreted as the start of a comment. The change
+ here causes that backslash to be removed, but does not create a
+ real escape system in the config parser. The biggest complication
+ with a real escape system would be breaking existing configs
+ everywhere (parsing \\ as \ and breaking on escaped non-semicolon
+ characters) even though it would be the "right" way to do things.
+ (closes issue ASTERISK-17121) Review:
+ https://reviewboard.asterisk.org/r/1724/
+
+ * channels/chan_sip.c: Fix parsing of SIP headers where compact and
+ non-compact headers are mixed Change parsing of SIP headers so
+ that compactness of the header no longer influences which header
+ will be chosen. Previously, a non-compact header would be chosen
+ instead of a preceeding compact-form header. (closes issue
+ ASTERISK-17192) Review: https://reviewboard.asterisk.org/r/1728/
+
+2012-02-09 22:01 +0000 [r354749] Terry Wilson <twilson at digium.com>
+
+ * funcs/func_cdr.c: Note that CDRs are immutable once a bridge is
+ torn down CDRs cannot be modified after a bridge is torn down,
+ (e.g. after Dial() returns) even though the CDR() function may be
+ called. Since modifying the CDR code to change this behavior
+ could very easily break all kinds of things, this patch just
+ documents this limitation. (closes issues ASTERISK-16923) Review:
+ https://reviewboard.asterisk.org/r/1720/
+
+2012-02-10 18:03 +0000 [r354835] Richard Mudgett <rmudgett at digium.com>
+
+ * main/manager.c: Fix AMI Redirect ExtraChannel not redirecting to
+ the same exten and context. The astman_get_header() never returns
+ NULL so the check by the code for NULL would never fail. (closes
+ issue ASTERISK-16974) Reported by: Nuno Borges Patches:
+ 0018325.patch (license #6116) patch uploaded by Nuno Borges
+ (modified)
+
+2012-02-10 21:45 +0000 [r354889] Jason Parker <jparker at digium.com>
+
+ * apps/app_voicemail.c: Fix a voicemail memory leak with
+ heard/deleted messages. open_mailbox() was changed quite a long
[... 37598 lines stripped ...]
More information about the asterisk-commits
mailing list