[asterisk-commits] tilghman: branch group/codec_bits r114380 - in /team/group/codec_bits: ./ app...

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Mon Apr 21 12:34:07 CDT 2008


Author: tilghman
Date: Mon Apr 21 12:33:59 2008
New Revision: 114380

URL: http://svn.digium.com/view/asterisk?view=rev&rev=114380
Log:
Merged revisions 114022,114024,114027,114030,114033,114036,114042,114046,114049,114052,114061,114064,114067,114073,114077,114080,114084-114085,114088,114090,114092-114093,114096,114098,114101,114104,114107,114109,114113,114115,114118,114121,114124,114127,114131,114134,114139,114141,114143,114146,114149-114152,114165,114168,114172,114174-114175,114181-114183,114185,114187-114188,114190,114192,114194,114196,114199,114201-114202,114205,114208,114212,114227,114229,114231,114233,114243,114246,114249,114253-114254,114259,114261,114271,114276,114279,114285,114295,114298,114300,114303,114314,114320,114323,114325,114327 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

................
r114022 | file | 2008-04-10 08:28:30 -0500 (Thu, 10 Apr 2008) | 14 lines

Merged revisions 114021 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114021 | file | 2008-04-10 10:27:11 -0300 (Thu, 10 Apr 2008) | 6 lines

Don't add custom URI options if they don't exist OR they are empty.
(closes issue #12407)
Reported by: homesick
Patches:
      uri_options-1.4.diff uploaded by homesick (license 91)

........

................
r114024 | file | 2008-04-10 08:45:45 -0500 (Thu, 10 Apr 2008) | 4 lines

Fix spelling of existent in a few places.
(closes issue #12409)
Reported by: candlerb

................
r114027 | file | 2008-04-10 09:53:19 -0500 (Thu, 10 Apr 2008) | 6 lines

Don't hardcode ru into the digits filename so that languageprefix can work.
(closes issue #12404)
Reported by: IgorG
Patches:
      voicemail_ru_hardcoded-v1.patch uploaded by IgorG (license 20)

................
r114030 | file | 2008-04-10 10:10:47 -0500 (Thu, 10 Apr 2008) | 14 lines

Merged revisions 114029 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114029 | file | 2008-04-10 12:09:04 -0300 (Thu, 10 Apr 2008) | 6 lines

Create the directory where name recordings will go if it does not exist.
(closes issue #12311)
Reported by: rkeene
Patches:
      12311-mkdir.diff uploaded by qwell (license 4)

........

................
r114033 | file | 2008-04-10 10:59:13 -0500 (Thu, 10 Apr 2008) | 13 lines

Blocked revisions 114032 via svnmerge

........
r114032 | file | 2008-04-10 12:58:39 -0300 (Thu, 10 Apr 2008) | 6 lines

Forgot the 1.4 branch for russian language fix.
(closes issue #12404)
Reported by: IgorG
Patches:
      voicemail_ru_hardcoded-v1.patch uploaded by IgorG (license 20)

........

................
r114036 | qwell | 2008-04-10 12:27:16 -0500 (Thu, 10 Apr 2008) | 18 lines

Merged revisions 114035 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114035 | qwell | 2008-04-10 12:26:10 -0500 (Thu, 10 Apr 2008) | 10 lines

Only try to prefix language if we are not using an absolute path (suffix it otherwise).

en/var/lib/asterisk/sounds/blah.gsm is a very silly path.

(closes issue #12379)
Reported by: kuj
Patches:
      12379-absolutepath.diff uploaded by qwell (license 4)
Tested by: kuj, qwell

........

................
r114042 | tilghman | 2008-04-10 14:04:29 -0500 (Thu, 10 Apr 2008) | 7 lines

The hydra grows yet another head...
(closes issue #12401)
 Reported by: davevg
 Patches: 
       astcli.diff2 uploaded by davevg (license 209)
 Tested by: davevg, Corydon76

................
r114046 | mmichelson | 2008-04-10 14:58:36 -0500 (Thu, 10 Apr 2008) | 14 lines

Merged revisions 114045 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114045 | mmichelson | 2008-04-10 14:55:33 -0500 (Thu, 10 Apr 2008) | 6 lines

Be sure that we're not about to set bridgepvt NULL prior to dereferencing it.

(closes issue #11775)
Reported by: fujin


........

................
r114049 | file | 2008-04-10 15:28:40 -0500 (Thu, 10 Apr 2008) | 2 lines

A 'b' option has been added which causes chan_local to return the actual channel that is behind it when queried. This is useful for transfer scenarios as the actual channel will be transferred, not the Local channel. If you have been using Local channels as queue members and having issues when the agent did a blind transfer this option may solve the issue.

................
r114052 | mmichelson | 2008-04-10 17:02:32 -0500 (Thu, 10 Apr 2008) | 11 lines

Merged revisions 114051 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114051 | mmichelson | 2008-04-10 15:59:49 -0500 (Thu, 10 Apr 2008) | 3 lines

Fix 1.4 build when LOW_MEMORY is enabled.


........

................
r114061 | tilghman | 2008-04-11 09:54:22 -0500 (Fri, 11 Apr 2008) | 6 lines

Errors are all greater than 0
(closes issue #12422)
 Reported by: nito
 Patches: 
       res_config_ldap_result_check_patch.diff uploaded by nito (license 340)

................
r114064 | mmichelson | 2008-04-11 10:49:35 -0500 (Fri, 11 Apr 2008) | 19 lines

Merged revisions 114063 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114063 | mmichelson | 2008-04-11 10:44:28 -0500 (Fri, 11 Apr 2008) | 11 lines

Fix a race condition that may happen between a sip hangup
and a "core show channel" command. This patch adds locking
to prevent the resulting crash.

(closes issue #12155)
Reported by: tsearle
Patches:
      show_channels_crash2.patch uploaded by tsearle (license 373)
Tested by: tsearle


........

................
r114067 | twilson | 2008-04-11 16:04:46 -0500 (Fri, 11 Apr 2008) | 3 lines

Fix the fact that global_variables 1) weren't being updated on reload (thanks for the report, Doug), and 2) weren't actually being appended to the list of profile variables because build_profile was called before the list was populated. Also needed to free the contents returned by load_file().


................
r114073 | qwell | 2008-04-11 16:39:44 -0500 (Fri, 11 Apr 2008) | 13 lines

Blocked revisions 114072 via svnmerge

Already fixed here.

........
r114072 | qwell | 2008-04-11 16:35:16 -0500 (Fri, 11 Apr 2008) | 4 lines

It's possible that a channel can have an async goto on the successful execution of an application as well.

Closes issue #12172.

........

................
r114077 | dbailey | 2008-04-11 17:09:04 -0500 (Fri, 11 Apr 2008) | 2 lines

Change the number of line keys per registration from 2 to 1 

................
r114080 | twilson | 2008-04-11 17:23:34 -0500 (Fri, 11 Apr 2008) | 2 lines

Make sure that ${LINE} is set even if linenumber is not set in users.conf

................
r114084 | twilson | 2008-04-11 17:48:52 -0500 (Fri, 11 Apr 2008) | 15 lines

Merged revisions 114083 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114083 | twilson | 2008-04-11 17:32:51 -0500 (Fri, 11 Apr 2008) | 7 lines

Several places in the code called find_callno() (which releases the lock on the pvt structure) and then immediately locked the call and did things with it. Unfortunately, the call can disappear between the find_callno and the lock, causing Bad Stuff(tm) to happen.

Added find_callno_locked() function to return the callno withtout unlocking for instances that it is needed.

(issue #12400)
Reported by: ztel

........

................
r114085 | tilghman | 2008-04-11 18:12:16 -0500 (Fri, 11 Apr 2008) | 7 lines

Use the correct function for free'ing objects, and maybe we won't crash.
(closes issue #12163)
 Reported by: gservat
 Patches: 
       20080411__bug12163.diff.txt uploaded by Corydon76 (license 14)
 Tested by: gservat

................
r114088 | tilghman | 2008-04-11 18:21:54 -0500 (Fri, 11 Apr 2008) | 3 lines

Make the sample config match the contributed LDAP schema
(Closes issue #12421)

................
r114090 | tilghman | 2008-04-11 18:26:56 -0500 (Fri, 11 Apr 2008) | 3 lines

If any field is not null, but has no default, then it must be set or the insert will fail.
(Closes issue #12285)

................
r114092 | mattf | 2008-04-12 11:13:25 -0500 (Sat, 12 Apr 2008) | 1 line

Make sure we start incoming calls on SS7 with echo cancellation enabled.  Also make sure when completing a COT we call ss7_start_call with the proper locks held.  Lastly, make sure if we fail to get a channel from zt_new that we don't assume it's there.
................
r114093 | mattf | 2008-04-12 11:21:29 -0500 (Sat, 12 Apr 2008) | 1 line

Make sure linkset is locked exiting ss7_start_call
................
r114096 | tilghman | 2008-04-13 09:35:43 -0500 (Sun, 13 Apr 2008) | 3 lines

Use ast_mkdir instead of mkdir
(Closes issue #12430)

................
r114098 | tilghman | 2008-04-13 21:55:41 -0500 (Sun, 13 Apr 2008) | 3 lines

Add tab command-line completion
(Closes issue #12428)

................
r114101 | file | 2008-04-14 08:53:33 -0500 (Mon, 14 Apr 2008) | 12 lines

Merged revisions 114100 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114100 | file | 2008-04-14 10:52:49 -0300 (Mon, 14 Apr 2008) | 4 lines

Don't change the SSRC when a new source comes into play, this might happen quite often and depending on the remote side... they might not like this.
(closes issue #12353)
Reported by: dimas

........

................
r114104 | file | 2008-04-14 09:53:33 -0500 (Mon, 14 Apr 2008) | 12 lines

Merged revisions 114103 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114103 | file | 2008-04-14 11:52:46 -0300 (Mon, 14 Apr 2008) | 4 lines

It is possible for the remote side to say they want T38 but not give any capabilities.
(closes issue #12414)
Reported by: MVF

........

................
r114107 | mmichelson | 2008-04-14 10:01:36 -0500 (Mon, 14 Apr 2008) | 13 lines

Merged revisions 114106 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114106 | mmichelson | 2008-04-14 09:58:02 -0500 (Mon, 14 Apr 2008) | 5 lines

Save a local copy of the generate callback prior to unlocking the channel in
case the generate callback goes NULL on us after the channel is unlocked. Thanks
to Russell for pointing this need out to me.


........

................
r114109 | file | 2008-04-14 10:36:02 -0500 (Mon, 14 Apr 2008) | 2 lines

During hangup it is possible for p->chan or p->owner to be NULL, so just return what the channel is bridged to instead of what they are *really* bridged to. Thanks Matt Nicholson!

................
r114113 | mmichelson | 2008-04-14 11:25:09 -0500 (Mon, 14 Apr 2008) | 17 lines

Merged revisions 114112 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114112 | mmichelson | 2008-04-14 11:24:22 -0500 (Mon, 14 Apr 2008) | 9 lines

If the datastore has been moved to another channel due to a masquerade, then
freeing the datastore here causes an eventual double free when the new channel
hangs up. We should only free the datastore if we were able to successfully remove
it from the channel we are referencing (i.e. the datastore was not moved).

(closes issue #12359)
Reported by: pguido


........

................
r114115 | tilghman | 2008-04-14 11:32:59 -0500 (Mon, 14 Apr 2008) | 2 lines

Make tab-completion work for all cases

................
r114118 | mmichelson | 2008-04-14 12:42:20 -0500 (Mon, 14 Apr 2008) | 19 lines

Merged revisions 114117 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114117 | mmichelson | 2008-04-14 12:41:03 -0500 (Mon, 14 Apr 2008) | 11 lines

Increase the retry count when attempting to show channels. This apparently
cleared an issue someone was seeing when attempting to show channels when
the load was high.

(closes issue #11667)
Reported by: falves11
Patches:
      11677.txt uploaded by russell (license 2)
Tested by: falves11


........

................
r114121 | qwell | 2008-04-14 13:34:17 -0500 (Mon, 14 Apr 2008) | 15 lines

Merged revisions 114120 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114120 | qwell | 2008-04-14 13:31:57 -0500 (Mon, 14 Apr 2008) | 7 lines

The call_token on the pvt can occasionally be NULL, causing a crash.

If it is NULL, we can skip this channel, since it can't the one we're looking for.

(closes issue #9299)
Reported by: vazir

........

................
r114124 | twilson | 2008-04-14 14:12:27 -0500 (Mon, 14 Apr 2008) | 2 lines

Don't unref user twice on failure.  Also, when adding sorted list of users, it is best to check the entry already in the list for a "next" entry instead of the newly created entry...

................
r114127 | twilson | 2008-04-14 14:58:52 -0500 (Mon, 14 Apr 2008) | 2 lines

Need a new buffer for each loop

................
r114131 | qwell | 2008-04-15 10:20:47 -0500 (Tue, 15 Apr 2008) | 7 lines

Convert several DEBUG logs into ast_debug.

(closes issue #12444)
Reported by: IgorG
Patches:
      channel_c_debug.diff uploaded by IgorG (license 20)

................
r114134 | qwell | 2008-04-15 11:18:38 -0500 (Tue, 15 Apr 2008) | 16 lines

Merged revisions 114133 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114133 | qwell | 2008-04-15 11:18:08 -0500 (Tue, 15 Apr 2008) | 8 lines

Allow autofill to work in the general section of queues.conf.
Additionally, don't try to (re)set options when they have empty values in realtime (all unset columns would have an empty value).

(closes issue #12445)
Reported by: atis
Patches:
      12445-autofill.diff uploaded by qwell (license 4)

........

................
r114139 | qwell | 2008-04-15 12:17:37 -0500 (Tue, 15 Apr 2008) | 15 lines

Merged revisions 114138 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114138 | qwell | 2008-04-15 12:17:18 -0500 (Tue, 15 Apr 2008) | 7 lines

Update Digium autosupport script, for more useful information.

(closes issue #12452)
Reported by: angler
Patches:
      autosupport.diff uploaded by angler (license 106)

........

................
r114141 | qwell | 2008-04-15 12:21:58 -0500 (Tue, 15 Apr 2008) | 8 lines

Shorten the mac address pattern, since some phones use different identifiers (such as the i2050 softphone).

(closes issue #12398)
Reported by: c_hans
Patches:
      chan_unistim_svn.diff uploaded by c (license 460)
Tested by: c_hans

................
r114143 | seanbright | 2008-04-15 12:54:38 -0500 (Tue, 15 Apr 2008) | 2 lines

I'm not sure why, but "this" bothers me.  Ba dum dum.

................
r114146 | murf | 2008-04-15 14:59:50 -0500 (Tue, 15 Apr 2008) | 8 lines

These changes: 

   a. fix a self-found problem with SPAWN-ing an extension,
      where matches were not being found
   b. correct some wording in a comment
   c. Add some debug for future debugging.


................
r114149 | qwell | 2008-04-15 15:27:04 -0500 (Tue, 15 Apr 2008) | 2 lines

If somebody enters a digit during ast_stream_and_wait, the return value is the digit, which we need to use later.

................
r114150 | oej | 2008-04-15 15:31:08 -0500 (Tue, 15 Apr 2008) | 2 lines

Adding chanvar to SIPPEER from 1.4 branch

................
r114151 | oej | 2008-04-15 15:39:29 -0500 (Tue, 15 Apr 2008) | 10 lines

Merged revisions 114148 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114148 | oej | 2008-04-15 22:26:05 +0200 (Tis, 15 Apr 2008) | 2 lines

Handle subscribe queues in all situations... Thanks to festr_ on irc for telling me about this bug.

........

................
r114152 | tilghman | 2008-04-15 15:51:08 -0500 (Tue, 15 Apr 2008) | 2 lines

Oops, buffer wasn't long enough for query

................
r114165 | seanbright | 2008-04-16 07:23:13 -0500 (Wed, 16 Apr 2008) | 1 line

Add the ability to disable channel technology name playback when speaking the current channel name
................
r114168 | file | 2008-04-16 09:11:18 -0500 (Wed, 16 Apr 2008) | 13 lines

Blocked revisions 114167 via svnmerge

........
r114167 | file | 2008-04-16 11:10:46 -0300 (Wed, 16 Apr 2008) | 6 lines

Include the proper headers for using mkdir on FreeBSD.
(closes issue #12430)
Reported by: ys
Patches:
      app_meetme.c.diff uploaded by ys (license 281)

........

................
r114172 | murf | 2008-04-16 12:14:18 -0500 (Wed, 16 Apr 2008) | 1 line

Introducing doubly linked lists to trunk from branch team/murf/bug11210.
................
r114174 | qwell | 2008-04-16 12:31:02 -0500 (Wed, 16 Apr 2008) | 14 lines

Blocked revisions 114173 via svnmerge

........
r114173 | qwell | 2008-04-16 12:30:09 -0500 (Wed, 16 Apr 2008) | 7 lines

Fix "fallthrough" behavior here, so config options in a previously configured user don't override settings in general.

(closes issue #12458)
Reported by: tzafrir
Patches:
      chanzap_users_sections.diff uploaded by tzafrir (license 46)

........

................
r114175 | murf | 2008-04-16 12:45:28 -0500 (Wed, 16 Apr 2008) | 1 line

Introducing various astobj2 enhancements, chief being a refcount tracing feature, and various documentation updates in astobj2.h, and the addition of standalone utility, refcounter, that will filter the trace output for unbalanced, unfreed objects. This comes from the team/murf/bug11210 branch.
................
r114181 | tilghman | 2008-04-16 15:00:27 -0500 (Wed, 16 Apr 2008) | 10 lines

Blocked revisions 114180 via svnmerge

........
r114180 | tilghman | 2008-04-16 14:59:37 -0500 (Wed, 16 Apr 2008) | 3 lines

Backport revisions for latest vpb drivers to 1.4
(Closes issue #12457)

........

................
r114182 | murf | 2008-04-16 15:09:39 -0500 (Wed, 16 Apr 2008) | 1 line

Introducing a small upgrade to the ast_sched_xxx  facility, to keep it from eating up lots of cpu cycles. See CHANGES. From the team/murf/bug11210 branch.
................
r114183 | murf | 2008-04-16 15:28:08 -0500 (Wed, 16 Apr 2008) | 1 line

Introducing a small optimization to event_unsubscribe; events now use a Doubly-Linked list for events, gives fast deletions, for the sake of channel driver mwi events. From team/murf/bug11210.
................
r114185 | kpfleming | 2008-04-16 15:47:30 -0500 (Wed, 16 Apr 2008) | 14 lines

Merged revisions 114184 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114184 | kpfleming | 2008-04-16 15:46:38 -0500 (Wed, 16 Apr 2008) | 6 lines

use the ZT_SET_DIALPARAMS ioctl properly by initializing the structure to all zeroes in case it contains fields that we don't write values into (which it does as of Zaptel 1.4.10)

(closes issue #12456)
Reported by: fnordian


........

................
r114187 | murf | 2008-04-16 15:54:41 -0500 (Wed, 16 Apr 2008) | 1 line

A small enhancement-- I added the routine log_show_lock to utils.c, which if the mentioned lock has been acquired, this routine will log to the console the normal info about that lock you'd see from the CLI when you do a 'core show locks'. It's solely for debug-- if the lock is NOT acquired, there is no output. I use it to show 'unexpected' locks, to see where/why a lock is pre-locked. This command is to be called from points of interest, like just before a trylock, and helps to spot fleeting, highly temporal locks that normally are not locked...
................
r114188 | tilghman | 2008-04-16 17:57:54 -0500 (Wed, 16 Apr 2008) | 2 lines

Standardized routines for forking processes (keeps all the specialized code in one place).

................
r114190 | murf | 2008-04-16 18:53:27 -0500 (Wed, 16 Apr 2008) | 1 line

This is the scariest commit I've done in a long time. This is the astobj2-ification of chan_sip. I've tested a number of scenarios like crazy. It used to have 4x the call setup/teardown performance of trunk, but now it's roughly at parity. I will attempt to find the bottlenecks and get it back to the 4x mark. The changes made were somewhat invasive, but the value to the community of these upgrades outweighs waiting further for more testing. Every change being made to chan_sip was lousing this code up when we tried to merge. Peers, Users, Dialogs, are all now astobj2 objects, indexed via hashtables. Refcounting is used to track objects and free them at the bitter end of their lives. Please file issues on bugs.digium.com, and PLEASE, please, please be patient. One natural advantage to all the hash-table work is that loading large sip.conf files full of thousands of peers now goes much faster. One more please: PLEASE help thrash this code and test it.
................
r114192 | seanbright | 2008-04-17 05:55:05 -0500 (Thu, 17 Apr 2008) | 9 lines

Merged revisions 114191 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114191 | seanbright | 2008-04-17 06:51:20 -0400 (Thu, 17 Apr 2008) | 1 line

Make sure we have enough room for the recording's filename.
........

................
r114194 | seanbright | 2008-04-17 07:25:23 -0500 (Thu, 17 Apr 2008) | 1 line

Update the CHANGES file with yesterday's ChanSpy change.  Sorry Kevin, just saw your e-mail.
................
r114196 | tilghman | 2008-04-17 07:59:04 -0500 (Thu, 17 Apr 2008) | 16 lines

Merged revisions 114195 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114195 | tilghman | 2008-04-17 07:56:38 -0500 (Thu, 17 Apr 2008) | 8 lines

Add special case for when the agi cannot be executed, to comply with the documentation that
we return failure in that case.
(closes issue #12462)
 Reported by: fmueller
 Patches: 
       20080416__bug12462.diff.txt uploaded by Corydon76 (license 14)
 Tested by: fmueller

........

................
r114199 | phsultan | 2008-04-17 08:46:17 -0500 (Thu, 17 Apr 2008) | 10 lines

Merged revisions 114198 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114198 | phsultan | 2008-04-17 15:42:23 +0200 (Thu, 17 Apr 2008) | 2 lines

Use keepalives effectively in order diagnose bug #12432.

........

................
r114201 | murf | 2008-04-17 09:45:16 -0500 (Thu, 17 Apr 2008) | 1 line

Thanks to snuff for finding these omissions
................
r114202 | tilghman | 2008-04-17 10:12:52 -0500 (Thu, 17 Apr 2008) | 2 lines

fileio.h does not exist; io.h does, though.

................
r114205 | russell | 2008-04-17 11:25:29 -0500 (Thu, 17 Apr 2008) | 11 lines

Merged revisions 114204 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114204 | russell | 2008-04-17 11:23:45 -0500 (Thu, 17 Apr 2008) | 3 lines

Fix the bininstall target to install from subdirs, as well.
(closes issue AST-8, patch from bmd at switchvox)

........

................
r114208 | mmichelson | 2008-04-17 11:40:12 -0500 (Thu, 17 Apr 2008) | 20 lines

Merged revisions 114207 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114207 | mmichelson | 2008-04-17 11:28:03 -0500 (Thu, 17 Apr 2008) | 12 lines

It was possible for a reference to a frame which was part of a freed DSP to still be
referenced, leading to memory corruption and eventual crashes. This code change ensures
that the dsp is freed when we are finished with the frame. This change is very similar
to a change Russell made with translators back a month or so ago.

(closes issue #11999)
Reported by: destiny6628
Patches:
      11999.patch uploaded by putnopvut (license 60)
Tested by: destiny6628, victoryure


........

................
r114212 | mmichelson | 2008-04-17 11:51:09 -0500 (Thu, 17 Apr 2008) | 11 lines

Blocked revisions 114211 via svnmerge

........
r114211 | mmichelson | 2008-04-17 11:50:46 -0500 (Thu, 17 Apr 2008) | 4 lines

Add prototype for ast_dsp_frame_freed. I'm not sure how this was
compiling before...


........

................
r114227 | mmichelson | 2008-04-17 16:04:40 -0500 (Thu, 17 Apr 2008) | 17 lines

Merged revisions 114226 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114226 | mmichelson | 2008-04-17 16:03:29 -0500 (Thu, 17 Apr 2008) | 9 lines

Declaration of the peer channel in this scope was making it so the peer variable defined
in the outer scope was never set properly, therefore making iterating through the channel
list always restart from the beginning. This bug would have affected anyone who called
chanspy without specifying a first argument.

(closes issue #12461)
Reported by: stever28


........

................
r114229 | jpeeler | 2008-04-17 16:09:37 -0500 (Thu, 17 Apr 2008) | 1 line

added info describing DNS manager
................
r114231 | russell | 2008-04-17 17:21:32 -0500 (Thu, 17 Apr 2008) | 3 lines

Make this file compile.  The variable str is never set anywhere.  Furthermore,
it duplicates a lot of code.  I will leave it to murf to clean up.

................
r114233 | russell | 2008-04-17 17:24:00 -0500 (Thu, 17 Apr 2008) | 14 lines

Merged revisions 114230 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114230 | russell | 2008-04-17 17:15:43 -0500 (Thu, 17 Apr 2008) | 6 lines

Remove redundant safety net.  The check for the autoservice channel list state
accomplishes the same goal in a better way.

(issue #12470)
Reported By: atis

........

................
r114243 | tilghman | 2008-04-18 01:53:47 -0500 (Fri, 18 Apr 2008) | 11 lines

Merged revisions 114242 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114242 | tilghman | 2008-04-18 01:49:16 -0500 (Fri, 18 Apr 2008) | 3 lines

For consistency sake, ensure that the values that ${CALLINGPRES} returns are valid as an
input to SetCallingPres.  (Closes issue #12472)

........

................
r114246 | seanbright | 2008-04-18 08:38:07 -0500 (Fri, 18 Apr 2008) | 9 lines

Merged revisions 114245 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114245 | seanbright | 2008-04-18 09:33:32 -0400 (Fri, 18 Apr 2008) | 1 line

Only complete the SIP channel name once for 'sip show channel <channel>'
........

................
r114249 | russell | 2008-04-18 10:25:50 -0500 (Fri, 18 Apr 2008) | 12 lines

Blocked revisions 114248 via svnmerge

........
r114248 | russell | 2008-04-18 10:24:09 -0500 (Fri, 18 Apr 2008) | 5 lines

Ensure that we don't ast_strdupa(NULL)
(closes issue #12476)
Reported by: davidw
Patch by me

........

................
r114253 | dbailey | 2008-04-18 11:05:29 -0500 (Fri, 18 Apr 2008) | 2 lines

Add g__object_unref to clean up gmime message object

................
r114254 | file | 2008-04-18 11:11:27 -0500 (Fri, 18 Apr 2008) | 4 lines

If the parsing of the config file fails make sure we unlock ldap_lock.
(closes issue #12477)
Reported by: IgorG

................
r114259 | mmichelson | 2008-04-18 13:03:06 -0500 (Fri, 18 Apr 2008) | 14 lines

Merged revisions 114257 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114257 | mmichelson | 2008-04-18 12:44:29 -0500 (Fri, 18 Apr 2008) | 6 lines

Clearing up error messages so they make a bit more sense. Also removing a redundant error
message.

Issue AST-15


........

................
r114261 | file | 2008-04-18 13:15:11 -0500 (Fri, 18 Apr 2008) | 6 lines

Add MEETME_INFO dialplan function that allows querying various properties of a Meetme conference.
(closes issue #11691)
Reported by: junky
Patches:
      meetme_info.patch uploaded by jpeeler (license 325)

................
r114271 | file | 2008-04-18 14:35:33 -0500 (Fri, 18 Apr 2008) | 4 lines

Make sure ADSI is marked as unavailable on Unistim channels so voicemail does not try to do some ADSI jazz.
(closes issue #12460)
Reported by: PerryB

................
r114276 | russell | 2008-04-18 14:59:17 -0500 (Fri, 18 Apr 2008) | 10 lines

Merged revisions 114275 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114275 | russell | 2008-04-18 14:58:55 -0500 (Fri, 18 Apr 2008) | 2 lines

SUBDIRS_INSTALL is already listed as a subtarget for bininstall

........

................
r114279 | russell | 2008-04-18 15:01:47 -0500 (Fri, 18 Apr 2008) | 10 lines

Merged revisions 114278 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114278 | russell | 2008-04-18 15:01:09 -0500 (Fri, 18 Apr 2008) | 2 lines

ensure directories are created before we try to install stuff into them

........

................
r114285 | russell | 2008-04-18 16:51:05 -0500 (Fri, 18 Apr 2008) | 10 lines

Merged revisions 114284 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114284 | russell | 2008-04-18 16:48:06 -0500 (Fri, 18 Apr 2008) | 2 lines

Don't destroy a manager session if poll() returns an error of EAGAIN.

........

................
r114295 | seanbright | 2008-04-18 19:15:31 -0500 (Fri, 18 Apr 2008) | 2 lines

Ignore refcounter

................
r114298 | tilghman | 2008-04-19 08:53:38 -0500 (Sat, 19 Apr 2008) | 11 lines

Blocked revisions 114297 via svnmerge

........
r114297 | tilghman | 2008-04-19 08:49:50 -0500 (Sat, 19 Apr 2008) | 4 lines

MOH usage information needs a terminating newline, or else
"asterisk -rx 'help moh reload'" will hang.  Reported via
-dev list, fixed by me.

........

................
r114300 | tilghman | 2008-04-19 08:58:56 -0500 (Sat, 19 Apr 2008) | 9 lines

Blocked revisions 114299 via svnmerge

........
r114299 | tilghman | 2008-04-19 08:57:17 -0500 (Sat, 19 Apr 2008) | 2 lines

Ensure that help text terminates with a newline

........

................
r114303 | mattf | 2008-04-19 11:58:24 -0500 (Sat, 19 Apr 2008) | 1 line

SS7:Added - Generic Name / Access Transport / Redirecting Number handling. #12425
................
r114314 | seanbright | 2008-04-20 09:52:07 -0500 (Sun, 20 Apr 2008) | 1 line

Minor logging cleanups
................
r114320 | file | 2008-04-21 09:34:06 -0500 (Mon, 21 Apr 2008) | 6 lines

Only print out the error message if ldap_modify_ext_s actually returns an error, and not success.
(closes issue #12438)
Reported by: gservat
Patches:
      res_config_ldap.c-patch-code uploaded by gservat (license 466)

................
r114323 | file | 2008-04-21 09:40:33 -0500 (Mon, 21 Apr 2008) | 12 lines

Merged revisions 114322 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/branches/1.4

........
r114322 | file | 2008-04-21 11:39:32 -0300 (Mon, 21 Apr 2008) | 4 lines

Only drop audio if we receive it without a progress indication. We allow other frames through such as DTMF because they may be needed to complete the call.
(closes issue #12440)
Reported by: aragon

........

................
r114325 | russell | 2008-04-21 10:01:04 -0500 (Mon, 21 Apr 2008) | 2 lines

Add a simple janitor project

................
r114327 | jpeeler | 2008-04-21 10:34:37 -0500 (Mon, 21 Apr 2008) | 2 lines

This removes an invalid warning message for an incorrectly entered pin, but more importantly removes an inapplicable check. If the first argument passed to app_authenticate does not contain a '/', the argument should be treated as the sole fixed "password" to match against and that is all. (Previous behavior was attempting to open a file based on the pin.)

................

Added:
    team/group/codec_bits/doc/chan_sip-perf-testing.txt
      - copied unchanged from r114327, trunk/doc/chan_sip-perf-testing.txt
    team/group/codec_bits/include/asterisk/dlinkedlists.h
      - copied unchanged from r114327, trunk/include/asterisk/dlinkedlists.h
    team/group/codec_bits/tests/test_dlinklists.c
      - copied unchanged from r114327, trunk/tests/test_dlinklists.c
    team/group/codec_bits/utils/refcounter.c
      - copied unchanged from r114327, trunk/utils/refcounter.c
Modified:
    team/group/codec_bits/   (props changed)
    team/group/codec_bits/CHANGES
    team/group/codec_bits/Makefile
    team/group/codec_bits/apps/app_authenticate.c
    team/group/codec_bits/apps/app_chanspy.c
    team/group/codec_bits/apps/app_dial.c
    team/group/codec_bits/apps/app_directory.c
    team/group/codec_bits/apps/app_externalivr.c
    team/group/codec_bits/apps/app_festival.c
    team/group/codec_bits/apps/app_ices.c
    team/group/codec_bits/apps/app_meetme.c
    team/group/codec_bits/apps/app_mp3.c
    team/group/codec_bits/apps/app_nbscat.c
    team/group/codec_bits/apps/app_queue.c
    team/group/codec_bits/apps/app_setcallerid.c
    team/group/codec_bits/apps/app_voicemail.c
    team/group/codec_bits/apps/app_zapras.c
    team/group/codec_bits/cdr/cdr_pgsql.c
    team/group/codec_bits/channels/chan_h323.c
    team/group/codec_bits/channels/chan_iax2.c
    team/group/codec_bits/channels/chan_local.c
    team/group/codec_bits/channels/chan_sip.c
    team/group/codec_bits/channels/chan_skinny.c
    team/group/codec_bits/channels/chan_unistim.c
    team/group/codec_bits/channels/chan_zap.c
    team/group/codec_bits/configs/res_ldap.conf.sample
    team/group/codec_bits/configs/sip.conf.sample
    team/group/codec_bits/contrib/scripts/astcli
    team/group/codec_bits/contrib/scripts/autosupport
    team/group/codec_bits/doc/CODING-GUIDELINES
    team/group/codec_bits/doc/janitor-projects.txt
    team/group/codec_bits/include/asterisk/app.h
    team/group/codec_bits/include/asterisk/astobj2.h
    team/group/codec_bits/include/asterisk/dsp.h
    team/group/codec_bits/include/asterisk/frame.h
    team/group/codec_bits/include/asterisk/lock.h
    team/group/codec_bits/include/asterisk/logger.h
    team/group/codec_bits/include/asterisk/sched.h
    team/group/codec_bits/main/app.c
    team/group/codec_bits/main/asterisk.c
    team/group/codec_bits/main/astobj2.c
    team/group/codec_bits/main/autoservice.c
    team/group/codec_bits/main/callerid.c
    team/group/codec_bits/main/channel.c
    team/group/codec_bits/main/dsp.c
    team/group/codec_bits/main/event.c
    team/group/codec_bits/main/features.c
    team/group/codec_bits/main/file.c
    team/group/codec_bits/main/frame.c
    team/group/codec_bits/main/logger.c
    team/group/codec_bits/main/manager.c
    team/group/codec_bits/main/pbx.c
    team/group/codec_bits/main/rtp.c
    team/group/codec_bits/main/sched.c
    team/group/codec_bits/main/utils.c
    team/group/codec_bits/phoneprov/polycom_line.xml
    team/group/codec_bits/res/res_agi.c
    team/group/codec_bits/res/res_config_ldap.c
    team/group/codec_bits/res/res_http_post.c
    team/group/codec_bits/res/res_jabber.c
    team/group/codec_bits/res/res_musiconhold.c
    team/group/codec_bits/res/res_phoneprov.c
    team/group/codec_bits/utils/   (props changed)
    team/group/codec_bits/utils/Makefile
    team/group/codec_bits/utils/ael_main.c
    team/group/codec_bits/utils/astman.c
    team/group/codec_bits/utils/check_expr.c
    team/group/codec_bits/utils/conf2ael.c
    team/group/codec_bits/utils/hashtest.c
    team/group/codec_bits/utils/hashtest2.c

Propchange: team/group/codec_bits/
------------------------------------------------------------------------------
Binary property 'branch-1.4-blocked' - no diff available.

Propchange: team/group/codec_bits/
------------------------------------------------------------------------------
Binary property 'branch-1.4-merged' - no diff available.

Propchange: team/group/codec_bits/
------------------------------------------------------------------------------
--- svnmerge-integrated (original)
+++ svnmerge-integrated Mon Apr 21 12:33:59 2008
@@ -1,1 +1,1 @@
-/trunk:1-113986
+/trunk:1-114357

Modified: team/group/codec_bits/CHANGES
URL: http://svn.digium.com/view/asterisk/team/group/codec_bits/CHANGES?view=diff&rev=114380&r1=114379&r2=114380
==============================================================================
--- team/group/codec_bits/CHANGES (original)
+++ team/group/codec_bits/CHANGES Mon Apr 21 12:33:59 2008
@@ -29,12 +29,24 @@
    messages, if the "messagewrap" option is set to a true value.
  * Dial has a new option: F(context^extension^pri), which permits a callee to
    continue in the dialplan, at the specified label, if the caller hangs up.
+ * ChanSpy and ExtenSpy have a new option, 's' which suppresses speaking the
+   technology name (e.g. SIP, IAX, etc) of the channel being spied on.
 
 SIP Changes
 -----------
  * The ATTENDED_TRANSFER_COMPLETE_SOUND can now be set using setvar to cause a given
    audio file to be played upon completion of an attended transfer.
  * Added DNS manager support to registrations for peers referencing peer entries.
+   DNS manager runs in the background which allows DNS lookups to be run asynchronously 
+   as well as periodically updating the IP address. These properties allow for
+   better performance as well as recovery in the event of an IP change.
+ * Performance improvements via using hash tables (astobj2) and doubly-linked lists to improve 
+   load/reload of large numbers of peers/users by ~40x (for large lists of peers.
+   Initially, we saw 4x improvement in call setup/destruction, but at the time
+   of merging, this gain has disappeared; further research will be done to try
+   and restore this performance improvement. Astobj2 refcounting is now used
+   for users, peers, and dialogs.  Users are encouraged to assist in regression
+   testing and problem reporting!
 
 IAX Changes
 -----------
@@ -257,6 +269,9 @@
      SIP call to Voicemail by putting a Local channel in the middle.  This
      feature is enabled by using the 'j' option in the Dial string to the Local
      channel in conjunction with the existing 'n' option for local channels.
+  * A 'b' option has been added which causes chan_local to return the actual channel
+     that is behind it when queried. This is useful for transfer scenarios as the
+     actual channel will be transferred, not the Local channel.
 
 Zaptel channel driver (chan_zap) Changes
 ----------------------------------------
@@ -411,6 +426,8 @@
      for the "meetme" command.
   * Added the ability to specify the music on hold class used to play into the
      conference when there is only one member and the M option is used.
+  * Added MEETME_INFO dialplan function which provides a way to query
+     various properties of a Meetme conference.
 
 Other Dialplan Application Changes
 ----------------------------------
@@ -612,6 +629,18 @@
   * A new option when starting a remote asterisk (rasterisk, asterisk -r) for
      specifying which socket to use to connect to the running Asterisk daemon
      (-s)
+  * Performance enhancements to the sched facility, which is used in
+    the channel drivers, etc. Added hashtabs and doubly-linked lists
+    to speed up deletion; start at the beginning or end of list to
+    speed up insertion.
+  * Added Doubly-linked lists after the fashion of linkedlists.h. They are in
+    dlinkedlists.h. Doubly-linked lists feature fast deletion times.
+    Added regression tests to the tests/ dir, also.
+  * Added a refcount trace feature to astobj2 for those trying to balance
+    object creation, deletion; work, play; space and time. See the
+    notes in astobj2.h. Also, see utils/refcounter as well, as a
+    quick way to find unbalanced refcounts in what could be a sea
+    of objects that were balanced.
   * Added logging to 'make update' command.  See update.log
   * Added strictrtp option to rtp.conf. If enabled this will drop RTP packets that
      do not come from the remote party.

Modified: team/group/codec_bits/Makefile
URL: http://svn.digium.com/view/asterisk/team/group/codec_bits/Makefile?view=diff&rev=114380&r1=114379&r2=114380
==============================================================================
--- team/group/codec_bits/Makefile (original)
+++ team/group/codec_bits/Makefile Mon Apr 21 12:33:59 2008
@@ -498,7 +498,7 @@
 NEWHEADERS=$(notdir $(wildcard include/asterisk/*.h))
 OLDHEADERS=$(filter-out $(NEWHEADERS),$(notdir $(wildcard $(DESTDIR)$(ASTHEADERDIR)/*.h)))
 
-bininstall: _all
+installdirs:
 	mkdir -p $(DESTDIR)$(MODULES_DIR)
 	mkdir -p $(DESTDIR)$(ASTSBINDIR)
 	mkdir -p $(DESTDIR)$(ASTETCDIR)
@@ -510,6 +510,8 @@
 	mkdir -p $(DESTDIR)$(ASTSPOOLDIR)/tmp
 	mkdir -p $(DESTDIR)$(ASTSPOOLDIR)/meetme
 	mkdir -p $(DESTDIR)$(ASTSPOOLDIR)/monitor
+
+bininstall: _all installdirs $(SUBDIRS_INSTALL)
 	$(INSTALL) -m 755 main/asterisk $(DESTDIR)$(ASTSBINDIR)/
 	$(LN) -sf asterisk $(DESTDIR)$(ASTSBINDIR)/rasterisk

[... 9727 lines stripped ...]



More information about the asterisk-commits mailing list