[asterisk-commits] seanbright: branch group/cdr_backend_ast_str r114317 - in /team/group/cdr_bac...

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Sun Apr 20 14:29:48 CDT 2008


Author: seanbright
Date: Sun Apr 20 14:29:47 2008
New Revision: 114317

URL: http://svn.digium.com/view/asterisk?view=rev&rev=114317
Log:
Merged revisions 112033,112035,112069,112071,112124,112126,112148,112155,112205,112207,112210,112234,112241,112252,112289,112321,112351,112357,112360,112394,112426,112431,112469,112520,112564,112600,112653,112656,112708,112710,112712,112714,112785,112821,112874,112906,112939,112972,113009,113013,113066,113119,113170,113172,113207,113241,113243,113245,113297,113349,113400,113403,113452,113455,113505,113508,113559,113597,113647,113649,113682,113731,113752,113785,113834,113836,113838,113840,113875,113928,113980,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 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

................
r112033 | twilson | 2008-03-31 16:45:05 -0400 (Mon, 31 Mar 2008) | 2 lines

Handle blank prefix= in http.conf

................
r112035 | twilson | 2008-03-31 17:01:59 -0400 (Mon, 31 Mar 2008) | 2 lines

Yeah, simplify that logic a bit...

................
r112069 | qwell | 2008-03-31 17:48:30 -0400 (Mon, 31 Mar 2008) | 13 lines

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

........
r112068 | qwell | 2008-03-31 16:48:05 -0500 (Mon, 31 Mar 2008) | 5 lines

Fix a silly infinite loop when choosing an invalid option.

(closes issue #12315)
Reported by: jmls

........

................
r112071 | qwell | 2008-03-31 18:16:34 -0400 (Mon, 31 Mar 2008) | 7 lines

I missed a place when this define was changed.

(closes issue #12334)
Reported by: ovi
Patches:
      12334-asterisk.patch uploaded by dimas (license 88)

................
r112124 | russell | 2008-04-01 12:35:04 -0400 (Tue, 01 Apr 2008) | 2 lines

Now that zaptel trunk has been removed, add the PSTN deprecation notice to chan_zap, as well.

................
r112126 | file | 2008-04-01 12:50:37 -0400 (Tue, 01 Apr 2008) | 13 lines

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

........
r112125 | file | 2008-04-01 13:45:14 -0300 (Tue, 01 Apr 2008) | 5 lines

Ensure that we do not exceed the hold's maximum size with a single frame.
(closes issue #12047)
Reported by: fabianoheringer
Tested by: fabianoheringer

........

................
r112148 | mmichelson | 2008-04-01 13:23:19 -0400 (Tue, 01 Apr 2008) | 18 lines

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

........
r112138 | mmichelson | 2008-04-01 12:21:21 -0500 (Tue, 01 Apr 2008) | 10 lines

Initialize the __res_state structure used for dns purposes
to all 0's prior to using it. This is due to valgrind's complaints
on issue #12284 as well as an excerpt found in "Description" portion
of the online man page found here:

http://www.iti.cs.tu-bs.de/cgi-bin/UNIXhelp/man-cgi?res_nquery+3RESOLV

(pertains to issue #12284 but does not necessarily close it)


........

................
r112155 | file | 2008-04-01 13:24:45 -0400 (Tue, 01 Apr 2008) | 6 lines

Demote a log message down to a warning.
(closes issue #12345)
Reported by: caio1982
Patches:
      limit_msg.diff uploaded by caio1982 (license 22)

................
r112205 | file | 2008-04-01 13:48:52 -0400 (Tue, 01 Apr 2008) | 12 lines

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

........
r112204 | file | 2008-04-01 14:43:46 -0300 (Tue, 01 Apr 2008) | 4 lines

Do not pass audio until the remote side has indicated they are providing early media, or if the channel has been answered.
(closes issue #11823)
Reported by: SDamm

........

................
r112207 | jpeeler | 2008-04-01 13:53:08 -0400 (Tue, 01 Apr 2008) | 3 lines

This adds DNS SRV record support to DNS manager.  If there is a SRV record for a given domain, the hostname and port listed in the SRV record will be used. If no SRV record exists or a SRV lookup is not attempted, the DNS lookup on the specified domain will be performed as normal. Chan_sip has been modified to take advantage of the new SRV support.


................
r112210 | file | 2008-04-01 14:06:13 -0400 (Tue, 01 Apr 2008) | 12 lines

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

........
r112209 | file | 2008-04-01 15:02:43 -0300 (Tue, 01 Apr 2008) | 4 lines

Disable Packet2Packet bridging when we need to feed DTMF frames into the core. Some implementations do not like how we switch between things.
(closes issue #12212)
Reported by: bamby

........

................
r112234 | tilghman | 2008-04-01 14:23:40 -0400 (Tue, 01 Apr 2008) | 2 lines

Fix last commit

................
r112241 | qwell | 2008-04-01 14:24:56 -0400 (Tue, 01 Apr 2008) | 7 lines

More voicemail doxygen additions/cleanup.

(issue #12343)
Reported by: travishein
Patches:
      app_voicemail_code_documentation.patch uploaded by travishein (license 385)

................
r112252 | qwell | 2008-04-01 14:27:08 -0400 (Tue, 01 Apr 2008) | 7 lines

Minor formatting cleanup.

(closes issue #12343)
Reported by: travishein
Patches:
      app_voicemail_code_convention.patch uploaded by travishein (license 385)

................
r112289 | murf | 2008-04-01 16:02:19 -0400 (Tue, 01 Apr 2008) | 21 lines

(closes issue #12298)
Reported by: falves11
Patches:
      12298.patch1 uploaded by murf (license 17)
Tested by: murf

I have hopes that the changes made over the last few days will
finalize and solidify this code. While there are bound to be 
small tweaks still needed, I feel that the job (at last) is
somewhat completed. Finally, I had a chance to comprehend how
the scoring of extension patterns was done in the previous
version, and I've come very close to using the exact same
criteria in the new pattern matching code. The left-right
sorting is now replicated in the trie structure itself, such
that the first match found will the 'best' match. Compared
the results against 1.4 for several extensions. Replicated
falves11's setup and it works. Used some devious patterns
provided by jsmith, supplemented with a few of my own.
Looks good.


................
r112321 | jpeeler | 2008-04-01 18:07:30 -0400 (Tue, 01 Apr 2008) | 2 lines

Existing DNS manager lookups extended to check for SRV records.

................
r112351 | russell | 2008-04-01 18:25:45 -0400 (Tue, 01 Apr 2008) | 2 lines

Fix a typo that prevented configuration of non-dynamic peers.

................
r112357 | murf | 2008-04-01 18:45:10 -0400 (Tue, 01 Apr 2008) | 1 line

Bumped across another test set for the new exten pattern matcher, which revealed a problem with the CANMATCH/MATCHMORE modes. Direct matches were getting in the way. Fixed.
................
r112360 | jpeeler | 2008-04-01 18:55:28 -0400 (Tue, 01 Apr 2008) | 2 lines

Added dnsmgr status output for sip show registry.

................
r112394 | mmichelson | 2008-04-02 10:32:43 -0400 (Wed, 02 Apr 2008) | 14 lines

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

........
r112393 | mmichelson | 2008-04-02 09:32:00 -0500 (Wed, 02 Apr 2008) | 6 lines

Ensure that there is no timeout if none is specified.

(closes issue #12349)
Reported by: johnlange


........

................
r112426 | twilson | 2008-04-02 11:25:48 -0400 (Wed, 02 Apr 2008) | 2 lines

Re-add HTTP post support by moving to res_http_post.c

................
r112431 | file | 2008-04-02 11:26:51 -0400 (Wed, 02 Apr 2008) | 7 lines

Since the SIP request structure gets reused multiple times with TCP handling we have to clear the debug state or else we will keep spitting out debug even after it has been turned off.
(closes issue #12169)
Reported by: pj
Patches:
      12169-debugoff-2.diff uploaded by qwell (license 4)
Tested by: pj

................
r112469 | mmichelson | 2008-04-02 13:36:49 -0400 (Wed, 02 Apr 2008) | 21 lines

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

........
r112468 | mmichelson | 2008-04-02 12:36:04 -0500 (Wed, 02 Apr 2008) | 13 lines

Fix a race condition in the manager. It is possible that a new manager event
could be appended during a brief time when the manager is not waiting for input.
If an event comes during this period, we need to set an indicator that there is an
event pending so that the manager doesn't attempt to wait forever for an event that
already happened.

(closes issue #12354)
Reported by: bamby
Patches:
      manager_race_condition.diff uploaded by bamby (license 430)
	  (comments added by me)


........

................
r112520 | tilghman | 2008-04-02 15:34:52 -0400 (Wed, 02 Apr 2008) | 6 lines

Make MISDN generate channel rename events when the name changes.
(closes issue #11142)
 Reported by: julianjm
 Patches: 
       chan_misdn_tmpchan_trunk_v1.diff uploaded by julianjm (license 99)

................
r112564 | tilghman | 2008-04-03 03:49:05 -0400 (Thu, 03 Apr 2008) | 7 lines

Use a 32k file buffer on recordings, which increases the efficiency of file recording.
(closes issue #11962)
 Reported by: garlew
 Patches: 
       recording.patch uploaded by garlew (license 376)
       bug-11962.diff uploaded by snuffy (license 35)

................
r112600 | mmichelson | 2008-04-03 10:35:47 -0400 (Thu, 03 Apr 2008) | 17 lines

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

........
r112599 | mmichelson | 2008-04-03 09:32:20 -0500 (Thu, 03 Apr 2008) | 9 lines

Fix the testing of the "res" variable so that it is more logically correct and 
makes the correct warning and debug messages print.

(closes issue #12361)
Reported by: one47
Patches:
      chan_zap_deferred_digit.patch uploaded by one47 (license 23)


........

................
r112653 | dhubbard | 2008-04-03 18:13:11 -0400 (Thu, 03 Apr 2008) | 1 line

add a Zaptel timer check to verify the timer is responding when Zaptel support is compiled into Asterisk and Zaptel drivers are loaded.  This will help people not waste their valuable time debugging side effects.
................
r112656 | dhubbard | 2008-04-03 18:19:43 -0400 (Thu, 03 Apr 2008) | 1 line

satisfy buildbot
................
r112708 | dhubbard | 2008-04-03 20:32:05 -0400 (Thu, 03 Apr 2008) | 1 line

blocked for trunk....woot
................
r112710 | file | 2008-04-03 20:49:01 -0400 (Thu, 03 Apr 2008) | 9 lines

Blocked revisions 112709 via svnmerge

........
r112709 | file | 2008-04-03 21:48:01 -0300 (Thu, 03 Apr 2008) | 2 lines

One thing at a time... let's get 1.4 building.

........

................
r112712 | file | 2008-04-03 20:53:19 -0400 (Thu, 03 Apr 2008) | 10 lines

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

........
r112711 | file | 2008-04-03 21:52:36 -0300 (Thu, 03 Apr 2008) | 2 lines

Pass in the path to Zaptel for systems that install Zaptel headers in a separate location.

........

................
r112714 | dhubbard | 2008-04-03 20:57:33 -0400 (Thu, 03 Apr 2008) | 1 line

sleep long enough for the zaptel timer error message to display before exit
................
r112785 | phsultan | 2008-04-04 13:32:46 -0400 (Fri, 04 Apr 2008) | 15 lines

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

........
r112766 | phsultan | 2008-04-04 19:16:59 +0200 (Fri, 04 Apr 2008) | 7 lines

Prevent call connections when codecs don't match.

(closes issue #10604)
Reported by: keepitcool
Patches:
      branch-1.4-10604-2.diff uploaded by phsultan (license 73)
Tested by: phsultan
........

................
r112821 | phsultan | 2008-04-04 15:28:49 -0400 (Fri, 04 Apr 2008) | 9 lines

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

........
r112820 | phsultan | 2008-04-04 21:26:15 +0200 (Fri, 04 Apr 2008) | 1 line

Free newly allocated channel before returning
........

................
r112874 | murf | 2008-04-04 21:33:13 -0400 (Fri, 04 Apr 2008) | 15 lines

Found a little problem with the sip request handling that could lead to a quick crash of asterisk, and a road to a DOS attack if left unfixed.

Attaching to a running asterisk with "telnet hostname 5060", I would input "something", then hit return three times, and asterisk crashes.

I traced it to handle_request_do(), which zeroes out the data (an ast_str ptr) if the string is too short. 
Instead of freeing the struct and nulling the pointer, it now just resets it, because this 
ast_str is expected by the calling routine to still be there after handle_request_do() returns.

This appears to fix the crash. I assume that it was introduced with ast_str's being adopted.  It's a subtle and easy-to-miss sort of problem.

I also found all the places where the req.data is freed, and made sure the ptr is Nulled out as well; 
no good leaving bad ptrs laying around-- I didn't need to do this, but it seemed a good thing to do...



................
r112906 | twilson | 2008-04-05 00:59:25 -0400 (Sat, 05 Apr 2008) | 2 lines

Multi-line support for phoneprov

................
r112939 | twilson | 2008-04-05 03:58:42 -0400 (Sat, 05 Apr 2008) | 2 lines

Clean up some memory leak/ref counting issues

................
r112972 | tilghman | 2008-04-05 09:24:12 -0400 (Sat, 05 Apr 2008) | 6 lines

AsyncAGI should not close the manager session on error.
(closes issue #12370)
 Reported by: srt
 Patches: 
       asterisk-12370.diff uploaded by srt (license 378)

................
r113009 | file | 2008-04-07 10:54:42 -0400 (Mon, 07 Apr 2008) | 2 lines

Put my slinfactory changes back in.

................
r113013 | jpeeler | 2008-04-07 11:18:10 -0400 (Mon, 07 Apr 2008) | 15 lines

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

........
r113012 | jpeeler | 2008-04-07 10:16:44 -0500 (Mon, 07 Apr 2008) | 7 lines

(closes issue #12362)
(closes issue #12372)
Reported by: vinsik
Tested by: tecnoxarxa

This one line change makes an if inside a for loop (in realtime_peer) check all the ast_variables the loop was intending to test rather than just the first one.

........

................
r113066 | mmichelson | 2008-04-07 12:12:30 -0400 (Mon, 07 Apr 2008) | 21 lines

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

........
r113065 | mmichelson | 2008-04-07 11:08:45 -0500 (Mon, 07 Apr 2008) | 13 lines

This fix prevents a deadlock that was experienced in chan_local. There was
deadlock prevention in place in chan_local, but it would not work in a specific
case because the channel was recursively locked. By unlocking the channel prior
to calling the generator's generate callback in ast_read_generator_actions(), we
prevent the recursive locking, and therefore the deadlock.

(closes issue #12307)
Reported by: callguy
Patches:
      12307.patch uploaded by putnopvut (license 60)
Tested by: callguy


........

................
r113119 | qwell | 2008-04-07 14:02:51 -0400 (Mon, 07 Apr 2008) | 16 lines

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

........
r113118 | qwell | 2008-04-07 13:00:09 -0500 (Mon, 07 Apr 2008) | 8 lines

Allow playback with noanswer (and add earlyrtp option).

(closes issue #9077)
Reported by: pj
Patches:
      earlyrtp.diff uploaded by wedhorn (license 30)
Tested by: pj, qwell, DEA, wedhorn

........

................
r113170 | twilson | 2008-04-07 14:57:21 -0400 (Mon, 07 Apr 2008) | 2 lines

atoi(NULL) is bad

................
r113172 | tilghman | 2008-04-07 15:06:46 -0400 (Mon, 07 Apr 2008) | 11 lines

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

........
r113117 | tilghman | 2008-04-07 12:51:49 -0500 (Mon, 07 Apr 2008) | 3 lines

Force ast_mktime() to check for DST, since strptime(3) does not.
(Closes issue #12374)

........

................
r113207 | mmichelson | 2008-04-07 16:22:06 -0400 (Mon, 07 Apr 2008) | 7 lines

This is a "fix" for something that's been bugging the crap out of me for a while.
The variable name "flag" to distinguish between whether a message is being forwarded or
is new is not a helpful name. The newly added doxygen documentation to app_voicemail is
tremendously helpful, but I still just...hate this variable name. I think is_new_message
is more indicative of what its purpose is.


................
r113241 | jpeeler | 2008-04-07 17:35:48 -0400 (Mon, 07 Apr 2008) | 23 lines

Merged revisions 113013 via svnmerge from 
https://origsvn.digium.com/svn/asterisk/trunk

................
r113013 | jpeeler | 2008-04-07 10:18:10 -0500 (Mon, 07 Apr 2008) | 15 lines

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

........
r113012 | jpeeler | 2008-04-07 10:16:44 -0500 (Mon, 07 Apr 2008) | 7 lines

(closes issue #12362)
(closes issue #12372)
Reported by: vinsik
Tested by: tecnoxarxa

This one line change makes an if inside a for loop (in realtime_peer) check all the ast_variables the loop was intending to test rather than just the first one.

........

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

................
r113243 | qwell | 2008-04-07 17:49:27 -0400 (Mon, 07 Apr 2008) | 1 line

Document 'originate' permission in manager sample config.
................
r113245 | tilghman | 2008-04-07 18:16:46 -0400 (Mon, 07 Apr 2008) | 2 lines

Additional note

................
r113297 | file | 2008-04-08 11:05:35 -0400 (Tue, 08 Apr 2008) | 12 lines

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

........
r113296 | file | 2008-04-08 12:03:43 -0300 (Tue, 08 Apr 2008) | 4 lines

If audio suddenly gets fed into one side of a channel after a lapse of frames flush the other factory so that old audio does not remain in the factory causing the sync code to not execute.
(closes issue #12296)
Reported by: jvandal

........

................
r113349 | tilghman | 2008-04-08 11:48:58 -0400 (Tue, 08 Apr 2008) | 15 lines

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

........
r113348 | tilghman | 2008-04-08 10:39:16 -0500 (Tue, 08 Apr 2008) | 7 lines

Move check for still-bridged channels out a little further, to avoid possible
deadlocks.  (Closes issue #12252)
Reported by: callguy
 Patches: 
       20080319__bug12252.diff.txt uploaded by Corydon76 (license 14)
 Tested by: callguy

........

................
r113400 | tilghman | 2008-04-08 12:54:21 -0400 (Tue, 08 Apr 2008) | 14 lines

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

........
r113399 | tilghman | 2008-04-08 11:51:28 -0500 (Tue, 08 Apr 2008) | 6 lines

Add security note on astgenkey's manpage.
(closes issue #12373)
 Reported by: lmamane
 Patches: 
       20080406__bug12373.diff.txt uploaded by Corydon76 (license 14)

........

................
r113403 | qwell | 2008-04-08 13:00:55 -0400 (Tue, 08 Apr 2008) | 9 lines

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

........
r113402 | qwell | 2008-04-08 11:56:52 -0500 (Tue, 08 Apr 2008) | 1 line

Work around some silliness caused by sys/capability.h - this should fix compile errors a number of users have been experiencing.
........

................
r113452 | qwell | 2008-04-08 13:32:42 -0400 (Tue, 08 Apr 2008) | 9 lines

Move AST_FEATURE_FLAG_* and FEATURE_RETURN_* to features.h so that they can be used by modules.

(closes issue #12384)
Reported by: fnordian
Patches:
      features.patch uploaded by fnordian (license 110)

(patch modified by me, to give FEATURE_RETURN_* an AST_ prefix)

................
r113455 | qwell | 2008-04-08 14:08:35 -0400 (Tue, 08 Apr 2008) | 12 lines

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

........
r113454 | qwell | 2008-04-08 13:07:49 -0500 (Tue, 08 Apr 2008) | 4 lines

Add support for several new(ish) devices - most notably, 7942/7945, 7962/7965, 7975.

Thanks to Greg Oliver for providing me the required information.

........

................
r113505 | qwell | 2008-04-08 14:49:21 -0400 (Tue, 08 Apr 2008) | 9 lines

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

........
r113504 | qwell | 2008-04-08 13:48:55 -0500 (Tue, 08 Apr 2008) | 1 line

Add a little more that is required for previously added devices.
........

................
r113508 | mmichelson | 2008-04-08 15:09:16 -0400 (Tue, 08 Apr 2008) | 15 lines

Blocked revisions 113507 via svnmerge

........
r113507 | mmichelson | 2008-04-08 14:07:38 -0500 (Tue, 08 Apr 2008) | 8 lines

Fix potential buffer overflow that could happen if more than 100 announce files
were specified when calling ParkAndAnnounce. This overflow is not exploitable remotely
and so there is no need for a security advisory.

(closes issue #12386)
Reported by: davidw


........

................
r113559 | tilghman | 2008-04-08 17:33:11 -0400 (Tue, 08 Apr 2008) | 6 lines

Add commandline tool for doing CLI commands through AMI (instead of using asterisk -rx)
(closes issue #12389)
 Reported by: davevg
 Patches: 
       astcli uploaded by davevg (license 209)

................
r113597 | twilson | 2008-04-08 21:36:58 -0400 (Tue, 08 Apr 2008) | 10 lines

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

........
r113596 | twilson | 2008-04-08 20:34:25 -0500 (Tue, 08 Apr 2008) | 2 lines

Initialize fr->cacheable to make valgrind happy

........

................
r113647 | tilghman | 2008-04-09 09:23:44 -0400 (Wed, 09 Apr 2008) | 6 lines

Additional enhancements
(closes issue #12390)
 Reported by: tzafrir
 Patches: 
       astcli_fixes.diff uploaded by tzafrir (license 46)

................
r113649 | tilghman | 2008-04-09 09:55:28 -0400 (Wed, 09 Apr 2008) | 6 lines

Permit callee to continue in the dialplan, after caller has hung up.
(closes issue #11954)
 Reported by: johan
 Patches: 
       app_dial_rev104031.patch uploaded by johan (license 334)

................
r113682 | mmichelson | 2008-04-09 10:41:58 -0400 (Wed, 09 Apr 2008) | 17 lines

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

........
r113681 | mmichelson | 2008-04-09 09:40:05 -0500 (Wed, 09 Apr 2008) | 9 lines

If Asterisk receives a 488 on an INVITE (not a reinvite), then
we should not send a BYE.

(closes issue #12392)
Reported by: fnordian
Patches:
      chan_sip.patch uploaded by fnordian (license 110) with small modification from me


........

................
r113731 | tilghman | 2008-04-09 12:16:44 -0400 (Wed, 09 Apr 2008) | 6 lines

Permit message wrap-around during message retrieval.
(closes issue #12254)
 Reported by: andrew
 Patches: 
       bug-12253.diff uploaded by snuffy (license 35)

................
r113752 | tilghman | 2008-04-09 12:23:30 -0400 (Wed, 09 Apr 2008) | 2 lines

Mark recent additions from #11954 and #12254

................
r113785 | file | 2008-04-09 12:52:04 -0400 (Wed, 09 Apr 2008) | 12 lines

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

........
r113784 | file | 2008-04-09 13:50:45 -0300 (Wed, 09 Apr 2008) | 4 lines

If we receive an AUTHREQ from the remote server and we are unable to reply (for example they have a secret configured, but we do not) then queue a hangup frame on the Asterisk channel. This will cause the channel to hangup and a HANGUP to be sent via IAX2 to the remote side which is the proper thing to do in this scenario.
(closes issue #12385)
Reported by: viraptor

........

................
r113834 | qwell | 2008-04-09 13:41:09 -0400 (Wed, 09 Apr 2008) | 7 lines

Move all messages wrapped in skinnydebug from debug to verbose.

(closes issue #12224)
Reported by: DEA
Patches:
      chan_skinny-debug-log.txt uploaded by DEA (license 3)

................
r113836 | mmichelson | 2008-04-09 13:48:33 -0400 (Wed, 09 Apr 2008) | 14 lines

There was a subtle logical difference between 1.4 and trunk with regards to how timeouts
were handled. In 1.4, if the absolute timeout were reached on a call, no matter what
the return value of ast_spawn_extension was, the pbx would attempt to go to the 'T'
extension or hangup otherwise. The rearrangement of this function in trunk made this check
only happen in the case that ast_spawn_extension returned 0. If ast_spawn_extension returned
1, then the fact that the timeout expired resulted in a no-op, and would cause an infinite
loop to occur in __ast_pbx_run. This change fixes this problem. Now timeouts will
behave as they did in 1.4

(closes issue #11550)
Reported by: pj
Tested by: putnopvut


................
r113838 | qwell | 2008-04-09 13:56:07 -0400 (Wed, 09 Apr 2008) | 2 lines

Fix a small file handle "leak" pointed out by jjshoe on #asterisk.

................
r113840 | file | 2008-04-09 14:05:40 -0400 (Wed, 09 Apr 2008) | 4 lines

Enable enough RTP bridging to allow P2P to work.
(closes issue #11901)
Reported by: pj

................
r113875 | tilghman | 2008-04-09 15:00:40 -0400 (Wed, 09 Apr 2008) | 12 lines

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

........
r113874 | tilghman | 2008-04-09 13:57:33 -0500 (Wed, 09 Apr 2008) | 4 lines

If the [csv] section does not exist in cdr.conf, then an unload/load sequence
is needed to correct the problem.  Track whether the load succeeded with a
variable, so we can fix this with a simple reload event, instead.

........

................
r113928 | mmichelson | 2008-04-09 16:56:14 -0400 (Wed, 09 Apr 2008) | 16 lines

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

........
r113927 | mmichelson | 2008-04-09 15:54:31 -0500 (Wed, 09 Apr 2008) | 8 lines

We need to set the persistant_route [sic] parameter for the sip_pvt
during the initial INVITE, no matter if we're building the route set from
an INVITE request or response.

(closes issue #12391)
Reported by: benjaminbohlmann
Tested by: benjaminbohlmann

........

................
r113980 | mmichelson | 2008-04-09 18:32:32 -0400 (Wed, 09 Apr 2008) | 8 lines

Fix a crash that happened due to accessing free'd memory

(closes issue #12396)
Reported by: tcalosi
Patches:
      12396.patch uploaded by putnopvut (license 60)
	  Tested by: tcalosi

................
r114022 | file | 2008-04-10 09:28:30 -0400 (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 09:45:45 -0400 (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 10:53:19 -0400 (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 11:10:47 -0400 (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 11:59:13 -0400 (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 13:27:16 -0400 (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 15:04:29 -0400 (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 15:58:36 -0400 (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 16:28:40 -0400 (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 18:02:32 -0400 (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 10:54:22 -0400 (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 11:49:35 -0400 (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 17:04:46 -0400 (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 17:39:44 -0400 (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 18:09:04 -0400 (Fri, 11 Apr 2008) | 2 lines

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

................
r114080 | twilson | 2008-04-11 18:23:34 -0400 (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 18:48:52 -0400 (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 19:12:16 -0400 (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 19:21:54 -0400 (Fri, 11 Apr 2008) | 3 lines

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

................
r114090 | tilghman | 2008-04-11 19:26:56 -0400 (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 12:13:25 -0400 (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 12:21:29 -0400 (Sat, 12 Apr 2008) | 1 line

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

Use ast_mkdir instead of mkdir
(Closes issue #12430)

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

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

................
r114101 | file | 2008-04-14 09:53:33 -0400 (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 10:53:33 -0400 (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 11:01:36 -0400 (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 11:36:02 -0400 (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 12:25:09 -0400 (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 12:32:59 -0400 (Mon, 14 Apr 2008) | 2 lines

Make tab-completion work for all cases

................
r114118 | mmichelson | 2008-04-14 13:42:20 -0400 (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 14:34:17 -0400 (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 15:12:27 -0400 (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 15:58:52 -0400 (Mon, 14 Apr 2008) | 2 lines

Need a new buffer for each loop

................
r114131 | qwell | 2008-04-15 11:20:47 -0400 (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 12:18:38 -0400 (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 13:17:37 -0400 (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 13:21:58 -0400 (Tue, 15 Apr 2008) | 8 lines


[... 17263 lines stripped ...]



More information about the asterisk-commits mailing list