8 years agochan_iax2: Restore codec choice behavior from media formats branch
Jonathan Rose [Mon, 21 Jul 2014 16:26:36 +0000 (16:26 +0000)]
chan_iax2: Restore codec choice behavior from media formats branch

After merging the media formats branch, chan_iax2 was discarding
codec preferences for the purpose of choosing which codec a
channel would use once a call started. This patch restores the
Asterisk 1.8-12 codec choice behaviors.

ASTERISK-23958 #close

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

8 years agochan_iax2: Only send mini frames if the underlying format has not changed, not if...
Joshua Colp [Mon, 21 Jul 2014 16:09:33 +0000 (16:09 +0000)]
chan_iax2: Only send mini frames if the underlying format has not changed, not if it has.

ASTERISK-24072 #close
Reported by: Matt Jordan

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

8 years agoFix build when pjproject is installed in a non-standard location.
Sean Bright [Mon, 21 Jul 2014 14:49:43 +0000 (14:49 +0000)]
Fix build when pjproject is installed in a non-standard location.

When configuring Asterisk to build against a version of pjproject installed
in a non-standard location, the checks for "PJSIP Transaction Group Lock
Support" and "PJSIP Media Stream Replacement Support" fail.  This is
because these secondary checks are not taking the CFLAGS and LIBS returned
by the pkg-config check into account.


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

8 years agores_smdi: convert to astobj2
Corey Farrell [Mon, 21 Jul 2014 08:41:29 +0000 (08:41 +0000)]
res_smdi: convert to astobj2

Remove functions:
ast_smdi_md_message destructor
ast_smdi_mwi_message destructor

Includes for astobj.h are removed everywhere it's possible.

ASTERISK-24066 #close

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

8 years agomedia formats: re-architect handling of media for performance improvements
Matthew Jordan [Sun, 20 Jul 2014 22:06:33 +0000 (22:06 +0000)]
media formats: re-architect handling of media for performance improvements

In the old times media formats were represented using a bit field. This was
fast but had a few limitations.
 1. Asterisk was limited in how many formats it could handle.
 2. Formats, being a bit field, could not include any attribute information.
    A format was strictly its type, e.g., "this is ulaw".
This was changed in Asterisk 10 (see for
notes on that work) which led to the creation of the ast_format structure.
This structure allowed Asterisk to handle attributes and bundle information
with a format.

Additionally, ast_format_cap was created to act as a container for multiple
formats that, together, formed the capability of some entity. Another
mechanism was added to allow logic to be registered which performed format
attribute negotiation. Everywhere throughout the codebase Asterisk was
changed to use this strategy.

Unfortunately, in software, there is no free lunch. These new capabilities
came at a cost.

Performance analysis and profiling showed that we spend an inordinate
amount of time comparing, copying, and generally manipulating formats and
their related structures. Basic prototyping has shown that a reasonably
large performance improvement could be made in this area. This patch is the
result of that project, which overhauled the media format architecture
and its usage in Asterisk to improve performance.

Generally, the new philosophy for handling formats is as follows:
 * The ast_format structure is reference counted. This removed a large amount
   of the memory allocations and copying that was done in prior versions.
 * In order to prevent race conditions while keeping things performant, the
   ast_format structure is immutable by convention and lock-free. Violate this
   tenet at your peril!
 * Because formats are reference counted, codecs are also reference counted.
   The Asterisk core generally provides built-in codecs and caches the
   ast_format structures created to represent them. Generally, to prevent
   inordinate amounts of module reference bumping, codecs and formats can be
   added at run-time but cannot be removed.
 * All compatibility with the bit field representation of codecs/formats has
   been moved to a compatibility API. The primary user of this representation
   is chan_iax2, which must continue to maintain its bit-field usage of formats
   for interoperability concerns.
 * When a format is negotiated with attributes, or when a format cannot be
   represented by one of the cached formats, a new format object is created or
   cloned from an existing format. That format may have the same codec
   underlying it, but is a different format than a version of the format with
   different attributes or without attributes.
 * While formats are reference counted objects, the reference count maintained
   on the format should be manipulated with care. Formats are generally cached
   and will persist for the lifetime of Asterisk and do not explicitly need
   to have their lifetime modified. An exception to this is when the user of a
   format does not know where the format came from *and* the user may outlive
   the provider of the format. This occurs, for example, when a format is read
   from a channel: the channel may have a format with attributes (hence,
   non-cached) and the user of the format may last longer than the channel (if
   the reference to the channel is released prior to the format's reference).

For more information on this work, see the API design notes:

Finally, this work was the culmination of a large number of developer's
efforts. Extra thanks goes to Corey Farrell, who took on a large amount of the
work in the Asterisk core, chan_sip, and was an invaluable resource in peer
reviews throughout this project.

There were a substantial number of patches contributed during this work; the
following issues/patch names simply reflect some of the work (and will cause
the release scripts to give attribution to the individuals who work on them).


ASTERISK-23114 #close
Reported by: mjordan
  media_formats_translation_core.diff uploaded by kharwell (License 6464)
  rb3506.diff uploaded by mjordan (License 6283)
  media_format_app_file.diff uploaded by kharwell (License 6464)
  misc-2.diff uploaded by file (License 5000)
  chan_mild-3.diff uploaded by file (License 5000)
  chan_obscure.diff uploaded by file (License 5000)
  jingle.diff uploaded by file (License 5000)
  funcs.diff uploaded by file (License 5000)
  formats.diff uploaded by file (License 5000)
  core.diff uploaded by file (License 5000)
  bridges.diff uploaded by file (License 5000)
  mf-codecs-2.diff uploaded by file (License 5000)
  mf-app_fax.diff uploaded by file (License 5000)
  mf-apps-3.diff uploaded by file (License 5000)
  media-formats-3.diff uploaded by file (License 5000)

  rb3713.patch uploaded by coreyfarrell (License 5909)
  rb3689.patch uploaded by mjordan (License 6283)

  rb3722.patch uploaded by mjordan (License 6283)
  mf-attributes-3.diff uploaded by file (License 5000)

Tested by: jrose
  rb3822.patch uploaded by coreyfarrell (License 5909)
  rb3800.patch uploaded by jrose (License 6182)
  chan_sip.diff uploaded by mjordan (License 6283)
  rb3747.patch uploaded by jrose (License 6182)

ASTERISK-23959 #close
Tested by: sgriepentrog, mjordan, coreyfarrell
  sip_cleanup.diff uploaded by opticron (License 6273)
  chan_sip_caps.diff uploaded by mjordan (License 6283)
  rb3751.patch uploaded by coreyfarrell (License 5909)
  chan_sip-3.diff uploaded by file (License 5000)

ASTERISK-23960 #close
Tested by: opticron
  direct_media.diff uploaded by opticron (License 6273)
  pjsip-direct-media.diff uploaded by file (License 5000)
  format_cap_remove.diff uploaded by opticron (License 6273)
  media_format_fixes.diff uploaded by opticron (License 6273)
  chan_pjsip-2.diff uploaded by file (License 5000)

ASTERISK-23966 #close
Tested by: rmudgett
  rb3803.patch uploaded by rmudgetti (License 5621)
  chan_dahdi.diff uploaded by file (License 5000)

ASTERISK-24064 #close
Tested by: coreyfarrell, mjordan, opticron, file, rmudgett, sgriepentrog, jrose
  rb3814.patch uploaded by rmudgett (License 5621)
  moh_cleanup.diff uploaded by opticron (License 6273)
  bridge_leak.diff uploaded by opticron (License 6273)
  translate.diff uploaded by file (License 5000)
  rb3795.patch uploaded by rmudgett (License 5621)
  tls_fix.diff uploaded by mjordan (License 6283)
  fax-mf-fix-2.diff uploaded by file (License 5000)
  rtp_transfer_stuff uploaded by mjordan (License 6283)
  rb3787.patch uploaded by rmudgett (License 5621)
  media-formats-explicit-translate-format-3.diff uploaded by file (License 5000)
  format_cache_case_fix.diff uploaded by opticron (License 6273)
  rb3774.patch uploaded by rmudgett (License 5621)
  rb3775.patch uploaded by rmudgett (License 5621)
  rtp_engine_fix.diff uploaded by opticron (License 6273)
  rtp_crash_fix.diff uploaded by opticron (License 6273)
  rb3753.patch uploaded by mjordan (License 6283)
  rb3750.patch uploaded by mjordan (License 6283)
  rb3748.patch uploaded by rmudgett (License 5621)
  media_format_fixes.diff uploaded by opticron (License 6273)
  rb3740.patch uploaded by mjordan (License 6283)
  rb3739.patch uploaded by mjordan (License 6283)
  rb3734.patch uploaded by mjordan (License 6283)
  rb3689.patch uploaded by mjordan (License 6283)
  rb3674.patch uploaded by coreyfarrell (License 5909)
  rb3671.patch uploaded by coreyfarrell (License 5909)
  rb3667.patch uploaded by coreyfarrell (License 5909)
  rb3665.patch uploaded by mjordan (License 6283)
  rb3625.patch uploaded by coreyfarrell (License 5909)
  rb3602.patch uploaded by coreyfarrell (License 5909)
  format_compatibility-2.diff uploaded by file (License 5000)
  core.diff uploaded by file (License 5000)

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

8 years agoari: Add a copy operation for stored recordings
Matthew Jordan [Fri, 18 Jul 2014 21:48:46 +0000 (21:48 +0000)]
ari: Add a copy operation for stored recordings

This patch adds a new operation for stored recordings, copy. It takes an
existing stored recording and makes a copy of it in the same directory
or a relative directory under the stored recording directory.


This is particularly useful for voicemail-esque applications, which may need to
copy or move recordings around a directory structure.


ASTERISK-24036 #close
Reported by: Sam Galarneau
Tested by: Sam Galarneau

Merged revisions 419021 from

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

8 years agostasis: fix call to ao2_t_alloc for stasis_message_router_create
Corey Farrell [Fri, 18 Jul 2014 21:25:14 +0000 (21:25 +0000)]
stasis: fix call to ao2_t_alloc for stasis_message_router_create

This fixes a build failure introduced by r3821.  struct stasis_topic is
opaque, so topic->name is unavailable.  Switch to using stasis_topic_name().

Merged revisions 419019 from

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

8 years agostasis: use ao2_t_alloc for certain object allocators
Corey Farrell [Fri, 18 Jul 2014 19:55:24 +0000 (19:55 +0000)]
stasis: use ao2_t_alloc for certain object allocators

Add tags to stasis objects using the name.  This makes it
easier to track the source of certain stasis ref leaks.


Merged revisions 418996 from

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

8 years agoFix build in dev-mode
Kinsey Moore [Fri, 18 Jul 2014 19:07:12 +0000 (19:07 +0000)]
Fix build in dev-mode

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

8 years agoastobj2: assert on invalid ref and backtrace cleanup
Scott Griepentrog [Fri, 18 Jul 2014 17:55:38 +0000 (17:55 +0000)]
astobj2: assert on invalid ref and backtrace cleanup

If a reference count goes negative, instead of
just logging that fact, be more helpful with a
backtrace and an assert that will DO_CRASH.

This patch also removes the duplicate ao2_bt()
function and cleans up extraneous usage of the
ast_log_backtrace() call.


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

8 years agomedia formats: fix ref leak of peer for mwi subscription
Scott Griepentrog [Fri, 18 Jul 2014 17:42:41 +0000 (17:42 +0000)]
media formats: fix ref leak of peer for mwi subscription

Holding a reference to the peer during mwi subscriptions
resulted in a circular reference because the final event
message would not be sent until destruction of the peer.

Instead, pass the name of the peer to the event callback
so that it can fail gracefully after the peer has gone.


Merged revisions 418636 from

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

8 years agofeature_config: insure featuregroups and applicationmaps are initialized
Scott Griepentrog [Fri, 18 Jul 2014 17:40:54 +0000 (17:40 +0000)]
feature_config: insure featuregroups and applicationmaps are initialized

If the features.conf is missing, the cfg->featurgroups
and cfg->applicationmaps is not initialized, resulting
in assert on ao2_find of a null container.  This patch
changes the initialization call and adds asserts for a


Merged revisions 418886 from

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

8 years agofunc_audiohookinherit.c: Fixup some XML documentation wording.
Richard Mudgett [Fri, 18 Jul 2014 16:47:23 +0000 (16:47 +0000)]
func_audiohookinherit.c: Fixup some XML documentation wording.

Merged revisions 418937 from

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

8 years agoChannels: Masquerades to automatically move frame/audio hooks
Jonathan Rose [Fri, 18 Jul 2014 16:28:10 +0000 (16:28 +0000)]
Channels: Masquerades to automatically move frame/audio hooks

Whenever possible, audiohooks and framehooks will now be copied over
to the channel that the masquerading channel gets cloned into. This
should occur for all audiohooks and most framehooks. As a result,
in Asterisk 12.5 and up, the AUDIOHOOK_INHERIT function is now
deprecated and its behavior is essentially the new default for all
audiohooks, plus some additional audiohooks/framehooks.


Merged revisions 418914 from

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

8 years agores_fax: Provide AMI equivalents for fax CLI commands
Jonathan Rose [Fri, 18 Jul 2014 15:49:46 +0000 (15:49 +0000)]
res_fax: Provide AMI equivalents for fax CLI commands

Specifically the following equivalents were created:
fax show session -> FAXSession
fax show sessions -> FAXSessions
fax show stats -> FAXStats


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

8 years agoUpdate config.guess and config.sub
Sean Bright [Fri, 18 Jul 2014 00:11:37 +0000 (00:11 +0000)]
Update config.guess and config.sub

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

8 years agoAdd missing file from previous commit.
Sean Bright [Fri, 18 Jul 2014 00:10:16 +0000 (00:10 +0000)]
Add missing file from previous commit.

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

8 years agoImport Asterisk's autoconf magic instead of using our own.
Sean Bright [Fri, 18 Jul 2014 00:07:48 +0000 (00:07 +0000)]
Import Asterisk's autoconf magic instead of using our own.

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

8 years agoconfigs: Move sample config files into a subdirectory of configs
Matthew Jordan [Thu, 17 Jul 2014 21:17:28 +0000 (21:17 +0000)]
configs: Move sample config files into a subdirectory of configs

This moves all samples configs from configs/ to configs/samples. This allows
for additional sets of sample configuration files to be added in the future.


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

8 years agochan_sip: Make progressinband=never really mean 'never'
Matthew Jordan [Thu, 17 Jul 2014 21:04:01 +0000 (21:04 +0000)]
chan_sip: Make progressinband=never really mean 'never'

progressinband=never in sip.conf is easily defeated if an onward trunk sends a
progress indication of its own. This is almost certain to happen if the onward
trunk is ISDN or IAX as these technologies send a progress indication even if
early media is not required. This progress message is passed to the caller,
and causes the "never" option to be rather badly named.

This patch changes the behaviour of this setting in the following ways:

1) In sip_write(), do not pass the media unless we have either progressed
   beyond INV_EARLY_MEDIA, or we are in INV_EARLY_MEDIA state, and early
   media is both set-up and wanted. This helps resolve double-ringing on some
   buggy handsets.

2) In sip_indicate(), if we see AST_CONTROL_PROGRESS, but
   SIP_PROG_INBAND_NEVER is set, send a 180 Ringing instead to avoid implicitly
   enabling early media. Avoid sending double ring indications.

NOTE: the meaning of the SIP_PROGRESS_SENT flag changes slightly in this patch
to also encapsulate the fact that a channel has *sent or received* a 183
Progress indication. This makes the updated code in sip_write() much more


ASTERISK-23972 #close
Reported by: Steve Davies
  inband_never_present_early_media2 uploaded by Steve Davies (License 5012)

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

8 years agoAdd svn:ignore property
Matthew Jordan [Thu, 17 Jul 2014 20:08:50 +0000 (20:08 +0000)]
Add svn:ignore property

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

8 years agoconfigure: Fix libxml2 development library dependency checking
Matthew Jordan [Thu, 17 Jul 2014 19:31:05 +0000 (19:31 +0000)]
configure: Fix libxml2 development library dependency checking

The commit that added libxml2 support didn't fully check for the libxml2
development script in the Asterisk configure file. As a result, Asterisk could
be configured, then fail on menuselect. This patch fixes it so that Asterisk
should detect the libxml2 dependency failure first.

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

8 years agomenuselect: Add libxml2 support (Patch 3)
Matthew Jordan [Thu, 17 Jul 2014 19:02:22 +0000 (19:02 +0000)]
menuselect: Add libxml2 support (Patch 3)

This is the final patch in adding menuselect to Asterisk.
 - The first patch (r418832) added menuselect along with mxml
 - The second patch (r418833) removed mxml from menuselect

This patch adds support for libxml2 to menuselect, and makes libxml2 a
required library for Asterisk.

Note that the libxml2 portion of this patch was written by Sean Bright,
and was made available on a team branch:


ASTERISK-20703 #close
  some_mysterious_team_branch uploaded by seanbright (License 5060)

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

8 years agomenuselect: Remove mxml from menuselect (Patch 2)
Matthew Jordan [Thu, 17 Jul 2014 18:46:06 +0000 (18:46 +0000)]
menuselect: Remove mxml from menuselect (Patch 2)

This is the second patch that adds menuselect to Asterisk trunk. The previous
commit (r418832) added menuselect along with mxml; this patch removes mxml
completely from Menuselect.

A subsequent patch will switch menuselect over to using libxml2, and make
libxml2 a required dependency for Asterisk.


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

8 years agomenuselect: Add menuselect to Asterisk trunk (Patch 1)
Matthew Jordan [Thu, 17 Jul 2014 18:42:43 +0000 (18:42 +0000)]
menuselect: Add menuselect to Asterisk trunk (Patch 1)

This is the first patch that adds menuselect to Asterisk trunk, and removes
the svn:externals property. This is being done for two reasons:
(1) The removal of external repositories eases a future migration to git
(2) Asterisk is now the only thing that uses menuselect; as a result, there's
    little need to keep it in an external repository

Subsequent patches will remove the mxml dependency from menuselect and tidy
up the build system.


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

8 years agoTEST_FRAMEWORK: Fix threewaytransfer reporting
Kinsey Moore [Thu, 17 Jul 2014 14:28:16 +0000 (14:28 +0000)]
TEST_FRAMEWORK: Fix threewaytransfer reporting

Ensure that three-way transfers can be reported even if featuremap is

Merged revisions 418810 from

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

8 years agoRemove include of astobj.h from channels/dahdi/bridge_native_dahdi.c.
Corey Farrell [Wed, 16 Jul 2014 23:08:19 +0000 (23:08 +0000)]
Remove include of astobj.h from channels/dahdi/bridge_native_dahdi.c.

The include was unneeded, this is split off from r3758 as it applies to 12.

Merged revisions 418787 from

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

8 years agores_pjsip: Support setting a default accountcode on endpoints
Matthew Jordan [Wed, 16 Jul 2014 14:03:51 +0000 (14:03 +0000)]
res_pjsip: Support setting a default accountcode on endpoints

Most channel drivers let you specify a default accountcode to be set on
channels associated with a particular peer/endpoint/object. Prior to this
patch, chan_pjsip/res_pjsip did not support such a setting.

This patch adds a new setting to the res_pjsip endpoint object, 'accountcode'.
When a channel is created that is associated with an endpoint with this value
set, the channel will automatically have its accountcode property set to the
value configured for the endpoint.


ASTERISK-24000 #close
Reported by: Matt Jordan

Merged revisions 418756 from

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

8 years agocel_pgsql, cdr_pgsql, res_config_pgsql: Add PostgreSQL application_name support
Matthew Jordan [Wed, 16 Jul 2014 13:55:36 +0000 (13:55 +0000)]
cel_pgsql, cdr_pgsql, res_config_pgsql: Add PostgreSQL application_name support

This patch adds support for the PostgreSQL application_name connection setting.
When the appropriate PostgreSQL module's configuration is set with an
application name, the name will be passed to PostgreSQL on connection and
displayed in the database's pg_stat_activity view, as well as in CSV logs. This
aids in managing which applications/servers are connected to a PostgreSQL
database, as well as tracing the activity of those connections.


ASTERISK-23737 #close
Reported by: Gergely Domodi
  pgsql_application_name.patch uploaded by Gergely Domodi (License 6610)

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

8 years agocodec_adpcm: Change description of codec "ADPCM" to "Dialogic ADPCM"
Matthew Jordan [Tue, 15 Jul 2014 23:29:29 +0000 (23:29 +0000)]
codec_adpcm: Change description of codec "ADPCM" to "Dialogic ADPCM"

Technically, ADPCM is a method that can be applied to several codecs.
Asterisk's ADPCM codec is the Dialogic ADPCM or VOX codec.

See for more information
about said codec.


  rb3744.patch uploaded by dennis.guse (License 6513)

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

8 years agomanager: Return ActionID on nominal responses to PresenceState action
Matthew Jordan [Tue, 15 Jul 2014 23:12:33 +0000 (23:12 +0000)]
manager: Return ActionID on nominal responses to PresenceState action

When the PresenceState action is executed, the nominal path fails to include
the ActionID in the successful response. This patch adds a call to
astman_start_ack, which guarantees that an ActionID (if provided) will be
sent back to the AMI client.

Unlike the Asterisk 11 and 12 patches, this patch also deprecates the
duplicate Message key in the response to the action, replacing it with the
key 'PresenceMessage'.


ASTERISK-23985 #close

Merged revisions 418713 from

Merged revisions 418714 from

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

8 years agoTEST_FRAMEWORK: Fix ref leak in feature activation
Kinsey Moore [Tue, 15 Jul 2014 23:03:40 +0000 (23:03 +0000)]
TEST_FRAMEWORK: Fix ref leak in feature activation

This fixes two reference leaks that would occur when TEST_FRAMEWORK was
enabled and features were successfully executed.

Merged revisions 418715 from

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

8 years agofunc_uri: URIENCODE/URIDECODE - allow empty strings as argument
Jonathan Rose [Tue, 15 Jul 2014 17:57:17 +0000 (17:57 +0000)]
func_uri: URIENCODE/URIDECODE - allow empty strings as argument

Previously these two dialplan functions would issue warnings and
return failure when an empty string is used as the argument. Now
they will not issue a warning and will successfully return an
empty string.

ASTERISK-23911 #close
Reported by: Matt Jordan

Merged revisions 418641 from

Merged revisions 418649 from

Merged revisions 418650 from

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

8 years agoUpdate Asterisk copyright year in main/asterisk.c
Sean Bright [Tue, 15 Jul 2014 12:11:25 +0000 (12:11 +0000)]
Update Asterisk copyright year in main/asterisk.c

It's been 2014 for like... 6 months.

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

8 years agologger.h: Extract DEBUG_ATLEAST() to complement VERBOSITY_ATLEAST().
Richard Mudgett [Mon, 14 Jul 2014 14:55:30 +0000 (14:55 +0000)]
logger.h: Extract DEBUG_ATLEAST() to complement VERBOSITY_ATLEAST().

Merged revisions 418586 from

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

8 years agoActually delete the removed files.
Richard Mudgett [Mon, 14 Jul 2014 01:45:01 +0000 (01:45 +0000)]
Actually delete the removed files.

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

8 years agoastobj2: work around REF_DEBUG race which causes out of order log entries
Corey Farrell [Sun, 13 Jul 2014 21:57:00 +0000 (21:57 +0000)]
astobj2: work around REF_DEBUG race which causes out of order log entries

* Update to use delta's to track the current reference count.
* Use result from internal_ao2_ref to write old_refcount to refs_log.


Merged revisions 418504 from

Merged revisions 418505 from

Merged revisions 418506 from

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

8 years agoastobj2: correct define for ao2_t_cleanup
Scott Griepentrog [Sun, 13 Jul 2014 20:08:37 +0000 (20:08 +0000)]
astobj2: correct define for ao2_t_cleanup

This change maps the ao2_t_cleanup() function to the
correct debug function so that it can be used.


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

8 years agoFix minor reference leaks in app_skel and TEST_FRAMEWORK
Corey Farrell [Sun, 13 Jul 2014 16:48:48 +0000 (16:48 +0000)]
Fix minor reference leaks in app_skel and TEST_FRAMEWORK

* Cleanup games object in app_skel.
* Cleanup stasis subscription to TEST_FRAMEWORK in manager.c (12+).


Merged revisions 418465 from

Merged revisions 418466 from

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

8 years agoRemove files left behind on removal of h323, jingle and jabber.
Corey Farrell [Sun, 13 Jul 2014 05:05:49 +0000 (05:05 +0000)]
Remove files left behind on removal of h323, jingle and jabber.

This change removes h323.conf.sample, jingle.h, jabber.h left behind by r3698.


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

8 years agoastobj2: Add tag variants for ao2_bump, ao2_cleanup, and ao2_replace
Matthew Jordan [Fri, 11 Jul 2014 23:00:21 +0000 (23:00 +0000)]
astobj2: Add tag variants for ao2_bump, ao2_cleanup, and ao2_replace

Tags are useful in hunting down ref imbalances; this patch adds tag variants
for these commonly used macros/functions.


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

8 years agoastobj2: tweak ao2_replace to do nothing when it would be a NoOp
Corey Farrell [Fri, 11 Jul 2014 21:10:45 +0000 (21:10 +0000)]
astobj2: tweak ao2_replace to do nothing when it would be a NoOp

This change causes ao2_replace to do nothing when src == dst. This
avoids REF_DEBUG logging when we're not actually doing anything.


Merged revisions 418396 from

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

8 years agoconfig: inform config hook of change when writing file
Scott Griepentrog [Fri, 11 Jul 2014 16:42:49 +0000 (16:42 +0000)]
config: inform config hook of change when writing file

When updated configuration is written back to the conf
file - for example when a user changes their voicemail
pin, make sure that any config hook that wants to know
of changes is informed.


Merged revisions 418366 from

Merged revisions 418369 from

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

8 years agoinclude/asterisk/xmpp.h: Convert indentation to tabs
Matthew Jordan [Thu, 10 Jul 2014 15:36:55 +0000 (15:36 +0000)]
include/asterisk/xmpp.h: Convert indentation to tabs

This is a whitespace only change.

Merged revisions 418323 from

Merged revisions 418324 from

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

8 years agochan_dahdi/sig_pri: Fix type mismatch in the idledial feature's channel creation.
Richard Mudgett [Thu, 10 Jul 2014 01:59:44 +0000 (01:59 +0000)]
chan_dahdi/sig_pri: Fix type mismatch in the idledial feature's channel creation.

Square pegs in round holes don't work very well.

Merged revisions 418261 from

Merged revisions 418262 from

Merged revisions 418263 from

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

8 years agoARI: Make mixing bridges propagate linkedids and accountcodes.
Richard Mudgett [Wed, 9 Jul 2014 16:34:51 +0000 (16:34 +0000)]
ARI: Make mixing bridges propagate linkedids and accountcodes.

* Create a Stasis bridge sub-class to propagate linkedids and

* Fixed the basic bridge sub-class to update peeraccount codes when the
number of channels in the bridge drops back down to two parties.

* Refactored ast_bridge_channel_update_accountcodes() to handle channels
joining/leaving the bridge.

* Fixed the basic bridge sub-class to not call the base bridge class pull
method twice.

AFS-105 #close
ASTERISK-23852 #close
Reported by: Richard Mudgett


Merged revisions 418225 from

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

8 years agomanager/ARI: Update version to 2.4.0/1.4.0; Update UPGRADE.txt
Matthew Jordan [Tue, 8 Jul 2014 14:48:30 +0000 (14:48 +0000)]
manager/ARI: Update version to 2.4.0/1.4.0; Update UPGRADE.txt

Merged revisions 418182 from

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

8 years agores_rtp_asterisk: Fix undefined function when PJPROJECT is not installed
Matthew Jordan [Tue, 8 Jul 2014 14:38:42 +0000 (14:38 +0000)]
res_rtp_asterisk: Fix undefined function when PJPROJECT is not installed

The dtls_perform_handshake function was mistakenly placed under the guards for
USE_PJPROJECT. If PJPROJECT was not installed, the function would not be
defined, while other functions would attempt to still use it. This prevented
res_rtp_asterisk from being loaded.

ASTERISK-24001 #close
Reported by: Don Fanning

Merged revisions 418172 from

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

8 years agores_pjsip_dialog_info_body_generator: Add dialog-info+xml support for presence.
Joshua Colp [Mon, 7 Jul 2014 16:08:47 +0000 (16:08 +0000)]
res_pjsip_dialog_info_body_generator: Add dialog-info+xml support for presence.

This module implements dialog-info+xml for the purposes of presence. This means
that phones such as Grandstreams can now subscribe to receive presence information
for an extension.

ASTERISK-21443 #close
Reported by: Matt Jordan


Merged revisions 418116 from

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

8 years agoARI/res_stasis: Subscribe to both Local channel halves when originating to app
Matthew Jordan [Mon, 7 Jul 2014 02:15:00 +0000 (02:15 +0000)]
ARI/res_stasis: Subscribe to both Local channel halves when originating to app

This patch fixes two bugs:

1. When originating a channel into a Stasis application, we already create a
   subscription for the channel that is going into our Stasis app.
   Unfortunately, when you create a Local channel and pass it off to a Stasis
   app, you really aren't creating just one channel: you're creating two. This
   patch snags the second half of the Local channel pair (assuming it is a
   Local channel pair, but luckily core_local is kind about such assumptions)
   and subscribes to it as well.

2. Subscriptions are a bit sticky right now. If a subscription is made, the
   'interest' count gets bumped on the Stasis subscription - but unless
   something explicitly unsubscribes the channel, said subscription sticks
   around. This is not much of a problem is a user is creating the subscription
   - if they made it, they must want it. However, when we are creating
   implicit subscriptions, we need to make sure something clears them out.
   This patch takes a pessimistic approach: it watches the cache updates
   coming from Stasis and, if we notice that the cache just cleared out an
   object, we delete our subscription object. This keeps our ao2 container of
   Stasis forwards in an application from growing out of hand; it also is a
   bit more forgiving for end users who may not realize they were supposed to
   unsubscribe from that channel that just hung up.

#ASTERISK-23939 #close

Merged revisions 418089 from

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

8 years agoCEL: Fix incorrect/missing extra field information
Kinsey Moore [Mon, 7 Jul 2014 01:22:44 +0000 (01:22 +0000)]
CEL: Fix incorrect/missing extra field information

This corrects two issues with the extra field information in Asterisk
12+ in channel event logs.

It is possible to inject custom values into the dialstatus provided by
ast_channel_dial_type() Stasis messages that fall outside the
enumeration allowed for the DIALSTATUS channel variable. CEL now
filters for the allowed values and ignores other values.

The "hangupsource" extra field key is always blank if the far end
channel is a chan_pjsip channel. This is because the hangupsource is
never set for the pjsip channel driver. This change sets the
hangupsource whenever a hangup is queued for chan_pjsip channels.

This corrects an issue with the pjsip channel driver where the
hangupcause information was not being set properly.


Merged revisions 418071 from

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

8 years agoHTTP: Fix build for gcc 4.10
Kinsey Moore [Mon, 7 Jul 2014 01:10:55 +0000 (01:10 +0000)]
HTTP: Fix build for gcc 4.10

Merged revisions 418066 from

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

8 years agomain/Makefile: fix compilation error of buildinfo occurring on 'make install'
Matthew Jordan [Fri, 4 Jul 2014 15:26:58 +0000 (15:26 +0000)]
main/Makefile: fix compilation error of buildinfo occurring on 'make install'

Egads. Another bad deletion of too much when attempting to remove h323 stuff.

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

8 years agoconfigure: Remove last vestiges of h323; DO create menuselect-deps
Matthew Jordan [Fri, 4 Jul 2014 15:11:51 +0000 (15:11 +0000)]
configure: Remove last vestiges of h323; DO create menuselect-deps

The previous patch (r418034) fixed the 'glitch' that the channels/h323
Makefile no longer existed. Unfortunately, removing the entire line was a bit
of a blunder, as it meant that build_tools/menuselect-deps was never
generated. Hilarity ensued when actually trying to compile.

But hey! At least configure worked.

This patch fixes *that* glitch, and removes some more of the vestiges of h323.
(It had tendrils in the main Makefile? Crazy.)

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

8 years agoconfigure: Update script to pass if channels/h323/ does not exist
Matthew Jordan [Fri, 4 Jul 2014 14:33:22 +0000 (14:33 +0000)]
configure: Update script to pass if channels/h323/ does not exist

This simply removes that check from the configure script, as r418019 removed

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

8 years agoRemove many deprecated modules
Matthew Jordan [Fri, 4 Jul 2014 13:26:37 +0000 (13:26 +0000)]
Remove many deprecated modules

Billing records are fair,
To get paid is quite bright,
You should really use ODBC;
Good-bye cdr_sqlite.

Microsoft did once push H.323,
Hell, we all remember NetMeeting.
But try to compile chan_h323 now
And you will take quite a beating.

The XMPP and SIP war was fierce,
And in the distant fray
Was birthed res_jabber/chan_jingle;
But neither to stay.

For everyone did care and chase what Google professed.
"Free Internet Calling" was what devotees cried,
But Google did change the specs so often
That the developers were happy the day chan_gtalk died.

And then there was that odd application
Dedicated to the Polish tongue.
app_saycountpl was subsumed by Say;
One could say its bell was rung.

To read and parse a file from the dialplan
You could (I guess) use an application.
app_readfile did fill that purpose, but I think
A function is perhaps better in its creation.

Barging is rude, I'm not sure why we do it.
Inwardly, the caller will probably sigh.
But if you really must do it,
Don't use app_dahdibarge, use ChanSpy.

We all despise the sound of tinny robots
It makes our queues so cold.
To control such an abomination
It's better to not use Wait/SetMusicOnHold.

It's often nice to know properties of a channel
It makes our calls right
We have a nice function called CHANNEL
And so SIPCHANINFO is sent off into the night.

And now things get odd;
Apparently one could delimit with a colon
Properties from the SIPPEER function!
Commas are in; all others are done.

Finally, a word on pipes and commas.
We're sorry. We can't say it enough.
But those compatibility options in asterisk.conf;
To maintain them forever was just too tough.

This patch removes:

* cdr_sqlite
* chan_gtalk
* chan_jingle
* chan_h323
* res_jabber
* app_saycountpl
* app_readfile
* app_dahdibarge

It removes the following applications/functions:

* WaitMusicOnHold
* SetMusicOnHold

It removes the colon delimiter from the SIPPEER function.

Finally, it also removes all compatibility options that were configurable from
asterisk.conf, as these all applied to compatibility with Asterisk 1.4 systems.


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

8 years agochan_dahdi: Add inband_on_setup_ack compatibility option.
Richard Mudgett [Thu, 3 Jul 2014 22:22:36 +0000 (22:22 +0000)]
chan_dahdi: Add inband_on_setup_ack compatibility option.

The new inband_on_setup_ack option causes Asterisk to assume inband audio
may be present when a SETUP_ACKNOWLEDGE message is received.

Q.931 Section 5.1.3 says that in scenarios with overlap dialing, when a
dialtone is sent from the network side, progress indicator 8 "Inband info
now available" MAY be sent to the CPE if no digits were received with the
SETUP.  It is thus implied that the ie is mandatory if digits came with
the SETUP and dialtone is needed.  This option should be enabled, when the
network sends dialtone and you want to hear it, but the network doesn't
send the progress indicator when needed.

NOTE: For Q.SIG setups this option should be enabled when outgoing overlap
dialing is also enabled because Q.SIG does not send the progress indicator
with the SETUP ACK.

The commit -r413714 (AST-1338) which causes this issue was dealing with a
SIP-to-ISDN interoperability issue.

This commit is a merge of the two patches indicated below.

ASTERISK-23897 #close
Reported by: Pavel Troller
      pri-4.diff (license #6302) patch uploaded by Pavel Troller
      jira_asterisk_23897_v11.patch (license #5621) patch uploaded by rmudgett


Merged revisions 417956 from

Merged revisions 417957 from

Merged revisions 417958 from

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

8 years agores_ari: Fix some off-nominal paths just dropping the HTTP connection.
Richard Mudgett [Thu, 3 Jul 2014 19:06:12 +0000 (19:06 +0000)]
res_ari: Fix some off-nominal paths just dropping the HTTP connection.

* Removed some incorrect newlines on ast_http_error() messages in

* Removed an incorrect newline in res_ari_channels.c.

Addendum to ASTERISK-23552

Merged revisions 417932 from

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

8 years agochan_dahdi: Add AMI commands for controlling PRI debugging output
Jonathan Rose [Thu, 3 Jul 2014 17:34:32 +0000 (17:34 +0000)]
chan_dahdi: Add AMI commands for controlling PRI debugging output

Adds the following AMI commands:
PRIDebugSet - Set PRI debug levels for a specific span
PRIDebugFileSet - Set the file used for PRI debug message output
PRIDebugFileUnset - Disables file output for PRI debug messages


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

8 years agopbx_config: Add manager actions to add/remove extensions
Jonathan Rose [Thu, 3 Jul 2014 17:20:00 +0000 (17:20 +0000)]
pbx_config: Add manager actions to add/remove extensions

Adds two new manager commands to pbx_config - DialplanExtensionAdd and
DialplanExtensionRemove which allow manager users to create and delete
extensions respectively.


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

8 years agoHTTP: Add persistent connection support.
Richard Mudgett [Thu, 3 Jul 2014 17:16:55 +0000 (17:16 +0000)]
HTTP: Add persistent connection support.

Persistent HTTP connection support is needed due to the increased usage of
the Asterisk core HTTP transport and the frequency at which REST API calls
are going to be issued.

* Add http.conf session_keep_alive option to enable persistent

* Parse and discard optional chunked body extension information and
trailing request headers.

* Increased the maximum application/json and
application/x-www-form-urlencoded body size allowed to 4k.  The previous
1k was kind of small.

* Removed a couple inlined versions of ast_http_manid_from_vars() by
calling the function.  manager.c:generic_http_callback() and

* Add missing va_end() in ast_ari_response_error().

* Eliminated unnecessary RAII_VAR() use in http.c:auth_create().

ASTERISK-23552 #close
Reported by: Scott Griepentrog


Merged revisions 417880 from

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

8 years agomain/tcptls: Add checks for OpenSSL Elliptic Curve support
Matthew Jordan [Thu, 3 Jul 2014 16:55:44 +0000 (16:55 +0000)]
main/tcptls: Add checks for OpenSSL Elliptic Curve support

The patch for ASTERISK-23905 that added PFS support in Asterisk depends on the
elliptic curve library support being present in OpenSSL. As it turns out, some
versions of OpenSSL don't have this library - notably the version running on
our build agents.

This patch fixes the build by providing a configure check for the specific
library calls that the PFS patch relies on.


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

8 years agoARI: Improvements to body parameters documentation
Sam Galarneau [Thu, 3 Jul 2014 16:14:39 +0000 (16:14 +0000)]
ARI: Improvements to body parameters documentation

The variables body parameter under the originate and originate with id
operations of the channel resource showed invalid JSON in its description.

The variables body parameter under the userEvent operation of the event
resource made no mention that the custom key/value pairs should be wrapped
in a variables key in order to be added to the custom user event.

ASTERISK-23975 #close


Merged revisions 417878 from

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

8 years Update wiki template to support body parameters
Sam Galarneau [Thu, 3 Jul 2014 15:35:22 +0000 (15:35 +0000)] Update wiki template to support body parameters

This patch updates the template and the swagger_model python
script to understand if an operation has a body parameter. If an operation
does have a body parameter, it will now be displayed in the corresponding
wiki entry.

Merged revisions 407389 from

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

8 years agodahdi_span_config_hook: automatically register new dahdi channels
Tzafrir Cohen [Thu, 3 Jul 2014 14:08:02 +0000 (14:08 +0000)]
dahdi_span_config_hook: automatically register new dahdi channels

Install a hook script for DAHDI to register new spans with Asterisk
automatically by running:

  asterisk -rx 'dahdi create channel FIRST LAST'


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

8 years agomain/tcptls: Add support for Perfect Forward Secrecy
Matthew Jordan [Thu, 3 Jul 2014 12:10:17 +0000 (12:10 +0000)]
main/tcptls: Add support for Perfect Forward Secrecy

This patch enables Perfect Forward Secrecy (PFS) in Asterisk's core TLS API.
Modules that wish to enable PFS should consider the following:

- Ephemeral ECDH (ECDHE) is enabled by default. To disable it, do not
  specify a ECDHE cipher suite in a module's configuration, for example:

- Ephemeral DH (DHE) is disabled by default. To enable it, add DH parameters
  into the private key file, i.e., tlsprivatekey. For an example, see the
  default dh2048.pem at

- Because clients expect the server to prefer PFS, and because OpenSSL sorts
  its cipher suites by bit strength, (see "openssl ciphers -v DEFAULT")
  consider re-ordering your cipher suites in the conf file. For example:
  will use PFS when offered by the client. Clients which do not offer PFS
  fall-back to AES-128 (or even 3DES as recommend by RFC 3261).


ASTERISK-23905 #close
Reported by: Alexander Traud
  tlsPFS_for_HEAD.patch uploaded by Alexander Traud (License 6520)
  tlsPFS.patch uploaded by Alexander Traud (License 6520)

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

8 years agomain/untils: Prevent potential infinite loop in ast_careful_fwrite
Matthew Jordan [Thu, 3 Jul 2014 11:27:25 +0000 (11:27 +0000)]
main/untils: Prevent potential infinite loop in ast_careful_fwrite

A loop in ast_careful_fwrite exists that will continually attempt to write to
a file stream, even in the presence of EAGAIN/EINTR errors. However, if a
connection that uses ast_careful_fwrite closes suddenly, ast_careful_fwrite's
call to fflush may return EAGAIN/EINTER along with EOF. A subsequent call to
fflush will return EOF but not clear errno, resulting in an infinite loop.

This patch clears errno after it is detected and handled the loop, such that
any subsequent call to fflush will not get erroneously stuck.


#ASTERISK-23984 #close
Reported by: Steve Davies
  fflush_loop_fix uploaded by one47 (License 5012)

Merged revisions 417797 from

Merged revisions 417798 from

Merged revisions 417799 from

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

8 years agoARI: Remove unnecessary \briefs from automatically generated documentation
Jonathan Rose [Wed, 2 Jul 2014 21:13:46 +0000 (21:13 +0000)]
ARI: Remove unnecessary \briefs from automatically generated documentation


Merged revisions 412653 from

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

8 years agores_rtp_asterisk: Don't leak memory or reset state if DTLS configuration is set multi...
Joshua Colp [Tue, 1 Jul 2014 14:42:46 +0000 (14:42 +0000)]
res_rtp_asterisk: Don't leak memory or reset state if DTLS configuration is set multiple times.

Merged revisions 417705 from

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

8 years agoRecorded merge of revisions 417677 from
Joshua Colp [Mon, 30 Jun 2014 19:51:28 +0000 (19:51 +0000)]
Recorded merge of revisions 417677 from

res_rtp_asterisk: Add SHA-256 support for DTLS and perform DTLS negotiation on RTCP.

This change fixes up DTLS support in res_rtp_asterisk so it can accept and provide
a SHA-256 fingerprint, so it occurs on RTCP, and so it occurs after ICE negotiation
completes. Configuration options to chan_sip and chan_pjsip have also been added to
allow behavior to be tweaked (such as forcing the AVP type media transports in SDP).

ASTERISK-22961 #close
Reported by: Jay Jideliov


Merged revisions 417678 from

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

8 years agoReverse logic during subscription persistence recreation.
Mark Michelson [Mon, 30 Jun 2014 18:39:56 +0000 (18:39 +0000)]
Reverse logic during subscription persistence recreation.

In the abstraction effort, this bit of logic got messed up. We
want to recreate the persistence if things go well, not if things

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

8 years agoapps/app_voicemail: Fix compilation error introduced in r417591
Matthew Jordan [Mon, 30 Jun 2014 13:02:43 +0000 (13:02 +0000)]
apps/app_voicemail: Fix compilation error introduced in r417591

Not sure why that change to ast_channel_alloc was made but ... okay.

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

8 years agoapp_voicemail, say: Add support for Japanese Language
Matthew Jordan [Mon, 30 Jun 2014 04:00:19 +0000 (04:00 +0000)]
app_voicemail, say: Add support for Japanese Language

This patch adds support for the Japanese language to both the say family of
applications, as well as for VoiceMail and VoiceMailMain. A new pack of
language sounds will be released at the same time as the next major version
of Asterisk to support the new language features.

The language features can be enabled using a language code of 'ja'.


ASTERISK-23324 #close
Reported by: Kevin McCoy
  app_voicemail.c.20140226.jb.patch uploaded by Kevin McCoy (License 6586)
  say.c.20140226.jb.patch uploaded by Kevin McCoy (License 6586)

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

8 years agochan_sip: be more tolerant of whitespace between attributes in SDP fmtp line
Matthew Jordan [Mon, 30 Jun 2014 03:27:00 +0000 (03:27 +0000)]
chan_sip: be more tolerant of whitespace between attributes in SDP fmtp line

This patch is essentially a backport of a small portion of r397526 from
ASTERISK-21981. In that patch, pass through support and format attribute
negotiation was added for Opus. Part of that included being more tolerant to
whitespace in the fmtp line of an SDP; that part of the patch is being
applied here.

As the author of the backport pointed out, in SDP, the fmtp line is allowed to
include whitespace between attributes. RFC 3267 chapter 8.3 (from 2001)
includes an example for this. This was not removed in the updated RFC 4867 in


#ASTERISK-23916 #close
Reported by: Alexander Traud
  sdpFMTPspace_Asterisk11.patch uploaded by Alexander Traud (License 6520)

Merged revisions 417587 from

Merged revisions 417588 from

Merged revisions 417589 from

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

8 years agoevent.c: Fix type mismatch errors in ie_maps[].
Richard Mudgett [Fri, 27 Jun 2014 23:21:44 +0000 (23:21 +0000)]
event.c: Fix type mismatch errors in ie_maps[].

In v12+ the type values from the table are only used by the CEL unit
tests.  Since the unit tests were only comparing a generated expected
event with a real event to see if the ie contents matched and using the
same table IE_PLTYPE values to read the event contents, the type
mismatches were not detected.

Merged revisions 417565 from

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

8 years agoEnsure REF_DEBUG records entrys for attempts to ao2_ref an invalid object
Corey Farrell [Fri, 27 Jun 2014 19:27:59 +0000 (19:27 +0000)]
Ensure REF_DEBUG records entrys for attempts to ao2_ref an invalid object

This change ensures that __ao2_ref_debug writes to ref_log when given a
non-NULL pointer to an invalid ao2 object.  This is to ensure that we
record any attempt manipulate references of already freed objects.

ASTERISK-23948 #close
Reported by: Corey Farrell

Merged revisions 417500 from

Merged revisions 417505 from

Merged revisions 417509 from

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

8 years prevent use of excessive RAM with large refs logs
Corey Farrell [Fri, 27 Jun 2014 19:18:32 +0000 (19:18 +0000)] prevent use of excessive RAM with large refs logs

When processing a 212MB refs file, used over 3GB of RAM.
This change greatly reduces memory usage in two ways:

* Saving object history in whole lines instead of separated values.
* Not saving normal/skewed/leaked object lists unless they are requested.

ASTERISK-23921 #close
Reported by: Corey Farrell

Merged revisions 417480 from

Merged revisions 417481 from

Merged revisions 417483 from

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

8 years agores_pjsip: Add ActionID to events created as a result of PJSIP AMI actions
Matthew Jordan [Fri, 27 Jun 2014 13:50:02 +0000 (13:50 +0000)]
res_pjsip: Add ActionID to events created as a result of PJSIP AMI actions

A number of various PJSIP AMI actions were failing to parse out and place the
ActionID into their responses. This patch updates the various PJSIP actions
such that the passed in ActionID is emitted on any event list complete events,
as well as any intermediate events created as a result of the action.

#ASTERISK-23947 #close
Reported by: Mark Michelson


Merged revisions 417460 from

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

8 years agoCEL: Update unit tests for bridge tech field
Kinsey Moore [Fri, 27 Jun 2014 02:04:50 +0000 (02:04 +0000)]
CEL: Update unit tests for bridge tech field

Update the CEL unit tests that handle BRIDGE_ENTER and BRIDGE_EXIT
events to expect the "bridge_technology" extra field key.

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

8 years agoCHANGES: Add missing changes
Kinsey Moore [Thu, 26 Jun 2014 19:15:10 +0000 (19:15 +0000)]
CHANGES: Add missing changes

Add missing CHANGES changes from r417361 and r417383.

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

8 years agores_http_websocket: Export symbol for ast_websocket_set_timeout
Matthew Jordan [Thu, 26 Jun 2014 18:27:53 +0000 (18:27 +0000)]
res_http_websocket: Export symbol for ast_websocket_set_timeout

Thanks to Sean Bright for pointing out that this was missed in #asterisk-dev.

Merged revisions 417419 from

Merged revisions 417420 from

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

8 years agochan_pjsip: Add a test event for fast picture updates
Matthew Jordan [Thu, 26 Jun 2014 17:17:25 +0000 (17:17 +0000)]
chan_pjsip: Add a test event for fast picture updates

This will drive the test on review r3419. Note that the patch for this was
done by Ben Ford, although it was slightly modified for this commit.

Reported by: Matt Jordan

Merged revisions 417399 from

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

8 years agoCEL: Add bridge tech to relevant CEL records
Kinsey Moore [Thu, 26 Jun 2014 14:48:21 +0000 (14:48 +0000)]
CEL: Add bridge tech to relevant CEL records

Add the "bridge_technology" extra field key to BRIDGE_ENTER and
BRIDGE_EXIT CEL events to convey the bridge technology in use at the
time the record was generated.

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

8 years agoBridging: Allow channels to define bridging hooks
Kinsey Moore [Thu, 26 Jun 2014 12:43:47 +0000 (12:43 +0000)]
Bridging: Allow channels to define bridging hooks

This patch allows the current owner of a channel to define various
feature hooks to be made available once the channel has entered a
bridge. This includes any hooks that are setup on the
ast_bridge_features struct such as DTMF hooks, bridge event hooks
(join, leave, etc.), and interval hooks.


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

8 years agoapp_jack: Support audio with a sampling rate higher than 8kHz
Matthew Jordan [Thu, 26 Jun 2014 12:43:05 +0000 (12:43 +0000)]
app_jack: Support audio with a sampling rate higher than 8kHz

This patch enables the jack-audiohook to cope with dynamic sampling rates from
and to Asterisk. Information from the channel is taken to derive the channel's
sampling rate, suiting SLINxx format and frame->datalen.

There are stil a few limitations after this patch:
* Required information is taken from the channel during initialization as
  the audiohook does not provide this information.
  Audiohook.internal_sampl_rate(...) is set later, but no callback is available
  to inform app_jack.

* Frame.datalen is computed using "rate / 50" assuming a ptime of 20ms.
  There is no internal API available to determine datalen for a SLINxx.

* Ringbuffer size is now dynamic depending on the value of frame.datalen
  (see above) and the number of frames, which are in RINGBUFFER_FRAME_CAPACITY,
  that need to fit.


Note that the patch being committed here is based on the patch posted on
ASTERISK-23836. However, Matthis Schmieder also provided a patch to enable
this functionality, and that patch is noted below.

ASTERISK-20696 #close
Reported by: Matthis Schmieder
  app_jack.patch uploaded by Matthis Schmieder (License 6445)

ASTERISK-23836 #close
Reported by: Dennis Guse
  patch-app_jack.c uploaded by Dennis Guse (License 6513)

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

8 years agoudptl: Correct FEC to not consider negative sequence numbers as missing
Matthew Jordan [Thu, 26 Jun 2014 12:24:25 +0000 (12:24 +0000)]
udptl: Correct FEC to not consider negative sequence numbers as missing

When using FEC, with span=3 and entries=4 Asterisk will attempt to repair
the packet with sequence number 5, as it will see that packet -4 is
missing. The result is Asterisk sending garbage packets that can kill a

This patch adds a check to see if the sequence number is valid before
checking if the packet is missing.


#ASTERISK-23908 #close
Reported by: Torrey Searle
  udptl_fec.patch uploaded by Torrey Searle (License 5334)

Merged revisions 417318 from

Merged revisions 417320 from

Merged revisions 417324 from

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

8 years agores_http_websocket: Close websocket correctly and use careful fwrite
Matthew Jordan [Thu, 26 Jun 2014 12:21:14 +0000 (12:21 +0000)]
res_http_websocket: Close websocket correctly and use careful fwrite

When a client takes a long time to process information received from Asterisk,
a write operation using fwrite may fail to write all information. This causes
the underlying file stream to be in an unknown state, such that the socket
must be disconnected. Unfortunately, there are two problems with this in
Asterisk's existing websocket code:
1. Periodically, during the read loop, Asterisk must write to the connected
   websocket to respond to pings. As such, Asterisk maintains a reference to
   the session during the loop. When ast_http_websocket_write fails, it may
   cause the session to decrement its ref count, but this in and of itself
   does not break the read loop. The read loop's write, on the other hand,
   does not break the loop if it fails. This causes the socket to get in a
   'stuck' state, preventing the client from reconnecting to the server.
2. More importantly, however, is that the fwrite in ast_http_websocket_write
   fails with a large volume of data when the client takes awhile to process
   the information. When it does fail, it fails writing only a portion of
   the bytes. With some debugging, it was shown that this was failing in a
   similar fashion to ASTERISK-12767. Switching this over to ast_careful_fwrite
   with a long enough timeout solved the problem.

Note that this version of the patch, unlike r417310 in Asterisk 11, exposes
configuration options beyond just chan_sip's sip.conf. Configuration options
to configure the write timeout have also been added to pjsip.conf and ari.conf.

#ASTERISK-23917 #close
Reported by: Matt Jordan


Merged revisions 417310 from

Merged revisions 417311 from

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

8 years agochan_sip: Fix handling of "From" headers longer than 256 characters
Corey Farrell [Thu, 26 Jun 2014 10:06:13 +0000 (10:06 +0000)]
chan_sip: Fix handling of "From" headers longer than 256 characters

From headers were processed using a 256 character buffer on the stack.
This change replaces that with a heap allocation by ast_strdup.

ASTERISK-23790 #close
Reported by: uniken1
Tested by: uniken1
    chan_sip-large-from-header-1.8-r3.patch uploaded by wdoekes (license 5674)

Merged revisions 417248 from

Merged revisions 417249 from

Merged revisions 417250 from

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

8 years agoAbstract PJSIP-specific elements from the pubsub API.
Mark Michelson [Wed, 25 Jun 2014 20:57:28 +0000 (20:57 +0000)]
Abstract PJSIP-specific elements from the pubsub API.

This helps to pave the way for RLS work that is to come.
Since this is a self-contained change and subscription
tests still pass, this work is being committed directly
to trunk instead of a working branch.

ASTERISK-23865 #close

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

8 years agoao2_container node object ignores REF_DEBUG in all places except one
Corey Farrell [Wed, 25 Jun 2014 18:57:04 +0000 (18:57 +0000)]
ao2_container node object ignores REF_DEBUG in all places except one

Almost every reference operation against container node's uses
__ao2_alloc or __ao2_ref, thereby preventing ref logging for
the nodes.  One node reference is released with ao2_t_ref, causing to falsely report skews and leaks for many nodes.

ASTERISK-23922 #close
Reported by: Corey Farrell

Merged revisions 417212 from

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

8 years agoSkinny: cleanup some log messages around sessions.
Damien Wedhorn [Wed, 25 Jun 2014 00:45:07 +0000 (00:45 +0000)]
Skinny: cleanup some log messages around sessions.

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

8 years agoMove eid functions to utils.c, mark netsock.h deprecated
Corey Farrell [Tue, 24 Jun 2014 02:50:15 +0000 (02:50 +0000)]
Move eid functions to utils.c, mark netsock.h deprecated

Move eid functions from netsock.c to utils.c.  These functions were
already published by utils.h.  Flag netsock.h as deprecated and switch
res_pjsip_session.h to use netsock2.h.  The only code that still uses
netsock.h is chan_iax2.

ASTERISK-23920 #close
Reported by: Corey Farrell

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

8 years agores_rtp_asterisk: Return the length of data written when sending via ICE instead...
Joshua Colp [Mon, 23 Jun 2014 18:50:18 +0000 (18:50 +0000)]
res_rtp_asterisk: Return the length of data written when sending via ICE instead of 0.

ASTERISK-23834 #close
Reported by: Richard Kenner

Merged revisions 417141 from

Merged revisions 417142 from

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

8 years agocore_unreal: Fix off by one buffer overwrite error.
Richard Mudgett [Mon, 23 Jun 2014 16:04:33 +0000 (16:04 +0000)]
core_unreal: Fix off by one buffer overwrite error.

Appending the ;2 to the user supplied ;1 uniqueid to create the ;2 version
if the user did not also supply an extra uniqueid for the ;2 channel
resulted in allocating a buffer that was one byte too small.

* Fix off by one error in ast_unreal_new_channels() when generating the ;2
uniqueid from the user suppled ;1 version.

* Pulled some long assignment lines from if tests to improve line break
readability in ast_unreal_new_channels().

Merged revisions 417119 from

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

8 years agosuspended destructions of pri spans on events
Tzafrir Cohen [Mon, 23 Jun 2014 07:44:19 +0000 (07:44 +0000)]
suspended destructions of pri spans on events

If a DAHDI span disappears, we wish for its representation in Asterisk
to be destroyed as well.

The information about the span's removal may come from several paths:

1. DAHDI sends DAHDI_EVENT_REMOVE on every channel.
2. An extra DAHDI_EVENT_REMOVED is sent on every subsequent call to
3. Every read (including the internal one by libpri on the D-channel)
   returns -ENODEV.

Asterisk responsds to DAHDI_EVENT_REMOVE on a channel by destroying it.

Destroying a channel requires holding the channel list lock (iflock).
Destroying a channel that is part of a span requires holding the span's
lock. Destroying a channel from a context that holds the span lock,
while at the same time another channel is destroyed directly, leads to a
deadlock. Solution: don't destroy span while holding the channels list

Thus changes in this patch:
* Deferring removal of PRI spans in response to events: doomed spans
  are collected on a list.
* Doomed spans are removed periodically by the monitor thread.
* ENODEV reads from the D-channel will warant the same deferred removal.


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

8 years agoastobj2: Add an ao2_replace macro to astobj2.h
George Joseph [Sun, 22 Jun 2014 18:53:26 +0000 (18:53 +0000)]
astobj2: Add an ao2_replace macro to astobj2.h

This macro replaces one object reference with another cleaning up the original.

param dst Pointer to the object that will be cleaned up.
param src Pointer to the object replacing it.

src's ref count is bumped if it's non-NULL.
dst's ref count is decremented if it's non-NULL.
src is assigned to dst,

This patch was reviewed on IRC by coreyfarrell and mjordan.

Tested by: George Joseph

Merged revisions 416995 from

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

8 years agobuild: Allow autoconf/ast_ext_tool_check to handle cross-compiling better.
George Joseph [Fri, 20 Jun 2014 23:18:36 +0000 (23:18 +0000)]
build: Allow autoconf/ast_ext_tool_check to handle cross-compiling better.

ast_ext_tool_check.m4 isn't handling cases where a path to a package is
provided (E.G. --with-mysqlclient=/some/sysroot) and the package has a config
tool (E.G. mysql_config) and the package has its own subdirectories in include
or lib.  For example, mysql's libraries are in ${MYSQLCLIENT_DIR}/usr/lib/mysql
but ast_ext_tool_check sets MYSQLCLIENT_LIB to ${MYSQLCLIENT_DIR}/usr/lib.
libxml2 has the same problem with its includes.  They're in
${LIBXML2_DIR}/usr/include/libxml2 not directly in ${LIBXML2_DIR}/usr/include.
Both cause configure to fail and there are others in the same boat.

The problem is caused by logic in ast_ext_tool_check that overrides the result
of the config tool's --cflags and --libs options if package_DIR is set.

This patch prepends package_DIR (if specified) to the -L and -I results from
the package's config tool instead of overriding them.

A regenerated ./configure and include/asterisk/ are included
but can be regenerated by running ./ at any time.

Tested by: George Joseph
Tested by: Matt Jordan

Merged revisions 416929 from

Merged revisions 416930 from

Merged revisions 416931 from

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

8 years agobuild: Allow autoconf/ast_ext_tool_check to handle cross-compiling better.
George Joseph [Fri, 20 Jun 2014 21:59:08 +0000 (21:59 +0000)]
build: Allow autoconf/ast_ext_tool_check to handle cross-compiling better.

ast_ext_tool_check.m4 isn't handling cases where a path to a package is
provided (E.G. --with-mysqlclient=/some/sysroot) and the package has a config
tool (E.G. mysql_config) and the package has its own subdirectories in include
or lib.  For example, mysql's libraries are in ${MYSQLCLIENT_DIR}/usr/lib/mysql
but ast_ext_tool_check sets MYSQLCLIENT_LIB to ${MYSQLCLIENT_DIR}/usr/lib.
libxml2 has the same problem with its includes.  They're in
${LIBXML2_DIR}/usr/include/libxml2 not directly in ${LIBXML2_DIR}/usr/include.
Both cause configure to fail and there are others in the same boat.

The problem is caused by logic in ast_ext_tool_check that overrides the result
of the config tool's --cflags and --libs options if package_DIR is set.

This patch prepends package_DIR (if specified) to the -L and -I results from
the package's config tool instead of overriding them.

Tested by: George Joseph
Tested by: Matt Jordan

Merged revisions 416870 from

Merged revisions 416871 from

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

8 years agores_parking: Make manager commands register with module information
Jonathan Rose [Fri, 20 Jun 2014 20:57:46 +0000 (20:57 +0000)]
res_parking: Make manager commands register with module information

Previously module information was not included due to an oversight.


Merged revisions 416849 from

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

8 years agoLogger: Add manager command 'LoggerRotate' to rotate logger
Jonathan Rose [Fri, 20 Jun 2014 20:29:45 +0000 (20:29 +0000)]
Logger: Add manager command 'LoggerRotate' to rotate logger

Part of a series of AMI command equivalents to existing CLI


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