8 years agoFix build in devmode for GCC 4.10
Kinsey Moore [Thu, 12 Jun 2014 20:17:37 +0000 (20:17 +0000)]
Fix build in devmode for GCC 4.10

Merged revisions 415980 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoAST-2014-007: Fix DOS by consuming the number of allowed HTTP connections.
Richard Mudgett [Thu, 12 Jun 2014 17:00:08 +0000 (17:00 +0000)]
AST-2014-007: Fix DOS by consuming the number of allowed HTTP connections.

Simply establishing a TCP connection and never sending anything to the
configured HTTP port in http.conf will tie up a HTTP connection.  Since
there is a maximum number of open HTTP sessions allowed at a time you can
block legitimate connections.

A similar problem exists if a HTTP request is started but never finished.

* Added http.conf session_inactivity timer option to close HTTP
connections that aren't doing anything.  Defaults to 30000 ms.

* Removed the undocumented manager.conf block-sockets option.  It
interferes with TCP/TLS inactivity timeouts.

* AMI and SIP TLS connections now have better authentication timeout
protection.  Though I didn't remove the bizzare TLS timeout polling code
from chan_sip.

* chan_sip can now handle SSL certificate renegotiations in the middle of
a session.  It couldn't do that before because the socket was non-blocking
and the SSL calls were not restarted as documented by the OpenSSL

* Fixed an off nominal leak of the ssl struct in
handle_tcptls_connection() if the FILE stream failed to open and the SSL
certificate negotiations failed.

The patch creates a custom FILE stream handler to give the created FILE
streams inactivity timeout and timeout after a specific moment in time
capability.  This approach eliminates the need for code using the FILE
stream to be redesigned to deal with the timeouts.

This patch indirectly fixes most of ASTERISK-18345 by fixing the usage of
the SSL_read/SSL_write operations.

ASTERISK-23673 #close
Reported by: Richard Mudgett

Merged revisions 415841 from

Merged revisions 415854 from

Merged revisions 415896 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoBlocked revisions 415838
Jonathan Rose [Thu, 12 Jun 2014 15:52:20 +0000 (15:52 +0000)]
Blocked revisions 415838

Correct UPGRADE.txt notes in r415825

The change was marked against the wrong version of Asterisk. My apologies.

Merged revisions 415837 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoapp_queue: delayed state can cause early leavewhenempty ringing
Scott Griepentrog [Thu, 12 Jun 2014 15:50:48 +0000 (15:50 +0000)]
app_queue: delayed state can cause early leavewhenempty ringing

In app_queue, device state changes arrive in event messages and
update the queue member status value.  That value is checked in
get_member_status() to decide that the caller should leave when
there are no available members.  Although event messages can be
delayed by other activity, there is no adverse affect by lagged
status except in one specific case: there is only one available
member, it was just rung, and leavewhenempty is enabled set for
ringing members.  This change adds a direct check of the device
state only under this condition where the caller may be dropped
incorrectly, resolving this issue without affecting performance
of app_queue normally.

AST-1248 #close
Reported by: Thomas Arimont

Merged revisions 415833 from

Merged revisions 415835 from

Merged revisions 415836 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoMixMontior: Add class authorization requirements to MixMonitor AMI commands
Jonathan Rose [Thu, 12 Jun 2014 15:39:52 +0000 (15:39 +0000)]
MixMontior: Add class authorization requirements to MixMonitor AMI commands

MixMonitor AMI commands StartMixMonitor and StopMixMonitor lacked class
authorization. StopMixMonitor now requires that the manager user either have
the call or system class authorization. StartMixMonitor is a slightly larger
issue since it can execute shell commands if the right arguments are passed
into it, and we consider this a permission escalation. A security release
will be issued for problem this shortly.

ASTERISK-23609 #close
Reported by: Corey Farrell


Merged revisions 415825 from

Merged revisions 415832 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agores_pjsip_pubsub: unauthenticated remote crash in PJSIP pub/sub framework
Kevin Harwell [Thu, 12 Jun 2014 14:39:29 +0000 (14:39 +0000)]
res_pjsip_pubsub: unauthenticated remote crash in PJSIP pub/sub framework

A remotely exploitable crash vulnerability exists in the PJSIP channel driver's
pub/sub framework. If an attempt is made to unsubscribe when not currently
subscribed and the endpoint's "sub_min_expiry" is set to zero, Asterisk tries
to create an expiration timer with zero seconds, which is not allowed, so an
assertion raised.

The fix was to reject a subscription that is attempting to unsubscribe when not
being already subscribed.  Asterisk now checks for this situation appropriately
and responds with a 400 instead of crashing.


ASTERISK-23489 #close

Merged revisions 415812 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoFix potential deadlock situation in res_pjsip.
Mark Michelson [Thu, 12 Jun 2014 14:15:51 +0000 (14:15 +0000)]
Fix potential deadlock situation in res_pjsip.

SIP transaction timeouts are handled in the PJSIP monitor thread. When
this happens on a subscription, and the subscription is destroyed, the
subscription destruction is dispatched synchronously to the threadpool.
The issue is that the PJSIP dialog is locked by the monitor thread,
and then the dispatched task attempts to lock the dialog. This leads
to a deadlock that causes SIP traffic to no longer be accepted on the
Asterisk server.

The fix here is to treat the monitor thread as if it were a threadpool
thread when it attempts to dispatch synchronous tasks. This way, the
dispatched task turns into a simple function call within the same thread,
and the locking issue is averted.


ASTERISK-23802 #close

Merged revisions 415794 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agores_pjsip_pubsub: Persist subscriptions in sorcery so they are recreated on startup.
Joshua Colp [Thu, 12 Jun 2014 11:34:36 +0000 (11:34 +0000)]
res_pjsip_pubsub: Persist subscriptions in sorcery so they are recreated on startup.

This change makes res_pjsip_pubsub persist inbound subscriptions in sorcery. By default
this uses the local astdb but it can also be configured to store within an outside
database. When Asterisk is started these subscriptions are recreated if they have not
expired. Notifications are sent to the devices which have subscribed and they are none
the wiser that the system has restarted.


Merged revisions 415766 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agosafe_asterisk: Overwrite old safe_asterisk on make install.
Walter Doekes [Thu, 12 Jun 2014 07:52:59 +0000 (07:52 +0000)]
safe_asterisk: Overwrite old safe_asterisk on make install.

From now on, make install will overwrite safe_asterisk with the
latest version. You need to move any local modifications to files
inside /etc/asterisk/startup.d, if you have any.

See also commits r394939 and r397938.

ASTERISK-21965 #close
  safe_asterisk.patch uploaded by jkister (License 6232, modified by me)

Merged revisions 415748 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoformat.c: Fix misuse of hash container function.
Richard Mudgett [Wed, 11 Jun 2014 23:01:19 +0000 (23:01 +0000)]
format.c: Fix misuse of hash container function.

The supplied hash function to a container must be idempotent given the
object's key value to figure out which container bucket the object belongs
in.  Returning a random number or the current container count is not
idempotent.  The "computed hash" value doesn't help find the object later
in those cases.

* Fixed the format_list container to actually be a list since that is how
the container is used.  Conceptually, if more than 283 formats were added
to the format_list then odd things may have happened before the fix.

Merged revisions 415728 from

Merged revisions 415729 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoCLI: correct presence information on core show hints
Scott Griepentrog [Wed, 11 Jun 2014 20:22:23 +0000 (20:22 +0000)]
CLI: correct presence information on core show hints

Adds presence to core show hint and changes presence
string conversion to use the correct function.

ASTERISK-23858 #close

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoCLI: add presence information to core show hints
Scott Griepentrog [Tue, 10 Jun 2014 22:31:04 +0000 (22:31 +0000)]
CLI: add presence information to core show hints

Adds presence state value to output of core show
hints.  Also reformats the output slightly so it
doesn't use as much space as it would otherwise.

                   1000@demo                : SIP/1000              State:Unavailable     Watchers  0

1000@demo           : SIP/1000              State:Unavailable     Presence:Idle            Watchers  0

AFS-53 #close

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoFix build in dev mode due to signed/unsigned mismatch
Kinsey Moore [Tue, 10 Jun 2014 18:32:12 +0000 (18:32 +0000)]
Fix build in dev mode due to signed/unsigned mismatch

Merged revisions 415678 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoPJSIP: PJSIPNotify - Strip content-length headers and add documentation
Jonathan Rose [Tue, 10 Jun 2014 16:06:12 +0000 (16:06 +0000)]
PJSIP: PJSIPNotify - Strip content-length headers and add documentation

Documentation for how to add custom headers/content to notifies created
with the PJSIPNotify manager action was a little sparse and it also
wasn't vetting application of Content-length headers like its chan_sip
equivalent was (so two Content-length headers could be applied... and
PJSIP determines the content length anyway, so it just opens people up
for error). This patch also flips the variable order so that the
variables are interpreted in the same order as they are put in the AMI


Merged revisions 415658 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agochan_ooh323: fix loading module failure if there no accessible h323_log or ooh323...
Alexandr Anikin [Tue, 10 Jun 2014 09:28:55 +0000 (09:28 +0000)]
chan_ooh323: fix loading module failure if there no accessible h323_log or ooh323 config file

change return 1 to return AST_MODULE_LOAD_FAILURE on module load routine
few cosmetic changes

ASTERISK-23814 #close

(closes issue ASTERISK-23814)

Reported by: Igor Goncharovsky

Merged revisions 415599 from

Merged revisions 415602 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agochan_pjsip: Fix bug where custom SIP headers could be duplicated on outgoing INVITEs.
Mark Michelson [Mon, 9 Jun 2014 20:21:42 +0000 (20:21 +0000)]
chan_pjsip: Fix bug where custom SIP headers could be duplicated on outgoing INVITEs.

When using PJSIP_HEADER() to add custom headers to outgoing INVITE requests, certain
situations could result in the headers being duplicated. For instance, if the request
were retransmitted, or if the INVITE were re-sent with authentication credentials,
the custom headers would be re-added to the request.

The fix here is to, after adding the custom headers to the outbound INVITE, remove
the datastore that holds the custom headers to add. This way, there is no risk in
accidentally adding them if the session supplement is called into a second or third

Merged revisions 415579 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agosafe_asterisk: Cleanup additions to r415132.
Walter Doekes [Mon, 9 Jun 2014 12:12:25 +0000 (12:12 +0000)]
safe_asterisk: Cleanup additions to r415132.

* Replaced a stray echo that should've been a message call in
  safe_asterisk. This replaces a conditional log message by a slightly
  different message. Please update your log parsing scripts.
* Made the $NOTIFY mail Subject more verbose by adding the machine name
  and exitstatus.

(Note that a 'make install' still won't overwrite your old safe_asterisk
if it exists. See ASTERISK-21965.)

ASTERISK-23492 #close

Merged revisions 415521 from

Merged revisions 415522 from

Merged revisions 415523 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoautoservice: stop thread on graceful shutdown
Corey Farrell [Mon, 9 Jun 2014 03:50:45 +0000 (03:50 +0000)]
autoservice: stop thread on graceful shutdown

This change adds thread shutdown to autoservice for graceful shutdowns only.
ast_register_cleanup is backported to 1.8 to allow this.  The logger callid
is also released on shutdown in 11+.

ASTERISK-23827 #close
Reported by: Corey Farrell

Merged revisions 415463 from

Merged revisions 415464 from

Merged revisions 415465 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agobridges/bridge_native_rtp: Reconfigure bridge on removal of framehook
Matthew Jordan [Sun, 8 Jun 2014 18:12:53 +0000 (18:12 +0000)]
bridges/bridge_native_rtp: Reconfigure bridge on removal of framehook

This patch is a re-do of r414122.

When r414122 was merged, a major problem with it was uncovered. UNBRIDGE soft
hangup flags have a catastrophic effect on the pbx core if they leak out from
the bridge layer: the channel gets hung up. With the number of threads
involved in a blind transfer, and with the initial patch, it was likely that
this would occur. This caused a large number of test failures

This patch is nearly identical with the one proposed in r414122, save for the
following changes:
 - We explicitly clear the UNBRIDGE flag when setting an after goto on a
   channel in a bridge
 - Defensively, if we encounter an UNBRIDGE flag in the pbx core, we handle it

Merged revisions 415443 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agobridge.h: Remove redundant struct ast_bridge_channel forward declaration.
Richard Mudgett [Sat, 7 Jun 2014 00:42:50 +0000 (00:42 +0000)]
bridge.h: Remove redundant struct ast_bridge_channel forward declaration.

Merged revisions 415427 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agochan_sip: Fix order of variables specified in SIPNotify action
Jonathan Rose [Fri, 6 Jun 2014 21:44:16 +0000 (21:44 +0000)]
chan_sip: Fix order of variables specified in SIPNotify action

Prior to this patch, sequential variables would be ordered in reverse
from the order specified in the manager action.


Merged revisions 415359 from

Merged revisions 415390 from

Merged revisions 415410 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agocore uri: Custom uri parsing error when no query parameters
Kevin Harwell [Fri, 6 Jun 2014 20:45:05 +0000 (20:45 +0000)]
core uri: Custom uri parsing error when no query parameters

If using the custom URI parsing code (not external uriparser lib) and there
was no query parameters the resulting pointer would be NULL and then an
attempt was made to subtract from it.  The pointer is now set to a valid
value if there is no query parameter(s).

Also, in the 'ast_uri_make_host_with_port' function when setting the terminator
on the resulting string it was writing it one past the end of allocated memory.
It now writes the string terminator appropriately.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoPJSIP: Remove premature write of raw formats
Kinsey Moore [Fri, 6 Jun 2014 19:13:08 +0000 (19:13 +0000)]
PJSIP: Remove premature write of raw formats

Currently, there are situations that can occur when using chan_pjsip
and certain dialplan applications (notably ChanSpy()) that can cause
the channel to get no audio with scrolling warnings about format
mismatches. This is caused by a failure to update translation paths on
a mid-call native format update since the raw formats have already
been updated by res_pjsip_sdp_rtp.c in set_caps(). Removing the
premature raw format updates allows the translation paths to be setup
correctly and the raw read and write formats with them.

AFS-63 #close

Merged revisions 415342 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoSplit astobj2.c into more maintainable components.
George Joseph [Fri, 6 Jun 2014 14:12:57 +0000 (14:12 +0000)]
Split astobj2.c into more maintainable components.

Split astobj2.c into the following files to improve maintainability.

astobj2.c - object primitives, object primitive misc and initialization code.
astobj2_private.h - internal object declarations needed by the containers.
astobj2_container.c - generic conainer and container misc code.
astobj2_container_hash.c - hash container specific code.
astobj2_container_rbtree.c - rbtree container specific code.
astobj2_container_private.h - generic container definitions and rtti prototypes.

Merged revisions 415317 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoconfigs/cli_aliases.conf: Two new aliases, plus enhancements for context names.
Rusty Newton [Fri, 6 Jun 2014 12:49:38 +0000 (12:49 +0000)]
configs/cli_aliases.conf: Two new aliases, plus enhancements for context names.

Changed naming of included alias templates to avoid confusion between version names. For example, asterisk12 was for asterisk 1.2, so I changed it to asterisk_1dot2, so that later we can use asterisk_12 for Asterisk 12.

Added alias for "features reload" to the template for Asterisk 11 style syntax template, as features reload was removed in 12, but you can still do "module reload features"

Added alias for "pjsip reload" to the friendly template. It is shorter than "module reload" and if some are like me; I constantly forget that reloading chan_pjsip doesn't parse config. Remembering "pjsip reload" is just easier.

ASTERISK-23654 #close
ASTERISK-23654 #comment Fixed by adding two new aliases and enhancements for context names.


Merged revisions 415301 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoconfig: Fix indentation and missing curlies in config_text_file_load().
Richard Mudgett [Thu, 5 Jun 2014 19:04:02 +0000 (19:04 +0000)]
config: Fix indentation and missing curlies in config_text_file_load().

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoconfig: Fix config files not reloading when only an included file changes.
Richard Mudgett [Thu, 5 Jun 2014 18:02:08 +0000 (18:02 +0000)]
config: Fix config files not reloading when only an included file changes.

The twisted logic determining if a config file should be reloaded was
mostly broken and disabled.  The incorrect test that ASTERISK-23383 fixed
actually reenabled the broken logic.  The incorrect test was causing the
timestamp to always be cleared which caused config files with includes to
always be reloaded.

* Made wildcard includes always cause a reload.  Determining if a file was
deleted cannot be determined without restructuring the cache to determine
if any files are missing from the last files actually loaded.  Also
without refactoring config_text_file_load(), the glob loop couldn't check
more than one file for changes anyway.

* Made remove the cache entry if the file no longer exists when trying to
get its timestamp or it is no longer a regular file.  This fixes the
corner case where the file was loaded, then deleted, then the config
reloaded, then the file restored with the same timestamp, and then the
config reloaded again.

* Made remove the cache entry include list when actually loading the file.
This gets rid of any stale includes the file had from the last time the
file was loaded.

ASTERISK-23683 #close
Reported by: tootai


Merged revisions 415225 from

Merged revisions 415229 from

Merged revisions 415230 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agores_http_websocket: Create a websocket client
Kevin Harwell [Thu, 5 Jun 2014 17:22:35 +0000 (17:22 +0000)]
res_http_websocket: Create a websocket client

Added a websocket server client in Asterisk. Asterisk has a websocket server,
but not a client. The ability to have Asterisk be able to connect to a websocket
server can potentially be useful for future work (for instance this could allow
ARI to connect back to some external system, although more work would be needed
in order to incorporate that).

Also a couple of things to note - proxy connection support has not been
implemented and there is limited http response code handling (basically, it is
connect or not).

Also added an initial new URI handling mechanism to core.  Internet type URI's
are parsed into a data structure that contains pointers to the various parts of
the URI.

(closes issue ASTERISK-23742)
Reported by: Kevin Harwell

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoapp_confbridge: Allow muting of users waiting to enter a ConfBridge
Matthew Jordan [Thu, 5 Jun 2014 14:49:20 +0000 (14:49 +0000)]
app_confbridge: Allow muting of users waiting to enter a ConfBridge

Prior to this patch, users waiting to enter a ConfBridge were not considered
when muted via the CLI or via AMI. Instead, a confusing message would be
emitted stating that the channel did not exist.

This patch allows a user to be muted when waiting to enter a ConfBridge
conference. This is equivalent to start when muted, only toggled via the CLI
or AMI.


#ASTERISK-23824 #close
  rb3582.patch uploaded by tm1000 (License 6524)

Merged revisions 415206 from

Merged revisions 415207 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoPJSIP: Send initial connected line information
Kinsey Moore [Thu, 5 Jun 2014 11:59:09 +0000 (11:59 +0000)]
PJSIP: Send initial connected line information

This makes chan_pjsip send connected line information when it is called
so that connected line information is available on the connected

(closes issue DPMA-442)
Reported by: John Bigelow

Merged revisions 415191 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agosafe_asterisk: Cleanup and debian compatibility.
Walter Doekes [Wed, 4 Jun 2014 20:16:40 +0000 (20:16 +0000)]
safe_asterisk: Cleanup and debian compatibility.

Cleans up the safe_asterisk script and adds the ASTSAFE_FOREGROUND
option that allows the debian asterisk init script to capture the
right pid.

* Drop the vim #modeline which wasn't used. Use test consistently
  without the odd configure xno syntax. Double quote all paths.
  General cleanup.
* Don't output message()s to the console but only to TTY if set.
* Allow TTY to be "no" as well as empty (debian compatibility with
* Add option to export ASTSAFE_FOREGROUND=1 from the init script
  that calls this to disable backgrounding. Debian uses a similar
  method in debian/patches/safe_asterisk-nobg).

ASTERISK-23492 #close

Merged revisions 415132 from

Merged revisions 415171 from

Merged revisions 415172 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agochan_pjsip: Add debug in RTP Engine glue callback
Matthew Jordan [Wed, 4 Jun 2014 14:13:07 +0000 (14:13 +0000)]
chan_pjsip: Add debug in RTP Engine glue callback

This patch adds some debug statements that aid with determining why a direct
media request may or may not be initiated.

Merged revisions 415117 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agores_pjsip_session: Add debug statement for session refreshes
Matthew Jordan [Wed, 4 Jun 2014 14:12:00 +0000 (14:12 +0000)]
res_pjsip_session: Add debug statement for session refreshes

This small patch adds a debug level 3 statement indicating how a session
refresh is being sent - either as a re-INVITE or as an UPDATE - and where
the session refresh is going.

Merged revisions 415115 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoapp_confbridge: Correct verification of conference name length
Corey Farrell [Wed, 4 Jun 2014 07:27:21 +0000 (07:27 +0000)]
app_confbridge: Correct verification of conference name length

Conference names were not checked for maximum length, allowing unexpected
behaviour.  This change adds checking to ensure the maximum length is not
exceeded.  The maximum length is also changed from 32 to AST_MAX_EXTENSION.

ASTERISK-23035 #close
Reported by: Iñaki Cívico
Tested by: Iñaki Cívico
    confbridge-enforce_max-1.8.patch uploaded by coreyfarrell (license 5909)
    confbridge-enforce_max-11up.patch uploaded by coreyfarrell (license 5909)

Merged revisions 415060 from

Merged revisions 415066 from

Merged revisions 415078 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agofunc_odbc: Fix fixed size buffers fix (r414968).
Walter Doekes [Tue, 3 Jun 2014 07:36:42 +0000 (07:36 +0000)]
func_odbc: Fix fixed size buffers fix (r414968).

The change that removed the fixed size buffers in odbc-related code --
removing arbitrary column width limits -- was incomplete. This change
adds: no segfault on writesql without insertsql and return value checks
after strdup.

While I was in the vicinity I cleaned up the linefeeds in the odbc
function descriptions, moved some code for clarity, removed some blobs
and noted (but didn't fix) that the 'odbc write ... exec' CLI command
doesn't behave as the dialplan equivalent when insertsql= is used.

ASTERISK-23582 #close

Merged revisions 414997 from

Merged revisions 414998 from

Merged revisions 414999 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agobridge_native_rtp: Take the bridge type choice of both channels into account.
Joshua Colp [Sun, 1 Jun 2014 15:32:34 +0000 (15:32 +0000)]
bridge_native_rtp: Take the bridge type choice of both channels into account.

The bridge_native_rtp module currently uses the bridge result of the first
channel that joins a bridge as the ultimate result. This means that if the
first channel has direct media enabled but the second does not a direct
media bridge will still occur.

This change makes it so that both sides are taken into account. If either
side forbids the bridge or responds with a local bridge result then
either a generic or local bridge occurs.

ASTERISK-23541 #close
Reported by: Justin E


Merged revisions 414975 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoPJSIP: Prevent crash on blind transfer
Kinsey Moore [Fri, 30 May 2014 14:53:44 +0000 (14:53 +0000)]
PJSIP: Prevent crash on blind transfer

Blind transfers don't go too well with NULL channels which can occur if
the channel has already been transferred away.

(closes issue ASTERISK-23718)
Reported by: Jonathan Rose

Merged revisions 414948 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoTALK_DETECT: A channel function that raises events when talking is detected
Matthew Jordan [Fri, 30 May 2014 12:42:57 +0000 (12:42 +0000)]
TALK_DETECT: A channel function that raises events when talking is detected

This patch adds a new channel function TALK_DETECT that, when set on a
channel, causes events indicating the start/stop of talking on a channel to be
emitted to both AMI and ARI clients.

The function allows setting both the silence threshold (the length of silence
after which we decide no one is talking) as well as the talking threshold (the
amount of energy that counts as talking). Parameters can be updated on a channel
after talk detection has been enabled, and talk detection can be removed at
any time.

The events raised by the function use a nomenclature similar to existing AMI/ARI
For AMI: ChannelTalkingStart/ChannelTalkingStop
For ARI: ChannelTalkingStarted/ChannelTalkingFinished


#ASTERISK-23786 #close
Reported by: Matt Jordan

Merged revisions 414934 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agomain/config.c: AMI action UpdateConfig EmptyCat clears all categories
Matthew Jordan [Fri, 30 May 2014 12:05:33 +0000 (12:05 +0000)]
main/config.c: AMI action UpdateConfig EmptyCat clears all categories

When invoking UpdateConfig AMI action with Action set to EmptyCat, Asterisk
will make all categories empty in the config but the one requested with a
Cat variable. This is due to a bug in ast_category_empty (main/config.c)
that makes an incorrect comparison for a category name.

This patch corrects the comparison such that only the requested category
is cleared.


#ASTERISK-23803 #close
Reported by: zvision
  manager.c.diff uploaded by zvision (License 5755)

Merged revisions 414880 from

Merged revisions 414881 from

Merged revisions 414882 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoPBX: Prevent incorrect hint parsing
Kinsey Moore [Thu, 29 May 2014 18:51:41 +0000 (18:51 +0000)]
PBX: Prevent incorrect hint parsing

Dynamic and pattern matching hints should not be checked for their last
known state until they are instantiated by subscribers.

(closes issue AFS-56)
Reported by: John Hardin
Patch AFS-56-pbx.diff submitted by Matt Jordan (license 6283)

Merged revisions 414813 from

Merged revisions 414859 from

Merged revisions 414860 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoLogger/CLI/etc.: Fix some aesthetic issues; reduce chatty verbose messages
Matthew Jordan [Wed, 28 May 2014 22:54:12 +0000 (22:54 +0000)]
Logger/CLI/etc.: Fix some aesthetic issues; reduce chatty verbose messages

This patch addresses some aesthetic issues in Asterisk. These are all just
minor tweaks to improve the look of the CLI when used in a variety of
settings. Specifically:
 * A number of chatty verbose messages were removed or demoted to DEBUG
   messages. Verbose messages with a verbosity level of 5 or higher were -
   if kept as verbose messages - demoted to level 4. Several messages
   that were emitted at verbose level 3 were demoted to 4, as announcement
   of dialplan applications being executed occur at level 3 (and so the
   effects of those applications should generally be less).
 * Some verbose messages that only appear when their respective 'debug'
   options are enabled were bumped up to always be displayed.
 * Prefix/timestamping of verbose messages were moved to the verboser
   handlers. This was done to prevent duplication of prefixes when the
   timestamp option (-T) is used with the CLI.
 * Verbose magic is removed from messages before being emitted to
   non-verboser handlers. This prevents the magic in multi-line verbose
   messages (such as SIP debug traces or the output of DumpChan) from
   being written to files.
 * _Slightly_ better support for the "light background" option (-W) was
   added. This includes using ast_term_quit in the output of XML
   documentation help, as well as changing the "Asterisk Ready" prompt to
   bright green on the default background (which stands a better chance of
   being displayed properly than bright white).


git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agopjsip.conf: privkey_file should be priv_key_file, mediaencryption=yes should be media...
Rusty Newton [Wed, 28 May 2014 20:53:05 +0000 (20:53 +0000)]
pjsip.conf: privkey_file should be priv_key_file, mediaencryption=yes should be mediaencryption=sdes

privkey_file was missed in the snake case update. An example included an invalid value for the mediaencryption option.

Merged revisions 414780 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoAMI/ARI: Update version numbers
Matthew Jordan [Wed, 28 May 2014 17:46:37 +0000 (17:46 +0000)]
AMI/ARI: Update version numbers

Update the semantic versioning of ARI to 1.3.0 and AMI to 2.3.0 to account for
backwards compatible changes going from 12.2.0 to 12.3.0.

Merged revisions 414765 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoast-db-manage/cdr/ Don't fail if a config file can't be loaded
Matthew Jordan [Wed, 28 May 2014 17:44:49 +0000 (17:44 +0000)]
ast-db-manage/cdr/ Don't fail if a config file can't be loaded

When generating SQL files via the repotools script, a
configuration object is used programatically with SQLAlechemy, as opposed to
a configuration file. This patch ignores failures to interpret a config file,
as ... there isn't one in this case.

Merged revisions 414763 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agores_pjsip_session: Fix leaked video RTP ports.
Richard Mudgett [Wed, 28 May 2014 16:56:07 +0000 (16:56 +0000)]
res_pjsip_session: Fix leaked video RTP ports.

Simply enabling PJSIP to negotiage a video codec (e.g., h264) would leak
video RTP ports if the codec were not negotiated by an incoming call.

* Made add_sdp_streams() associate the handler with the media stream if
the handler handled the media stream.  Otherwise, when the
ast_sip_session_media object was destroyed it didn't know how to clean up
the RTP resources.

* Fixed sdp_requires_deferral() associating the handler with the media
stream when deciding if the SDP processing needs to be deferred for T.38.
Like the leaked video RTP ports, the T.38 handler needs to clean up
allocated resources from deciding if SDP processing needs to be deffered.

* Cleaned up some dead code in handle_incoming_sdp() and

ASTERISK-23721 #close
Reported by: cervajs


Merged revisions 414749 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoapp_agent_pool: Return to dialplan if the agent fails to ack the call.
Richard Mudgett [Wed, 28 May 2014 16:34:47 +0000 (16:34 +0000)]
app_agent_pool: Return to dialplan if the agent fails to ack the call.

Improvements to the agent pool functionality.

* AgentRequest no longer hangs up the caller if the agent fails to connect
with the caller.  It now continues in the dialplan.

* AgentRequest returns AGENT_STATUS set to NOT_CONNECTED if the agent
failed to connect with the call.  Most likely because the agent did not
acknowledge the call in time or got disconnected.

* The agent alerting play file configured by the agent.conf custom_beep
option can now be disabled by setting the option to an empty string.  The
agent is effectively alerted to a call presence when MOH stops.

* Fixed bridge reference leak when the agent connects with a caller.

ASTERISK-23499 #close
Reported by: Matt Jordan


Merged revisions 414747 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agores_config_odbc: Use dynamically sized buffers to store row data so values do not...
Joshua Colp [Wed, 28 May 2014 11:37:50 +0000 (11:37 +0000)]
res_config_odbc: Use dynamically sized buffers to store row data so values do not get truncated.

ASTERISK-23582 #close
ASTERISk-23582 #comment Reported by: Walter Doekes


Merged revisions 414693 from

Merged revisions 414694 from

Merged revisions 414695 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agochan_unistim: Unlock mutex in rare OOM condition.
Walter Doekes [Wed, 28 May 2014 09:43:53 +0000 (09:43 +0000)]
chan_unistim: Unlock mutex in rare OOM condition.

#ASTERISK-23792 #close
Reported by: Peter Whisker


Merged revisions 414677 from

Merged revisions 414678 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agochan_sip: Start session timer at 200, not at INVITE.
Walter Doekes [Tue, 27 May 2014 21:23:16 +0000 (21:23 +0000)]
chan_sip: Start session timer at 200, not at INVITE.

Asterisk started counting the session timer at INVITE while the other
end correctly started at 200. This meant that for short session-expiries
(90 seconds) combined with long ringing times (e.g. 30 seconds), asterisk
would wrongly assume that the timer was hit before the other end thought
it was time to send a session refresh. This resulted in prematurely
ended calls.

This changes the session timer to start counting first at 200 like RFC
says it should.

(Also removed a few excess NULL checks that would never hit, because if
they did, asterisk would have crashed already.)

ASTERISK-22551 #close
Reported by: i2045


Merged revisions 414620 from

Merged revisions 414628 from

Merged revisions 414636 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agores_config_odbc: Fix old and new ast_string_field memory leaks.
Walter Doekes [Tue, 27 May 2014 20:03:00 +0000 (20:03 +0000)]
res_config_odbc: Fix old and new ast_string_field memory leaks.

The ODBC realtime driver uses ^NN parameter encoding to cope with the
special meaning of the semi-colon. A semi-colon in a field is
interpreted as if the key was supplied twice, something which isn't
otherwise possible with fixed database columns. E.g. allow=alaw;ulaw
is parsed as allow=alaw and allow=ulaw. A literal semi-colon is
rewritten to ^3B when stored in the database.

The module uses a stringfield to efficiently store the encoded
parameters. However, this stringfield wasn't always freed in some
off-nominal cases.

Commit r413241 fixed initialization so the encoding for INSERT and
DELETE queries wouldn't crash. (Only SELECTs and UPDATEs worked
apparently.) But that commit forgot the frees. This change cleans
that up.


Merged revisions 414564 from

Merged revisions 414565 from

Merged revisions 414566 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agocore_unreal: Prevent double free of core_unreal pvt
Matthew Jordan [Sun, 25 May 2014 02:37:03 +0000 (02:37 +0000)]
core_unreal: Prevent double free of core_unreal pvt

When a channel is destroyed (such as via ast_channel_release in off nominal
paths in core_unreal), it will attempt to free (via ast_free) the channel tech
pvt. This is problematic for a few reasons:
1. The channel tech pvt is an ao2 object in core_unreal. Free'ing the pvt
   directly is no good.
2. The channel tech pvt's reference count is dropped just prior to calling
   ast_channel_release, resulting in the pvt's destruction. Hence, the
   channel destructor is free'ing an invalid pointer.

This patch keeps the dropping of the reference count, but sets the pvt to
NULL on the channel prior to releasing it. This models what would occur if the
channel was hung up directly.

Merged revisions 414542 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agotest_cel: Fix unit tests broken due to event def changes from res_corosync
Matthew Jordan [Fri, 23 May 2014 17:36:35 +0000 (17:36 +0000)]
test_cel: Fix unit tests broken due to event def changes from res_corosync

This patch instructs test_cel to skip any IE types it doesn't care about. The
addition of the raw and bitfield types caused the tests to fail.

Merged revisions 414528 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoFix signed/unsigned build warnings
Kinsey Moore [Fri, 23 May 2014 14:36:40 +0000 (14:36 +0000)]
Fix signed/unsigned build warnings

Merged revisions 414474 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoapp_meetme: Don't interrupt MOH for waitmarked users.
Richard Mudgett [Thu, 22 May 2014 16:19:13 +0000 (16:19 +0000)]
app_meetme: Don't interrupt MOH for waitmarked users.

Occasionally, when the last marked user leaves the conference, waitmarked
users don't get MOH if MOH is supposed to be played while a waitmarked
user is waiting for another marked user.

* Made not interrupt MOH when the user is a waitmarked user.  The
waitmarked user doesn't need to hear any leave announcements from the
conference as the user would have already heard different leave
announcements if they were enabled.  Apparently DAHDI occasionally sends
unending non-silent streams to these users or a normal user still in the
conference has continuous high background noise.  These non-silent streams
cause MOH to be suspended while the never ending "announcement" is played.

Issue caused by ASTERISK-13680.

AST-1349 #close
Reported by: Tyler Stewart


Merged revisions 414401 from

Merged revisions 414402 from

Merged revisions 414404 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoARI: Add ability to raise arbitrary User Events
Scott Griepentrog [Thu, 22 May 2014 16:09:51 +0000 (16:09 +0000)]
ARI: Add ability to raise arbitrary User Events

User events can now be generated from ARI.  Events can be signalled with
arbitrary json variables, and include one or more of channel, bridge, or
endpoint snapshots.  An application must be specified which will receive
the event message (other applications can subscribe to it).  The message
will also be delivered via AMI provided a channel is attached.  Dialplan
generated user event messages are still transmitted via the channel, and
will only be received by a stasis application they are attached to or if
the channel is subscribed to.

This change also introduces the multi object blob mechanism used to send
multiple snapshot types in a single message.  The dialplan app UserEvent
was also changed to use multi object blob, and a new stasis message type
created to handle them.

ASTERISK-22697 #close

Merged revisions 414405 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agores_pjsip_refer: Fix bugs involving Parking/PJSIP/transfers
Jonathan Rose [Thu, 22 May 2014 15:52:30 +0000 (15:52 +0000)]
res_pjsip_refer: Fix bugs involving Parking/PJSIP/transfers

PJSIP would never send the final 200 Notify for a blind transfer
when transferring to parking. This patch fixes that. In addition,
it fixes a reference leak when performing blind transfers to
non-bridging extensions.


Merged revisions 414400 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoUPGRADE: Add note for REF_DEBUG flag
Matthew Jordan [Thu, 22 May 2014 14:02:19 +0000 (14:02 +0000)]
UPGRADE: Add note for REF_DEBUG flag

Merged revisions 414345 from

Merged revisions 414346 from

Merged revisions 414347 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agores_corosync: Update module to work with Stasis (and compile)
Matthew Jordan [Thu, 22 May 2014 12:01:37 +0000 (12:01 +0000)]
res_corosync: Update module to work with Stasis (and compile)

This patch fixes res_corosync such that it works with Asterisk 12. This
restores the functionality that was present in previous versions of
Asterisk, and ensures compatibility with those versions by restoring the
binary message format needed to pass information from/to them.

The following changes were made in the core to support this:
 * The event system has been partially restored. All event definition and
   event types in this patch were pulled from Asterisk 11. Previously, we had
   hoped that this information would live in res_corosync; however, the
   approach in this patch seems to be better for a few reasons:
   (1) Theoretically, ast_events can be used by any module as a binary
       representation of a Stasis message. Given the structure of an ast_event
       object, that information has to live in the core to be used universally.
       For example, defining the payload of a device state ast_event in
       res_corosync could result in an incompatible device state representation
       in another module.
   (2) Much of this representation already lived in the core, and was not
       easily extensible.
   (3) The code already existed. :-)
 * Stasis message types now have a message formatter that converts their
   payload to an ast_event object.
 * Stasis message forwarders now handle forwarding to themselves. Previously
   this would result in an infinite recursive call. Now, this simply creates a
   new forwarding object with no forwards set up (as it is the thing it is
   forwarding to). This is advantageous for res_corosync, as returning NULL
   would also imply an unrecoverable error. Returning a subscription in this
   case allows for easier handling of message types that are published directly
   to an aggregate topic that has forwarders.


ASTERISK-22912 #close
ASTERISK-22372 #close

Merged revisions 414330 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agocore_unreal: Only block media frames when a generator is on both ends of an unreal...
Richard Mudgett [Wed, 21 May 2014 22:24:40 +0000 (22:24 +0000)]
core_unreal: Only block media frames when a generator is on both ends of an unreal channel.

The fix for ASTERISK-12292 was a bit too aggressive.  You could have
generators pointed at each other on local channels but need to get other
kinds of frames such as DTMF or CONNECTED_LINE frames accross.

Merged revisions 414269 from

Merged revisions 414270 from

Merged revisions 414272 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agopbx.c: prevent potential crash from recursive replace()
Scott Griepentrog [Wed, 21 May 2014 19:08:39 +0000 (19:08 +0000)]
pbx.c: prevent potential crash from recursive replace()

Recurisve usage of replace() resulted in corruption of the
temporary string storage and potential crash.  By changing
the string to be allocated separtely per instance, this is

ASTERISK-23650 #comment Reported by: Roel van Meer
ASTERISK-23650 #close


Merged revisions 414214 from

Merged revisions 414215 from

Merged revisions 414216 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoReplace __ast_answer with ast_raw_answer in app_control_answer
Paul Belanger [Mon, 19 May 2014 19:52:34 +0000 (19:52 +0000)]
Replace __ast_answer with ast_raw_answer in app_control_answer

While load testing an ARI application, I noticed asterisk was returning HTTP 500
internal server errors on channels/:id/answer.  After talking to #asterisk-dev,
the issue appeared to be a lack of media flowing after __ast_answer() was
called.  So now, we call ast_raw_answer instead and no longer wait for media.

ASTERISK-23758 #close

Merged revisions 414195 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoUndo r414123
Matthew Jordan [Mon, 19 May 2014 01:10:23 +0000 (01:10 +0000)]
Undo r414123

The Test Suite caught a few problems, undoing until those are resolved

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agobridge_native_rtp/bridge_channel: Fix direct media issues due to frame hook
Matthew Jordan [Sun, 18 May 2014 20:38:02 +0000 (20:38 +0000)]
bridge_native_rtp/bridge_channel: Fix direct media issues due to frame hook

This patch fixes issues with direct media bridges that occur after a blind
transfer. These issues were caught by the (currently failing)
pjsip/transfers/blind_transfer/caller_direct_media test.

The test currently fails primarily for two reasons:
(1) When Bob and Charlie (the transfer target and the transfer destination)
    enter a bridge together, the framehook remains on the transfer target
    channel until both channels are in the bridge. As it consumes voice frames,
    the initial bridge type is a simple bridge. The framehook is removed when
    both channels are in the bridge; however, this does not currently cause the
    bridging framework to re-evaluate the bridge. This patch adds a
    AST_SOFTHANGUP_UNBRIDGE poke to the transfer target channel when a
    framehook is removed so the bridge can re-evaluate itself.

(2) When a channel leaves a native RTP bridge, it may be leaving due to being
    hung up. Sending a re-INVITE to a channel that is about to be hung up is
    not nice - in fact, there's a good chance we'll send the BYE request before
    the channel has had a chance to send back a 200 OK. To be somewhat nicer,
    this patch adds a function to channel.h that allows the bridging framework
    to query for exactly why a channel is leaving a bridge via the channel's
    soft hangup flags. This allows it to only send the re-INVITE if there's a
    chance the channel will survive the native bridging experience.


Merged revisions 414122 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agochan_dahdi: Fix analog dialtone detection.
Richard Mudgett [Fri, 16 May 2014 20:06:59 +0000 (20:06 +0000)]
chan_dahdi: Fix analog dialtone detection.

* Check if waitingfordt (waitfordialtone) is enabled in dahdi_read() to
allow the DSP to operate early enough to detect dialtone.

* Made use the correct variable in my_check_waitingfordt().

ASTERISK-23709 #close
Reported by: Steve Davies
      dialtone_detect_fix (license #5012) patch uploaded by Steve Davies


Merged revisions 414067 from

Merged revisions 414068 from

Merged revisions 414069 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agosig_pri.c: Pull the pri_dchannel() PRI_EVENT_RING case into its own function.
Richard Mudgett [Fri, 16 May 2014 17:32:44 +0000 (17:32 +0000)]
sig_pri.c: Pull the pri_dchannel() PRI_EVENT_RING case into its own function.

* Populate the CALLERID(ani2) value (and the special CALLINGANI2 channel
variable) with the ANI2 value in addition to the PRI specific ANI2 channel

* Made complete snapshot staging with the channel lock held.  All channel
snapshots need to be done while the channel lock is held.

Merged revisions 414050 from

Merged revisions 414051 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoapp_meetme: Fix overwrite of DAHDI conference data structure.
Richard Mudgett [Thu, 15 May 2014 22:02:32 +0000 (22:02 +0000)]
app_meetme: Fix overwrite of DAHDI conference data structure.

Starting a conference recording using the admin menu overwrites the DAHDI
conference data structure used to modify the admin user's conference mute

* Made no longer pass the user's DAHDI conference data structure into the
menu functions.  The menu now uses its own DAHDI conference data
structure to start the recording channel.

* Moved the unlock conf->playlock to before playing the conf-full message.
No sense keeping the lock while that prompt is playing.  The user is never
going to get into the conference at that point.

Merged revisions 413991 from

Merged revisions 413992 from

Merged revisions 413993 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agores_musiconhold: Minor cleanup.
Walter Doekes [Wed, 14 May 2014 15:41:42 +0000 (15:41 +0000)]
res_musiconhold: Minor cleanup.

Fix a few free()'s that should be ast_free()'s. Reverted an old
workaround that isn't necessary. Reorder a tiny bit of code.
Remove a bit of commented-out code.


Merged revisions 413894 from

Merged revisions 413895 from

Merged revisions 413896 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agochan_sip: Add TLS and SRTP status to CLI command 'sip show channel'
Jonathan Rose [Tue, 13 May 2014 18:09:13 +0000 (18:09 +0000)]
chan_sip: Add TLS and SRTP status to CLI command 'sip show channel'

ASTERISK-23564 #close
Reported by: Patrick Laimbock

Merged revisions 413876 from

Merged revisions 413877 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoh264: Fix H264 SDP payload format.
Walter Doekes [Tue, 13 May 2014 13:53:28 +0000 (13:53 +0000)]
h264: Fix H264 SDP payload format. says profile-level-id
takes 3 bytes in base16 (6 hex digits).

This fixes video setup in certain cases.

ASTERISK-23664 #close
ASTERISK-23664 #comment Patch r3530.patch uploaded by Guillaume Maudoux.

Merged revisions 413791 from

Merged revisions 413792 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agortp: Fix case typo in H263+ mime.
Walter Doekes [Tue, 13 May 2014 13:39:21 +0000 (13:39 +0000)]
rtp: Fix case typo in H263+ mime. says the canonical
mime subtype is "H263-1998", not "h263-1998". Original code was added
in r183101 on 2009-03-19 02:26:50 +0100.

This fixes issues with Polycom phones.

ASTERISK-23665 #close
ASTERISK-23665 #comment Patch r3529.patch uploaded by Guillaume Maudoux, backported by me.

Merged revisions 413787 from

Merged revisions 413788 from

Merged revisions 413789 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agochan_dahdi/sig_pri: Prevent unnecessary PROGRESS events when overlap dialing is enabled.
Richard Mudgett [Tue, 13 May 2014 00:35:31 +0000 (00:35 +0000)]
chan_dahdi/sig_pri: Prevent unnecessary PROGRESS events when overlap dialing is enabled.

When overlap dialing is enabled, the lack of inband audio available
information in the SETUP_ACKNOWLEDGE events causes an interoperability
problem with SIP.  sig_pri doesn't know if there is dialtone present when
a SETUP_ACKNOWLEDGE is received so it assumes it is there and posts an
AST_CONTROL_PROGRESS frame.  The SIP channel driver then sends out a 183
Session Progress and blocks the desired 180 Ringing message when the
ALERTING message comes in.

* Made the configure script detect if the installed version of libpri
supports the SETUP_ACKNOWLEDGE enhancements.

* Using the new API, made generate an AST_CONTROL_PROGRESS frame on an
incoming SETUP_ACKNOWLEDGE message when the message indicates inband audio
is present instead of assuming that dialtone is present.

* Using the new API, made SETUP_ACKNOWLEDGE send out an inband audio
available indication only if dialtone is expected.  The change also makes
the fallback behaviour of sending the PROGRESS message better by sending
it only if dialtone is expected.

* Changed receiving a PROCEEDING message to not generate an
AST_CONTROL_PROGRESS frame if the progress indication ie indicates
non-end-to-end-ISDN.  This helps interoperability with SIP.

* Changed sending a PROCEEDING message in response to an
AST_CONTROL_PROCEEDING frame to not indicate inband audio available.  It
was silly to do so anyway because the channel driver doesn't know if
inband audio is even available.  This helps interoperability with SIP.

This patch and a corresponding change in libpri work together to allow
Asterisk to control the inband audio available progress indication ie on
the SETUP_ACKNOWLEDGE message when dialtone is present.

AST-1338 #close
Reported by: Tyler Stewart


Merged revisions 413714 from

Merged revisions 413765 from

Merged revisions 413771 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoFix compiler warning from GCC 4.10 fixup.
Richard Mudgett [Tue, 13 May 2014 00:23:45 +0000 (00:23 +0000)]
Fix compiler warning from GCC 4.10 fixup.

Merged revisions 413766 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoapp_chanspy: Fix a test that was failing on account of r413551
Jonathan Rose [Mon, 12 May 2014 22:33:28 +0000 (22:33 +0000)]
app_chanspy: Fix a test that was failing on account of r413551

ASTERISK-23381 #close
ASTERISK-23381 #comment Reported by: Robert Moss

Merged revisions 413710 from

Merged revisions 413712 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoframehooks: Add callback for determining if a hook is consuming frames of a specific...
Joshua Colp [Sun, 11 May 2014 02:09:10 +0000 (02:09 +0000)]
framehooks: Add callback for determining if a hook is consuming frames of a specific type.

In the past framehooks have had no capability to determine what frame types a hook
is actually interested in consuming. This has meant that code has had to assume they
want all frames, thus preventing native bridging.

This change adds a callback which allows a framehook to be queried for whether it
is consuming a frame of a specific type. The native RTP bridging module has also
been updated to take advantange of this, allowing native bridging to occur when
previously it would not.

ASTERISK-23497 #comment Reported by: Etienne Lessard
ASTERISK-23497 #close


Merged revisions 413681 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoUndoing framehook support. Issues were uncovered by Bamboo.
Joshua Colp [Sun, 11 May 2014 01:09:06 +0000 (01:09 +0000)]
Undoing framehook support. Issues were uncovered by Bamboo.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoframehooks: Add callback for determining if a hook is consuming frames of a specific...
Joshua Colp [Sat, 10 May 2014 18:50:17 +0000 (18:50 +0000)]
framehooks: Add callback for determining if a hook is consuming frames of a specific type.

In the past framehooks have had no capability to determine what frame types a hook
is actually interested in consuming. This has meant that code has had to assume they
want all frames, thus preventing native bridging.

This change adds a callback which allows a framehook to be queried for whether it
is consuming a frame of a specific type. The native RTP bridging module has also
been updated to take advantange of this, allowing native bridging to occur when
previously it would not.

ASTERISK-23497 #comment Reported by: Etienne Lessard
ASTERISK-23497 #close


Merged revisions 413650 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoFix 32bit build for func_env
Kinsey Moore [Fri, 9 May 2014 23:18:21 +0000 (23:18 +0000)]
Fix 32bit build for func_env

Merged revisions 413592 from

Merged revisions 413595 from

Merged revisions 413597 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoAllow Asterisk to compile under GCC 4.10
Kinsey Moore [Fri, 9 May 2014 22:49:26 +0000 (22:49 +0000)]
Allow Asterisk to compile under GCC 4.10

This resolves a large number of compiler warnings from GCC 4.10 which
cause the build to fail under dev mode. The vast majority are
signed/unsigned mismatches in printf-style format strings.

Merged revisions 413586 from

Merged revisions 413587 from

Merged revisions 413588 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agohttp.c: Remove dead code.
Richard Mudgett [Fri, 9 May 2014 18:15:34 +0000 (18:15 +0000)]
http.c: Remove dead code.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoapp_chanspy: Fix a bug where Barge mode could fail
Jonathan Rose [Fri, 9 May 2014 17:03:41 +0000 (17:03 +0000)]
app_chanspy: Fix a bug where Barge mode could fail

If the barge audiohook was attached prior to the spyee and its peer
actually being bridged, the audiohook would not be applied and the
connected peer would not be able to hear audio from the spy when the
spy is in barge mode.

(closes issue ASTERISK-23381)
Reported by: Robert Moss

Merged revisions 413551 from

Merged revisions 413556 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoapp_queue: Extend documentation for various Manager actions and events.
Joshua Colp [Thu, 8 May 2014 00:36:38 +0000 (00:36 +0000)]
app_queue: Extend documentation for various Manager actions and events.

Merged revisions 413485 from

Merged revisions 413486 from

Merged revisions 413487 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoEnsure that presence state is decoded properly on Asterisk startup.
Mark Michelson [Wed, 7 May 2014 21:58:37 +0000 (21:58 +0000)]
Ensure that presence state is decoded properly on Asterisk startup.

The CustomPresence provider callback will automatically base64 decode
stored data if the 'e' option was present when the state was set. However,
since the provider callback was bypassed on Asterisk startup, encoded
presence subtypes and messages were being sent instead. This fix makes
it so the provider callback is always used when providing presence
state updates.

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoapp_confbridge: Fixed "CBAnn" channels not going away.
Richard Mudgett [Wed, 7 May 2014 20:59:13 +0000 (20:59 +0000)]
app_confbridge: Fixed "CBAnn" channels not going away.

Fixed a ref leak in conf_handle_talker_cb() everytime the conference
bridge was found to report a channel's talker status change.  The
resulting leak caused the "CBAnn" channels and the conference bridge to
never be destroyed.

Thanks to Richard Kenner on the asterisk-user's list for locating the

Reported by: Richard Kenner

Merged revisions 413454 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoapp_confbridge: Fix ref leak in CLI "confbridge kick" command.
Richard Mudgett [Wed, 7 May 2014 20:39:29 +0000 (20:39 +0000)]
app_confbridge: Fix ref leak in CLI "confbridge kick" command.

Fixed ref leak in the CLI "confbridge kick" command when the channel to be
kicked was not in the conference.

Merged revisions 413451 from

Merged revisions 413452 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoFix encoding of custom prepare extra data.
Mark Michelson [Wed, 7 May 2014 17:56:04 +0000 (17:56 +0000)]
Fix encoding of custom prepare extra data.

res_config_odbc-take2.patch by John Hardin (License #6512)

Merged revisions 413396 from

Merged revisions 413397 from

Merged revisions 413398 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoImprove XML sanitization in NOTIFYs, especially for presence subtypes and messages.
Mark Michelson [Wed, 7 May 2014 15:29:18 +0000 (15:29 +0000)]
Improve XML sanitization in NOTIFYs, especially for presence subtypes and messages.

Embedded carriage return line feed combinations may appear in presence subtypes
and messages since they may be derived from user input in an instant messenger
client. As such, they need to be properly escaped so that XML parsers do not
vomit when the messages are received.

Merged revisions 413372 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoCheck for an act on failures to update contacts during registration.
Mark Michelson [Tue, 6 May 2014 17:47:20 +0000 (17:47 +0000)]
Check for an act on failures to update contacts during registration.

There was an underlying issue in a realtime backend where database updates
would fail. Since we were not checking for failure, we would end up in a
strange state where the old database entry was still present but Asterisk
thought that it had been updated. Now when an entry fails to update, we
print a warning and delete the old contact from sorcery so there is no
mismatch between foreground and backend state.

res_pjsip_registrar.patch by John Hardin (License #6512)

Merged revisions 413358 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoEnsure that all parts of SQL UPDATEs and DELETEs are encoded.
Mark Michelson [Tue, 6 May 2014 17:12:19 +0000 (17:12 +0000)]
Ensure that all parts of SQL UPDATEs and DELETEs are encoded.

res_config_odbc.patch by John Hardin (License #6512)

Merged revisions 413304 from

Merged revisions 413305 from

Merged revisions 413306 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoBlocked revisions 413282
Mark Michelson [Fri, 2 May 2014 20:37:00 +0000 (20:37 +0000)]
Blocked revisions 413282

Correct variable traversal logic in res_config_odbc's update_odbc function.

Closes issue ASTERISK-23675
Reported by Leando Dardini
asterisk-23675-odbc-linkedlist-traversal_12.diff uploaded by Michael L. Young (license #5026)

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoPrevent crashes in res_config_odbc due to uninitialized string fields.
Mark Michelson [Fri, 2 May 2014 20:28:23 +0000 (20:28 +0000)]
Prevent crashes in res_config_odbc due to uninitialized string fields.

    odbc-crash.patch by John Hardin (License #6512)

Merged revisions 413241 from

Merged revisions 413251 from

Merged revisions 413258 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoReturn the number of rows affected by a SQL insert, rather than an object ID.
Mark Michelson [Fri, 2 May 2014 20:07:08 +0000 (20:07 +0000)]
Return the number of rows affected by a SQL insert, rather than an object ID.

The realtime API specifies that the store callback is supposed to return the number
of rows affected. res_config_pgsql was instead returning an Oid cast as an int, which
during any nominal execution would be cast to 0. Returning 0 when more than 0 rows were
inserted causes problems to the function's callers.

To give an idea of how strange code can be, this is the necessary code change to fix
a device state issue reported against chan_pjsip in Asterisk 12+. The issue was that
the registrar would attempt to insert contacts into the database. Because of the 0
return from res_config_pgsql, the registrar would think that the contact was not successfully
inserted, even though it actually was. As such, even though the contact was query-able
and it was possible to call the endpoint, Asterisk would "think" the endpoint was unregistered,
meaning it would report the device state as UNAVAILABLE instead of NOT_INUSE.

The necessary fix applies to all versions of Asterisk, so even though the bug reported
only applies to Asterisk 12+, the code correction is being inserted into 1.8+.

Closes issue ASTERISK-23707
Reported by Mark Michelson

Merged revisions 413224 from

Merged revisions 413225 from

Merged revisions 413226 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agores_pjsip_refer: Add Referred-By header on INVITE for blind transfers.
Richard Mudgett [Fri, 2 May 2014 16:39:58 +0000 (16:39 +0000)]
res_pjsip_refer: Add Referred-By header on INVITE for blind transfers.

Per rfc3892, the Referred-By header in a REFER must be copied into the
referenced request (IE.  The outgoing INVITE to the transfer target).

* Automatically put the Referred-By header in the outgoing INVITE message
if the SIPREFERREDBYHDR channel variable is defined with a value.

* Made chan_sip.c:get_refer_info() set SIPREFERREDBYHDR for inheritance so
chan_pjsip has a better chance to interoperate.

* Fixed refer_blind_callback() and refer_incoming_refer_request() to not
modify the data in the pointer returned by pjsip_msg_find_hdr_by_name().
It seems wrong to modify that data since the calling routine doesn't own
the buffer.

ASTERISK-23501 #close
Reported by: John Bigelow


Merged revisions 413210 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoParking: Add 'AnnounceChannel' argument to manager action 'Park'
Jonathan Rose [Fri, 2 May 2014 16:06:40 +0000 (16:06 +0000)]
Parking: Add 'AnnounceChannel' argument to manager action 'Park'

(closes ASTERISK-23397)
Reported by: Denis

Merged revisions 413196 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoMake behavior of the PRESENCE_STATE 'e' option more consistent.
Mark Michelson [Thu, 1 May 2014 16:21:09 +0000 (16:21 +0000)]
Make behavior of the PRESENCE_STATE 'e' option more consistent.

When writing presence state, if 'e' is specified, then the presence state will
be stored in the astdb encoded. However, consumers of presence state events or those
that query for the presence state will be given decoded information. If base64 encoding
is desired for consumers, then the information can be base64-encoded manually and the
'e' option can be omitted.

closes issue ASTERISK-23671
Reported by Mark Michelson


git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoRemove unnecessary repetition checks from res_pjsip_exten_state
Mark Michelson [Thu, 1 May 2014 15:47:49 +0000 (15:47 +0000)]
Remove unnecessary repetition checks from res_pjsip_exten_state

The PBX core already takes care of ensuring that repeated state changes
are not communicated to exten state consumers. Because the check in res_pjsip_exten_state
was incomplete, it was causing valid presence state changes not to be sent out. For instance,
if the presence state did not change but the message or subtype did, then no presence-related
NOTIFY request would be sent out.

closes issue ASTERISK-23672
Reported by Mark Michelson

Merged revisions 413173 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agores_pjsip: Add the ability to configure ciphers based on name.
Joshua Colp [Thu, 1 May 2014 12:31:20 +0000 (12:31 +0000)]
res_pjsip: Add the ability to configure ciphers based on name.

Previously this code would only accept the OpenSSL identifier instead
of the documented name.

ASTERISK-23498 #close
ASTERISK-23498 #comment Reported by: Anthony Messina


Merged revisions 413159 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agochan_sip.c: Fixed off-nominal message iterator ref count and alloc fail issues.
Richard Mudgett [Wed, 30 Apr 2014 21:03:29 +0000 (21:03 +0000)]
chan_sip.c: Fixed off-nominal message iterator ref count and alloc fail issues.

* Fixed early exit in sip_msg_send() not destroying the message iterator.

* Made ast_msg_var_iterator_next() and ast_msg_var_iterator_destroy()
tolerant of a NULL iter parameter in case ast_msg_var_iterator_init()

* Made ast_msg_var_iterator_destroy() clean up any current message data

* Made struct ast_msg_var_iterator, ast_msg_var_iterator_init(),
ast_msg_var_iterator_next(), ast_msg_var_unref_current(), and
ast_msg_var_iterator_destroy() use iter instead of i.

* Eliminated RAII_VAR usage in res_pjsip_messaging.c:vars_to_headers().

Merged revisions 413139 from

Merged revisions 413142 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agochan_pjsip: Fix deadlock when retrieving call-id of channel.
Joshua Colp [Wed, 30 Apr 2014 20:39:17 +0000 (20:39 +0000)]
chan_pjsip: Fix deadlock when retrieving call-id of channel.

If a task was in-flight which required the channel or bridge lock
it was possible for the synchronous task retrieving the call-id
to deadlock as it holds those locks.

After discussing with Mark Michelson the synchronous task was
removed and the call-id accessed directly. This should be safe as
each object involved is guaranteed to exist and the call-id will
never change.

Merged revisions 413140 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agoWebsocket: Add session locking and delay close
Kinsey Moore [Wed, 30 Apr 2014 13:08:07 +0000 (13:08 +0000)]
Websocket: Add session locking and delay close

This resolves a race condition where data could be written to a NULL
FILE pointer causing a crash as a websocket connection was in the
process of shutting down by adding locking to websocket session writes
and by deferring session teardown until session destruction.

(closes issue ASTERISK-23605)
Reported by: Matt Jordan

Merged revisions 413123 from

Merged revisions 413124 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3

8 years agores_stasis: Add progress indications to operations which perform media.
Joshua Colp [Wed, 30 Apr 2014 12:42:42 +0000 (12:42 +0000)]
res_stasis: Add progress indications to operations which perform media.

This change fixes operations which did not account for the fact that they may
be executed on channels which have not been answered. These operations will
now indicate progress when invoked.

ASTERISK-23560 #close
ASTERISk-23560 #comment Reported by: Jan Svoboda


Merged revisions 413121 from

git-svn-id: 65c4cc65-6c06-0410-ace0-fbb531ad65f3