6 years agoMerge "configure: Fix check for libunbound to require v1.5.0 as minimum."
Joshua Colp [Fri, 9 Oct 2015 17:08:29 +0000 (12:08 -0500)]
Merge "configure: Fix check for libunbound to require v1.5.0 as minimum."

6 years agoconfigure: Fix check for libunbound to require v1.5.0 as minimum.
Richard Mudgett [Thu, 8 Oct 2015 21:43:10 +0000 (16:43 -0500)]
configure: Fix check for libunbound to require v1.5.0 as minimum.

Versions of libunbound before v1.4.21 do not compile with Asterisk.
However, since v1.4.21 has a configure script bug that fails to detect the
ldns library (which is fixed in v1.4.22) and v1.4.22 is not an easily
detectable version we will require v1.5.0 as a minimum version of the
library to work with Asterisk.

ASTERISK-25108 #close
Reported by: Richard Mudgett

Change-Id: Ieb228bfb01467573fc121c7356a9dde27128894d

6 years agoMerge "chan_pjsip: Fix crash on reINVITE before initial INVITE completes."
Joshua Colp [Thu, 8 Oct 2015 18:48:33 +0000 (13:48 -0500)]
Merge "chan_pjsip: Fix crash on reINVITE before initial INVITE completes."

6 years agores_pjsip: Fix deadlock when sending out-of-dialog requests.
Richard Mudgett [Tue, 6 Oct 2015 23:01:37 +0000 (18:01 -0500)]
res_pjsip: Fix deadlock when sending out-of-dialog requests.

The struct send_request_wrapper has a pjsip lock associated with it that
is created non-recursive.  There is a code path for the struct
send_request_wrapper lock that will attempt to lock it recursively.  The
reporter's deadlock showed that the thread calling endpt_send_request()
deadlocked itself right after the wrapper object got created.

Out-of-dialog requests such as MESSAGE, qualify OPTIONS, and unsolicited
MWI NOTIFY messages can hit this deadlock.

* Replaced the struct send_request_wrapper pjsip lock with the mutex lock
that can come with an ao2 object since all of Asterisk's mutexes are
recursive.  Benefits include removal of code maintaining the pjsip
non-recursive lock since ao2 objects already know how to maintain their
own lock and the lock will show up in the CLI "core show locks" output.

ASTERISK-25435 #close
Reported by: Dmitriy Serov

Change-Id: I458e131dd1b9816f9e963f796c54136e9e84322d

6 years agoMerge "res/res_rtp_asterisk.c: Fix incorrect assignment of frame->subclass.frame_ending"
Joshua Colp [Wed, 7 Oct 2015 16:58:58 +0000 (11:58 -0500)]
Merge "res/res_rtp_asterisk.c: Fix incorrect assignment of frame->subclass.frame_ending"

6 years agores/res_rtp_asterisk.c: Fix incorrect assignment of frame->subclass.frame_ending
StefanEng86 [Tue, 6 Oct 2015 16:05:00 +0000 (18:05 +0200)]
res/res_rtp_asterisk.c: Fix incorrect assignment of frame->subclass.frame_ending

In ast_rtp_read, the value of the variable 'mark' which we try to assign to a
frame->subclass.frame_ending may be 0, 1 or (1<<23), but we should translate
it to 0 or 1.

ASTERISK-25451 #close
Change-Id: I53bdf5c026041730184a6a809009c028549ce626

6 years agofunc_presencestate: Return "not_set" when no data is set in AstDB
Ivan Poddubny [Wed, 7 Oct 2015 06:24:31 +0000 (09:24 +0300)]
func_presencestate: Return "not_set" when no data is set in AstDB

Return AST_PRESENCE_NOT_SET when CustomPresence AstDB key does not
exist, i.e. when a new CustomPresence is added in the dialplan.

ASTERISK-25400 #close
Reported by: Andrew Nagy

Change-Id: I6fb17b16591b5a55fbffe96f3994ec26b1b1723a

6 years agores/res_rtp_asterisk: Fix assignment after ao2 decrement
Matt Jordan [Wed, 7 Oct 2015 01:43:58 +0000 (20:43 -0500)]
res/res_rtp_asterisk: Fix assignment after ao2 decrement

When we decide we will no longer schedule an RTCP write, we remove the
reference to the RTP instance, then assign -1 to the stored scheduler ID
in case something else comes along and wants to see if anything is scheduled.

That scheduler ID is on the RTP instance. After 60a9172d7ef2 was merged to
fix the regression introduced by 3cf0f29310, this improper assignment on a
potentially destroyed object started getting tripped on the build agents.

Frankly, this should have been crashing a lot more often earlier. I can only
assume that the timing was changed just enough by both changes to start
actually hitting this problem.

As it is, simply moving the assignment prior to the ao2 deference is sufficient
to keep the RTP instance from being referenced when it is very, truly,
aboslutely dead.

(Note that it is still good practice to assign -1 to the scheduler ID when we
know we won't be scheduling it again, as the ao2 deref *may* not always destroy
the ao2 object.)


Change-Id: Ie6d3cb4adc7b1a6c078b1c38c19fc84cf787cda7

6 years agochan_sip: Fix port parsing for IPv6 addresses in SIP Via headers.
Florian Sauerteig [Tue, 6 Oct 2015 17:40:22 +0000 (19:40 +0200)]
chan_sip: Fix port parsing for IPv6 addresses in SIP Via headers.

If a Via header containes an IPv6 address and a port number is ommitted,
as it is the standard port, we now leave the port empty and to not set it
to the value after the first colon of the IPv6 address.

ASTERISK-25443 #close

Change-Id: Ie3c2f05471cd006bf04ed15598589c09577b1e70

6 years agochan_pjsip: Fix crash on reINVITE before initial INVITE completes.
Richard Mudgett [Mon, 5 Oct 2015 21:53:44 +0000 (16:53 -0500)]
chan_pjsip: Fix crash on reINVITE before initial INVITE completes.

Apparently some endpoints attempt to send a reINVITE before completing the
initial INVITE transaction.  In this case PJSIP responds appropriately to
the reINVITE with a 491 INVITE request pending.  Unfortunately chan_pjsip
is using the initial INVITE transaction state to determine if an INVITE is
the initial INVITE or a reINVITE.  Since the initial INVITE transaction
has not been confirmed yet chan_pjsip thinks the reINVITE is an initial
INVITE and starts another PBX thread on the channel.  The extra PBX thread
ensures that hilarity ensues.

* Fix checks for a reINVITE on incoming requests to look for the presence
of a to-tag instead of the initial INVITE transaction state.

* Made caller_id_incoming_request() determine what to do if there is a
channel on the session or not.  After a channel is created it is too late
to just store the new party id on the session because the session's party
id has already been copied to the channel's caller id.

ASTERISK-25404 #close
Reported by: Chet Stevens

Change-Id: Ie78201c304a2b13226f3a4ce59908beecc2c68be

6 years agoMerge "Fix improper usage of scheduler exposed by 5c713fdf18f"
Matt Jordan [Tue, 6 Oct 2015 13:30:13 +0000 (08:30 -0500)]
Merge "Fix improper usage of scheduler exposed by 5c713fdf18f"

6 years agoFix improper usage of scheduler exposed by 5c713fdf18f
Matt Jordan [Tue, 6 Oct 2015 02:34:41 +0000 (21:34 -0500)]
Fix improper usage of scheduler exposed by 5c713fdf18f

When 5c713fdf18f was merged, it allowed for scheduled items to have an ID of
'0' returned. While this was valid per the documentation for the API, it was
apparently never returned previously. As a result, several users of the
scheduler API viewed the result as being invalid, causing them to reschedule
already scheduled items or otherwise fail in interesting ways.

This patch corrects the users such that they view '0' as valid, and a returned
ID of -1 as being invalid.

Note that the failing HEP RTCP tests now pass with this patch. These tests
failed due to a duplicate scheduling of the RTCP transmissions.

ASTERISK-25449 #close

Change-Id: I019a9aa8b6997584f66876331675981ac9e07e39

6 years agochan_pjsip: Add Referred-By header to the PJSIP REFER packet.
Debian Amtelco [Wed, 26 Aug 2015 21:58:04 +0000 (21:58 +0000)]
chan_pjsip: Add Referred-By header to the PJSIP REFER packet.

Some systems require the REFER packet to include a Referred-By header.
If the channel variable SIPREFERREDBYHDR is set, it passes that value as the
Referred-By header value.  Otherwise, it adds the current dialog’s local info.

Reported by: Dan Cropp
Tested by: Dan Cropp

Change-Id: I3d17912ce548667edf53cb549e88a25475eda245

6 years agomanager: Fix GetConfigJSON returning invalid JSON
Ivan Poddubny [Sat, 3 Oct 2015 11:27:27 +0000 (14:27 +0300)]
manager: Fix GetConfigJSON returning invalid JSON

When GetConfigJSON was introduced back in 1.6, it returned each
section as an array of strings: ["key=value", "key2=value2"].
Afterwards, it was changed a few times and became
["key": "value", "key2": "value2"], which is not a correct JSON.
This patch fixes that by constructing a JSON object {} instead of
an array [].

Also, the keys "istemplate" and "tempates" that are used to
indicate templates and their inherited categories are now wrapped in

ASTERISK-25391 #close
Reported by: Bojan Nemčić

Change-Id: Ibbe93c6a227dff14d4a54b0d152341857bcf6ad8

6 years agoMerge "res_sorcery_memory_cache.c: Fix deadlock with scheduler."
Joshua Colp [Fri, 2 Oct 2015 21:28:09 +0000 (16:28 -0500)]
Merge "res_sorcery_memory_cache.c: Fix deadlock with scheduler."

6 years agoMerge "res_sorcery_memory_cache.c: Replace inline code with function."
Joshua Colp [Fri, 2 Oct 2015 21:28:03 +0000 (16:28 -0500)]
Merge "res_sorcery_memory_cache.c: Replace inline code with function."

6 years agoMerge "res_sorcery_memory_cache.c: Shutdown in a less crash potential order."
Joshua Colp [Fri, 2 Oct 2015 21:27:57 +0000 (16:27 -0500)]
Merge "res_sorcery_memory_cache.c: Shutdown in a less crash potential order."

6 years agoMerge "res_sorcery_memory_cache.c: Misc tweaks."
Joshua Colp [Fri, 2 Oct 2015 21:27:51 +0000 (16:27 -0500)]
Merge "res_sorcery_memory_cache.c: Misc tweaks."

6 years agoMerge "res_sorcery_memory_cache.c: Made use OBJ_SEARCH_MASK."
Joshua Colp [Fri, 2 Oct 2015 21:27:46 +0000 (16:27 -0500)]
Merge "res_sorcery_memory_cache.c: Made use OBJ_SEARCH_MASK."

6 years agoMerge "sched.c: Add warning about negative time interval request."
Joshua Colp [Fri, 2 Oct 2015 21:27:07 +0000 (16:27 -0500)]
Merge "sched.c: Add warning about negative time interval request."

6 years agores_sorcery_memory_cache.c: Fix deadlock with scheduler.
Richard Mudgett [Wed, 30 Sep 2015 22:28:19 +0000 (17:28 -0500)]
res_sorcery_memory_cache.c: Fix deadlock with scheduler.

A deadlock can happen when a sorcery object is being expired from the
memory cache when at the same time another object is being placed into the
memory cache.  There are a couple other variations on this theme that
could cause the deadlock.  Basically if an object is being expired from
the sorcery memory cache at the same time as another thread tries to
update the next object expiration timer the deadlock can happen.

* Add a deadlock avoidance loop in expire_objects_from_cache() to check if
someone is trying to remove the scheduler callback from the scheduler.

ASTERISK-25441 #close

Change-Id: Iec7b0bdb81a72b39477727b1535b2539ad0cf4dc

6 years agores_sorcery_memory_cache.c: Replace inline code with function.
Richard Mudgett [Thu, 1 Oct 2015 19:30:06 +0000 (14:30 -0500)]
res_sorcery_memory_cache.c: Replace inline code with function.

Make sorcery_memory_cache_close() call remove_all_from_cache() instead of
partially inlining it.


Change-Id: I1aa6cb425b1a4307096f3f914d17af8ec179a74c

6 years agores_sorcery_memory_cache.c: Shutdown in a less crash potential order.
Richard Mudgett [Thu, 1 Oct 2015 19:27:34 +0000 (14:27 -0500)]
res_sorcery_memory_cache.c: Shutdown in a less crash potential order.

Basically you should shutdown in the opposite order of how you setup since
later setup pieces likely depend on earlier setup pieces.  e.g.,
Registering your external API with the rest of the system should be the
last thing setup and the first thing unregistered during shutdown.

Change-Id: I5715765b723100c8d3c2642e9e72cc7ad5ad115e

6 years agores_sorcery_memory_cache.c: Misc tweaks.
Richard Mudgett [Wed, 30 Sep 2015 22:27:54 +0000 (17:27 -0500)]
res_sorcery_memory_cache.c: Misc tweaks.

Change-Id: I8cd32dffbb4f33bb0c39518d6e4c991e73573160

6 years agores_sorcery_memory_cache.c: Made use OBJ_SEARCH_MASK.
Richard Mudgett [Wed, 30 Sep 2015 22:27:23 +0000 (17:27 -0500)]
res_sorcery_memory_cache.c: Made use OBJ_SEARCH_MASK.

Change-Id: Ibca6574dc3c213b29cc93486e01ccd51f5caa46c

6 years agores_rtp_asterisk: Move "Set role" warning to be debug.
Joshua Colp [Wed, 30 Sep 2015 18:42:40 +0000 (15:42 -0300)]
res_rtp_asterisk: Move "Set role" warning to be debug.

In practice the set_role API callback can be invoked even
when no ICE is present on an RTP instance. This can occur
if ICE has not been enabled on it.

ASTERISK-25438 #close

Change-Id: I0e17e4316f0f0d7f095c78c3d4fd73a913b6ba69

6 years agosched.c: Add warning about negative time interval request.
Richard Mudgett [Mon, 28 Sep 2015 20:31:38 +0000 (15:31 -0500)]
sched.c: Add warning about negative time interval request.

Change-Id: Ib91435fb45b7f5f7c0fc83d0eec20b88098707bc

6 years agores/ari/config.c: Fix user sort compare function.
Richard Mudgett [Fri, 25 Sep 2015 23:37:04 +0000 (18:37 -0500)]
res/ari/config.c: Fix user sort compare function.

Made use the ao2 sort compare template function and OBJ_SEARCH_xxx

Change-Id: Ic53005dc5aafa7a36c72300dd89b75fb63c92f4c

6 years agores/ari/config.c: Optimize conf_alloc() object init.
Richard Mudgett [Fri, 25 Sep 2015 22:26:08 +0000 (17:26 -0500)]
res/ari/config.c: Optimize conf_alloc() object init.

* Now conf_alloc() has more off nominal error checking.

* Eliminated RAII_VAR() use in conf_alloc().

* Eliminated a dubius shortcut when destroying cfg->general in
conf_destructor() that would cause a crash if cfg->general failed to get

* Add some ACO registration section comments.

Change-Id: Ia40c2b1b2d0777d641605118ae019c5a73865e1a

6 years agores/ari/config.c: Fix conf_alloc() object init.
Richard Mudgett [Fri, 25 Sep 2015 21:48:10 +0000 (16:48 -0500)]
res/ari/config.c: Fix conf_alloc() object init.

Need to finish initializing the string fields in the ao2 object before
putting any default strings into them.

ASTERISK-25383 #close
Reported by:  yaron nahum

Change-Id: I9f7f3a03f0c4991a01593abf8697b9a587c0ea84

6 years agomain/logger: Add log formatters and JSON structured logs
Matt Jordan [Mon, 21 Sep 2015 12:26:00 +0000 (07:26 -0500)]
main/logger: Add log formatters and JSON structured logs

When Asterisk is part of a larger distributed system, log files are often
gathered using tools (such as logstash) that prefer to consume information
and have it rendered using other tools (such as Kibana) that prefer a
structured format, e.g., JSON. This patch adds support for JSON formatted
logs by adding support for an optional log format specifier in Asterisk's
logging subsystem. By adding a format specifier of '[json]':

full => [json]debug,verbose,notice,warning,error

Log messages will be output to the 'full' channel in the following

  "hostname": Hostname or name specified in asterisk.conf
  "timestamp": Date/Time
  "identifiers": {
    "lwp": Thread ID,
    "callid": Call Identifier
  "logmsg": {
    "location": {
      "filename": Name of the file that generated the log statement
      "function": Function that generated the log statement
      "line": Line number that called the logging function
    "level": Log level, e.g., DEBUG, VERBOSE, etc.
    "message": Actual text of the log message

ASTERISK-25425 #close

Change-Id: I8649bfedf3fb7bf3138008cc11565553209cc238

6 years agoMerge "app_queue.c: Force COLP update if outgoing channel name changed."
Matt Jordan [Tue, 29 Sep 2015 12:27:50 +0000 (07:27 -0500)]
Merge "app_queue.c: Force COLP update if outgoing channel name changed."

6 years agoMerge "app_queue.c: Factor out a connected line update routine."
Matt Jordan [Tue, 29 Sep 2015 12:27:13 +0000 (07:27 -0500)]
Merge "app_queue.c: Factor out a connected line update routine."

6 years agoMerge "app_dial.c: Make 'A' option pass COLP updates."
Matt Jordan [Tue, 29 Sep 2015 12:27:02 +0000 (07:27 -0500)]
Merge "app_dial.c: Make 'A' option pass COLP updates."

6 years agoMerge "app_dial.c: Force COLP update if outgoing channel name changed."
Matt Jordan [Tue, 29 Sep 2015 12:26:20 +0000 (07:26 -0500)]
Merge "app_dial.c: Force COLP update if outgoing channel name changed."

6 years agoMerge "translate: Fix transcoding while different in frame size."
Matt Jordan [Mon, 28 Sep 2015 21:24:19 +0000 (16:24 -0500)]
Merge "translate: Fix transcoding while different in frame size."

6 years agoMerge "app_dial.c: Factor out a connected line update routine."
Joshua Colp [Mon, 28 Sep 2015 19:07:05 +0000 (14:07 -0500)]
Merge "app_dial.c: Factor out a connected line update routine."

6 years agoMerge "res/res_stasis: Fix accidental subscription to 'all' bridge topic"
Matt Jordan [Mon, 28 Sep 2015 13:36:09 +0000 (08:36 -0500)]
Merge "res/res_stasis: Fix accidental subscription to 'all' bridge topic"

6 years agoMerge "Scripts: check file versions of Asterisk and dependencies"
Joshua Colp [Mon, 28 Sep 2015 11:50:12 +0000 (06:50 -0500)]
Merge "Scripts: check file versions of Asterisk and dependencies"

6 years agores/res_stasis: Fix accidental subscription to 'all' bridge topic
Matt Jordan [Mon, 28 Sep 2015 01:45:50 +0000 (20:45 -0500)]
res/res_stasis: Fix accidental subscription to 'all' bridge topic

When b99a7052621700a1aa641a1c24308f5873275fc8 was merged, subscribing to a
NULL bridge will now cause app_subscribe_bridge to implicitly subscribe to
all bridges. Unfortunately, the res_stasis control loop did not check that
a bridge changing on a channel's control object was actually also non-NULL.
As a result, app_subscribe_bridge will be called with a NULL bridge when a
channel leaves a bridge. This causes a new subscription to be made to the
bridge. If an application has also subscribed to the bridge, the application
will now have two subscriptions:
(1) The explicit one created by the app
(2) The implicit one accidentally created by the control structure

As a result, the 'BridgeDestroyed' event can be sent multiple times. This
patch corrects the control loop such that it only subscribes an application
to a new bridge if the bridge pointer is non-NULL.


Change-Id: I3510e55f6bc36517c10597ead857b964463c9f4f

6 years agoScripts: check file versions of Asterisk and dependencies
Scott Griepentrog [Fri, 4 Sep 2015 18:51:46 +0000 (13:51 -0500)]
Scripts: check file versions of Asterisk and dependencies

To help in diagnosing mismatched modules and libraries, this
script scans for version, repository, and source information
and reports what is found.

ASTERISK-25376 #close
Reported by: Ashley Sanders

Change-Id: Ib0642d0fb96712476f59760d6d137a24633fe2d6

6 years agoapp_queue.c: Force COLP update if outgoing channel name changed.
Richard Mudgett [Thu, 24 Sep 2015 19:56:24 +0000 (14:56 -0500)]
app_queue.c: Force COLP update if outgoing channel name changed.

* When a call is answered and the outgoing channel name has changed then
force a connected line update because the channel is no longer the same.
The channel was masqueraded into by another channel.  This is usually
because of a call pickup.

Note: Forwarded calls are handled in a controlled manner so the original
channel name is replaced with the forwarded channel.

ASTERISK-25423 #close
Reported by: John Hardin

Change-Id: Ie275ea9e99c092ad369db23e0feb08c44498c172

6 years agoapp_queue.c: Factor out a connected line update routine.
Richard Mudgett [Thu, 24 Sep 2015 19:20:24 +0000 (14:20 -0500)]
app_queue.c: Factor out a connected line update routine.

Replace inlined code with update_connected_line_from_peer().

Reported by: John Hardin

Change-Id: I33bbd033596fcb0208d41d8970369b4e87b806f3

6 years agoapp_dial.c: Make 'A' option pass COLP updates.
Richard Mudgett [Thu, 24 Sep 2015 18:27:44 +0000 (13:27 -0500)]
app_dial.c: Make 'A' option pass COLP updates.

While the 'A' option is playing the announcement file allow the caller and
peer to exchange COLP update frames.

Reported by: John Hardin

Change-Id: Iac6cf89b56d26452c6bb88e9363622bbf23895f9

6 years agoapp_dial.c: Force COLP update if outgoing channel name changed.
Richard Mudgett [Thu, 24 Sep 2015 17:59:08 +0000 (12:59 -0500)]
app_dial.c: Force COLP update if outgoing channel name changed.

* When a call is answered and the outgoing channel name has changed then
force a connected line update because the channel is no longer the same.
The channel was masqueraded into by another channel.  This is usually
because of a call pickup.

Note: Forwarded calls are handled in a controlled manner so the original
channel name is replaced with the forwarded channel.

Reported by: John Hardin

Change-Id: I2e01f7a698fbbc8c26344a59c2be40c6cd98b00c

6 years agoapp_dial.c: Factor out a connected line update routine.
Richard Mudgett [Thu, 24 Sep 2015 17:37:07 +0000 (12:37 -0500)]
app_dial.c: Factor out a connected line update routine.

Replace inlined code with update_connected_line_from_peer().

Reported by: John Hardin

Change-Id: Ia14f18def417645cd7fb453e1bdac682630a5091

6 years agoapp_dial.c: Remove some no-op code.
Richard Mudgett [Wed, 23 Sep 2015 22:41:35 +0000 (17:41 -0500)]
app_dial.c: Remove some no-op code.

Change-Id: Ice1884a94315d3cb7e3bbd47a9fba76a27276c54

6 years agoMerge "logger: Prevent duplicate dynamic channels from being added."
Joshua Colp [Fri, 25 Sep 2015 15:57:40 +0000 (10:57 -0500)]
Merge "logger: Prevent duplicate dynamic channels from being added."

6 years agologger: Prevent duplicate dynamic channels from being added.
Mark Michelson [Wed, 23 Sep 2015 19:02:15 +0000 (14:02 -0500)]
logger: Prevent duplicate dynamic channels from being added.

There was a problem observed where the "logger add channel" CLI command
would allow for a channel with the same name to be added multiple times.
This would result in each message being written out to the same file
multiple times.

The problem was due to the difference in how logger channel filenames
are stored versus the format they are allowed to be presented when they
are added. For instance, if adding the logger channel "foo" through the
CLI, the result would be a logger channel with the file name
/var/log/asterisk/foo being stored. So when trying to add another "foo"
channel, "foo" would not match "/var/log/asterisk/foo" so we'd happily
add the duplicate channel.

The fix presented here is to introduce two new methods in the logger
 * make_filename(): given a logger channel name, this creates the
   filename for that logger channel.
 * find_logchannel(): given a logger channel name, this calls
   make_filename() and then traverses the list of logchannels in order
   to find a match.

This change has made use of make_filename() and find_logchannel()
throughout to more consistently behave.

ASTERISK-25305 #close
Reported by Mark Michelson

Change-Id: I892d52954d6007d8bc453c3cbdd9235dec9c4a36

6 years agoDo not swallow frames on channels leaving bridges.
Mark Michelson [Thu, 24 Sep 2015 19:49:46 +0000 (14:49 -0500)]
Do not swallow frames on channels leaving bridges.

When leaving a bridge, indications on a channel could be swallowed by
the internal indication logic because it appears that the channel is on
its way to be hung up anyway. One such situation where this is
detrimental is when channels on hold are redirected out of a bridge. The
AST_CONTROL_UNHOLD indication from the bridging code is swallowed,
leaving the channel in question to still appear to be on hold.

The fix here is to modify the logic inside ast_indicate_data() to not
drop the indication if the channel is simply leaving a bridge. This way,
channels on hold redirected out of a bridge revert to their expected "in
use" state after the redirection.

ASTERISK-25418 #close
Reported by Mark Michelson

Change-Id: If6115204dfa0551c050974ee138fabd15f978949

6 years agoMerge "ARI: Add the ability to subscribe to all events"
Matt Jordan [Wed, 23 Sep 2015 17:56:36 +0000 (12:56 -0500)]
Merge "ARI: Add the ability to subscribe to all events"

6 years agoMerge "res/res_stasis_device_state: Allow for subscribing to 'all' device state"
Matt Jordan [Wed, 23 Sep 2015 17:56:31 +0000 (12:56 -0500)]
Merge "res/res_stasis_device_state: Allow for subscribing to 'all' device state"

6 years agoMerge "ARI: Add events for Contact and Peer Status changes"
Matt Jordan [Wed, 23 Sep 2015 17:56:26 +0000 (12:56 -0500)]
Merge "ARI: Add events for Contact and Peer Status changes"

6 years agoapp_page.c: Fix crash when forwarding with a predial handler.
Richard Mudgett [Tue, 22 Sep 2015 22:08:49 +0000 (17:08 -0500)]
app_page.c: Fix crash when forwarding with a predial handler.

Page uses the async method of dialing with the dial API.  When a call gets
forwarded there is no calling channel available.  If the predial handler
was set then the calling channel could not be put into auto-service
for the forwarded call because it doesn't exist.  A crash is the result.

* Moved the callee predial parameter string processing to before the
string is passed to the dial API rather than having the dial API do it.
There are a few benefits do doing this.  The first is the predial
parameter string processing doesn't need to be done for each channel
called by the dial API.  The second is in async mode and the forwarded
channel is to have the predial handler executed on it then the
non-existent calling channel does not need to be present to process the
predial parameter string.

* Don't start auto-service on a non-existent calling channel to execute
the predial handler when the dial API is in async mode and forwarding a

ASTERISK-25384 #close
Reported by: Chet Stevens

Change-Id: If53892b286d29f6cf955e2545b03dcffa2610981

6 years agoMerge "core/logging: Fix logging to more than one syslog channel"
Matt Jordan [Tue, 22 Sep 2015 18:20:39 +0000 (13:20 -0500)]
Merge "core/logging: Fix logging to more than one syslog channel"

6 years agoARI: Add the ability to subscribe to all events
Matt Jordan [Fri, 4 Sep 2015 17:25:07 +0000 (12:25 -0500)]
ARI: Add the ability to subscribe to all events

This patch adds the ability to subscribe to all events. There are two possible
ways to accomplish this:
(1) On initial WebSocket connection. This patch adds a new query parameter,
    'subscribeAll'. If present and True, Asterisk will subscribe the
    applications to all ARI events.
(2) Via the applications resource. When subscribing in this manner, an ARI
    client should merely specify a blank resource name, i.e., 'channels:'
    instead of 'channels:12354'. This will subscribe the application to all
    resources of the 'channels' type.

ASTERISK-24870 #close

Change-Id: I4a943b4db24442cf28bc64b24bfd541249790ad6

6 years agoMerge "app_record: RECORDED_FILE variable not being populated"
Matt Jordan [Tue, 22 Sep 2015 12:40:35 +0000 (07:40 -0500)]
Merge "app_record: RECORDED_FILE variable not being populated"

6 years agoMerge "pbx: Update device and presence state when changing a hint extension."
Joshua Colp [Tue, 22 Sep 2015 10:29:52 +0000 (05:29 -0500)]
Merge "pbx: Update device and presence state when changing a hint extension."

6 years agoapp_record: RECORDED_FILE variable not being populated
Kevin Harwell [Mon, 21 Sep 2015 23:06:15 +0000 (18:06 -0500)]
app_record: RECORDED_FILE variable not being populated

The RECORDED_FILE variable is empty unless a '%d' is specified in the filename.
This patch makes it so the variable is always set to the filename.

ASTERISK-25410 #close

Change-Id: I4ec826d8eb582ae2ad184e717be8668b74d37653

6 years agoMerge "astfd: Adds a timestamp for each entry."
Joshua Colp [Mon, 21 Sep 2015 13:43:45 +0000 (08:43 -0500)]
Merge "astfd: Adds a timestamp for each entry."

6 years agocore/logging: Fix logging to more than one syslog channel
Elazar Broad [Mon, 21 Sep 2015 13:16:46 +0000 (09:16 -0400)]
core/logging: Fix logging to more than one syslog channel

Currently, Asterisk will log to the last configured syslog
channel in logger.conf. This is due to the fact that the
final call to openlog() supersedes all of the previous calls.
This commit removes the call to openlog() and passes the
facility to ast_log_vsyslog(), along with utilizing the
LOG_MAKEPRI macro to ensure that the message is routed to
the correct facility and with the correct priority.

ASTERISK-25407 #close
Reported by: Elazar Broad
Tested by: Elazar Broad

Change-Id: Ie2a2416bc00cce1b04e99ef40917c2011953ddd2

6 years agores/res_stasis_device_state: Allow for subscribing to 'all' device state
Matt Jordan [Fri, 4 Sep 2015 17:24:57 +0000 (12:24 -0500)]
res/res_stasis_device_state: Allow for subscribing to 'all' device state

This patch adds support for subscribing to all device state changes. This is
done either by subscribing to an empty device, e.g., 'eventSource=deviceState:',
or by the WebSocket connection specifying that it wants all state in the


Change-Id: I9cfeca1c9e2231bd7ea73e45919111d44d2eda32

6 years agoARI: Add events for Contact and Peer Status changes
Matt Jordan [Fri, 4 Sep 2015 02:19:21 +0000 (21:19 -0500)]
ARI: Add events for Contact and Peer Status changes

This patch adds support for receiving events regarding Peer status changes
and Contact status changes. This is particularly useful in scenarios where
we are subscribed to all endpoints and channels, where we often want to know
more about the state of channel technology specific items than a single
endpoint's state.


Change-Id: I6137459cdc25ce27efc134ad58abf065653da4e9

6 years agoMerge "main/config_options: Check for existance of internal object before derefing"
Matt Jordan [Mon, 21 Sep 2015 13:08:36 +0000 (08:08 -0500)]
Merge "main/config_options: Check for existance of internal object before derefing"

6 years agoMerge "app_queue: AgentComplete event has wrong reason"
Matt Jordan [Sat, 19 Sep 2015 21:26:33 +0000 (16:26 -0500)]
Merge "app_queue: AgentComplete event has wrong reason"

6 years agoastfd: Adds a timestamp for each entry.
Alexander Traud [Sat, 19 Sep 2015 17:49:46 +0000 (19:49 +0200)]
astfd: Adds a timestamp for each entry.

Now with menuselect "DEBUG_FD_LEAKS" and CLI "core show fd", a timestamp is
shown with each file descriptor. This helps to debug leaked UDP/TCP ports on
long-lived servers, for example.

ASTERISK-25405 #close

Change-Id: I968339e5155a512eba1032a5263f1ec8b5e1f80b

6 years agoMerge "app_queue: Crash when transferring"
Matt Jordan [Sat, 19 Sep 2015 14:15:37 +0000 (09:15 -0500)]
Merge "app_queue: Crash when transferring"

6 years agoMerge "CHAOS: res_pjsip_diversion avoid crash if allocation fails"
Matt Jordan [Sat, 19 Sep 2015 14:15:21 +0000 (09:15 -0500)]
Merge "CHAOS: res_pjsip_diversion avoid crash if allocation fails"

6 years agoMerge "dr_adaptive_odbc.c, cel_odbc.c, cel_pgsql.c: REFACTOR Macro LENGTHEN_BUF"
Matt Jordan [Sat, 19 Sep 2015 14:15:02 +0000 (09:15 -0500)]
Merge "dr_adaptive_odbc.c, cel_odbc.c, cel_pgsql.c: REFACTOR Macro LENGTHEN_BUF"

6 years agoMerge "chan_sip: Fix From header truncation for extremely long CALLERID(name)."
Joshua Colp [Sat, 19 Sep 2015 13:31:52 +0000 (08:31 -0500)]
Merge "chan_sip: Fix From header truncation for extremely long CALLERID(name)."

6 years agoMerge "CHAOS: avoid crash if string create fails"
Joshua Colp [Sat, 19 Sep 2015 13:22:45 +0000 (08:22 -0500)]
Merge "CHAOS: avoid crash if string create fails"

6 years agopbx: Update device and presence state when changing a hint extension.
Joshua Colp [Wed, 16 Sep 2015 13:22:35 +0000 (10:22 -0300)]
pbx: Update device and presence state when changing a hint extension.

When changing a hint extension without removing the hint first the
device state and presence state is not updated. This causes the state
of the hint to be that of the previous extension and not the current
one. This state is kept until a state change occurs as a result of
something (presence state change, device state change).

This change updates the hint with the current device and presence
state of the new extension when it is changed. Any state callbacks
which may have been added before the hint extension is changed are
also informed of the new device and presence state if either have

ASTERISK-25394 #close

Change-Id: If268f1110290e502c73dd289c9e7e7b27bc8432f

6 years agoCHAOS: avoid crash if string create fails
Scott Griepentrog [Thu, 17 Sep 2015 21:34:41 +0000 (16:34 -0500)]
CHAOS: avoid crash if string create fails

Validate string buffer allocation before using them.


Change-Id: Ib9c338bdc1e53fb8b81366f0b39482b83ef56ce0

6 years agodr_adaptive_odbc.c, cel_odbc.c, cel_pgsql.c: REFACTOR Macro LENGTHEN_BUF
Rodrigo Ramírez Norambuena [Fri, 11 Sep 2015 06:52:15 +0000 (03:52 -0300)]
dr_adaptive_odbc.c, cel_odbc.c, cel_pgsql.c: REFACTOR Macro LENGTHEN_BUF

Remove repeated code on macro of assigned buffer to SQL vars

Change-Id: Icb19ad013124498e172ea1d0b29ccd0ed17deef0

6 years agochan_sip: Fix From header truncation for extremely long CALLERID(name).
Walter Doekes [Thu, 17 Sep 2015 09:52:09 +0000 (11:52 +0200)]
chan_sip: Fix From header truncation for extremely long CALLERID(name).

The CALLERID(num) and CALLERID(name) and other info are placed into the
`char from[256]` in initreqprep. If the name was too long, the addr-spec
and params wouldn't fit.

Code is moved around so the addr-spec with params is placed there first,
and then fitting in as much of the display-name as possible.

ASTERISK-25396 #close

Change-Id: I33632baf024f01b6a00f8c7f35c91e5f68c40260

6 years agoCHAOS: res_pjsip_diversion avoid crash if allocation fails
Richard Mudgett [Thu, 17 Sep 2015 21:59:18 +0000 (16:59 -0500)]
CHAOS: res_pjsip_diversion avoid crash if allocation fails

Validate ast_malloc buffer returned before using it in


Change-Id: I15d2ed7cb0546818264c0bf251aa40adeae83253

6 years agoapp_queue: AgentComplete event has wrong reason
Kevin Harwell [Thu, 17 Sep 2015 21:47:33 +0000 (16:47 -0500)]
app_queue: AgentComplete event has wrong reason

When a queued caller transfers an agent to another extension sometimes the
raised AgentComplete event has a reason of "caller" and sometimes "transfer".
Since a transfer has taken place this should always be transfer. This occurs
because sometimes the stasis hangup event arrives before the transfer event
thus writing a different reason out.

With this patch, when a hangup event is received during a transfer it will
check to see if the channel that is hanging up is part of a transfer. If so
it will return and let the subsequently received transfer event handler take
care of the cleanup.

ASTERISK-25399 #close

Change-Id: Ic63c49bd9a5ed463ea7a032fd2ea3d63bc81a50d

6 years agoPJSIP: avoid crash when getting rtp peer
Scott Griepentrog [Thu, 17 Sep 2015 18:09:56 +0000 (13:09 -0500)]
PJSIP: avoid crash when getting rtp peer

Although unlikely, if the tech private is returned as
a NULL, chan_pjsip_get_rtp_peer() would crash.


Change-Id: Ie231369bfa7da926fb2b9fdaac228261a3152e6a

6 years agoapp_queue: Crash when transferring
Kevin Harwell [Thu, 17 Sep 2015 16:31:15 +0000 (11:31 -0500)]
app_queue: Crash when transferring

During some transfer scenarios involving queues Asterisk would sometimes
crash when trying to obtain a channel snapshot (could happen on caller or
member channels). This occurred because the underlying channel had already
disappeared when trying to obtain the latest snapshot.

This patch adds a reference to both the member and caller channels that
extends to the lifetime of the queue'd call, thus making sure the channels
will always exist when retrieving the latest snapshots.

ASTERISK-25185 #close
Reported by: Etienne Lessard

Change-Id: Ic397fa68fb4ff35fbc378e745da9246a7b552128

6 years agores_pjsip_pubsub: Eliminate race during initial NOTIFY.
Mark Michelson [Wed, 16 Sep 2015 22:36:32 +0000 (17:36 -0500)]
res_pjsip_pubsub: Eliminate race during initial NOTIFY.

There is a slim chance of a race condition occurring where two threads
can both attempt to manipulate the same area.

Thread A can be handling an incoming initial SUBSCRIBE request. Thread A
lets the specific subscription handler know that the subscription has
been established.

At this point, Thread B may detect a state change on the subscribed
resource and queue up a notification task on Thread C, the subscription
serializer thread.

Now Thread A attempts to generate the initial NOTIFY request to send to
the subscriber at the same time that Thread C attempts to generate a
state change NOTIFY request to send to the subscriber.

The result is that Threads A and C can step on the same memory area,
resulting in a crash. The crash has been observed as happening when
attempting to allocate more space to hold the body for the NOTIFY.

The solution presented here is to queue the subscription establishment
and initial NOTIFY generation onto the subscription serializer thread
(Thread C in the above scenario). This way, there is no way that a state
change notification can occur before the initial NOTIFY is sent, and if
there is a quick succession of NOTIFYs, we can guarantee that the two
NOTIFY requests will be sent in succession.

Change-Id: I5a89a77b5f2717928c54d6efb9955e5f6f5cf815

6 years agotranslate: Fix transcoding while different in frame size.
Alexander Traud [Fri, 28 Aug 2015 20:42:23 +0000 (22:42 +0200)]
translate: Fix transcoding while different in frame size.

When Asterisk translates between codecs, each with a different frame size (for
example between iLBC 30 and Speex-WB), too large frames were created by
ast_trans_frameout. Now, ast_trans_frameout is called with the correct frame
length, creating several frames when necessary. Affects all transcoding modules
which used ast_trans_frameout: GSM, iLBC, LPC10, and Speex.

ASTERISK-25353 #close

Change-Id: I2e229569d73191d66a4e43fef35432db24000212

6 years agoscheduler: Use queue for allocating sched IDs.
Mark Michelson [Thu, 10 Sep 2015 22:19:26 +0000 (17:19 -0500)]
scheduler: Use queue for allocating sched IDs.

It has been observed that on long-running busy systems, a scheduler
context can eventually hit INT_MAX for its assigned IDs and end up
overflowing into a very low negative number. When this occurs, this can
result in odd behaviors, because a negative return is interpreted by
callers as being a failure. However, the item actually was successfully
scheduled. The result may be that a freed item remains in the scheduler,
resulting in a crash at some point in the future.

The scheduler can overflow because every time that an item is added to
the scheduler, a counter is bumped and that counter's current value is
assigned as the new item's ID.

This patch introduces a new method for assigning scheduler IDs. Instead
of assigning from a counter, a queue of available IDs is maintained.
When assigning a new ID, an ID is pulled from the queue. When a
scheduler item is released, its ID is pushed back onto the queue. This
way, IDs may be reused when they become available, and the growth of ID
numbers is directly related to concurrent activity within a scheduler
context rather than the uptime of the system.

Change-Id: I532708eef8f669d823457d7fefdad9a6078b99b2

6 years agomain/config_options: Check for existance of internal object before derefing
Matt Jordan [Fri, 4 Sep 2015 02:15:13 +0000 (21:15 -0500)]
main/config_options: Check for existance of internal object before derefing

Asterisk can load and register an object type while still having an invalid
sorcery mapping. This can cause an issue when a creation call is invoked.
For example, mis-configuring PJSIP's endpoint identifier by IP address mapping
in sorcery.conf will cause the sorcery mechanism to be invalidated; however, a
subsequent ARI invocation to create the object will cause a crash, as the
internal type may not be registered as sorcery expects.

Merely checking for a NULL pointer here solves the issue.

Change-Id: I54079fb94a1440992f4735a9a1bbf1abb1c601ac

6 years agoMerge "res/res_pjsip_nat: Ignore REGISTER requests when looking for a Record-Route"
Matt Jordan [Fri, 11 Sep 2015 21:13:53 +0000 (16:13 -0500)]
Merge "res/res_pjsip_nat: Ignore REGISTER requests when looking for a Record-Route"

6 years agoMerge "chan_sip.c: Validation on module reload"
Matt Jordan [Fri, 11 Sep 2015 17:40:43 +0000 (12:40 -0500)]
Merge "chan_sip.c: Validation on module reload"

6 years agochan_sip.c: Validation on module reload
Rodrigo Ramírez Norambuena [Sat, 22 Aug 2015 02:50:31 +0000 (22:50 -0400)]
chan_sip.c: Validation on module reload

Change validation on reload module because now used the cli function for
reload. The sip_reload() function never fail and ever return NULL for this
reason on reload() now use the call the sip_reload() and return

This problem is dectected on reload by PUT method on ARI, getting always
404 http code when the module is reloaded.

ASTERISK-25325 #close
Reporte by: Rodrigo Ramírez Norambuena

Change-Id: I41215877fb2cfc589e0d4d464000cf6825f4d7fb

6 years agores_pjsip_pubsub.c: Mark ast_sip_create_subscription() as not used.
Richard Mudgett [Fri, 21 Aug 2015 22:39:01 +0000 (17:39 -0500)]
res_pjsip_pubsub.c: Mark ast_sip_create_subscription() as not used.

Change-Id: I2b8db18eac36c01a5c7eb9467699124e203fd093

6 years agores_pjsip_pubsub.c: Add some notification comments.
Richard Mudgett [Wed, 9 Sep 2015 17:24:31 +0000 (12:24 -0500)]
res_pjsip_pubsub.c: Add some notification comments.

Change-Id: Ie62ff1f4b7adc1a12fa0303f53926af249b25e20

6 years agores_pjsip_pubsub.c: Set dlg_status code instead of sending SIP response.
Richard Mudgett [Fri, 21 Aug 2015 23:01:59 +0000 (18:01 -0500)]
res_pjsip_pubsub.c: Set dlg_status code instead of sending SIP response.

We should not try to send a SIP response message because we may be
restoring a persistent subscription where we are not responding to a SIP

Change-Id: Id89167ef90320c5563f37e632db0dda6cb9e7dec

6 years agores_pjsip_pubsub.c: Fix off-nominal memory leak.
Richard Mudgett [Fri, 21 Aug 2015 22:40:39 +0000 (17:40 -0500)]
res_pjsip_pubsub.c: Fix off-nominal memory leak.

Fix off-nominal visited vector leak in build_resource_tree().

Change-Id: If0399c7941c9c0b1038bcfb7b9a371760977831c

6 years agores_pjsip_pubsub.c: Fix one byte buffer overrun error.
Richard Mudgett [Fri, 21 Aug 2015 20:26:30 +0000 (15:26 -0500)]
res_pjsip_pubsub.c: Fix one byte buffer overrun error.

ast_sip_pubsub_register_body_generator() did not account for the null
terminator set by sprintf() in the allocated output buffer.

Change-Id: I388688a132e479bca6ad1c19275eae0070969ae2

6 years agores_pjsip_pubsub.c: Use ast_alloca() instead of alloca().
Richard Mudgett [Fri, 21 Aug 2015 20:25:11 +0000 (15:25 -0500)]
res_pjsip_pubsub.c: Use ast_alloca() instead of alloca().

Change-Id: Ia396096b4fedc2874649ca11137612c3f55e83e3

6 years agores_pjsip_pubsub.c: Add missing error return in load_module().
Richard Mudgett [Fri, 21 Aug 2015 16:04:08 +0000 (11:04 -0500)]
res_pjsip_pubsub.c: Add missing error return in load_module().

Change-Id: I15debd0f717f16ee2f78e7f56151c3b3b97b72fc

6 years agores_pjsip/location.c: Use the builtin ao2_callback() match function instead.
Richard Mudgett [Fri, 21 Aug 2015 16:03:02 +0000 (11:03 -0500)]
res_pjsip/location.c: Use the builtin ao2_callback() match function instead.

Change-Id: I364906d6d2bad3472929986704a0286b9a2cbe3f

6 years agores_pjsip: Copy default_from_user to avoid crash.
Mark Michelson [Thu, 10 Sep 2015 14:49:45 +0000 (09:49 -0500)]
res_pjsip: Copy default_from_user to avoid crash.

The default_from_user retrieval function was pulling the
default_from_user from the global configuration struct in an unsafe way.
If using a database as a backend configuration store, the global
configuration struct is short-lived, so grabbing a pointer from it
results in referencing freed memory.

The fix here is to copy the default_from_user value out of the global
configuration struct.

Thanks go to John Hardin for discovering this problem and proposing the
patch on which this fix is based.

ASTERISK-25390 #close
Reported by Mark Michelson

Change-Id: I6b96067a495c1259da768f4012d44e03e7c6148c

6 years agores/res_pjsip_nat: Ignore REGISTER requests when looking for a Record-Route
Matt Jordan [Thu, 10 Sep 2015 13:39:21 +0000 (08:39 -0500)]
res/res_pjsip_nat: Ignore REGISTER requests when looking for a Record-Route

We will only rewrite the Contact header if there is no Record-Route header in
the received request. If a malfunctioning proxy places a Record-Route header
into a REGISTER request, we will decide that we shouldn't update the IP/port
in the Contact header, and we will end up storing a contact with an AoR that
contains the NAT'd IP address.

While it is nice to have the proxy *not* send a Record-Route in a REGISTER
request, it's also a good idea to not process the header in a non-dialog
message. This patch updates the code to explicitly ignore the Record-Route
header in REGISTER requests.

ASTERISK-25387 #close

Change-Id: I4bd3bcccc4003d460cc354d986b0dea2e433ef3f

6 years agoMerge "ParkAndAnnounce: Add variable inheritance"
Joshua Colp [Thu, 10 Sep 2015 12:25:02 +0000 (07:25 -0500)]
Merge "ParkAndAnnounce: Add variable inheritance"

6 years agoMerge "chan_ooh323: Add ProgressIndicator IE with inband info available"
Matt Jordan [Thu, 10 Sep 2015 00:12:09 +0000 (19:12 -0500)]
Merge "chan_ooh323: Add ProgressIndicator IE with inband info available"

6 years agoMerge "pjsip: avoid possible crash req_caps allocation failure"
Joshua Colp [Wed, 9 Sep 2015 22:22:22 +0000 (17:22 -0500)]
Merge "pjsip: avoid possible crash req_caps allocation failure"

6 years agochan_ooh323: Add ProgressIndicator IE with inband info available
Alexander Anikin [Wed, 9 Sep 2015 21:46:44 +0000 (01:46 +0400)]
chan_ooh323: Add ProgressIndicator IE with inband info available

Add ProgressIndicator IE with inband info present to Progress and
Alerting Q.931 message

ASTERISK-25227 #close
Reported by: Alexandr Dranchuk

Change-Id: I326ad13cb1db9a72b3fd902bafed3c28a3684203