Commit Graph

21130 Commits

Author SHA1 Message Date
Richard Mudgett
f6c540157f * Add missing channel lock to handle_cli_agi_add_cmd().
* Flush any Async AGI commands left over from earlier Async AGI control of
the call.


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@313615 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-13 17:18:49 +00:00
Richard Mudgett
b183d64131 Merged revisions 313579 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.6.2

................
  r313579 | rmudgett | 2011-04-13 11:29:49 -0500 (Wed, 13 Apr 2011) | 48 lines
  
  Merged revisions 313545 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r313545 | rmudgett | 2011-04-13 11:21:24 -0500 (Wed, 13 Apr 2011) | 41 lines
    
    Asterisk does not hangup a channel after endpoint hangs up.
    
    If the call that the dialplan started an AGI script for is hungup while
    the AGI script is in the middle of a command then the AGI script is not
    notified of the hangup.  There are many AGI Exec commands that this can
    happen with.  The reported applications have been: Background, Wait, Read,
    and Dial.  Also the AGI Get Data command.
    
    * Don't wait on the Asterisk channel after it has hung up.  The channel is
    likely to never need servicing again.
    
    * Restored the AGI script's ability to return the AGI_RESULT_HANGUP value
    in run_agi().  It previously only could return AGI_RESULT_SUCCESS or
    AGI_RESULT_FAILURE after the DeadAGI and AGI applications were merged.
    
    (closes issue #17954)
    Reported by: mn3250
    Patches:
          issue17954_v1.8.patch uploaded by rmudgett (license 664)
          issue17954_v1.6.2.patch uploaded by rmudgett (license 664)
          issue17954_v1.4.patch uploaded by rmudgett (license 664)
    Tested by: rmudgett
    JIRA SWP-2171
    
    (closes issue #18492)
    Reported by: devmod
    Tested by: rmudgett
    JIRA SWP-2761
    
    (closes issue #18935)
    Reported by: nvitaly
    Tested by: astmiv, rmudgett
    JIRA SWP-3216
    
    (closes issue #17393)
    Reported by: siby
    Tested by: rmudgett
    JIRA SWP-2727
    
    Review: https://reviewboard.asterisk.org/r/1165/
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@313588 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-13 16:31:50 +00:00
Richard Mudgett
42882cd3bc Bring the dumpchan application inline with "core show channel".
* Added fields that are in "core show channel" to dumpchan output.

* Fixed reuse of formatbuf before the previous string stored there was
used by snprintf.  All output strings now have their own buffer.

* Adjusted the buffer sizes to not be so abusive of the stack now that
there are more buffers.

Change requested by oej.


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@313517 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-12 22:35:53 +00:00
Jonathan Rose
2600de8c9f fixing stupid mistake with putting code before variable declaration
........

  Merged revisions 313435 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.6.2
	
  ........
	  
    r313432 | jrose | 2011-04-12 13:12:29 -0500 (Tue, 12 Apr 2011) | 14 lines
		      
	reload Chan_dahdi memory leak caused by variables
			
	chan_dahdi reloading with variables set via setvar in chan_dahdi.conf would
	stay in the dahdi_pvt structs for individual channels (causing them to just
	continue adding the new ones to the list) and also there was a memory leak
	causes by the conf objects. This patch resolves both of these by using 
	ast_variables_destroy during the loading process.
									
	(closes issue #17450)
	Reported by: nahuelgreco
	Patches:
		patch.diff uploaded by jrose (license 1225)
	Tested by: tilghman, jrose
	Review: https://reviewboard.asterisk.org/r/1170/
																	
  ........

........
																	  
																	  																																	














git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@313436 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-12 18:47:05 +00:00
Jonathan Rose
833c42ce4b Merged revisions 313432 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.6.2

........

  r313432 | jrose | 2011-04-12 13:12:29 -0500 (Tue, 12 Apr 2011) | 14 lines
  
  reload Chan_dahdi memory leak caused by variables

  chan_dahdi reloading with variables set via setvar in chan_dahdi.conf would
  stay in the dahdi_pvt structs for individual channels (causing them to just
  continue adding the new ones to the list) and also there was a memory leak
  causes by the conf objects. This patch resolves both of these by using 
  ast_variables_destroy during the loading process.

  (closes issue #17450)
  Reported by: nahuelgreco
  Patches:
	  patch.diff uploaded by jrose (license 1225)
	  Tested by: tilghman, jrose

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

........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@313434 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-12 18:25:48 +00:00
Richard Mudgett
dde33a1e01 Frames from the inbound channel should go to all outbound channels in app_dial.c.
In app_dial.c:wait_for_answer() frames from the inbound channel should be
sent to all outbound channels instead of only if there is just one
outbound channel.

Control frames like AST_CONTROL_CONNECTED_LINE need to be passed to all of
the the outbound channels.  This can happen if a blond transfer is done by
a remote switch on the inbound channel.

JIRA AST-443
JIRA SWP-2730


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@313369 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-11 23:08:02 +00:00
Richard Mudgett
6dc376082d Backport a restructuring change from trunk to make the next change stand out.
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@313368 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-11 23:03:02 +00:00
Richard Mudgett
46cc4405f4 Added "Connected Line ID" and "Connected Line ID Name" to "core show channel" output.
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@313366 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-11 22:27:25 +00:00
Leif Madsen
55aa84a6e9 Merged revisions 313278 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.6.2

................
  r313278 | lmadsen | 2011-04-11 14:33:03 -0500 (Mon, 11 Apr 2011) | 14 lines
  
  Merged revisions 313277 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r313277 | lmadsen | 2011-04-11 14:30:20 -0500 (Mon, 11 Apr 2011) | 6 lines
    
    Fix detection of OpenSSL 1.0
    
    (closes issue #19093)
    Reported by: tzafrir
    Patches: 
          detect_openssl_10.diff uploaded by tzafrir (license 46)
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@313279 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-11 19:36:40 +00:00
Richard Mudgett
46067d2dc4 Merged revisions 313189 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.6.2

................
  r313189 | rmudgett | 2011-04-11 10:32:53 -0500 (Mon, 11 Apr 2011) | 32 lines
  
  Merged revisions 313188 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r313188 | rmudgett | 2011-04-11 10:27:52 -0500 (Mon, 11 Apr 2011) | 25 lines
    
    Stuck channel using FEATD_MF if caller hangs up at the right time.
    
    The cause was actually a caller hanging up just at the end of the Feature
    Group D DTMF tones that setup the call.  The reason for this is a "guard
    timer" that's implemented using ast_safe_sleep(100).  If the caller
    happens to hang up AFTER the final tone of the DTMF string but BEFORE the
    end of that ast_safe_sleep(), then ast_safe_sleep() will return non-zero.
    This causes the code to bounce to the end of ss_thread(), but it does NOT
    tear down the call properly.
    
    This should be a rare occurrence because the caller has to hang up at
    EXACTLY the right time.  Nonetheless, it was happening quite regularly on
    the reporter's system.  It's not easily reproducible, unless you purposely
    increase the guard-time to 2000 or more.  Once you do that, you can
    reproduce it every time by watching the DTMF debug and hanging up just as
    it ends.
    
    Simply add an ast_hangup() before goto quit.
    
    (closes issue #15671)
    Reported by: jcromes
    Patches:
          issue15671.patch uploaded by pabelanger (license 224)
    Tested by: jcromes
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@313190 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-11 15:40:30 +00:00
Alexandr Anikin
ce3d97c483 fix trivial bug in ooh323_indicate on AST_CONTROL_SRC...
check p->rtp is not null


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@313142 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-09 20:56:17 +00:00
Jonathan Rose
9bb44d964f Merged revisions 313047 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.6.2

........
  r313047 | jrose | 2011-04-07 08:23:01 -0500 (Thu, 07 Apr 2011) | 9 lines
  
  Makes parking lots clear and rebuild properly when features reload is invoked from CLI
  
  Before, default parkinglot in context parkedcalls with ext 700 would always be present and when reload was invoked, the previous parkinglots would not be cleared.
  
  (closes issue #18801)
  Reported by: mickecarlsson
  
  Review: https://reviewboard.asterisk.org/r/1161/
........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@313048 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-07 13:35:33 +00:00
Alec L Davis
8fe6967f1d app_voicemail: close_mailbox change LOG_WARNING to LOG_NOTICE
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@313002 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-07 10:24:51 +00:00
Alec L Davis
20ef1e9c95 Fix ISDN calling subaddr User Specified Odd/Even Flag
Calculation of the Odd/Even flag was wrong.
Implement correct algo, and set odd/even=0 if data would be truncated.
Only allow automatic calculation of the O/E flag, don't let dialplan influence.

(closes issue #19062)
Reported by: festr
Patches: 
      bug19062.diff2.txt uploaded by alecdavis (license 585)
Tested by: festr, alecdavis, rmudgett



git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@313001 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-07 10:19:31 +00:00
Richard Mudgett
4242cb82f4 Crash if ISDN span layer 1 is down on initial load.
Regression from -r312575 B channel shifting during negotiation.

* Also combine updating the alarm flag with clearing the resetting flag.


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@312949 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-05 18:45:24 +00:00
Richard Mudgett
ddc3fac28b Add 416 response to OPTIONS packet.
RFC3261 Section 11.2 says the response code to an OPTIONS packet needs to
be the same as if it were an INVITE.


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@312889 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-05 16:19:35 +00:00
Richard Mudgett
0acdb60dbd Responding to OPTIONS packet with 404 because Asterisk not looking for "s" extension.
The get_destination() function was not using the "s" extension when the
request URI did not specify an extension.  This is a regression caused
when the URI parsing code was extracted into parse_uri().

Made get_destination() substitute the "s" extension when the parsed URI
results in an empty string.

(closes issue #18348)
Reported by: shmaize
Patches:
      issue18348_v1.8.patch uploaded by rmudgett (license 664)
Tested by: shmaize


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@312866 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-05 15:38:14 +00:00
Matthew Nicholson
b5e04b75dc Merged revisions 312764 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.6.2

................
  r312764 | mnicholson | 2011-04-05 09:13:07 -0500 (Tue, 05 Apr 2011) | 15 lines
  
  Merged revisions 312761 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r312761 | mnicholson | 2011-04-05 09:10:34 -0500 (Tue, 05 Apr 2011) | 8 lines
    
    Limit the number of unauthenticated manager sessions and also limit the time they have to authenticate.
    
    AST-2011-005
    
    (closes issue #18996)
    Reported by: tzafrir
    Tested by: mnicholson
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@312766 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-05 14:14:50 +00:00
Jonathan Rose
f6f5340777 Merged revisions 312762 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.6.2

........
  r312762 | jrose | 2011-04-05 09:11:36 -0500 (Tue, 05 Apr 2011) | 1 line
  
  Backporting trunk change to add verbosity to 'L' option in meetme
........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@312765 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-05 14:13:15 +00:00
Richard Mudgett
458a57d1d3 Merged revisions 312574 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.6.2

................
  r312574 | rmudgett | 2011-04-04 11:00:02 -0500 (Mon, 04 Apr 2011) | 45 lines
  
  Merged revisions 312573 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r312573 | rmudgett | 2011-04-04 10:49:30 -0500 (Mon, 04 Apr 2011) | 38 lines
    
    Issues with ISDN calls changing B channels during call negotiations.
    
    The handling of the PROCEEDING message was not using the correct call
    structure if the B channel was changed.  (The same for PROGRESS.) The call
    was also not hungup if the new B channel is not provisioned or is busy.
    
    * Made all call connection messages (SETUP_ACKNOWLEDGE, PROCEEDING,
    PROGRESS, ALERTING, CONNECT, CONNECT_ACKNOWLEDGE) ensure that they are
    using the correct structure and B channel.  If there is any problem with
    the operations then the call is now hungup with an appropriate cause code.
    
    * Made miscellaneous messages (INFORMATION, FACILITY, NOTIFY) find the
    correct structure by looking for the call and not using the channel ID.
    NOTIFY is an exception with versions of libpri before v1.4.11 because a
    call pointer is not available for Asterisk to use.
    
    * Made all hangup messages (DISCONNECT, RELEASE, RELEASE_COMPLETE) find
    the correct structure by looking for the call and not using the channel
    ID.
    
    (closes issue #18313)
    Reported by: destiny6628
    Tested by: rmudgett
    JIRA SWP-2620
    
    (closes issue #18231)
    Reported by: destiny6628
    Tested by: rmudgett
    JIRA SWP-2924
    
    (closes issue #18488)
    Reported by: jpokorny
    JIRA SWP-2929
    
    JIRA AST-437 (The issues fixed here are most likely causing this JIRA issue.)
    JIRA DAHDI-406
    JIRA LIBPRI-33 (Stuck resetting flag likely fixed)
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@312575 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-04 16:10:50 +00:00
Richard Mudgett
d01f7b6dd8 When a call going out an NT-PTMP port gets rejected, Asterisk crashes.
If a call is sent to an ISDN phone that rejects the call with
RELEASE_COMPLETE(cause: call reject(21), or busy(17)) Asterisk crashes.

I could not get my setup to crash.  However, I could see the possibility
from a race condition between queuing an AST_CONTROL_BUSY to the core and
then queueing an AST_CONTROL_HANGUP.  If the AST_CONTROL_BUSY is processed
before the AST_CONTROL_HANGUP is queued, the ast_channel could be
destroyed out from under chan_misdn.

Avoid this particular crash scenario by not queueing the
AST_CONTROL_HANGUP if the AST_CONTROL_BUSY was queued.

(closes issue #18408)
Reported by: wimpy
Patches:
      issue18408_v1.8.patch uploaded by rmudgett (license 664)
Tested by: rmudgett, wimpy

JIRA SWP-2679


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@312509 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-01 23:15:42 +00:00
Richard Mudgett
e59d5151ab CallCompletionRequest()/CallCompletionCancel() exit non-zero if fail.
The CallCompletionRequest()/CallCompletionCancel() dialplan applications
exit nonzero on normal failure conditions.  The nonzero exit causes the
dialplan to hangup immediately.  The dialplan author has no opportunity to
report success/failure to the user.

* Made always return zero so the dialplan can continue.

* Made set CC_REQUEST_RESULT/CC_REQUEST_REASON and
CC_CANCEL_RESULT/CC_CANCEL_REASON channel variables respectively.  Also
documented the values set.

* Reduced the warning about no core instance in CallCompletionCancel() to
a debug message.  It is a normal event and should not be output at the
WARNING level.

(closes issue #18763)
Reported by: p_lindheimer
Patches:
      ccss.patch uploaded by p lindheimer (license 558) Modified
Tested by: p_lindheimer, rmudgett

JIRA SWP-3042


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@312461 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-01 21:31:39 +00:00
Tilghman Lesher
b17b0a7fa8 Merged revisions 312287 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.6.2

................
  r312287 | tilghman | 2011-04-01 05:51:24 -0500 (Fri, 01 Apr 2011) | 14 lines
  
  Merged revisions 312285 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r312285 | tilghman | 2011-04-01 05:36:42 -0500 (Fri, 01 Apr 2011) | 7 lines
    
    Found some leaking file descriptors while looking at ast_FD_SETSIZE dead code.
    
    (issue #18969)
     Reported by: oej
     Patches: 
           20110315__issue18969__14.diff.txt uploaded by tilghman (license 14)
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@312288 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-01 10:58:45 +00:00
Tilghman Lesher
479b3fed00 Reload must react correctly against a possibly changed table, so dropping the conditional reload flag.
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@312286 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-01 10:44:33 +00:00
Alec L Davis
62e679f784 Merged revisions 312210 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.6.2

................
  r312210 | alecdavis | 2011-04-01 21:47:29 +1300 (Fri, 01 Apr 2011) | 29 lines
  
  Merged revisions 312174 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r312174 | alecdavis | 2011-04-01 21:29:49 +1300 (Fri, 01 Apr 2011) | 23 lines
    
    voicemail: get real last_message_index and count_messages, ODBC resequence
    
    change last_message_index to read the max msgnum stored in the database
    change count_messages to actually count the number of messages.
    
    last_message_index change:
      This fixed overwriting of the last message if msgnum=0 was missing.
      Previously every incoming message would overwrite msgnum=1.
    count_messages change:
      allows us to detect when requencing is required in opneA_mailbox.
    resequence enabled for ODBC storage:
      Assists with fixing up corrupt databases with gaps, but only when
      a user actively opens there mailboxes.
    
    (closes issue #18692,#18582,#19032)
    Reported by: elguero
    Patches: 
          based on odbc_resequence_mailbox2.1.diff uploaded by elguero (license 37)
    Tested by: elguero, nivek, alecdavis
    
    Review: https://reviewboard.asterisk.org/r/1153/
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@312211 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-01 09:03:11 +00:00
Alec L Davis
83aeb52dd0 Merged revisions 312103 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.6.2

................
  r312103 | alecdavis | 2011-04-01 20:25:54 +1300 (Fri, 01 Apr 2011) | 22 lines
  
  Merged revisions 312070 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r312070 | alecdavis | 2011-04-01 19:46:56 +1300 (Fri, 01 Apr 2011) | 16 lines
    
    app_voicemail: close_mailbox needs to respect additional messages while mailbox is open.
    
    close_mailbox leave gaps in message sequence if messages are deleted and new messages
    arrive during this time, this is because the shuffle down to slot 0, only shuffles
    the number of pre-existing messages when mailbox is opened, ignoring new arrivals.
    
    Fix: in close_mailbox re-evaluate number of messages before the shuffle, this then includes new arrivals.
    
    Happens on filebased or ODBC storage.
    
    (issues #19032,#18582,#18692,#18998)
    Reported by: alecdavis,tootai,afosorio
    
    Review: https://reviewboard.asterisk.org/r/1153/
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@312117 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-04-01 07:32:12 +00:00
Richard Mudgett
32e0a3510c chan_misdn segfaults when DEBUG_THREADS is enabled.
The segfault happens because jb->mutexjb is uninitialized from the
ast_malloc().  The internals of ast_mutex_init() were assuming a nonzero
value meant mutex tracking initialization had already happened.  Recent
changes to mutex tracking code to reduce excessive memory consumption
exposed this uninitialized value.

Converted misdn_jb_init() to use ast_calloc() instead of ast_malloc().
Also eliminated redundant zero initialization code in the routine.

(closes issue #18975)
Reported by: irroot


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@312022 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-31 20:11:40 +00:00
Tilghman Lesher
8439e8344c Incorrect default example; the field is actually internally named "clid", not "callerid".
(closes issue #19040)
Reported by: wcselby
Tested by: tilghman


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@311930 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-31 06:43:18 +00:00
Richard Mudgett
28bfbccfb7 Update some setup_dahdi_int() comments.
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@311874 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-30 01:56:05 +00:00
Tilghman Lesher
b8aef91ce8 Remove extraneous check from integer-type fields.
(closes issue #19027)
 Reported by: mlehner
 
Review: https://reviewboard.asterisk.org/r/1149/


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@311799 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-29 07:08:39 +00:00
Russell Bryant
0a186e3f4f Cross-reference VoiceMail() and VoiceMailMain() in the xml docs.
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@311751 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-28 22:00:01 +00:00
Alexandr Anikin
9b64fbc06c correct return values in ooh323_indicate for AST_CONTROL_T38_PARAMETERS
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@311687 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-27 21:47:13 +00:00
Brett Bryant
51ce432d07 This patch fixes a bug with MeetMe behavior where the 'P' option for always
prompting for a pin is ignored for the first caller.

(closes issue #18070)
Reported by: mav3rick

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


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@311615 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-23 21:54:11 +00:00
Brett Bryant
a54ab29087 Fix a possible crash in sip/reqresp_parser.c that is caused by a possible null
value.

(closes issue #18821)
Reported by: cmaj
Patches: 
      patch-reqresp_parser_sip_uri_domain_cmp_c_locale-crash-1.8.3-rc2.diff.tx
      uploaded by cmaj (license 830)


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@311612 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-23 21:45:46 +00:00
Terry Wilson
2f95620a2f Don't use static declared buf in parse_name_andor_addr
This function isn't used anywhere yet, but we definitely don't want
to keep the same value for buf between calls to the function.


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@311558 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-23 02:24:53 +00:00
David Vossel
a00e99ec56 Merged revisions 311496 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.6.2

........
  r311496 | dvossel | 2011-03-22 10:24:45 -0500 (Tue, 22 Mar 2011) | 2 lines
  
  Fixes memory leak in MeetMe AMI action
........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@311497 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-22 15:25:24 +00:00
Jonathan Rose
7cf95da39a Changes some print statements/events to use a blank string in place of NULL if the string in question is NULL.
This is supposed to improve Solaris compatibility since Solaris goes berserk when trying to output NULL strings.

(closes issue #18759)
Reported by: bklang
Patches:
      null-strings.patch uploaded by bklang (license 919)



git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@311352 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-18 16:19:05 +00:00
Matthew Nicholson
87b246e421 Properly populate the LOCALSTATIONID channel variable.
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@311342 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-18 16:02:50 +00:00
Richard Mudgett
4f3cf039f4 Race condition when ISDN CallRerouting/CallDeflection invoked.
The queued AST_CONTROL_BUSY could sometimes be processed before the
call_forward dial string is recognized.

* Moved setting the call_forwarding dial string after sending a response
to the initiator and just queue an empty frame to wake up the media thread
instead of an AST_CONTROL_BUSY.

* Added check for empty rerouting/deflection number and respond with an
error.


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@311297 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-18 02:59:05 +00:00
Richard Mudgett
93601856b6 Merged revision 310986 from
https://origsvn.digium.com/svn/asterisk/be/branches/C.3-bier

..........
  r310986 | rmudgett | 2011-03-16 13:56:28 -0500 (Wed, 16 Mar 2011) | 28 lines

  Dial() o option broke when connected line feature added.

  The patch restores the o option behavior and adds the ability to specify
  the CallerID.  The Dial o and f options are complementary to each other.
  The o option stores the CallerID on the outgoing channel as the channel's
  CallerID.  The f option forces the CallerID sent by the outgoing channel.

  o(x) - The argument 'x' is optional.  If not present, then specify that
  the CallerID that was present on the *calling* channel be stored as the
  CallerID on the *called* channel.  This was the behavior of Asterisk 1.0
  and earlier.  If present, then specify the CallerID stored on the *called*
  channel.  Note that o(${CALLERID(all)}) is similar to option o without
  parameters.

  f(x) - The argument 'x' is optional and its presence changes the behavior
  of this option.  If not present, then force the outgoing CallerID on a
  call-forward or deflection to the dialplan extension for this Dial() using
  a dialplan 'hint'.  For example, some PSTNs do not allow CallerID to be
  set to anything other than the numbers assigned to you.  If present, then
  force the outgoing CallerID to 'x'.

  Patches:
	jira_abe_2752_dial_fo_options.patch uploaded by rmudgett (license 664)
  Tested by: rmudgett

  JIRA ABE-2752
  JIRA SWP-3096
..........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@311295 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-18 02:22:07 +00:00
Jonathan Rose
ef01ba5ff2 This fixes a nasty chanspy bug which was causing a channel leak every time a spied on channel made a call.
In addition to the above, it makes certain channel destruction occurs so that applications don't get stuck waiting for datastore destruction while monitored by chanspy.

(closes issue #18742)
Reported by: jkister
Tested by: jkister, jcovert, jrose

Review: http://reviewboard.digium.internal/r/106/



git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@311197 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-17 19:03:34 +00:00
Matthew Nicholson
0e86babe2f Merged revisions 311140 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.6.2

........
  r311140 | mnicholson | 2011-03-17 09:58:52 -0500 (Thu, 17 Mar 2011) | 4 lines
  
  Don't write items to the manager socket twice.
  
  AST-2011-003
  
  (closes issue 0018987)
  Reported by: ks-steven
........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@311141 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-17 15:00:33 +00:00
Alec L Davis
d0829474b2 Merged revisions 311049 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.6.2

................
  r311049 | alecdavis | 2011-03-17 23:45:47 +1300 (Thu, 17 Mar 2011) | 17 lines
  
  Merged revisions 311048 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r311048 | alecdavis | 2011-03-17 23:43:35 +1300 (Thu, 17 Mar 2011) | 12 lines
    
    Remove extra quote in indications.conf 
    
    Picking low hanging fruit.
    
    (closes issue #18971)
    Reported by: IgorG
    Patches: 
          based on indications.conf.sample.diff uploaded by IgorG (license 20)
    Tested by: IgorG
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@311050 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-17 10:49:41 +00:00
Terry Wilson
3cc8ae22cd Merged revisions 310998 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.6.2

........
  r310998 | twilson | 2011-03-16 14:46:36 -0500 (Wed, 16 Mar 2011) | 11 lines
  
  Fix crash on fdopen failure
  
  See security advisory AST-2011-004
  
  (closes issue #18845)
  Reported by: cmaj
  Patches: 
      patch-main-tcptls-1.8.3-rc2-open-session-crash-take2.diff.txt uploaded by cmaj (license 830)
      patch-main-tcptls-1.8.3-rc2-open-session-crash-take3.diff.txt uploaded by cmaj (license 830)
  Tested by: cmaj, twilson
........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@310999 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-16 19:47:59 +00:00
Terry Wilson
d37bdd02dc Merged revisions 310992 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.6.2

........
  r310992 | twilson | 2011-03-16 14:23:03 -0500 (Wed, 16 Mar 2011) | 4 lines
  
  Don't keep trying to write to a closed connection
  
  See security advisory AST-2011-003.
........


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@310993 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-16 19:26:57 +00:00
Terry Wilson
bec22e5c1f Merged revisions 310889 via svnmerge from
https://origsvn.digium.com/svn/asterisk/branches/1.6.2

................
  r310889 | twilson | 2011-03-16 12:03:27 -0500 (Wed, 16 Mar 2011) | 36 lines
  
  Merged revisions 310888 via svnmerge from 
  https://origsvn.digium.com/svn/asterisk/branches/1.4
  
  ........
    r310888 | twilson | 2011-03-16 11:58:42 -0500 (Wed, 16 Mar 2011) | 29 lines
    
    Don't delay DTMF in core bridge while listening for DTMF features
    
    This patch is mostly the work of Olle Johansson. I did some cleanup and
    added the silence generating code if transmit_silence is set.
    
    When a channel listens for DTMF in the core bridge, the outbound DTMF is not
    sent until we have received DTMF_END. For a long DTMF, this is a disaster. We
    send 4 seconds of DTMF to Asterisk, which sends no audio for those 4 seconds.
    Some products see this delay and the time skew on RTP packets that results and
    start ignoring the audio that is sent afterward.
    
    With this change, the DTMF_BEGIN frame is inspected and checked. If it matches
    a feature code, we wait for DTMF_END and activate the feature as before. If
    transmit_silence=yes in asterisk.conf, silence is sent if we paritally match a
    multi-digit feature. If it doesn't match a feature, the frame is forwarded
    along with the DTMF_END without delay. By doing it this way, DTMF is not delayed.
    
    (closes issue #15642)
    Reported by: jasonshugart
    Patches: 
          issue_15652_dtmf_ast-1.4.patch.txt uploaded by twilson (license 396)
    Tested by: globalnetinc, jde
    
    (closes issue #16625)
    Reported by: sharvanek
    
    Review: https://reviewboard.asterisk.org/r/1092/
    Review: https://reviewboard.asterisk.org/r/1125/
  ........
................


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@310902 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-16 17:19:57 +00:00
Tilghman Lesher
feb0df1202 Fix branch compile.
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@310834 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-15 01:48:25 +00:00
Alec L Davis
7ac0138d31 core show locks: display ThreadID in hexadecimal
Allow easier cross referencing of thread ID's with GDB backtraces

(closes issue #18968)
Reported by: alecdavis
Patches: 
      bug18968.diff.txt uploaded by alecdavis (license 585)



git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@310781 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-15 01:00:55 +00:00
Alexandr Anikin
5082c5d44b Introduce t.38 parameters control functionality not full but enough for
Send/RcvFax support

Introduce t.38 controls between asterisk core and channel/proto layers.
Not all parameters are transferred from proto layers but *Fax apps
tested and work ok.

(issue #18693)
Reported by: benngard2
Patches: 
      issue-18693.patch uploaded by may213 (license 454)


git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@310734 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-14 21:45:53 +00:00
Jonathan Rose
1a7d232a5a Undoes 310726 for further analysis
git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/1.8@310733 65c4cc65-6c06-0410-ace0-fbb531ad65f3
2011-03-14 21:30:25 +00:00