[asterisk-commits] russell: tag 1.4.22-rc3 r140811 - /tags/1.4.22-rc3/

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Wed Sep 3 07:33:05 CDT 2008


Author: russell
Date: Wed Sep  3 07:33:04 2008
New Revision: 140811

URL: http://svn.digium.com/view/asterisk?view=rev&rev=140811
Log:
Importing files for 1.4.22-rc3 release

Added:
    tags/1.4.22-rc3/.lastclean   (with props)
    tags/1.4.22-rc3/.version   (with props)
    tags/1.4.22-rc3/ChangeLog   (with props)

Added: tags/1.4.22-rc3/.lastclean
URL: http://svn.digium.com/view/asterisk/tags/1.4.22-rc3/.lastclean?view=auto&rev=140811
==============================================================================
--- tags/1.4.22-rc3/.lastclean (added)
+++ tags/1.4.22-rc3/.lastclean Wed Sep  3 07:33:04 2008
@@ -1,0 +1,1 @@
+32

Propchange: tags/1.4.22-rc3/.lastclean
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: tags/1.4.22-rc3/.lastclean
------------------------------------------------------------------------------
    svn:keywords = none

Propchange: tags/1.4.22-rc3/.lastclean
------------------------------------------------------------------------------
    svn:mime-type = text/plain

Added: tags/1.4.22-rc3/.version
URL: http://svn.digium.com/view/asterisk/tags/1.4.22-rc3/.version?view=auto&rev=140811
==============================================================================
--- tags/1.4.22-rc3/.version (added)
+++ tags/1.4.22-rc3/.version Wed Sep  3 07:33:04 2008
@@ -1,0 +1,1 @@
+1.4.22-rc3

Propchange: tags/1.4.22-rc3/.version
------------------------------------------------------------------------------
    svn:eol-style = native

Propchange: tags/1.4.22-rc3/.version
------------------------------------------------------------------------------
    svn:keywords = none

Propchange: tags/1.4.22-rc3/.version
------------------------------------------------------------------------------
    svn:mime-type = text/plain

Added: tags/1.4.22-rc3/ChangeLog
URL: http://svn.digium.com/view/asterisk/tags/1.4.22-rc3/ChangeLog?view=auto&rev=140811
==============================================================================
--- tags/1.4.22-rc3/ChangeLog (added)
+++ tags/1.4.22-rc3/ChangeLog Wed Sep  3 07:33:04 2008
@@ -1,0 +1,19093 @@
+2008-09-03  Russell Bryant <russell at digium.com>
+
+	* Asterisk 1.4.22-rc3 released.
+
+2008-09-02 23:47 +0000 [r140751]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_voicemail.c: After adding the context checking to
+	  app_voicemail for IMAP storage, I left out a crucial place to
+	  copy the context to the vm_state structure. This is the
+	  correction.
+
+2008-09-02 23:36 +0000 [r140670-140747]  Steve Murphy <murf at digium.com>
+
+	* main/cdr.c: I am turning the warnings generated in ast_cdr_free
+	  and post_cdr into verbose level 2 messages. Really, they matter
+	  little to end users. You either get the CDR's you wanted, or you
+	  don't, and it is a bug.
+
+	* main/channel.c: After reconsidering, with respect to 13409,
+	  ast_cdr_detach should be OK, better in fact, than ast_cdr_free,
+	  which generates lots of useless warnings that will undoubtably
+	  generate complaints.
+
+	* main/channel.c, main/pbx.c: (closes issue #13409) Reported by:
+	  tomaso Patches: asterisk-1.6.0-rc2-cdrmemleak.patch uploaded by
+	  tomaso (license 564) I basically spent the day, verifying that
+	  this patch solves the problem, and doesn't hurt in non-problem
+	  cases. Why valgrind did not plainly reveal this leak absolutely
+	  mystifies and stuns me. Many, many thanks to tomaso for finding
+	  and providing the fix.
+
+2008-09-02 18:14 +0000 [r140605]  Sean Bright <sean.bright at gmail.com>
+
+	* channels/chan_iax2.c: Make sure to use the correct length of the
+	  mohinterpret and mohsuggest buffers when copying configuration
+	  values. (closes issue #13336) Reported by:
+	  decryptus_proformatique Patches:
+	  chan_iax2_mohinterpret_mohsuggest_general_settings.patch uploaded
+	  by decryptus (license 555)
+
+2008-08-29 17:34 +0000 [r140417-140488]  Mark Michelson <mmichelson at digium.com>
+
+	* main/manager.c, apps/app_queue.c, channels/chan_iax2.c: After
+	  working on the ao2_containers branch, I noticed something a bit
+	  strange. In all cases where we provide a callback function to
+	  ao2_container_alloc, the callback function would only return 0 or
+	  CMP_MATCH. After inspecting the ao2_callback() code carefully, I
+	  found that if you're only looking for one specific item, then you
+	  should return CMP_MATCH | CMP_STOP. Otherwise, astobj2 will
+	  continue traversing the current bucket until the end searching
+	  for more matches. In cases like chan_iax2 where in 1.4, all the
+	  peers are shoved into a single bucket, this makes for potentially
+	  terrible performance since the entire bucket will be traversed
+	  even if the peer is one of the first ones come across in the
+	  bucket. All the changes I have made were for cases where the
+	  callback function defined was passed to ao2_container_alloc so
+	  that calls to ao2_find could find a unique instance of whatever
+	  object was being stored in the container.
+
+	* apps/app_voicemail.c: Add context checking when retrieving a
+	  vm_state. This was causing a problem for people who had
+	  identically named mailboxes in separate voicemail contexts. This
+	  commit affects IMAP storage only. (closes issue #13194) Reported
+	  by: moliveras Patches: 13194.patch uploaded by putnopvut (license
+	  60) Tested by: putnopvut, moliveras
+
+	* channels/chan_sip.c: Fix SIP's parsing so that if a port is
+	  specified in a string to Dial(), it is not ignored. (closes issue
+	  #13355) Reported by: acunningham Patches: 13355v2.patch uploaded
+	  by putnopvut (license 60) Tested by: acunningham
+
+2008-08-27 19:49 +0000 [r140299]  Mark Michelson <mmichelson at digium.com>
+
+	* channels/chan_sip.c: Fix tag checking in get_sip_pvt_byid_locked
+	  when in pedantic mode. The problem was that the wrong tags would
+	  be compared depending on the direction of the call. (closes issue
+	  #13353) Reported by: flefoll Patches:
+	  chan_sip.c.br14.139015.patch-refer-pedantic uploaded by flefoll
+	  (license 244)
+
+2008-08-26 16:49 +0000 [r140115]  Jeff Peeler <jpeeler at digium.com>
+
+	* channels/chan_dahdi.c: add HAVE_PRI if define around
+	  dahdi_close_pri_fd
+
+2008-08-26 16:07 +0000 [r140060]  Russell Bryant <russell at digium.com>
+
+	* channels/chan_sip.c: Fix some bogus scheduler usage in chan_sip.
+	  This code used the return value of a completely unrelated
+	  function to determine whether the scheduler should be run or not.
+	  This would have caused the scheduler to not run in cases where it
+	  should have. Also, leave a note about another scheduler issue
+	  that needs to be addressed at some point.
+
+2008-08-26 15:57 +0000 [r140056]  Jeff Peeler <jpeeler at digium.com>
+
+	* channels/chan_dahdi.c: (closes issue #12071) Reported by: tzafrir
+	  Patches: dahdi_close.diff uploaded by tzafrir (license 46) Tested
+	  by: tzafrir, jpeeler This patch fixes closing open file
+	  descriptors in the case of an error.
+
+2008-08-26 15:27 +0000 [r140051]  Russell Bryant <russell at digium.com>
+
+	* channels/chan_iax2.c: Fix a race condition with the IAX scheduler
+	  thread. A lock and condition are used here to allow newly
+	  scheduled tasks to wake up the scheduler just in case the new
+	  task needs to run sooner than the current wakeup time when the
+	  thread is sleeping. However, there was a race condition such that
+	  a newly scheduled task would not properly wake up the scheduler
+	  or affect the wake up period. The order of execution would have
+	  been: 1) Scheduler thread determines wake up time of N ms. 2)
+	  Another thread schedules a task and signals the condition, with
+	  an execution time of < N ms. 3) Scheduler thread locks and goes
+	  to sleep for N ms. By moving the sleep time determination to
+	  inside the critical section, this possibility is avoided.
+
+2008-08-26 15:22 +0000 [r140050]  Terry Wilson <twilson at digium.com>
+
+	* Makefile: sounds/Makefile installs sounds using the "new"
+	  language directory structure, but languageprefix needs to be set
+	  = yes for sounds in subdirectories (digits/1, etc.) to play as
+	  the correct language. Fix the generation of asterisk.conf to
+	  include languageprefix=yes
+
+2008-08-26 14:09 +0000 [r140029]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* channels/chan_dahdi.c: correct a file location in an error
+	  message
+
+2008-08-25 21:47 +0000 [r139927]  Jeff Peeler <jpeeler at digium.com>
+
+	* main/manager.c: Fix a typo I made. Lesson learned, apply the
+	  patch if one exists.
+
+2008-08-25 21:31 +0000 [r139909]  Sean Bright <sean.bright at gmail.com>
+
+	* build_tools/get_moduleinfo, build_tools/get_makeopts: Some
+	  versions of awk (nawk, for example) don't like empty regular
+	  expressions so be slightly more verbose. (closes issue #13374)
+	  Reported by: dougm Patches: 13374.diff uploaded by seanbright
+	  (license 71) Tested by: dougm
+
+2008-08-25 20:46 +0000 [r139869]  Terry Wilson <twilson at digium.com>
+
+	* channels/chan_sip.c: Make SIPADDHEADER() propagate indefinitely
+
+2008-08-25 15:52 +0000 [r139769]  Mark Michelson <mmichelson at digium.com>
+
+	* main/config.c: Fix the logic in config_text_file_save so that if
+	  an UpdateConfig manager action is issued and the file specified
+	  in DstFileName does not yet exist, an error is not returned.
+	  (closes issue #13341) Reported by: vadim Patches: 13341.patch
+	  uploaded by putnopvut (license 60) (with small modification from
+	  seanbright)
+
+2008-08-25 15:33 +0000 [r139764]  Steve Murphy <murf at digium.com>
+
+	* main/pbx.c, res/res_features.c: This patch reverts the changes
+	  made via 139347, and 139635, as users are seeing adverse
+	  difference. I will un-close 13251. Back to the drawing board/
+	  concept/ beginning/ whatever!
+
+2008-08-22 22:24 +0000 [r139635]  Steve Murphy <murf at digium.com>
+
+	* res/res_features.c: I found some problems with the code I
+	  committed earlier, when I merged them into trunk, so I'm coming
+	  back to clean up. And, in the process, I found an error in the
+	  code I added to trunk and 1.6.x, that I'll fix using this patch
+	  also.
+
+2008-08-22 21:36 +0000 [r139621]  Jeff Peeler <jpeeler at digium.com>
+
+	* main/manager.c: (closes issue #13359) Reported by: Laureano
+	  Patches: originate_channel_check.patch uploaded by Laureano
+	  (license 265)
+
+2008-08-22 19:45 +0000 [r139456-139553]  Mark Michelson <mmichelson at digium.com>
+
+	* include/asterisk/threadstorage.h: Fix compilation when
+	  DEBUG_THREAD_LOCALS is selected (closes issue #13298) Reported
+	  by: snuffy Patches: bug13298_20080822.diff uploaded by snuffy
+	  (license 35)
+
+	* main/frame.c: Remove show_frame_stats_deprecated since it is not
+	  used anywhere and causes build errors if building under dev-mode
+	  with TRACE_FRAMES selected in menuselect. (closes issue #13362)
+	  Reported by: snuffy
+
+	* channels/chan_iax2.c: Fix the build. Thanks, mvanbaak!
+
+	* channels/chan_iax2.c: Prevent a deadlock in chan_iax2 resulting
+	  from incorrect locking order between iax2_pvt and ast_channel
+	  structures. AST-13
+
+2008-08-21 23:39 +0000 [r139387]  Jeff Peeler <jpeeler at digium.com>
+
+	* channels/chan_dahdi.c: Fixes loop that could possibly never exit
+	  in the event of a channel never being able to be opened or
+	  specify after a restart. (closes issue #11017)
+
+2008-08-21 23:03 +0000 [r139347]  Steve Murphy <murf at digium.com>
+
+	* main/pbx.c, res/res_features.c: (closes issue #13251) Reported
+	  by: sergee Tested by: murf THis is a bold move for a static
+	  release fix, but I wouldn't have made it if I didn't feel
+	  confident (at least a *bit* confident) that it wouldn't mess
+	  everyone up. The reasoning goes something like this: 1. We simply
+	  cannot do anything with CDR's at the current point (in pbx.c,
+	  after the __ast_pbx_run loop). It's way too late to have any
+	  affect on the CDRs. The CDR is already posted and gone, and the
+	  remnants have been cleared. 2. I was very much afraid that moving
+	  the running of the 'h' extension down into the bridge code (where
+	  it would be now practical to do it), would result in a lot more
+	  calls to the 'h' exten, so I implemented it as another exten
+	  under another name, but found, to my pleasant surprise, that
+	  there was a 1:1 correspondence to the running of the 'h' exten in
+	  the pbx_run loop, and the new spot at the end of the bridge. So,
+	  I ifdef'd out the current 'h' loop, and moved it into the bridge
+	  code. The only difference I can see is the stuff about the
+	  AST_PBX_KEEPALIVE, and hopefully, if this is still an important
+	  decision point, I can replicate it if there are complaints. To be
+	  perfectly honest, the KEEPALIVE situation is not totally clear to
+	  me, and how it relates to a post-bridge situation is less clear.
+	  I suspect the users will point out everything in total clarity if
+	  this steps on anyone's toes! 3. I temporarily swap the bridge_cdr
+	  into the channel before running the 'h' exten, which makes it
+	  possible for users to edit the cdr before it goes out the door.
+	  And, of course, with the endbeforehexten config var set, the
+	  users can also get at the billsec/duration vals. After the h
+	  exten finishes, the cdr is swapped back and processing continues
+	  as normal. Please, all who deal with CDR's, please test this
+	  version of Asterisk, and file bug reports as appropriate!
+
+2008-08-21 10:11 +0000 [r139283]  Philippe Sultan <philippe.sultan at gmail.com>
+
+	* channels/chan_gtalk.c: Apply fix for issue #13310 to branch 1.4,
+	  too.
+
+2008-08-20 22:14 +0000 [r139213]  Russell Bryant <russell at digium.com>
+
+	* apps/app_chanspy.c: Fix a crash in the ChanSpy application. The
+	  issue here is that if you call ChanSpy and specify a spy group,
+	  and sit in the application long enough looping through the
+	  channel list, you will eventually run out of stack space and the
+	  application with exit with a seg fault. The backtrace was always
+	  inside of a harmless snprintf() call, so it was tricky to track
+	  down. However, it turned out that the call to snprintf() was just
+	  the biggest stack consumer in this code path, so it would always
+	  be the first one to hit the boundary. (closes issue #13338)
+	  Reported by: ruddy
+
+2008-08-20 19:52 +0000 [r139151]  Shaun Ruffell <sruffell at digium.com>
+
+	* codecs/codec_dahdi.c: Fix bug where the samples were not accurate
+	  when in G723 mode, which would cause the timestamp field of the
+	  RTP header to be invalid.
+
+2008-08-20 19:35 +0000 [r139145]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* channels/chan_dahdi.c, configure,
+	  include/asterisk/autoconfig.h.in, configure.ac: Backport support
+	  for Zaptel/DAHDI channel-level alarms from trunk/1.6, because not
+	  doing so just makes it difficult for people with channels that
+	  are in alarm when Asterisk starts up to get them going once the
+	  alarm is cleared (closes issue #12160) Reported by: tzafrir
+	  Patches: asterisk-chanalarms_14.patch uploaded by tzafrir
+	  (license 46) Tested by: tzafrir
+
+2008-08-20 17:14 +0000 [r139074]  Steve Murphy <murf at digium.com>
+
+	* main/cdr.c: (closes issue #13263) Reported by: brainy Tested by:
+	  murf The specialized reset routine is tromping on the flags field
+	  of the CDR. I made a change to not reset the DISABLED bit. This
+	  should get rid of this problem.
+
+2008-08-20 15:37 +0000 [r139015]  Mark Michelson <mmichelson at digium.com>
+
+	* channels/chan_sip.c: sip_read should properly handle a NULL
+	  return from sip_rtp_read. (closes issue #13257) Reported by:
+	  travishein
+
+2008-08-19 23:22 +0000 [r138949]  Jeff Peeler <jpeeler at digium.com>
+
+	* include/asterisk/dahdi_compat.h: add DAHDI_POLICY_WHEN_FULL
+	  compatability define for Zaptel
+
+2008-08-19 23:17 +0000 [r138942]  Mark Michelson <mmichelson at digium.com>
+
+	* channels/chan_agent.c: Reset agent_pvt variables back to the
+	  values in agents.conf (from what the corresponding channel
+	  variables were set to) when the agent logs out. (closes issue
+	  #13098) Reported by: davidw Patches:
+	  20080731__issue13098_agent_ackcall_not_reset.diff uploaded by
+	  bbryant (license 36) Tested by: davidw
+
+2008-08-19 22:56 +0000 [r138938]  Jeff Peeler <jpeeler at digium.com>
+
+	* channels/chan_dahdi.c: Add configuration option to
+	  chan_dahdi.conf to allow buffering policy and number of buffers
+	  to be configured per channel. Syntax: buffers=<num of
+	  buffers>,<policy> Where the number of buffers is some
+	  non-negative integer and the policy is either "full", "half", or
+	  "immediate".
+
+2008-08-19 18:50 +0000 [r138685-138886]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_chanspy.c: Add a lock and unlock prior to the
+	  destruction of the chanspy_ds lock to ensure that no other
+	  threads still have it locked. While this should not happen under
+	  normal circumstances, it appears that if the spyer and spyee hang
+	  up at nearly the same time, the following may occur. 1.
+	  ast_channel_free is called on the spyee's channel. 2. The chanspy
+	  datastore is removed from the spyee's channel in
+	  ast_channel_free. 3. In the spyer's thread, the spyer attempts to
+	  remove and destroy the datastore from the spyee channel, but the
+	  datastore has already been removed in step 2, so the spyer
+	  continues in the code. 4. The spyee's thread continues and calls
+	  the datastore's destroy callback, chanspy_ds_destroy. This
+	  involves locking the chanspy_ds. 5. Now the spyer attempts to
+	  destroy the chanspy_ds lock. The problem is that in step 4, the
+	  spyee has locked this lock, meaning that the spyer is attempting
+	  to destroy a lock which is currently locked by another thread.
+	  The backtrace provided in issue #12969 supports the idea that
+	  this is possible (and has even occurred). This commit does not
+	  close the issue, but should help in preventing one type of crash
+	  associated with the use of app_chanspy.
+
+	* apps/app_queue.c: Change the inequalities used in app_queue with
+	  regards to timeouts from being strict to non-strict for more
+	  accuracy. (closes issue #13239) Reported by: atis Patches:
+	  app_queue_timeouts_v2.patch uploaded by atis (license 242)
+
+2008-08-18 16:57 +0000 [r138663]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* codecs/codec_dahdi.c: look for transcoder in proper place based
+	  on build against Zaptel or DAHDI
+
+2008-08-18 11:57 +0000 [r138569]  Sean Bright <sean.bright at gmail.com>
+
+	* channels/chan_dahdi.c: You know what's awesome? Code that
+	  compiles... ;)
+
+2008-08-18 02:05 +0000 [r138516]  Jeff Peeler <jpeeler at digium.com>
+
+	* channels/chan_dahdi.c: fix compilation warnings
+
+2008-08-16 01:12 +0000 [r138309-138360]  Jeff Peeler <jpeeler at digium.com>
+
+	* channels/chan_dahdi.c: fixes use count to properly decrement if
+	  an active dahdi channel is destroyed allowing module to be
+	  unloaded
+
+	* channels/chan_dahdi.c: add forgotten locks around ss_thread_count
+	  in ss_thread for dahdi restart
+
+2008-08-15 22:33 +0000 [r138258]  Tilghman Lesher <tlesher at digium.com>
+
+	* channels/chan_sip.c, configs/sip.conf.sample: More fixes for
+	  realtime peers. (closes issue #12921) Reported by: Nuitari
+	  Patches: 20080804__bug12921.diff.txt uploaded by Corydon76
+	  (license 14) 20080815__bug12921.diff.txt uploaded by Corydon76
+	  (license 14) Tested by: Corydon76
+
+2008-08-15 21:28 +0000 [r138119-138238]  Jeff Peeler <jpeeler at digium.com>
+
+	* channels/chan_dahdi.c: initialize condition variable
+	  ss_thread_complete using ast_cond_init
+
+	* channels/chan_dahdi.c: declared static mutexes using
+	  AST_MUTEX_DEFINE_STATIC macro
+
+	* channels/chan_dahdi.c: Fixes the dahdi restart functionality.
+	  Dahdi restart allows one to restart all DAHDI channels, even if
+	  they are currently in use. This is different from unloading and
+	  then loading the module since unloading requires the use count to
+	  be zero. Reloading the module is different in that the signalling
+	  is not changed from what it was originally configured. Also, this
+	  fixes not closing all the file descriptors for D-channels upon
+	  module unload (which would prevent loading the module
+	  afterwards). (closes issue #11017)
+
+2008-08-15 15:07 +0000 [r138027]  Russell Bryant <russell at digium.com>
+
+	* main/autoservice.c: Ensure that when a hangup occurs in
+	  autoservice, that a hangup frame gets properly deferred to be
+	  read from the channel owner when it gets taken out of
+	  autoservice. (closes issue #12874) Reported by: dimas Patches:
+	  v1-12874.patch uploaded by dimas (license 88)
+
+2008-08-15 14:51 +0000 [r137847-138023]  Tilghman Lesher <tlesher at digium.com>
+
+	* funcs/func_strings.c: Additional check for more string specifiers
+	  than arguments. (closes issue #13299) Reported by: adomjan
+	  Patches: 20080813__bug13299.diff.txt uploaded by Corydon76
+	  (license 14) func_strings.c-sprintf.patch uploaded by adomjan
+	  (license 487) Tested by: adomjan
+
+	* channels/chan_dahdi.c: Oops, wrong direction
+
+	* channels/chan_dahdi.c: When creating the secondary subchannel
+	  name, it is necessary to compare to the existing channel name
+	  without the "Zap/" or "DAHDI/" prefix, since our test string is
+	  also without that prefix. (closes issue #13027) Reported by:
+	  dferrer Patches: chan_zap-1.4.21.1_fix2.patch uploaded by dferrer
+	  (license 525) (Slightly modified by me, to compensate for both
+	  names)
+
+2008-08-14 14:05 +0000 [r137731]  Russell Bryant <russell at digium.com>
+
+	* configs/sip.conf.sample: Comments in this config file were
+	  aligned only if your tab size was set to 8. So, convert tabs to
+	  spaces so that things should be aligned regardless of what tab
+	  size you use in your editor.
+
+2008-08-14 02:03 +0000 [r137677-137679]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* Zaptel-to-DAHDI.txt: forgot one module name that changed
+
+	* include/asterisk/dahdi_compat.h, channels/chan_dahdi.c,
+	  build_tools/menuselect-deps.in, configure, configure.ac,
+	  codecs/codec_dahdi.c: add support for Zaptel versions that
+	  contain the new transcoder interface
+
+2008-08-13 21:35 +0000 [r137580]  Jeff Peeler <jpeeler at digium.com>
+
+	* channels/chan_dahdi.c: Register DAHDISendKeypadFacility
+	  application if dahdi_chan_mode is set to DAHDI + Zap. Mark
+	  ZapSendKeypadFacility application as deprecated on usage.
+
+2008-08-13 20:46 +0000 [r137527-137530]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* Zaptel-to-DAHDI.txt (added): add document describing what users
+	  will need to be aware of when upgrading to this version and using
+	  DAHDI
+
+	* apps/app_meetme.c: remove some more chan_zap references
+
+	* doc/asterisk-conf.txt, channels/chan_dahdi.c: document
+	  dahdichanname option in doc/asterisk-conf.txt make chan_dahdi
+	  read its configuration from zapata.conf if dahdichanname has been
+	  set to 'no'
+
+2008-08-13 14:33 +0000 [r137348-137405]  Sean Bright <sean.bright at gmail.com>
+
+	* doc/cdrdriver.txt: Update docs to reflect the change to cdr_tds
+
+	* cdr/cdr_tds.c: Bring cdr_tds in line with the other CDR backends
+	  and have it try to store CDR(userfield) if it is set. The new
+	  behavior is to check for the userfield column on module load, and
+	  if it exists, we will store CDR(userfield) when CDRs are written.
+	  A similar patch already went into trunk and 1.6.0. (closes issue
+	  #13290) Reported by: falves11
+
+2008-08-11 13:33 +0000 [r137188]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* apps/app_meetme.c: convert this module to be able to handle DAHDI
+	  or Zaptel (reported on asterisk-users, don't know how this got
+	  missed before)
+
+2008-08-11 00:20 +0000 [r137138]  Tilghman Lesher <tlesher at digium.com>
+
+	* res/res_odbc.c: Deallocate database connection handle on
+	  disconnect, as we allocate another one on connect. (closes issue
+	  #13271) Reported by: dveiga
+
+2008-08-09 17:11 +0000 [r136999]  Russell Bryant <russell at digium.com>
+
+	* configure, configure.ac: Ensure PBX_DAHDI_TRANSCODE will evaluate
+	  to 0 if not found instead of empty. pointed out by tzafrir on
+	  #asterisk-dev
+
+2008-08-09 15:25 +0000 [r136946]  Tilghman Lesher <tlesher at digium.com>
+
+	* /, include/asterisk/compat.h, include/asterisk/astobj2.h: Merged
+	  revisions 136945 via svnmerge from
+	  https://origsvn.digium.com/svn/asterisk/branches/1.2 ........
+	  r136945 | tilghman | 2008-08-09 10:24:36 -0500 (Sat, 09 Aug 2008)
+	  | 2 lines Regression fixes for Solaris ........
+
+2008-08-08 00:15 +0000 [r136726]  Steve Murphy <murf at digium.com>
+
+	* pbx/ael/ael-test/ref.ael-test8, pbx/ael/ael-test/ref.ael-test18,
+	  pbx/ael/ael-test/ref.ael-vtest13,
+	  pbx/ael/ael-test/ref.ael-ntest10, pbx/pbx_ael.c,
+	  include/asterisk/ael_structs.h: (closes issue #13236) Reported
+	  by: korihor Wow, this one was a challenge! I regrouped and ran a
+	  new strategy for setting the ~~MACRO~~ value; I set it once per
+	  extension, up near the top. It is only set if there is a switch
+	  in the extension. So, I had to put in a chunk of code to detect a
+	  switch in the pval tree. I moved the code to insert the set of
+	  ~~exten~~ up to the beginning of the gen_prios routine, instead
+	  of down in the switch code. I learned that I have to push the
+	  detection of the switches down into the code, so everywhere I
+	  create a new exten in gen_prios, I make sure to pass onto it the
+	  values of the mother_exten first, and the exten next. I had to
+	  add a couple fields to the exten struct to accomplish this, in
+	  the ael_structs.h file. The checked field makes it so we don't
+	  repeat the switch search if it's been done. I also updated the
+	  regressions.
+
+2008-08-07 18:25 +0000 [r136560]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* build_tools/menuselect-deps.in, configure, configure.ac: change
+	  the required dependency for codec_dahdi to only be satisfied by
+	  DAHDI and not Zaptel, as the new transcoder interface is only in
+	  DAHDI
+
+2008-08-07 18:14 +0000 [r136544]  Shaun Ruffell <sruffell at digium.com>
+
+	* codecs/codec_dahdi.c: Updated codec_dahdi to use the new
+	  transcoder interface in the first DAHDI release. Codec dahdi no
+	  longer functions with the transcoder interface in zaptel at this
+	  time (which the last zaptel release was 1.4.11). NOTE: Still
+	  needs an update to the configure script to make sure that
+	  codec_dahdi is only built if the new transcoder interface is
+	  present in the drivers. (Issue: DAHDI-42)
+
+2008-08-07 16:50 +0000 [r136488]  Tilghman Lesher <tlesher at digium.com>
+
+	* apps/app_queue.c: Update persistent state on all exit conditions.
+	  (closes issue #12916) Reported by: sgenyuk Patches:
+	  app_queue.patch.txt uploaded by neutrino88 (license 297) Tested
+	  by: sgenyuk, aragon
+
+2008-08-07 16:30 +0000 [r136404-136484]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* include/asterisk/doxyref.h: add a raw list of all libraries that
+	  any part of Asterisk links directly to
+
+	* apps/app_voicemail.c: work around a bug in gcc-4.2.3 that
+	  incorrectly ignores the casting away of 'const' for pointers when
+	  the developer knows it is safe to do so
+
+	* Makefile: remove config.cache during distclean, in case the user
+	  is using autoconf caching
+
+2008-08-07 01:31 +0000 [r136304-136348]  Tilghman Lesher <tlesher at digium.com>
+
+	* channels/chan_dahdi.c: Also, parse
+	  useincomingcalleridonzaptransfer (and add appropriate deprecation
+	  warnings).
+
+	* channels/chan_dahdi.c: For backwards compatibility with previous
+	  1.4 versions which used "zapchan" in users.conf, ensure that we
+	  still support it.
+
+2008-08-06 21:18 +0000 [r136241]  Richard Mudgett <rmudgett at digium.com>
+
+	* channels/misdn_config.c, channels/chan_misdn.c,
+	  configs/misdn.conf.sample: * The allowed_bearers setting in
+	  misdn.conf misspelled one of its options: digital_restricted. *
+	  Fixed some other spelling errors and typos.
+
+2008-08-06 20:42 +0000 [r136238]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_queue.c: We only need to unregister the QueueStatus
+	  manager command once on an unload
+
+2008-08-06 20:14 +0000 [r136190]  Tilghman Lesher <tlesher at digium.com>
+
+	* contrib/init.d/rc.redhat.asterisk: -C option takes a filename,
+	  not a directory path. (closes issue #13007) Reported by:
+	  klaus3000
+
+2008-08-06 18:58 +0000 [r136168]  Russell Bryant <russell at digium.com>
+
+	* Makefile: Remove the use of --no-print-directory when compiling
+	  subdirectories. This allows vim :make functionality to work
+	  properly when errors have occurred in the build. Without printing
+	  the directories, vim did not know how to find the file that the
+	  error occurred in. If the extra bit of build noise annoys anyone,
+	  just let me know, and I'll make this optional.
+
+2008-08-06 15:58 +0000 [r136062]  Mark Michelson <mmichelson at digium.com>
+
+	* main/rtp.c, channels/chan_skinny.c: Since adding the
+	  AST_CONTROL_SRCUPDATE frame type, there are places where
+	  ast_rtp_new_source may be called where the tech_pvt of a channel
+	  may not yet have an rtp structure allocated. This caused a crash
+	  in chan_skinny, which was fixed earlier, but now the same crash
+	  has been reported against chan_h323 as well. It seems that the
+	  best solution is to modify ast_rtp_new_source to not attempt to
+	  set the marker bit if the rtp structure passed in is NULL. This
+	  change to ast_rtp_new_source also allows the removal of what is
+	  now a redundant pointer check from chan_skinny. (closes issue
+	  #13247) Reported by: pj
+
+2008-08-06 03:53 +0000 [r135899-135949]  Tilghman Lesher <tlesher at digium.com>
+
+	* main/channel.c: Fix a longstanding bug in channel walking logic,
+	  and fix the explanation to make sense. (Closes issue #13124)
+
+	* main/translate.c: Since powerof() can return an error condition,
+	  it's foolhardy not to detect and deal with that condition.
+	  (Related to issue #13240)
+
+	* include/asterisk/threadstorage.h, include/asterisk/utils.h: 1)
+	  Bugfix for debugging code 2) Reduce compiler warnings for another
+	  section of debugging code (Closes issue #13237)
+
+2008-08-06 00:29 +0000 [r135841-135850]  Mark Michelson <mmichelson at digium.com>
+
+	* /: Remove properties that should not be here
+
+	* apps/app_skel.c: Revert inadvertent changes to app_skel that
+	  occurred when I was testing for a memory leak
+
+	* include/asterisk/abstract_jb.h, main/channel.c, /,
+	  apps/app_skel.c, main/abstract_jb.c, main/fixedjitterbuf.h:
+	  Merging the issue11259 branch. The purpose of this branch was to
+	  take into account "burps" which could cause jitterbuffers to
+	  misbehave. One such example is if the L option to Dial() were
+	  used to inject audio into a bridged conversation at regular
+	  intervals. Since the audio here was not passed through the
+	  jitterbuffer, it would cause a gap in the jitterbuffer's
+	  timestamps which would cause a frames to be dropped for a brief
+	  period. Now ast_generic_bridge will empty and reset the
+	  jitterbuffer each time it is called. This causes injected audio
+	  to be handled properly. ast_generic_bridge also will empty and
+	  reset the jitterbuffer if it receives an AST_CONTROL_SRCUPDATE
+	  frame since the change in audio source could negatively affect
+	  the jitterbuffer. All of this was made possible by adding a new
+	  public API call to the abstract_jb called ast_jb_empty_and_reset.
+	  (closes issue #11259) Reported by: plack Tested by: putnopvut
+
+2008-08-05 23:13 +0000 [r135799]  Steve Murphy <murf at digium.com>
+
+	* apps/app_dial.c, main/cdr.c, main/channel.c, res/res_features.c,
+	  include/asterisk/cdr.h: (closes issue #12982) Reported by: bcnit
+	  Tested by: murf I discovered that also, in the previous bug fixes
+	  and changes, the cdr.conf 'unanswered' option is not being
+	  obeyed, so I fixed this. And, yes, there are two 'answer' times
+	  involved in this scenario, and I would agree with you, that the
+	  first answer time is the time that should appear in the CDR. (the
+	  second 'answer' time is the time that the bridge was begun). I
+	  made the necessary adjustments, recording the first answer time
+	  into the peer cdr, and then using that to override the bridge
+	  cdr's value. To get the 'unanswered' CDRs to appear, I purposely
+	  output them, using the dial cmd to mark them as DIALED (with a
+	  new flag), and outputting them if they bear that flag, and you
+	  are in the right mode. I also corrected one small mention of the
+	  Zap device to equally consider the dahdi device. I heavily tested
+	  10-sec-wait macros in dial, and without the macro call; I tested
+	  hangups while the macro was running vs. letting the macro
+	  complete and the bridge form. Looks OK. Removed all the
+	  instrumentation and debug.
+
+2008-08-05 21:34 +0000 [r135747]  Tilghman Lesher <tlesher at digium.com>
+
+	* channels/chan_iax2.c: In a conversion to use ast_strlen_zero, the
+	  meaning of the flag IAX_HASCALLERID was perverted. This change
+	  reverts IAX2 to the original meaning, which was, that the
+	  callerid set on the client should be overridden on the server,
+	  even if that means the resulting callerid is blank. In other
+	  words, if you set "callerid=" in the IAX config, then the
+	  callerid should be overridden to blank, even if set on the
+	  client. Note that there's a distinction, even on realtime,
+	  between the field not existing (NULL in databases) and the field
+	  existing, but set to blank (override callerid to blank).
+
+2008-08-05 13:25 +0000 [r135597]  Sean Bright <sean.bright at gmail.com>
+
+	* main/cli.c: Use PATH_MAX for filenames
+
+2008-08-04 20:15 +0000 [r135536]  Russell Bryant <russell at digium.com>
+
+	* configs/chan_dahdi.conf.sample: fix a config sample typo
+
+2008-08-04 17:07 +0000 [r135479-135482]  Tilghman Lesher <tlesher at digium.com>
+
+	* contrib/init.d/rc.mandrake.asterisk: Define ASTSBINDIR for script
+
+	* apps/app_voicemail.c: Memory leak on unload (closes issue #13231)
+	  Reported by: eliel Patches: app_voicemail.leak.patch uploaded by
+	  eliel (license 64)
+
+2008-08-04 16:26 +0000 [r135473]  Russell Bryant <russell at digium.com>
+
+	* configs/chan_dahdi.conf.sample: Add a minor clarification to the
+	  documentation of mohinterpret and mohsuggest
+
+2008-08-01 11:43 +0000 [r135055-135058]  Michiel van Baak <michiel at vanbaak.info>
+
+	* apps/app_ices.c: make app_ices compile on OpenBSD.
+
+	* channels/chan_skinny.c: fix some potential deadlocks in
+	  chan_skinny (closes issue #13215) Reported by: qwell Patches:
+	  2008080100_bug13215.diff.txt uploaded by mvanbaak (license 7)
+	  Tested by: mvanbaak
+
+2008-07-31 22:18 +0000 [r134983]  Kevin P. Fleming <kpfleming at digium.com>
+
+	* main/http.c: accomodate users who seem to lack a sense of humor
+	  :-)
+
+2008-07-31 21:53 +0000 [r134976]  Tilghman Lesher <tlesher at digium.com>
+
+	* sample.call, main/manager.c, pbx/pbx_spool.c: Specify codecs in
+	  callfiles and manager, to allow video calls to be set up from
+	  callfiles and AMI. (closes issue #9531) Reported by: Geisj
+	  Patches: 20080715__bug9531__1.4.diff.txt uploaded by Corydon76
+	  (license 14) 20080715__bug9531__1.6.0.diff.txt uploaded by
+	  Corydon76 (license 14) Tested by: Corydon76
+
+2008-07-31 19:37 +0000 [r134915]  Russell Bryant <russell at digium.com>
+
+	* apps/app_ices.c: Get app_ices working again (closes issue #12981)
+	  Reported by: dlogan Patches:
+	  20080709__app_ices_v2_update_trunk.diff uploaded by bbryant
+	  (license 36) 20080709__app_ices_v2_update_14.diff uploaded by
+	  bbryant (license 36) Tested by: bbryant
+
+2008-07-31 19:23 +0000 [r134883]  Steve Murphy <murf at digium.com>
+
+	* res/res_features.c: (closes issue #11849) Reported by: greyvoip
+	  Tested by: murf OK, a few days of debugging, a bunch of
+	  instrumentation in chan_sip, main/channel.c, main/pbx.c, etc. and
+	  5 solid notebook pages of notes later, I have made the small
+	  tweek necc. to get the start time right on the second CDR when: A
+	  Calls B B answ. A hits Xfer button on sip phone, A dials C and
+	  hits the OK button, A hangs up C answers ringing phone B and C
+	  converse B and/or C hangs up But does not harm the scenario
+	  where: A Calls B B answ. B hits xfer button on sip phone, B dials
+	  C and hits the OK button, B hangs up C answers ringing phone A
+	  and C converse A and/or C hangs up The difference in start times
+	  on the second CDR is because of a Masquerade on the B channel
+	  when the xfer number is sent. It ends up replacing the CDR on the
+	  B channel with a duplicate, which ends up getting tossed out. We
+	  keep a pointer to the first CDR, and update *that* after the
+	  bridge closes. But, only if the CDR has changed. I hope this
+	  change is specific enough not to muck up any current CDR-based
+	  apps. In my defence, I assert that the previous information was
+	  wrong, and this change fixes it, and possibly other similar
+	  scenarios. I wonder if I should be doing the same thing for the
+	  channel, as I did for the peer, but I can't think of a scenario
+	  this might affect. I leave it, then, as an exersize for the
+	  users, to find the scenario where the chan's CDR changes and
+	  loses the proper start time.
+
+2008-07-31 16:45 +0000 [r134814]  Russell Bryant <russell at digium.com>
+
+	* channels/iax2-parser.c: In case we have some processing threads
+	  that free more frames than they allocate, do not let the frame
+	  cache grow forever. (closes issue #13160) Reported by: tavius
+	  Tested by: tavius, russell
+
+2008-07-31 15:56 +0000 [r134758]  Mark Michelson <mmichelson at digium.com>
+
+	* apps/app_queue.c: Add more timeout checks into app_queue,
+	  specifically targeting areas where an unknown and potentially
+	  long time has just elapsed. Also added a check to try_calling()
+	  to return early if the timeout has elapsed instead of potentially
+	  setting a negative timeout for the call (thus making it have *no*
+	  timeout at all). (closes issue #13186) Reported by:
+	  miquel_cabrespina Patches: 13186.diff uploaded by putnopvut
+	  (license 60) Tested by: miquel_cabrespina
+
+2008-07-30 22:39 +0000 [r134704]  Tilghman Lesher <tlesher at digium.com>
+
+	* main/sched.c, include/asterisk/sched.h: Oops, wrong define
+
+2008-07-30 22:02 +0000 [r134652]  Steve Murphy <murf at digium.com>
+
+	* pbx/pbx_ael.c: (closes issue #13197) Reported by: pj (closes
+	  issue #13051) Reported by: pj This patch substitutes commas in
+	  the expr supplied to the if () statement, as in if ( expr ) ...
+	  This solves both the bugs above, and makes the source symmetric
+	  with switch statements, which were earlier reported to need this
+	  sort of treatment. I tested this using the examples, both for the
+	  compiler and at run time. Looks good.
+
+2008-07-30 21:38 +0000 [r134649]  Tilghman Lesher <tlesher at digium.com>
+
+	* configure, configure.ac: Qwell pointed out, via IRC, that the
+	  previous fix only worked when explicitly set. When nothing is
+	  set, and the option is implied, it breaks, because configure sets
+	  the prefix to 'NONE'. Fixing.
+
+2008-07-30 20:37 +0000 [r134540-134595]  Russell Bryant <russell at digium.com>
+
+	* pbx/pbx_dundi.c: Reduce stack consumption by 12.5% of the max
+	  stack size to fix a crash when compiled with LOW_MEMORY. (closes
+	  issue #13154) Reported by: edantie
+
+	* funcs/func_curl.c: Fix a memory leak in func_curl. Every thread
+	  that used this function leaked an allocation the size of a
+	  pointer. (reported by jmls in #asterisk-dev)
+
+2008-07-30 19:47 +0000 [r134480-134536]  Tilghman Lesher <tlesher at digium.com>
+
+	* configure, configure.ac: Only override sysconfdir and mandir when
+	  prefix=/usr (closes issue #13093) Reported by: pabelanger
+
+	* res/res_agi.c: launch_netscript sometimes returns -1, which fails
+	  to set AGISTATUS. Map failure to -1, so that AGISTATUS is always

[... 18311 lines stripped ...]



More information about the asterisk-commits mailing list