4 years agoMerge "build: Warn if asterisk is installed in both 32 and 64 bit sys dirs"
Joshua Colp [Tue, 28 Feb 2017 23:47:46 +0000 (17:47 -0600)]
Merge "build:  Warn if asterisk is installed in both 32 and 64 bit sys dirs"

4 years agoMerge "res_pjsip_pubsub: Remove unneeded endpoint unref"
Joshua Colp [Tue, 28 Feb 2017 23:32:25 +0000 (17:32 -0600)]
Merge "res_pjsip_pubsub:  Remove unneeded endpoint unref"

4 years agoMerge "bridge_native_rtp: Handle case where channel joins already suspended."
zuul [Tue, 28 Feb 2017 23:14:25 +0000 (17:14 -0600)]
Merge "bridge_native_rtp: Handle case where channel joins already suspended."

4 years agoMerge "config: Improve documentation and behavior of outbound_proxy option."
Joshua Colp [Tue, 28 Feb 2017 20:44:29 +0000 (14:44 -0600)]
Merge "config: Improve documentation and behavior of outbound_proxy option."

4 years agoMerge "res_pjsip: Fix crash when contact has no status"
Joshua Colp [Tue, 28 Feb 2017 16:24:45 +0000 (10:24 -0600)]
Merge "res_pjsip: Fix crash when contact has no status"

4 years agopjsip.conf.sample: user_agent: not a specific version
Tzafrir Cohen [Tue, 28 Feb 2017 11:41:18 +0000 (13:41 +0200)]
pjsip.conf.sample: user_agent: not a specific version

Use the description of useragent from sip.conf here.

ASTERISK-26825 #close

Change-Id: I5b33a4aaa0ae1d793289d05e3bc09521affbf755

4 years agores_pjsip_pubsub: Remove unneeded endpoint unref
George Joseph [Tue, 28 Feb 2017 02:07:06 +0000 (19:07 -0700)]
res_pjsip_pubsub:  Remove unneeded endpoint unref

When a subscription was being recreated and the endpoint wasn't
found, we were trying to unref the endpoint.  This was causing
FRACKs.  Removed the unref.

ASTERISK-26823 #close

Change-Id: If86d2aecff8fe853c7f38a1bfde721fcef3cd164

4 years agores_pjsip: Fix crash when contact has no status
Jørgen H [Thu, 16 Feb 2017 10:16:47 +0000 (10:16 +0000)]
res_pjsip: Fix crash when contact has no status

This change fixes an assumption in res_pjsip that a contact will
always have a status. There is a race condition where this is
not true and would crash. The status will now be unknown when
this situation occurs.

ASTERISK-26623 #close

Change-Id: Id52d3ca4d788562d236da49990a319118f8d22b5

4 years agobuild: Warn if asterisk is installed in both 32 and 64 bit sys dirs
George Joseph [Mon, 27 Feb 2017 18:25:01 +0000 (11:25 -0700)]
build:  Warn if asterisk is installed in both 32 and 64 bit sys dirs

... and clean them both up on uninstall.

We've fixed the issue where 'make install' was installing to
/usr/lib on 64-bit systems that use /usr/lib64.  Now we need
to clean up the remnants in /usr/lib.

* 'make install' now prints a warning if DESTDIR/ASTLIBDIR
  contains 'lib64' and libasterisk* shared libraries or modules
  are also found in DESTDIR/ASTLIBDIR with 'lib64' transformed
  to 'lib'.

* 'make uninstall' ALWAYS cleans up both DESTDIR/ASTLIBDIR and
  DESTDIR/ASTLIBDIR with 'lib64' transformed to 'lib'.


Change-Id: I6edddeb3c07a51e7c7ba7cac3c05e4bf3ec3f01f

4 years agobridge_native_rtp: Handle case where channel joins already suspended.
Joshua Colp [Mon, 27 Feb 2017 13:02:38 +0000 (13:02 +0000)]
bridge_native_rtp: Handle case where channel joins already suspended.

The bridge_native_rtp module did not properly handle the case where
a smart bridge operation occurs while a channel is suspended. In this
scenario the module would incorrectly set up local or remote RTP
bridging despite the media having to flow through Asterisk. The remote
endpoint would see two media streams and experience wonky audio.

The module has been changed so that it ensures both channels are
not suspended when performing the native RTP bridging and this
requirement has been documented in the bridge technology.


Change-Id: Id4022d73ace837d4a293106445e3ade10dbc7c7c

4 years agoMerge "channel: Add ast_read_stream function for reading frames from all streams."
George Joseph [Mon, 27 Feb 2017 14:51:26 +0000 (08:51 -0600)]
Merge "channel: Add ast_read_stream function for reading frames from all streams."

4 years agoMerge "Binaural synthesis (confbridge): DTMF conference management."
zuul [Fri, 24 Feb 2017 23:35:43 +0000 (17:35 -0600)]
Merge "Binaural synthesis (confbridge): DTMF conference management."

4 years agoBinaural synthesis (confbridge): DTMF conference management.
frahaase [Fri, 12 Aug 2016 16:23:40 +0000 (18:23 +0200)]
Binaural synthesis (confbridge): DTMF conference management.

DTMF configuration options for the binaural softmix bridge:
toggle binaural rendering (per channel).


Change-Id: Ibfe708b9fe26097c1798fcbfcc4dc461267d8af8

4 years agoconfig: Improve documentation and behavior of outbound_proxy option.
Joshua Colp [Fri, 24 Feb 2017 17:49:59 +0000 (17:49 +0000)]
config: Improve documentation and behavior of outbound_proxy option.

This change updates the documentation for the outbound_proxy option
to ensure it is consistently stated that a full SIP URI must be
provided for the option.

The res_pjsip_outbound_registration module has also been changed so
that the provided outbound_proxy value is checked to ensure it is a
URI and if not an error is output stating so.


Change-Id: I6c239a32274846fd44e65b44ad9bf6373479b593

4 years agoMerge "pjproject_bundled: Update for pjproject 2.6"
Joshua Colp [Fri, 24 Feb 2017 18:49:07 +0000 (12:49 -0600)]
Merge "pjproject_bundled: Update for pjproject 2.6"

4 years agoMerge "Binaural synthesis (confbridge): Adds binaural synthesis to bridge_softmix."
Joshua Colp [Fri, 24 Feb 2017 18:49:00 +0000 (12:49 -0600)]
Merge "Binaural synthesis (confbridge): Adds binaural synthesis to bridge_softmix."

4 years agoMerge "build: Execute ldconfig to build cache. (take two)"
zuul [Fri, 24 Feb 2017 18:12:13 +0000 (12:12 -0600)]
Merge "build: Execute ldconfig to build cache. (take two)"

4 years agoMerge "channel: Add support for writing to a specific stream."
zuul [Fri, 24 Feb 2017 17:16:13 +0000 (11:16 -0600)]
Merge "channel: Add support for writing to a specific stream."

4 years agochannel: Add ast_read_stream function for reading frames from all streams.
Joshua Colp [Thu, 23 Feb 2017 19:03:15 +0000 (19:03 +0000)]
channel: Add ast_read_stream function for reading frames from all streams.

This change introduces an ast_read_stream function and callback in
the channel technology which allows reading frames from all streams
and not just the default streams.

The stream number has also been added to frames. This is to allow the
case where frames are queued onto the channel instead of being read
directly from the driver.

This change does impose a restriction on reading though: a chain of
frames can only contain frames from the same stream.


Change-Id: I5d7dc35e86694df91fd025126f6cfe0453aa38ce

4 years agopjproject_bundled: Update for pjproject 2.6
George Joseph [Fri, 10 Feb 2017 00:05:01 +0000 (17:05 -0700)]
pjproject_bundled: Update for pjproject 2.6

 * Removed all 2.5.5 functional patches.
 * Updated usages of pj_release_pool to be "safe".
 * Updated configure options to disable webrtc.
 * Updated config_site.h to disable webrtc in pjmedia.
 * Added Richard Mudgett's recent resolver patches.

Change-Id: Ib400cc4dfca68b3d07ce14d314e829bfddc252c7

4 years agobuild: Execute ldconfig to build cache. (take two)
George Joseph [Thu, 23 Feb 2017 21:49:17 +0000 (14:49 -0700)]
build: Execute ldconfig to build cache. (take two)

On some platforms a multiarch approach is used for libraries.
The build system does not take this into account and still
places libraries into the lib directory if no --libdir is
specified to configure. On initial startup this results in not being found, as it is not in the multiarch
lib directory.  To make matters worse, options were being passed
to ldconfig on both Linux and FreeBSD that actually prevented
the rebuild of the cache.

 * Fedora has a /usr/share/ that automatically tells
   autoconf to use /usr/lib64 but CentOS does not. This logic was
   copied to and modified so systems like Ubuntu,
   which still use /usr/lib for 64-bit systems, aren't affected.

Now that we have them in the correct directory...

In order for the system loader to find libasteriskssl and
libasteriskpj, one of 3 things has to happen...

  - The linker cache must be rebuilt including the directory
    where the libasterisk* libraries were installed.  Only root
    can rebuild the cache.  This was busted.
  - We have to link the asterisk binary with an rpath pointing
    to the directrory where the libasterisk* libraries were
    installed.  This makes things very complicated and will happen
    over the collective dead bodies of everyone who's had to
    package a distribution with an rpath.
  - Finally, you can start asterisk with LD_LIBRARY_PATH set to the
    directrory where the libasterisk* libraries were installed.

There are no other options. So...

 * The invokation of ldconfig has been moved from main/Makefile
   to ASTTOPDIR/Makefile, the options have been removed, and
   DESTDIR/ASTLIBDIR appended.  If you aren't root, you will be
   warned after the "Asterisk Installation Compete" banner that
   you must re-run 'make install' as root, manually run
   'ldconfig DESTDIR/ASTLIBDIR' as root, or run asterisk with


Change-Id: I2a64b7c33a7d3e9bde20f47e3d3ab771977af982

4 years agores_config_pgsql: Fix thread safety problems
Sean Bright [Thu, 23 Feb 2017 20:48:53 +0000 (15:48 -0500)]
res_config_pgsql: Fix thread safety problems

* A missing AST_LIST_UNLOCK() in find_table()

* The ESCAPE_STRING() macro uses pgsqlConn under the hood and we were
  not consistently locking before calling it.

* There were a handful of other places where pgsqlConn was accessed
  directly without appropriate locking.

Change-Id: Iea63f0728f76985a01e95b9912c3c5c6065836ed

4 years agoMerge "res_config_ldap: Various code improvements"
Joshua Colp [Thu, 23 Feb 2017 20:47:54 +0000 (14:47 -0600)]
Merge "res_config_ldap: Various code improvements"

4 years agochannel: Add support for writing to a specific stream.
Joshua Colp [Wed, 22 Feb 2017 11:00:57 +0000 (11:00 +0000)]
channel: Add support for writing to a specific stream.

This change adds an ast_write_stream function which allows
writing a frame to a specific media stream. It also moves
ast_write() to using this underneath by writing media
frames provided to it to the default streams of the channel.
Existing functionality (such as audiohooks, framehooks, etc)
are limited to being applied to the default stream only.

Unit tests have also been added which test the behavior of
both non-multistream and multistream channels to confirm that
the write() and write_stream() callbacks are invoked


Change-Id: I4df20d1b65bd4d787fce0b4b478e19d2dfea245c

4 years agoBinaural synthesis (confbridge): Adds binaural synthesis to bridge_softmix.
frahaase [Fri, 12 Aug 2016 16:23:18 +0000 (18:23 +0200)]
Binaural synthesis (confbridge): Adds binaural synthesis to bridge_softmix.

Adds binaural synthesis to bridge_softmix (via convolution using libfftw3).
Binaural synthesis is conducted at 48kHz.
For a conference, only one spatial representation is rendered.
The default rendering is applied for mono-capable channels.


Change-Id: Iecdb381b6adc17c961049658678f6219adae1ddf

4 years agores_config_ldap: Various code improvements
Sean Bright [Wed, 22 Feb 2017 14:53:25 +0000 (09:53 -0500)]
res_config_ldap: Various code improvements

The initial motivation for this patch was to properly handle memory
allocation failures - we weren't checking the return values from the
various LDAP library allocation functions.

In the process, because update_ldap() and update2_ldap() were
substantially the same code, they've been consolidated.

Change-Id: Iebcfe404177cc6860ee5087976fe97812221b822

4 years agobuild_tools: Fix download_externals to allow the use of curl or wget
Michael L. Young [Wed, 22 Feb 2017 19:08:05 +0000 (14:08 -0500)]
build_tools:  Fix download_externals to allow the use of curl or wget

Not sure if this is really a bug versus an improvement. I can see it being
viewed as a bug though by some.

The current build_tools/download_externals file depends on wget in order to
download external modules.  The current build system is able to discover
which tool to use for fetching remote files - either wget or curl.

This patch takes advantage of this capability by modifying the two calls to
the wget binary to instead use what was discovered by the build system.

ASTERISK-26812 #close

Change-Id: If9411a2554f009274d377445613ae91192d948a1

4 years agoMerge "Revert "build: Execute ldconfig to build cache.""
zuul [Wed, 22 Feb 2017 19:56:48 +0000 (13:56 -0600)]
Merge "Revert "build: Execute ldconfig to build cache.""

4 years agoMerge "pbx_realtime: Prevent premature extension matching"
zuul [Wed, 22 Feb 2017 18:08:50 +0000 (12:08 -0600)]
Merge "pbx_realtime: Prevent premature extension matching"

4 years agoMerge "core: Show streams in "core show channel"."
zuul [Wed, 22 Feb 2017 17:40:02 +0000 (11:40 -0600)]
Merge "core: Show streams in "core show channel"."

4 years agoRevert "build: Execute ldconfig to build cache."
Joshua Colp [Wed, 22 Feb 2017 17:12:54 +0000 (11:12 -0600)]
Revert "build: Execute ldconfig to build cache."

This reverts commit 28c8e4f58f0f38792c7c79a05bd07788ebf15332.

Change-Id: Ie2e1aaf61fd49045994974a4581545ac8348fe4c

4 years agoMerge "Add initial SDP state code."
Joshua Colp [Wed, 22 Feb 2017 16:56:02 +0000 (10:56 -0600)]
Merge "Add initial SDP state code."

4 years agopbx_realtime: Prevent premature extension matching
Sean Bright [Tue, 21 Feb 2017 16:47:41 +0000 (11:47 -0500)]
pbx_realtime: Prevent premature extension matching

The patterns provided by pbx_realtime were checked in the order in
which they were returned from the realtime backend. If there was
overlap between multiple patterns, the first one to correctly match was
chosen even though it may not have been the best match.

We now sort the patterns descending by their length and compare in that
order. There may be cases where this still results in a sub-optimal
match, but this patch should improve the overall behavior.

ASTERISK-18271 #close
Reported by: Charlie Smurthwaite

Change-Id: I56d9ac15810eb1775966b669c3028e32cc7bd809

4 years agoMerge "pbx_dundi: DUNDi weight parameter not processed correctly"
zuul [Wed, 22 Feb 2017 16:19:45 +0000 (10:19 -0600)]
Merge "pbx_dundi: DUNDi weight parameter not processed correctly"

4 years agoMerge "realtime: Fix ast_load_realtime_multientry handling"
zuul [Wed, 22 Feb 2017 16:11:49 +0000 (10:11 -0600)]
Merge "realtime: Fix ast_load_realtime_multientry handling"

4 years agocore: Show streams in "core show channel".
Joshua Colp [Wed, 22 Feb 2017 14:32:23 +0000 (14:32 +0000)]
core: Show streams in "core show channel".

The "core show channel" CLI command will now output the streams
present on the channel with their details.


Change-Id: I9c95b57aa09415005f0677a1949a0feb07e4987a

4 years agoMerge "realtime: Centralize some common realtime backend code"
Joshua Colp [Wed, 22 Feb 2017 11:53:50 +0000 (05:53 -0600)]
Merge "realtime: Centralize some common realtime backend code"

4 years agoMerge "res_pjsip_authenticator_digest.c: Fix sorcery's immutable contract violation."
zuul [Wed, 22 Feb 2017 05:13:44 +0000 (23:13 -0600)]
Merge "res_pjsip_authenticator_digest.c: Fix sorcery's immutable contract violation."

4 years agoMerge "res_pjsip: Update artificial auth whenever default_realm changes."
zuul [Wed, 22 Feb 2017 04:55:13 +0000 (22:55 -0600)]
Merge "res_pjsip: Update artificial auth whenever default_realm changes."

4 years agoMerge "res_pjsip: Update authentication realm documentation."
zuul [Wed, 22 Feb 2017 04:39:09 +0000 (22:39 -0600)]
Merge "res_pjsip: Update authentication realm documentation."

4 years agoMerge "pjproject: Fixes to resolve DNS SRV crashes."
zuul [Wed, 22 Feb 2017 02:23:20 +0000 (20:23 -0600)]
Merge "pjproject: Fixes to resolve DNS SRV crashes."

4 years agoMerge "pjproject: Increase SENDER_WIDTH column size for 64-bit system logs."
zuul [Wed, 22 Feb 2017 02:14:26 +0000 (20:14 -0600)]
Merge "pjproject: Increase SENDER_WIDTH column size for 64-bit system logs."

4 years agoMerge "pjsip_distributor.c: Update some debug messages to get transaction name."
zuul [Wed, 22 Feb 2017 02:01:08 +0000 (20:01 -0600)]
Merge "pjsip_distributor.c: Update some debug messages to get transaction name."

4 years agoMerge "res_config_ldap: Don't try to delete non-existent attributes"
zuul [Wed, 22 Feb 2017 00:58:28 +0000 (18:58 -0600)]
Merge "res_config_ldap: Don't try to delete non-existent attributes"

4 years agoMerge "res_config_ldap: Remove extraneous line numbers from log messages"
zuul [Wed, 22 Feb 2017 00:58:27 +0000 (18:58 -0600)]
Merge "res_config_ldap: Remove extraneous line numbers from log messages"

4 years agoMerge "res_config_ldap: Make memory allocation more consistent"
zuul [Wed, 22 Feb 2017 00:58:26 +0000 (18:58 -0600)]
Merge "res_config_ldap: Make memory allocation more consistent"

4 years agoMerge "res_config_ldap: Fix configuration inheritance from _general"
zuul [Wed, 22 Feb 2017 00:24:05 +0000 (18:24 -0600)]
Merge "res_config_ldap: Fix configuration inheritance from _general"

4 years agoMerge "res_config_ldap: Fix erroneous LDAP_MOD_REPLACE in LDAP modify"
zuul [Tue, 21 Feb 2017 23:57:24 +0000 (17:57 -0600)]
Merge "res_config_ldap: Fix erroneous LDAP_MOD_REPLACE in LDAP modify"

4 years agopbx_dundi: DUNDi weight parameter not processed correctly
Sean Bright [Tue, 21 Feb 2017 21:09:47 +0000 (16:09 -0500)]
pbx_dundi: DUNDi weight parameter not processed correctly

The DUNDi weight field is not always converted from network byte order
to host byte order. This can result in incorrect weight values and
incorrect selection of DUNDi destinations.

ASTERISK-18731 #close
Reported by: Peter Racz
dundi_weight.patch (license #6290) patch uploaded by Peter Racz

Change-Id: Iba3e1a700ff539db57211a7bbc26f7b22ea9a1be

4 years agoAdd initial SDP state code.
Mark Michelson [Wed, 15 Feb 2017 20:43:36 +0000 (14:43 -0600)]
Add initial SDP state code.

This establishes the basic allocation/destruction of an SDP state
object, plus some of the simpler getter methods involved. Subsequent
tasks will deal with adding a state machine, creating SDPs from
capabilities and options, and merging SDPs into a joint SDP.

Change-Id: Ie3757ce186f04b65e9d1883f5aace53f24e53709

4 years agoMerge changes from topic 'sdp_state_beginnings'
Joshua Colp [Tue, 21 Feb 2017 19:37:03 +0000 (13:37 -0600)]
Merge changes from topic 'sdp_state_beginnings'

* changes:
  Add SDP translator and PJMEDIA implementation.
  Add initial SDP options.

4 years agoMerge "build: Execute ldconfig to build cache."
Joshua Colp [Tue, 21 Feb 2017 19:36:38 +0000 (13:36 -0600)]
Merge "build: Execute ldconfig to build cache."

4 years agorealtime: Fix ast_load_realtime_multientry handling
Sean Bright [Tue, 21 Feb 2017 16:47:28 +0000 (11:47 -0500)]
realtime: Fix ast_load_realtime_multientry handling

ast_load_realtime_multientry() returns an ast_config structure whose
ast_categorys are keyed with the empty strings. Several modules were
giving semantic meaning to the category names causing problems at

* app_directory: Treated the category name as the mailbox name, and
  would fail to direct calls to the appropriate extension after an
  entry was chosen.

* app_queue: Queues, queue members, and queue rules were all affected
  and needed to be updated.

* pbx_realtime: Pattern matching would never succeed because the
  extension entered by the user was always compared to the empty

Change-Id: Ie7e44986344b0b76ea8f6ddb5879f5040c6ca8a7

4 years agorealtime: Centralize some common realtime backend code
Sean Bright [Tue, 21 Feb 2017 14:56:54 +0000 (09:56 -0500)]
realtime: Centralize some common realtime backend code

All of the realtime backends create artificial ast_categorys to pass
back into the core as query results. These categories have no filename
or line number information associated with them and the backends differ
slightly on how they create them. So create a couple helper macros to
help make things more consistent.

Also updated the call sites to remove redundant error messages about
memory allocation failure.

Note that res_config_ldap sets the category filename to the 'table name'
but that is not read by anything in the core, so I've dropped it.

Change-Id: I3a1fd91e0c807dea1ce3b643b0a6fe5be9002897

4 years agoMerge "realtime: Fix LIKE escaping in SQL backends"
zuul [Tue, 21 Feb 2017 12:26:30 +0000 (06:26 -0600)]
Merge "realtime: Fix LIKE escaping in SQL backends"

4 years agobuild: Execute ldconfig to build cache.
Joshua Colp [Thu, 16 Feb 2017 16:30:00 +0000 (16:30 +0000)]
build: Execute ldconfig to build cache.

On some platforms a multiarch approach is used for libraries.
The build system does not take this into account and still
places libraries into the lib directory if no --libdir is
specified to configure. On initial startup this results in not being found, as it is not in the multiarch
lib directory.

This change does the minimally invasive thing and executes
ldconfig so that the libraries in the lib directory are found
and their location cached. By doing so Asterisk starts up fine.

If DESTDIR is specified, however, the old logic is executed as
the install process may not have permission to alter the ldconfig


Change-Id: If4eca46ac510c6fea5568256280ffdb3888d7bb4

4 years agores_pjsip_authenticator_digest.c: Fix sorcery's immutable contract violation.
Richard Mudgett [Mon, 9 Jan 2017 02:32:17 +0000 (20:32 -0600)]
res_pjsip_authenticator_digest.c: Fix sorcery's immutable contract violation.

The inbound authentication object is supposed to be immutable when it is
stored in sorcery.  However, the immutable property is violated if the
authentication object does not have a realm set.

The immutable contract violation has a different effect depending upon
what sorcery back end is used.  If it is the config file back end you
would get the same object back until res_pjsip is reloaded.  If it is the
real-time or AstDB back end you would get a new object on each query.  If
it is cached you would get the same object back until it is refreshed from
the database.

Once an inbound authentication object has its realm set it may or may not
get updated again if the default_realm changes.

If the same authentication object is used for inbound and outbound
authentication then the immutable violation can make it very hard to
determine why the outbound authentication now fails.  The only diagnostic
message is a complaint about no realms matching when it had worked
earlier.  It fails because of the difference in behaviour for an empty
realm setting between inbound and outbound authentication objects.

* Fixed the sorcery object immutable violation by creating a new object
and setting the default_realm on it instead.  The new object is a shallow
copy for speed.

* The auth_store thread storage no longer holds an auth ref.  It
interferes with the shallow copy and never needed a ref anyway.

ASTERISK-26799 #close

Change-Id: I2328a52f61b78ed5fbba38180b7f183ee7e08956

4 years agores_pjsip: Update artificial auth whenever default_realm changes.
Richard Mudgett [Sun, 5 Feb 2017 02:17:36 +0000 (20:17 -0600)]
res_pjsip: Update artificial auth whenever default_realm changes.

There was code attempting to update the artificial authentication object
whenever the default_realm changed.  However, once the artificial
authentication object was created it would never get updated.  The
artificial authentication object would require a system restart for a
change to the default_realm to take effect.


Change-Id: Id59036e9529c2d3ed728af2ed904dc36e7094802

4 years agores_pjsip: Update authentication realm documentation.
Richard Mudgett [Sun, 1 Jan 2017 14:02:17 +0000 (08:02 -0600)]
res_pjsip: Update authentication realm documentation.

Using the same auth section for inbound and outbound authentication is not
recommended.  There is a difference in meaning for an empty realm setting
between inbound and outbound authentication uses.

An empty inbound auth realm represents the global section's default_realm
value when the authentication object is used to challenge an incoming
request.  An empty outgoing auth realm is treated as a don't care wildcard
when the authentication object is used to respond to an incoming
authentication challenge.


Change-Id: Id3952f7cfa1b6683b9954f2c5d2352d2f11059ce

4 years agoMerge "app_voicemail: vm_authenticate accesses uninitialized memory"
zuul [Tue, 21 Feb 2017 00:29:24 +0000 (18:29 -0600)]
Merge "app_voicemail: vm_authenticate accesses uninitialized memory"

4 years agoMerge "tcptls.c: Add some missing allocation failure checks."
zuul [Mon, 20 Feb 2017 23:43:27 +0000 (17:43 -0600)]
Merge "tcptls.c: Add some missing allocation failure checks."

4 years agopjproject: Fixes to resolve DNS SRV crashes.
Richard Mudgett [Mon, 13 Feb 2017 23:11:06 +0000 (17:11 -0600)]
pjproject: Fixes to resolve DNS SRV crashes.

* Re #1945 (misc): Don't trigger SRV complete callback when there is a
parse error.

* srv_resolver.c: Don't try to send query if already considered resolved.

** In resolve_hostnames() don't try to resolve a query that is already
considered resolved.

** In resolve_hostnames() fix DNS typo in comments.

** In build_server_entries() move a common expression assigning to cnt

* sip_transport.c: Fix tdata object name to actually contain the pointer.

It helps if the logs referencing a tdata object buffer actually have a
name that includes the correct pointer as part of the name.  Also since
the tdata has its own pool it helps if any logs referencing the pool have
the same name as the tdata object.  This change brings tdata logging in
line with how tsx objects are named.

ASTERISK-26669 #close
ASTERISK-26738 #close

Change-Id: I56af2ded25476b3e870ca586ee69ed6954ef75af

4 years agopjproject: Increase SENDER_WIDTH column size for 64-bit system logs.
Richard Mudgett [Mon, 20 Feb 2017 19:38:58 +0000 (13:38 -0600)]
pjproject: Increase SENDER_WIDTH column size for 64-bit system logs.


Change-Id: Ibae6fc8cae69a1f04df0c577c4c11200499d6fe0

4 years agopjsip_distributor.c: Update some debug messages to get transaction name.
Richard Mudgett [Mon, 6 Feb 2017 20:26:30 +0000 (14:26 -0600)]
pjsip_distributor.c: Update some debug messages to get transaction name.

* Removed overloaded unmatched response ignore.  We obviously sent the
request so we shouldn't ignore it because it isn't new work.


Change-Id: I55fb5cadc83a8e6699b347c6dc7fa32c5a617d37

4 years agoMerge "pjproject cli: Add object count after object lists"
zuul [Mon, 20 Feb 2017 22:25:34 +0000 (16:25 -0600)]
Merge "pjproject cli:  Add object count after object lists"

4 years agoMerge "res_config_sqlite3: Properly create missing columns when necessary"
zuul [Mon, 20 Feb 2017 21:43:16 +0000 (15:43 -0600)]
Merge "res_config_sqlite3: Properly create missing columns when necessary"

4 years agoapp_voicemail: vm_authenticate accesses uninitialized memory
Sean Bright [Mon, 20 Feb 2017 12:28:23 +0000 (07:28 -0500)]
app_voicemail: vm_authenticate accesses uninitialized memory

vm_authenticate doesn't always set the passed ast_vm_user argument, so
we initialize to 0 before passing it in.

ASTERISK-25893 #close
Reported by: Filip Jenicek

Change-Id: Ia3cc0128f93d352ed9add8d5c2f0f7232c2cbe4a

4 years agoMerge "Revert "build: Execute ldconfig to build cache.""
zuul [Mon, 20 Feb 2017 20:09:27 +0000 (14:09 -0600)]
Merge "Revert "build: Execute ldconfig to build cache.""

4 years agoRevert "build: Execute ldconfig to build cache."
Joshua Colp [Mon, 20 Feb 2017 17:19:55 +0000 (11:19 -0600)]
Revert "build: Execute ldconfig to build cache."

This reverts commit 8851c3e0885cb704a5a6159a51768ea5297e9b10.

Change-Id: I124380be5e3bd57da978428a2a93604336ccd0db

4 years agoMerge "Binaural synthesis (confbridge): Adds utils/conf_bridge_binaural_hrir_importer"
Joshua Colp [Mon, 20 Feb 2017 16:24:55 +0000 (10:24 -0600)]
Merge "Binaural synthesis (confbridge): Adds utils/conf_bridge_binaural_hrir_importer"

4 years agopjproject cli: Add object count after object lists
George Joseph [Mon, 20 Feb 2017 14:04:25 +0000 (07:04 -0700)]
pjproject cli:  Add object count after object lists

When listing a container, we now print the number of objects
in the container at the end of the list.

Change-Id: I791cbc3ee9da9a2af9adc655164b5d32953df812

4 years agores_config_ldap: Don't try to delete non-existent attributes
Sean Bright [Mon, 20 Feb 2017 11:53:47 +0000 (06:53 -0500)]
res_config_ldap: Don't try to delete non-existent attributes

OpenLDAP will raise an error when we try to delete an LDAP attribute
that doesn't exist. We need to filter out LDAP_MOD_DELETE requests
based on which attributes the current LDAP entry actually has. There
is of course a small window of opportunity for this to still fail,
but it is much less likely now.

Change-Id: I3fe1b04472733e43151563aaf9f8b49980273e6b

4 years agores_config_ldap: Remove extraneous line numbers from log messages
Sean Bright [Mon, 20 Feb 2017 11:49:17 +0000 (06:49 -0500)]
res_config_ldap: Remove extraneous line numbers from log messages

Extraneous line numbers were being output in many log messages. These
have been removed.

Change-Id: Ice9efa3d252ee87f37fa8f5ea852fda482675431

4 years agores_config_ldap: Make memory allocation more consistent
Sean Bright [Mon, 20 Feb 2017 11:45:54 +0000 (06:45 -0500)]
res_config_ldap: Make memory allocation more consistent

The code in update_ldap() and update2_ldap() was using both Asterisk's
memory allocation routines as well as OpenLDAP's. I've changed it so
that everything that is passed to OpenLDAP's functions are allocated
with their routines.

Change-Id: Iafec9c1fd8ea49ccc496d6316769a6a426daa804

4 years agores_config_ldap: Fix configuration inheritance from _general
Sean Bright [Mon, 20 Feb 2017 11:30:31 +0000 (06:30 -0500)]
res_config_ldap: Fix configuration inheritance from _general

The "_general" configuration section allows administrators to provide
both general configuration options (host, port, url, etc.) as well as a
global realtime-to-LDAP-attribute mapping that is a fallback if one of
the later sections do not override it. This neglected to exclude the
general configuration options from the mapping. As an example, during
my testing, chan_sip requested 'port' from realtime, and because I did
not have it defined, it pulled in the 'port' configuration option from
"_general." We now filter those out explicitly.

Change-Id: I1fc61560bf96b8ba623063cfb7e0a49c4690d778

4 years agores_config_ldap: Fix erroneous LDAP_MOD_REPLACE in LDAP modify
Sean Bright [Mon, 20 Feb 2017 11:27:38 +0000 (06:27 -0500)]
res_config_ldap: Fix erroneous LDAP_MOD_REPLACE in LDAP modify

We always treat the first change of our modification batch as a
replacement when it sometimes is actually a delete. So we have to pass
the correct arguments to the OpenLDAP library.

ASTERISK-26580 #close
Reported by: Nicholas John Koch
res_config_ldap.c-11.24.1.patch (license #6833) patch uploaded
by Nicholas John Koch

Change-Id: I0741d25de07c9539f1edc6eff3696165dfb64fbe

4 years agoMerge "res_config_sqlite3: Fix crash when loading with invalid config"
zuul [Sun, 19 Feb 2017 19:24:44 +0000 (13:24 -0600)]
Merge "res_config_sqlite3: Fix crash when loading with invalid config"

4 years agoMerge "pjproject-bundled: Fix checksum verification when using cURL"
Joshua Colp [Sun, 19 Feb 2017 18:46:09 +0000 (12:46 -0600)]
Merge "pjproject-bundled: Fix checksum verification when using cURL"

4 years agores_config_sqlite3: Fix crash when loading with invalid config
Sean Bright [Wed, 15 Feb 2017 17:55:19 +0000 (12:55 -0500)]
res_config_sqlite3: Fix crash when loading with invalid config

When ast_config_load() fails with CONFIG_STATUS_FILEINVALID, it has
already destroyed the ast_config struct for us. Trying to do it again
results in a crash.

Change-Id: If6a5c0ca718ad428e01a1fb25beb209a9ac18bc6

4 years agoMerge "Remove extra ast_iostream_close() calls."
zuul [Fri, 17 Feb 2017 23:41:06 +0000 (17:41 -0600)]
Merge "Remove extra ast_iostream_close() calls."

4 years agopjproject-bundled: Fix checksum verification when using cURL
Sean Bright [Fri, 17 Feb 2017 23:06:47 +0000 (18:06 -0500)]
pjproject-bundled: Fix checksum verification when using cURL

ASTERISK-26802 #close
Reported by: Michael L. Young

Change-Id: Iad293080f55d4d69ab615717a15211d916eed613

4 years agotcptls.c: Add some missing allocation failure checks.
Richard Mudgett [Fri, 17 Feb 2017 22:57:54 +0000 (16:57 -0600)]
tcptls.c: Add some missing allocation failure checks.

* Fix tcptls_session ref and fd leak in ast_tcptls_server_root().

Change-Id: I0ddf01cd3c10d3b6666d7bf68d4e206a37f4fbdb

4 years agoRemove extra ast_iostream_close() calls.
Mark Michelson [Fri, 17 Feb 2017 20:58:28 +0000 (14:58 -0600)]
Remove extra ast_iostream_close() calls.

When AMI encounters an error at the beginning of a session, it would
explicitly call ast_iostream_close() on its tcptls session's iostream.
It then would jump to a label where it would shut down the tcptls
session instance. The tcptls session instance would again attempt to
close the iostream.

Under normal circumstances, this might go by unnoticed. However, when
MALLOC_DEBUG is enabled, all fields on the iostream get set to
0xdeaddead when the iostream is freed. Thus a second call to
ast_iostream_close() after the iostream has been freed would reslt in an
attempt to call SSL_shutdown on 0xdeaddead, which would crash and burn

The fix here is to not directly close the iostream from the dangerous
scenarios. The specific scenarios are:
* Exceeding the configured authlimit
* Failing to build a mansession on a new connection

Change-Id: I908f98d516afd5a263bd36b072221008a4731acd

4 years agoAdd SDP translator and PJMEDIA implementation.
Mark Michelson [Tue, 14 Feb 2017 15:54:43 +0000 (09:54 -0600)]
Add SDP translator and PJMEDIA implementation.

This creates the following:
* Asterisk's internal representation of an SDP
* An API for translating SDPs from one format to another
* An implementation of a translator for PJMEDIA

Change-Id: Ie2ecd3cbebe76756577be9b133e84d2ee356d46b

4 years agoAdd initial SDP options.
Mark Michelson [Tue, 7 Feb 2017 15:50:52 +0000 (09:50 -0600)]
Add initial SDP options.

This is step one of adding an SDP API: defining some
configurable settings for SDPs. This is based on options
that are currently supported in Asterisk.

Change-Id: I1ede91aafed403b12a9ccdfb91a88389baa7e5d7

4 years agobuild: Execute ldconfig to build cache.
Joshua Colp [Thu, 16 Feb 2017 16:30:00 +0000 (16:30 +0000)]
build: Execute ldconfig to build cache.

On some platforms a multiarch approach is used for libraries.
The build system does not take this into account and still
places libraries into the lib directory if no --libdir is
specified to configure. On initial startup this results in not being found, as it is not in the multiarch
lib directory.

This change does the minimally invasive thing and executes
ldconfig so that the libraries in the lib directory are found
and their location cached. By doing so Asterisk starts up fine.


Change-Id: I6d30b6427e9d5e69470e11327c7ff203fa7da519

4 years agoMerge "stream: Rename creates/destroys to allocs/frees"
zuul [Thu, 16 Feb 2017 19:24:30 +0000 (13:24 -0600)]
Merge "stream:  Rename creates/destroys to allocs/frees"

4 years agorealtime: Fix LIKE escaping in SQL backends
Sean Bright [Thu, 16 Feb 2017 14:38:06 +0000 (09:38 -0500)]
realtime: Fix LIKE escaping in SQL backends

The realtime framework allows for components to look up values using a
LIKE clause with similar syntax to SQL's. pbx_realtime uses this
functionality to search for pattern matching extensions that start with
an underscore (_).

When passing an underscore to SQL's LIKE clause, it will be interpreted
as a wildcard matching a single character and therefore needs to be
escaped. It is (for better or for worse) the responsibility of the
component that is querying realtime to escape it with a backslash before
passing it in. Some RDBMs support escape characters by default, but the
SQL92 standard explicitly says that there are no escape characters
unless they are specified with an ESCAPE clause, e.g.

SELECT * FROM table WHERE column LIKE '\_%' ESCAPE '\'

This patch instructs 3 backends - res_config_mysql, res_config_pgsql,
and res_config_sqlite3 - to use the ESCAPE clause where appropriate.

Looking through documentation and source tarballs, I was able to
determine that the ESCAPE clause is supported in:

MySQL 5.0.15   (released 2005-10-22 - earliest version available from
PostgreSQL 7.1 (released 2001-04-13)
SQLite 3.1.0   (released 2005-01-21)

The versions of the relevant libraries that we depend on to access MySQL
and PostgreSQL will not work on versions that old, and I've added an
explicit check in res_config_sqlite3 to only use the ESCAPE clause when
we have a sufficiently new version of SQLite3.

res_config_odbc already handles the escape characters appropriately, so
no changes were required there.

ASTERISK-15858 #close
Reported by: Humberto Figuera

ASTERISK-26057 #close
Reported by: Stepan

Change-Id: I93117fbb874189ae819f4a31222df7c82cd20efa

4 years agoMerge "stream: Add unit tests for channel stream usage."
Joshua Colp [Thu, 16 Feb 2017 17:34:40 +0000 (11:34 -0600)]
Merge "stream: Add unit tests for channel stream usage."

4 years agoMerge "chan_unistim: fix char type to have consistent behavior on ARM"
zuul [Thu, 16 Feb 2017 17:32:32 +0000 (11:32 -0600)]
Merge "chan_unistim: fix char type to have consistent behavior on ARM"

4 years agoMerge "http: Ensure capath is defined on all http creations"
Joshua Colp [Thu, 16 Feb 2017 16:40:10 +0000 (10:40 -0600)]
Merge "http: Ensure capath is defined on all http creations"

4 years agoMerge "res_pjsip_pubsub: Correctly implement persisted subscriptions"
Joshua Colp [Thu, 16 Feb 2017 15:48:52 +0000 (09:48 -0600)]
Merge "res_pjsip_pubsub:  Correctly implement persisted subscriptions"

4 years agostream: Rename creates/destroys to allocs/frees
George Joseph [Thu, 16 Feb 2017 14:28:33 +0000 (07:28 -0700)]
stream:  Rename creates/destroys to allocs/frees

To be consistent with sdp implementation.

Change-Id: I714e300939b4188f58ca66ce9d1e84b287009500

4 years agoMerge "pjsip_distributor.c: Fix off-nominal tdata ref leak."
zuul [Thu, 16 Feb 2017 13:08:29 +0000 (07:08 -0600)]
Merge "pjsip_distributor.c: Fix off-nominal tdata ref leak."

4 years agores_config_sqlite3: Properly create missing columns when necessary
Sean Bright [Thu, 16 Feb 2017 11:46:04 +0000 (06:46 -0500)]
res_config_sqlite3: Properly create missing columns when necessary

There were two specific issues resolved here:

1) The code that iterated over the required fields
   (via ast_realtime_require) was broken for the RQ_INTEGER1 field
   type. Iteration would stop when the first RQ_INTEGER1 (0) field
   was encountered.

2) sqlite3_changes() was used to try and count the number of rows
   returned by a SELECT statement. sqlite3_changes() only counts
   affected rows, so this was always returning the value from the
   most recent data modification statement. We now separate read-only
   queries from data modification queries and count rows appropriately
   in both cases.

ASTERISK-23457 #close
Reported by: Scott Griepentrog

Change-Id: I91ed20494efc3fcfbc2a96ac7646999a49814884

4 years agohttp: Ensure capath is defined on all http creations
Joshua Elson [Wed, 15 Feb 2017 20:44:32 +0000 (13:44 -0700)]
http: Ensure capath is defined on all http creations

ASTERISK-26794 #close

Change-Id: I9cbc3b6b6a8aab590f5ccde9c262a98e4d5253a1

4 years agochan_unistim: fix char type to have consistent behavior on ARM
Igor Goncharovsky [Thu, 16 Feb 2017 05:09:35 +0000 (08:09 +0300)]
chan_unistim: fix char type to have consistent behavior on ARM

There is difference exists in behaviour of char type on x86 and ARM.
On x86 by default char variable type means signed char, but in ARM
unsigned char used. This make binary calculations and negative values
works wrong on ARM.

This patch change type of char variables used for store negative
values and binary calculations to signed char.


Change-Id: Id78716dee9568a58419d4ef63c038affc3dfc7ab

4 years agoMerge "stream: Add stream topology to channel"
George Joseph [Thu, 16 Feb 2017 01:29:52 +0000 (19:29 -0600)]
Merge "stream:  Add stream topology to channel"

4 years agores_pjsip_pubsub: Correctly implement persisted subscriptions
George Joseph [Tue, 7 Feb 2017 19:17:12 +0000 (12:17 -0700)]
res_pjsip_pubsub:  Correctly implement persisted subscriptions

This patch fixes 2 original issues and more that those 2 exposed.

* When we send a NOTIFY, and the client either doesn't respond or
  responds with a non OK, pjproject only calls our
  pubsub_on_evsub_state callback, no others.  Since
  pubsub_on_evsub_state (which does the sub_tree cleanup) does not
  expect to be called back without the other callbacks being called
  first, it just returns leaving the sub_tree orphaned.  Now
  pubsub_on_evsub_state checks the event for PJSIP_EVENT_TSX_STATE
  which is what pjproject will set to tell us that it was the
  transaction that timed out or failed and not the subscription
  itself timing our or being terminated by the client. If is
  TSX_STATE, pubsub_on_evsub_state now does the proper cleanup
  regardless of the state of the subscription.

* When a client renews a subscription, we don't update the
  persisted subscription with the new expires timestamp.  This causes
  subscription_persistence_recreate to prune the subscription if/when
  asterisk restarts.  Now, pubsub_on_rx_refresh calls
  subscription_persistence_update to apply the new expires timestamp.
  This exposed other issues however...

* When creating a dialog from rdata (which sub_persistence_recreate
  does from the packet buffer) there must NOT be a tag on the To
  header (which there will be when a client refreshes a
  subscription).  If there is one, pjsip_dlg_create_uas will fail.
  To address this, subscription_persistence_update now accepts a flag
  that indicates that the original packet buffer must not be updated.
  New subscribes don't set the flag and renews do.  This makes sure
  that when the rdata is recreated on asterisk startup, it's done
  from the original subscribe packet which won't have the tag on To.

* When creating a dialog from rdata, we were setting the dialog's
  remote (SUBSCRIBE) cseq to be the same as the local (NOTIFY) cseq.
  When the client tried to resubscribe after a restart with the
  correct cseq, we'd reject the request with an Invalid CSeq error.

* The acts of creating a dialog and evsub by themselves when
  recreating a subscription does NOT restart pjproject's subscription
  timer.  The result was that even if we did correctly recreate the
  subscription, we never removed it if the client happened to go away
  or send a non-OK response to a NOTIFY.  However, there is no
  pjproject function exposed to just set the timer on an evsub that
  wasn't created by an incoming subscribe request.  To address this,
  we create our own timer using ast_sip_schedule_task.  This timer is
  used only for re-establishing subscriptions after a restart.

  An earlier approach was to add support for setting pjproject's
  timer (via a pjproject patch) and while that patch is still included
  here, we don't use that call at the moment.

While addressing these issues, additional debugging was added and
some existing messages made more useful.  A few formatting changes
were also made to 'pjsip show scheduled tasks' to make displaying
the subscription timers a little more friendly.


Change-Id: I8c605fc1e3923f466a74db087d5ab6f90abce68e

4 years agores_rtp_asterisk: Use PJ_ICE_MAX_CAND instead of hard-coding 16
Sean Bright [Wed, 15 Feb 2017 17:03:00 +0000 (12:03 -0500)]
res_rtp_asterisk: Use PJ_ICE_MAX_CAND instead of hard-coding 16

pjsip limits the total number of ICE candidates to PJ_ICE_MAX_CAND,
which is a compile-time constant. Instead of hard-coding 16 when we
enumerate local interfaces, use PJ_ICE_MAX_CAND so that we can
potentially collect more interfaces if the compile time options are

Tangentially related to ASTERISK~24464

Change-Id: I1b85509e39e33b1fed63c86261fc229ba14bbabd