Commit Graph

13786 Commits

Author SHA1 Message Date
Jeff Peeler
09c3005ca6 Do not modify the gain settings on data calls.
(The digital flag actually represents a data call.)

(closes issue #15972)
Reported by: udosw
Patches: 
      transcap_digital_fix.diff.txt uploaded by alecdavis (license 585)
Tested by: alecdavis


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@232090 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-12-02 00:42:58 +00:00
Russell Bryant
42d9c2c466 Fix a warning pointed out by buildbot.
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@232007 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-12-01 23:25:36 +00:00
Jeff Peeler
8d6faec4b6 log channel name in dev mode as well
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@231926 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-12-01 21:52:39 +00:00
Jeff Peeler
9405b9e2b7 Fix crash with invalid frame data
The crash was happening as a result of a frame containing an invalid data
pointer, but was set with data length of zero. The few times the issue was
reproduced it _seemed_ that the frame was queued properly, that is the data
pointer was set to NULL. I never could reproduce the crash so as a last resort
the crash has been fixed, but a check in __ast_read has been added to give as
much information about the source of problematic frames in the future.

(closes issue #16058)
Reported by: atis


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@231911 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-12-01 21:29:31 +00:00
David Vossel
031d3b8d1f WaitExten m option with no parameters generates frame with zero datalen but non-null data ptr
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@231853 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-12-01 21:14:31 +00:00
Matthew Nicholson
64c5037988 Ignore unknown formats in ast_format_str_reduce() and return an error if no know formats are found.
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@231740 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-12-01 15:34:57 +00:00
Matthew Nicholson
6a680d9b28 Remove duplicate entries from voicemail format lists. This prevents app_voicemail from entering an infinite loop when the same format is specified twice in the format list.
(closes issue #15625)
Reported by: Shagg63
Tested by: mnicholson

Review: https://reviewboard.asterisk.org/r/429/


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@231614 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-30 21:11:44 +00:00
David Vossel
4f89878b7f fixes crash caused by RTP comfort noise payload greater than 24 bytes
AST-2009-010

(closes issue #16242)
Reported by: amorsen
Patches:
      issue16242.diff uploaded by oej (license 306)
Tested by: amorsen, oej, dvossel



git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@231441 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-30 17:14:08 +00:00
David Vossel
264a1a78a9 app_queue crashes randomly, often during call-transfers
In app_queue, it is possible for a call_queue to be destroyed
while another object still holds a pointer to it.  This patch
converts call_queue objects to ao2 objects allowing them to be
ref counted.  This makes it safe for the queue_ent object in
queue_exec() to reference it's parent call_queue even after it
has left the queue.

(closes issue #15686)
Reported by: Hatrix
Patches:
      v2_queue_ao2.diff uploaded by dvossel (license 671)
Tested by: dvossel, aragon

Review: https://reviewboard.asterisk.org/r/427/



git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@231437 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-30 16:32:58 +00:00
Tilghman Lesher
85cce5cb4b After a frame duplication failure, unlock the channel before returning.
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@231298 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-25 22:31:57 +00:00
David Vossel
c79e242ada fixes solaris segfault on dial with verbosity >= 3
(closes issue #16193)
Reported by: asgaroth
Patches:
      bug_16193_1.4.21.2_vers.diff uploaded by snuffy (license 35)
Tested by: asgaroth, snuffy



git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@231235 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-25 21:38:32 +00:00
David Vossel
8730e034d3 fixes conditional jump or move depending on uninitialised STACK value
(closes issue #16261)
Reported by: edguy3
Patches:
      edguy16261.patch uploaded by edguy3 (license 917)



git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@231233 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-25 21:23:41 +00:00
Kevin P. Fleming
851de5edaa When 'sip set debug' is enabled, and the last line of an incoming SIP message
is not properly newline terminated, ensure that that line is included in the
debug output.

(part of issue #16268)



git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@230875 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-23 15:31:02 +00:00
Kevin P. Fleming
afd2b5e203 Correct fix for issue #16268... the reporter's original patch was very close to correct.
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@230839 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-23 15:09:24 +00:00
Kevin P. Fleming
800a1df304 Ensure that SDP parsing does not ignore the last line of the SDP.
(closes issue #16268)
Reported by: sgimeno


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@230772 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-23 14:13:56 +00:00
Matthew Nicholson
f32239bff8 Copy the peer CDR's userfield to the bridge CDR if it exists. This is necessary for the recordagentcalls option in chan_agent to store the recorded file name in the bridge CDR.
(closes issue #14590)
Reported by: msetim
Patches:
      queue_agent_userfield.patch uploaded by Laureano (license 265)
Tested by: Laureano, mnicholson


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@230627 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-20 20:53:06 +00:00
David Vossel
182c57ce17 fixes MixMonitor thread not exiting when StopMixMonitor is used
(closes issue #16152)
Reported by: AlexMS
Patches:
      stopmixmonitor_1.4.diff uploaded by dvossel (license 671)
Tested by: dvossel, AlexMS

Review: https://reviewboard.asterisk.org/r/424/


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@230508 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-19 21:22:46 +00:00
Michiel van Baak
9d6f7502ac Update copyright year in visible output. (cli)
Spotted by Stuart Henderson


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@230469 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-19 16:09:51 +00:00
Kevin P. Fleming
708e058915 Correct mistaken option name in error message.
The configuration option for allowing hosts to make non-token-based calls
is 'calltokenoptional', not 'calltokenignore'. (reported on asterisk-users)



git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@230246 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-15 17:19:06 +00:00
Joshua Colp
4950fbeebc Respect the maddr parameter in the Via header.
(closes issue #14446)
Reported by: frawd
Patches:
      via_maddr.patch uploaded by frawd (license 610)
Tested by: frawd


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@230144 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-13 22:00:19 +00:00
Joshua Colp
607defd94a Fix a crash caused by two threads thinking they should both free the
chan_local private structure when only one should.

(closes issue #15314)
Reported by: sroberts
Patches:
      Issue15314_Move_Nulling_owner.patch uploaded by davidw (license 780)
Tested by: davidw, lottc


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@230038 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-13 19:44:07 +00:00
Joshua Colp
2ef94c5196 Document a limitation in the AVAILSTATUS variable from ChanIsAvail and provide
a workaround for it that does not change existing behavior.

(closes issue #14426)
Reported by: macli


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@229965 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-13 17:19:59 +00:00
David Vossel
251f196760 fixes merging error, datastore was being freed in the wrong function.
(closes issue #16219)
Reported by: aragon



git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@229669 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-12 16:41:49 +00:00
David Brooks
933ad675be Solaris doesn't like NULL going to ast_log
Solaris will crash if NULL is passed to ast_log. This simple patch simply uses S_OR to
get around this.

(closes issue #15392)
Reported by: yrashk


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@229498 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-11 19:46:19 +00:00
Tilghman Lesher
e04cade814 If two pattern classes start with the same digit and have the same number of characters, they will compare equal.
The example given in the issue report is that of [234] and [246], which have
these characteristics, yet they are clearly not equivalent.  The code still
uses these two characteristics, yet when the two scores compare equal, an
additional check will be done to compare all characters within the class to
verify equality.
(closes issue #15421)
 Reported by: jsmith
 Patches: 
       20091109__issue15421__2.diff.txt uploaded by tilghman (license 14)
 Tested by: jsmith, thedavidfactor


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@229360 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-10 22:09:16 +00:00
David Ruggles
4d366f8147 Fix ExternalIVR Documentation
Remove documentation for event that doesn't function

(closes issue #16220)
Reported by: thedavidfactor
Patches:
      externalivr.txt.20091110.1622.patch uploaded by thedavidfactor (license 903)


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@229355 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-10 21:45:15 +00:00
Joshua Colp
258a49e0dc Remove broken support for direct transcoding between G.726 RFC3551 and G.726 AAL2.
On some systems the translation core would actually consider g726aal2 -> g726 -> signed linear
to be a quicker path then g726aal2 -> signed linear which exposed this problem.

(closes issue #15504)
Reported by: globalnetinc


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@229281 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-10 20:03:14 +00:00
David Ruggles
24261d3fba Document ExternalIVR event tag collision
ExternalIVR uses the D tag for two different event types. This documents that
behavior and how to differentiate between the two cases. Also includes a minor
spelling fix and clarification

(closes issue #16211)
Reported by: thedavidfactor
Patches:
      externalivr.txt.20091109.1507.patch uploaded by thedavidfactor (license 903)


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@229191 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-10 17:23:59 +00:00
David Vossel
cbd0215153 don't crash on log message in solaris
AST-2009-006

(closes issue #16206)
Reported by: bklang
Tested by: bklang



git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@229167 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-10 17:15:57 +00:00
Matthew Nicholson
d9ef686bc3 Reverted revision 202022.
(closes issue #16175)
Reported by: paul-tg


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@229091 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-10 15:22:13 +00:00
Leif Madsen
446f9a0e47 Update WARNING message.
Update a WARNING message to give a suggested fix when encountered.

(closes issue #16198)
Reported by: atis
Tested by: atis

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@228896 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-09 15:37:43 +00:00
Matthew Nicholson
a3887d4511 Perform limited bounds checking when destroying ast_mutex_t structures to make sure we don't try to use negative indices.
(closes issue #15588)
Reported by: zerohalo
Patches:
      20090820__issue15588.diff.txt uploaded by tilghman (license 14)
Tested by: zerohalo


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@228827 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-09 14:16:03 +00:00
David Vossel
8939eece6a fixes audiohook write crash occuring in chan_spy whisper mode.
After writing to the audiohook list in ast_write(), frames
were being freed incorrectly.  Under certain conditions this
resulted in a double free crash.

(closes issue #16133)
Reported by: wetwired

(closes issue #16045)
Reported by: bluecrow76
Patches:
      issue16045.diff uploaded by dvossel (license 671)
Tested by: bluecrow76, dvossel, habile


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@228692 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-06 22:33:27 +00:00
Joshua Colp
0eb5bea853 Don't overwrite caller ID name on a trunk with the configured fullname when using users.conf
(issue ABE-1989)


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@228547 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-06 18:32:58 +00:00
David Vossel
5f6bf9a820 fixes segfault in iLBC
For reasons not yet known, it appears possible for an ast_frame
to have a datalen greater than zero while the actual data is NULL
during Packet Loss Concealment.  Most codecs don't support PLC so
this doesn't affect them.  This patch catches the malformed frame
and prevents the crash from occuring.  Additional efforts to determine
why it is possible for a frame to look like this are still being
investigated.

(issue #16979)



git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@228418 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-06 17:07:13 +00:00
Joshua Colp
3e10834b22 Fix a bug caused by a partially invalid frame (from the jitterbuffer) passing through the Asterisk core.
(closes issue #15560)
Reported by: jvandal
(closes issue #15709)
Reported by: covici


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@228409 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-06 16:41:20 +00:00
Matthew Nicholson
aa51d30173 Properly handle '=' while decoding base64 messages and null terminate strings returned from BASE64_DECODE.
(closes issue #15271)
Reported by: chappell
Patches:
      base64_fix.patch uploaded by chappell (license 8)
Tested by: kobaz


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@228378 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-06 16:26:59 +00:00
David Vossel
15d78bc88b fixes crash in astfd.c
(closes issue #15981)
Reported by: slavon


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@228338 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-06 15:41:41 +00:00
David Vossel
25968fe49b fixes memory leak in func_audiohookinherit.c
(closes issue 0015394)
Reported by: boroda
Patches:
      bug15394_memoryleak_diff2.txt uploaded by dbrooks (license 790)
Tested by: dbrooks, boroda



git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@228272 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-06 15:07:31 +00:00
Jason Parker
d7dfd99014 Fix crash on VPB exception when no hardware is present.
(closes issue #14970)
Reported by: tzafrir
Patches:
      vpb_exception.diff uploaded by tzafrir (license 46)
Tested by: markwaters


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@228079 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-05 19:14:25 +00:00
David Brooks
50c0d05b8a chan_misdn Asterisk 1.4.27-rc2 crash
Crash related to chan_misdn connection. Patch submitted by gknispel_proformatique, tested
by francesco_r. "I have many crash since i have upgraded to Asterisk 1.4.27-rc2. Attached
a full bt." This patch zeros out an ast_frame.

(closes issue #16041)
Reported by: francesco_r


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@228078 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-05 18:59:41 +00:00
Jeff Peeler
cd447927e4 Fix incorrect filename comparsion after monitor file change
The logic to detect if a requested file is indeed a different file from the
current file was incorrect. The main issue being confusion of the use of
filename_base which was previously set without pathing information and then
compared to another full path. Robust file comparison logic has been added
to properly check if two files are the same even if symlinks are used.

(closes issue #15313)
Reported by: caspy
Patches: 
      20091103__issue15313__1.4.diff.txt uploaded by jpeeler (license 325)
      but mostly tilghman's work


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@227944 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-04 23:47:08 +00:00
Matthew Nicholson
3c256882d6 This patch modifies the Dial application to monitor the calling channel for hangups while playing back announcements.
(closes issue #16005)
Reported by: falves11
Patches:
      dial-announce-hangup-fix1.diff uploaded by mnicholson (license 96)
Tested by: mnicholson, falves11

Review: https://reviewboard.asterisk.org/r/407/


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@227827 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-04 20:52:27 +00:00
Matthew Nicholson
841a1d5ed5 Modify the SDP parsing code to parse session and media level items separately.
With the new code, media level proprieties should no longer be confused with session level proprieties. This change also reorganizes some of the SDP parsing code which should make it easier to manage in the future.

(closes issue #14994)
Reported by: frawd
Tested by: frawd, mnicholson, file

Review: https://reviewboard.asterisk.org/r/385/


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@227758 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-04 19:55:44 +00:00
Joshua Colp
d1ec8f8be6 Fix a security issue where it may be possible for someone to execute a cross-site
AJAX request exploit.

(AST-2009-009)


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@227735 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-04 19:25:37 +00:00
Joshua Colp
7f8c4f7278 Fix a security issue where sending a REGISTER with a differing username in the From
URI and Authorization header would reveal whether it was valid or not.

(AST-2009-008)


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@227700 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-04 19:17:39 +00:00
Richard Mudgett
dc898f35c9 Make sure the outgoing flag is cleared if a new channel fails to get created for outgoing calls.
This is the relevant portion of asterisk/trunk -r226648


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@227275 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-03 17:55:47 +00:00
Joshua Colp
f4298a49f0 Fix a bug where an RPID header could be generated with a blank username in the URI.
(closes issue #15909)
Reported by: kobaz


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@227166 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-03 15:36:16 +00:00
Olle Johansson
6ad9ff8acc Fixing bug before someone reports it...
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@227090 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-03 10:48:41 +00:00
Olle Johansson
8239b12ab7 Adding IP address in Contact ACL log message and removing redundant message
(based on kpfleming's feedback)


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.4@227089 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2009-11-03 10:41:45 +00:00