asterisk/asterisk.git
4 years agochan_sip: option 'notifyringing' change and doc fix
Ward van Wanrooij [Sat, 26 Dec 2015 15:24:09 +0000 (16:24 +0100)]
chan_sip: option 'notifyringing' change and doc fix

In the sample sip.conf this is written with regard to notifyringing:
;notifyringing = no ; Control whether subscriptions already INUSE get sent
RINGING when another call is sent (default: yes)

However, this setting changes whether or not any RINGING indications are sent
to subscriptions. There is no separate configurable setting that allows
to control whether INUSE subscriptions also get sent RINGING. This is however
a useful option, to see (using BLF) if somebody else is able to handle an
incoming call or if everybody is busy.

This patch corrects the documentation for notifyringing (so the documentation
matches the functionality) and make notifyringing a tri-state option, by adding
the value 'notinuse' (in addition to 'yes' and 'no'). When notifyringing =
notinuse, only subscriptions that are not INUSE are sent the RINGING signal.

The default setting for notifyringing remains set to yes, so the default
behaviour is not affected.

ASTERISK-25558

Change-Id: I88f7036ee084bb3f43b74f15612695c6708f74aa

4 years agoMerge "res_pjsip_rfc3326.c: Fix crash when channel goes away."
Matt Jordan [Wed, 18 Nov 2015 13:33:57 +0000 (07:33 -0600)]
Merge "res_pjsip_rfc3326.c: Fix crash when channel goes away."

4 years agoapp_queue: (try_calling): mutex 'qe->chan' freed more times than we've locked!
Alec Davis [Wed, 18 Nov 2015 06:20:22 +0000 (19:20 +1300)]
app_queue: (try_calling): mutex 'qe->chan' freed more times than we've locked!

commit aae45acbd (Mark Michelson 2015-04-15 10:38:02 -0500 6525)
refer ASTERISK-24958

above commit removed ast_channel_lock(qe->chan);
but failed to remove corresponding ast_channel_unlock(qe->chan);

ASTERISK-25561 #close
Reported Alec Davis

Change-Id: Ie05f4e2d08912606178bf1fded57cc022c7a2e1a

4 years agoMerge "format: Register format-attribute module with cached formats."
Matt Jordan [Tue, 17 Nov 2015 20:35:22 +0000 (14:35 -0600)]
Merge "format: Register format-attribute module with cached formats."

4 years agoMerge "res/res_pjsip: Fix off nominal crash with requests that fail and have a timer"
Matt Jordan [Tue, 17 Nov 2015 18:59:35 +0000 (12:59 -0600)]
Merge "res/res_pjsip: Fix off nominal crash with requests that fail and have a timer"

4 years agoMerge "Confbridge: Add a user timeout option"
Joshua Colp [Tue, 17 Nov 2015 14:12:26 +0000 (08:12 -0600)]
Merge "Confbridge: Add a user timeout option"

4 years agodns: Fix pointer increment in dns_parse_answer_ex
George Joseph [Mon, 16 Nov 2015 22:10:20 +0000 (15:10 -0700)]
dns: Fix pointer increment in dns_parse_answer_ex

When dns_parse_answer_ex was iterating over the answers it
wasn't incrementing the answer pointer correctly after the first
answer.  The result was that no answers after the first
were being returned.  For results where multiple records should
have been sorted by priority, weight, etc., there was nothing
to sort so the only the first record was returned even if it
wouldn't have been the correct record based on the sort.

ASTERISK-25565 #close
Reported-by: Daniel Tryba
Tested-by George Joseph

Change-Id: I8622604fefdcd3c11e2c5609a6382e53b1467b0b

4 years agoConfbridge: Add a user timeout option
Mark Michelson [Fri, 13 Nov 2015 20:03:35 +0000 (14:03 -0600)]
Confbridge: Add a user timeout option

This option adds the ability to specify a timeout, in seconds, for a
participant in a ConfBridge. When the user's timeout has been reached,
the user is ejected from the conference with the CONFBRIDGE_RESULT
channel variable set to "TIMEOUT".

The rationale for this change is that there have been times where we
have seen channels get "stuck" in ConfBridge because a network issue
results in a SIP BYE not being received by Asterisk. While these
channels can be hung up manually via CLI/AMI/ARI, adding some sort of
automatic cleanup of the channels is a nice feature to have.

ASTERISK-25549 #close
Reported by Mark Michelson

Change-Id: I2996b6c5e16a3dda27595f8352abad0bda9c2d98

4 years agores/res_pjsip: Fix off nominal crash with requests that fail and have a timer
Matt Jordan [Mon, 16 Nov 2015 19:56:49 +0000 (13:56 -0600)]
res/res_pjsip: Fix off nominal crash with requests that fail and have a timer

When a request is sent using pjsip_endpt_send_request and fails, a condition
exists where the request wrapper, which is an AO2 object, may be de-ref'd
more times than it should. This occurs when the request's callback is called,
and, in the callback, the timer on the PJSIP heap is cancelled. When that
occurs, the request wrapper's lifetime is decremented. When
pjsip_endpt_send_request fails, we unilaterally decrement the lifetime of
the request wrapper again, even though we've already cancelled the reference
associated with the timer.

This patch checks the return result of pj_timer_heap_cancel_if_active before
removing the reference associated with the timer. We now only decrement it
in this case if a timer is cancelled as a result of the function call.

Change-Id: I21332343a1a019c1117076f9bf2df27be2850102

4 years agohashtab: Add NULL check when destroying iterator.
Joshua Colp [Sat, 14 Nov 2015 13:02:10 +0000 (09:02 -0400)]
hashtab: Add NULL check when destroying iterator.

The hashtab API is pretty NULL tolerant which has resulted
in remaining callers not doing much checks themselves.
Unfortunately the function to destroy an iterator does not
do a NULL check and will result in a crash if passed NULL.
This change fixes that.

ASTERISK-25552 #close

Change-Id: Ic1bf8eec3639e5a440f1c941d3ae3893ac6ed619

4 years agores_pjsip_rfc3326.c: Fix crash when channel goes away.
Richard Mudgett [Fri, 13 Nov 2015 20:32:10 +0000 (14:32 -0600)]
res_pjsip_rfc3326.c: Fix crash when channel goes away.

If an authenticated incoming caller does not respond to our 200 OK INVITE
response with an ACK then PJSIP will hangup the call.  Unfortunately,
there is a chance that the session's channel will go away between one use
of the channel pointer and another when building the BYE request because
the BYE is being built by the monitor thread and not the call's serializer
thread.

* Added a check to ensure that the thread trying to add the Reason header
is the call's serializer thread.  This ensures that the channel will not
go away on us.

Change-Id: I866388d2b97ea2032eaae3f3ab3f1ca6cbd2df89

4 years agoTaskprocessors: Increase high-water mark
Mark Michelson [Fri, 13 Nov 2015 20:19:35 +0000 (14:19 -0600)]
Taskprocessors: Increase high-water mark

In practical tests, we have seen certain taskprocessors, specifically
Stasis subscription taskprocessors, cross the recently-added high-water
mark and emit a warning. This high-water mark warning is only intended
to be emitted when things have tanked on the system and things are
heading south quickly. In the practical tests, the Stasis taskprocessors
sometimes had a max depth of 180 tasks in them, and Asterisk wasn't in
any danger at all.

As such, this ups the high-water mark to 500 tasks instead. It also
redefines the SIP threadpool request denial number to be a multiple of
the taskprocessor high-water mark.

Change-Id: Ic8d3e9497452fecd768ac427bb6f58aa616eebce

4 years agoformat: Register format-attribute module with cached formats.
Alexander Traud [Wed, 11 Nov 2015 13:00:42 +0000 (14:00 +0100)]
format: Register format-attribute module with cached formats.

In Asterisk 13, cached formats are created before their corresponding format-
attribute module is registered. Cached formats are involved when a local
extension is called. Therefore, ast_format_generate_sdp_fmtp did not work
on local extensions. This change affects the Opus Codec, H.263 (Plus), H.264,
and format-attribute modules provided externally.

ASTERISK-25160 #close

Change-Id: I1ea1f0483e5261e2a050112e4ebdfc22057d1354

4 years agores_pjsip distributor: Don't send 503 response to responses.
Mark Michelson [Thu, 12 Nov 2015 17:17:51 +0000 (11:17 -0600)]
res_pjsip distributor: Don't send 503 response to responses.

When the SIP threadpool is backed up with tasks, we send 503 responses
to ensure that we don't try to overload ourselves. The problem is that
we were not insuring that we were not trying to send a 503 to an
incoming SIP response.

This change makes it so that we only send the 503 on incoming requests.

Change-Id: Ie2b418d89c0e453cc6c2b5c7d543651c981e1404

4 years agoMerge "res_pjsip: Deny requests when threadpool queue is backed up."
Joshua Colp [Thu, 12 Nov 2015 16:56:09 +0000 (10:56 -0600)]
Merge "res_pjsip: Deny requests when threadpool queue is backed up."

4 years agoMerge "format_cap: Don't append the 'none' format when appending all."
Matt Jordan [Thu, 12 Nov 2015 16:54:01 +0000 (10:54 -0600)]
Merge "format_cap: Don't append the 'none' format when appending all."

4 years agores_pjsip: Deny requests when threadpool queue is backed up.
Mark Michelson [Wed, 11 Nov 2015 23:11:53 +0000 (17:11 -0600)]
res_pjsip: Deny requests when threadpool queue is backed up.

We have observed situations where the SIP threadpool may become
deadlocked. However, because incoming traffic is still arriving, the SIP
threadpool's queue can continue to grow, eventually running the system
out of memory.

This change makes it so that incoming traffic gets rejected with a 503
response if the queue is backed up too much.

Change-Id: I4e736d48a2ba79fd1f8056c0dcd330e38e6a3816

4 years agoMerge "Further fixes to improper usage of scheduler"
Joshua Colp [Thu, 12 Nov 2015 13:56:36 +0000 (07:56 -0600)]
Merge "Further fixes to improper usage of scheduler"

4 years agoformat_cap: Don't append the 'none' format when appending all.
Joshua Colp [Thu, 12 Nov 2015 12:24:06 +0000 (08:24 -0400)]
format_cap: Don't append the 'none' format when appending all.

When appending all formats of a type all the codecs are iterated
and added. This operation was incorrectly adding the ast_format_none
format which is special in that it is supposed to be used when no
format is present. It shouldn't be appended.

ASTERISK-25535

Change-Id: I7b00f3bdf4a5f3022e483d6ece602b1e8b12827c

4 years agoFurther fixes to improper usage of scheduler
Steve Davies [Wed, 11 Nov 2015 10:16:22 +0000 (10:16 +0000)]
Further fixes to improper usage of scheduler

When ASTERISK-25449 was closed, a number of scheduler issues mentioned in
the comments were missed. These have since beed raised in ASTERISK-25476
and elsewhere.

This patch attempts to collect all of the scheduler issues discovered so
far and address them sensibly.

ASTERISK-25476 #close

Change-Id: I87a77d581e2e0d91d33b4b2fbff80f64a566d05b

4 years agothreadpool: Handle worker thread transitioning to dead when going active.
Joshua Colp [Wed, 11 Nov 2015 17:04:08 +0000 (13:04 -0400)]
threadpool: Handle worker thread transitioning to dead when going active.

This change adds handling of dead worker threads when moving them
to be active. When this happens the worker thread is removed from
both the active and idle threads container. If no threads are able
to be moved to active then the pool grows as configured.

A unit test has also been added which thrashes the idle timeout
and thread activation to exploit any race conditions between the
two.

ASTERISK-25546 #close

Change-Id: I6c455f9a40de60d9e86458d447b548fb52ba1143

4 years agoMerge "rtp_engine: Init a format-attribute module to its RFC defaults."
Matt Jordan [Wed, 11 Nov 2015 14:19:39 +0000 (08:19 -0600)]
Merge "rtp_engine: Init a format-attribute module to its RFC defaults."

4 years agoMerge "Increase account code maximum length to 80."
Matt Jordan [Wed, 11 Nov 2015 14:09:18 +0000 (08:09 -0600)]
Merge "Increase account code maximum length to 80."

4 years agoMerge "dns: Use ntohl for ans->ttl in dns_parse_answer_ex"
Matt Jordan [Wed, 11 Nov 2015 14:09:06 +0000 (08:09 -0600)]
Merge "dns: Use ntohl for ans->ttl in dns_parse_answer_ex"

4 years agoMerge "res_pjsip_sdp_rtp: Enable Opus to be negotiated via SIP/SDP."
Matt Jordan [Wed, 11 Nov 2015 14:08:55 +0000 (08:08 -0600)]
Merge "res_pjsip_sdp_rtp: Enable Opus to be negotiated via SIP/SDP."

4 years agoMerge "ast_format_cap: Avoid format creation on module load, use cache instead."
Matt Jordan [Wed, 11 Nov 2015 14:08:02 +0000 (08:08 -0600)]
Merge "ast_format_cap: Avoid format creation on module load, use cache instead."

4 years agoMerge "xmldoc: Improve xmldoc wrapping of 'core show ...' output."
Matt Jordan [Wed, 11 Nov 2015 14:06:54 +0000 (08:06 -0600)]
Merge "xmldoc: Improve xmldoc wrapping of 'core show ...' output."

4 years agortp_engine: Init a format-attribute module to its RFC defaults.
Alexander Traud [Tue, 10 Nov 2015 15:24:31 +0000 (16:24 +0100)]
rtp_engine: Init a format-attribute module to its RFC defaults.

Previously, format-attribute modules relied on an existing fmtp line in SDP
negotiation. However, fmtp is optional for several formats like the Opus Codec.
Now, the format-attribute module is called with an empty fmtp, which allows the
module to initialise itself to RFC defaults. Furthermore now, Asterisk is able
to differentiate between internally and externally created formats.

ASTERISK-25537 #close

Change-Id: I28f680cef7fdf51c0969ff8da71548edad72ec52

4 years agoMerge "taskprocessor: Add high water mark warnings"
Joshua Colp [Wed, 11 Nov 2015 13:10:41 +0000 (07:10 -0600)]
Merge "taskprocessor: Add high water mark warnings"

4 years agoMerge "Remove ABI compatibility stub functions."
Joshua Colp [Tue, 10 Nov 2015 22:11:07 +0000 (16:11 -0600)]
Merge "Remove ABI compatibility stub functions."

4 years agoMerge "ast_format_cap_get_names: To display all formats, the buffer was increased."
Joshua Colp [Tue, 10 Nov 2015 20:58:17 +0000 (14:58 -0600)]
Merge "ast_format_cap_get_names: To display all formats, the buffer was increased."

4 years agoRemove ABI compatibility stub functions.
Corey Farrell [Tue, 10 Nov 2015 00:19:04 +0000 (19:19 -0500)]
Remove ABI compatibility stub functions.

ABI compatibility stubs existed for ast_app_separate_args and ast_verbose,
this is not needed in master.

Change-Id: I07b4d2c16079da3c2c6efa55df4a74368e0bd453

4 years agoRemove execute permission from dns_system_resolver.c
Corey Farrell [Tue, 10 Nov 2015 13:51:39 +0000 (08:51 -0500)]
Remove execute permission from dns_system_resolver.c

Change-Id: I3185735db42064bab00d3e073aed703385a00bf4

4 years agoMerge "func_callerid: Document that CALLERID(pres) is available."
Joshua Colp [Tue, 10 Nov 2015 16:04:50 +0000 (10:04 -0600)]
Merge "func_callerid: Document that CALLERID(pres) is available."

4 years agoast_format_cap_get_names: To display all formats, the buffer was increased.
Alexander Traud [Mon, 9 Nov 2015 09:01:41 +0000 (10:01 +0100)]
ast_format_cap_get_names: To display all formats, the buffer was increased.

ASTERISK-25533 #close

Change-Id: Ie1a9d1a6511b3f1a56b93d04475fbf8a4e40010a

4 years agoast_format_cap: Avoid format creation on module load, use cache instead.
Alexander Traud [Mon, 9 Nov 2015 13:04:43 +0000 (14:04 +0100)]
ast_format_cap: Avoid format creation on module load, use cache instead.

Since Asterisk 13, formats are immutable and cached. However while loading a
module like chan_sip, some formats were created instead using cached ones.

ASTERISK-25535 #close

Change-Id: I479cdc220d5617c840a98f3389b3bd91e91fbd9b

4 years agofunc_callerid: Document that CALLERID(pres) is available.
Walter Doekes [Fri, 6 Nov 2015 13:54:59 +0000 (14:54 +0100)]
func_callerid: Document that CALLERID(pres) is available.

CALLERPRES() says that it's deprecated in favor of CALLERID(num-pres)
and CALLERID(name-pres).  But for channel driver that don't make a
distinction between the two (e.g. SIP), it makes more sense to get/set
both at once.  This change reveals the availability of CALLERID(pres),
CONNECTEDLINE(pres), REDIRECTING(orig-pres), REDIRECTING(to-pres) and
REDIRECTING(from-pres).

ASTERISK-25373 #close

Change-Id: I5614ae4ab7d3bbe9c791c1adf147e10de8698d7a

4 years agodocs: Fix a few typo's in app docs (more then, resourse).
Walter Doekes [Fri, 6 Nov 2015 13:52:00 +0000 (14:52 +0100)]
docs: Fix a few typo's in app docs (more then, resourse).

Change-Id: Iba57efadf6c0b822e762c7a001bc89611d98afd7

4 years agodns: Use ntohl for ans->ttl in dns_parse_answer_ex
George Joseph [Fri, 6 Nov 2015 20:19:11 +0000 (13:19 -0700)]
dns: Use ntohl for ans->ttl in dns_parse_answer_ex

dns_parse_answer_ex was not converting ans->ttl from network
by order to host byte order which was causing certain ttls
it to go negative. In turn this was causing answer edit checks
to fail.

ASTERISK-25528 #close
Reported-by: Daniel Tryba
Tested-by: George Joseph

Change-Id: I31505132d6321c46d2f39fd06c20ee808a864037

4 years agoxmldoc: Improve xmldoc wrapping of 'core show ...' output.
Walter Doekes [Fri, 6 Nov 2015 13:36:40 +0000 (14:36 +0100)]
xmldoc: Improve xmldoc wrapping of 'core show ...' output.

Previously, the wrapping did both lookahead and lookback, which,
together with color escape sequences, caused some lines to be wrapped
way earlier than other lines.  This led to inconsistent output.

This simplifies the wrapping code and makes it more sane: if maxcolumns
is hit, we simply jump back to the last space and wrap there.

ASTERISK-25527 #close

Change-Id: I56d01c6f9a812642b1b05535c98d4db48d17c957

4 years agores_pjsip_sdp_rtp: Enable Opus to be negotiated via SIP/SDP.
Alexander Traud [Fri, 6 Nov 2015 12:57:15 +0000 (13:57 +0100)]
res_pjsip_sdp_rtp: Enable Opus to be negotiated via SIP/SDP.

In SIP/SDP, Opus has two channels always (see RFC 7587 section 7). The actual
amount of channels is negotiated in-band. Therefore now, the Opus codec and its
attribute rtpmap are registered with two channels.

ASTERISK-24779 #close
Reported by: PowerPBX
Tested by: Alexander Traud
patches:
  asterisk-24779.patch submitted by Sean Bright (license #5060)

Change-Id: Ic7ac13cafa1d3450b4fa4987350924b42cbb657b

4 years agotaskprocessor: Add high water mark warnings
Jonathan Rose [Tue, 3 Nov 2015 22:19:43 +0000 (16:19 -0600)]
taskprocessor: Add high water mark warnings

If a taskprocessor's queue grows large, this can indicate that there
may be a problem with tasks not leaving the processor or else that
the number of available task processors for a given type of task is
too low. This patch makes it so that if a taskprocessor's task queue
grows above 100 queued tasks that it will emit a warning message.
Warning messages are emitted only once per task processor.

ASTERISK-25518 #close
Reported by: Jonathan Rose

Change-Id: Ib1607c35d18c1d6a0575b3f0e3ff5d932fd6600c

4 years agoIncrease account code maximum length to 80.
Corey Farrell [Tue, 3 Nov 2015 02:11:11 +0000 (21:11 -0500)]
Increase account code maximum length to 80.

This increases the maximum length of account code's to match
extensions.  This ensures it is always possible to set an
accountcode to ${EXTEN} without truncation.

ASTERISK-23904
Reported by: Ben Merrills

Change-Id: If122602304ce03362722eb213a3111b32da5eeb9

4 years agoMerge "StatsD: Add res_statsd compatibility"
Joshua Colp [Wed, 4 Nov 2015 23:47:09 +0000 (17:47 -0600)]
Merge "StatsD: Add res_statsd compatibility"

4 years agoStatsD: Add res_statsd compatibility
tcambron [Tue, 3 Nov 2015 20:36:43 +0000 (14:36 -0600)]
StatsD: Add res_statsd compatibility

Added a new api to res_statsd.c to allow it to receive a
character pointer for the value argument. This allows for a
'+' and a '-' to easily be sent with the value.

ASTERISK-25419
Reported By: Ashley Sanders

Change-Id: Id6bb53600943d27347d2bcae26c0bd5643567611

4 years agomain/dial: Protect access to the format_cap structure of the requesting channel
Matt Jordan [Wed, 4 Nov 2015 20:31:28 +0000 (14:31 -0600)]
main/dial: Protect access to the format_cap structure of the requesting channel

When a dial attempt is made that involves a requesting channel, we previously
were not:
a) Protecting access to the native format capabilities structure on the
   requesting channel. That is inherently unsafe.
b) Reference bumping the lifetime of the format capabilities structure.

In both cases, something else could sneak in, blow away the format
capabilities, and we'd be holding onto an invalid format_cap structure. When
the newly created channel attempts to construct its format capabilities, things
go poorly.

This patch:
a) Ensures that we get a reference to the native format capabilities while
   the requesting channel is locked
b) Holds a reference to the native format capabilities during the creation
   of the new channel.

ASTERISK-25522 #close

Change-Id: I0bfb7ba8b9711f4158cbeaae96edf9626e88a54f

4 years agoFix cli display of build options.
Corey Farrell [Sat, 31 Oct 2015 03:57:58 +0000 (23:57 -0400)]
Fix cli display of build options.

A previous commit reduced the AST_BUILDOPTS compiler define to
only include options that affected ABI.  This included some options
that were previously displayed by cli "core show settings".  This
change corrects the CLI display while still restricting buildopts.h
to ABI effecting options only.

ASTERISK-25434 #close
Reported by: Rusty Newton

Change-Id: Id07af6bedd1d7d325878023e403fbd9d3607e325

4 years agoMerge "res_pjsip/location: Destroy contact_status objects on contact deletion"
Matt Jordan [Wed, 4 Nov 2015 13:44:26 +0000 (07:44 -0600)]
Merge "res_pjsip/location: Destroy contact_status objects on contact deletion"

4 years agoMerge "pjsip_configuration: On delete, remove the persistent version of an endpoint"
Matt Jordan [Wed, 4 Nov 2015 13:44:06 +0000 (07:44 -0600)]
Merge "pjsip_configuration: On delete, remove the persistent version of an endpoint"

4 years agopjsip_configuration: On delete, remove the persistent version of an endpoint
Matt Jordan [Tue, 3 Nov 2015 16:58:47 +0000 (10:58 -0600)]
pjsip_configuration: On delete, remove the persistent version of an endpoint

When an endpoint is deleted (such as through an API), the persistent endpoint
currently continues to lurk around. While this isn't harmful from a memory
consumption perspective - as all persistent endpoints are reclaimed on
shutdown - it does cause Stasis endpoint related operations to continue
to believe that the endpoint may or may not exist.

This patch causes the persistent endpoint related to a PJSIP endpoint to be
destroyed if the PJSIP endpoint is deleted.

Change-Id: I85ac707b4d5e6aad882ac275b0c2e2154affa5bb

4 years agores_pjsip/location: Destroy contact_status objects on contact deletion
Matt Jordan [Tue, 3 Nov 2015 17:15:09 +0000 (11:15 -0600)]
res_pjsip/location: Destroy contact_status objects on contact deletion

The contact_status Sorcery objects are currently not destroyed when a contact
is deleted. This causes the contact's last known RTT/status to be 'sticky'
when the contact itself may no longer exist. This patch causes the
contact_status objects associated with both dynamic and static contacts to
be destroyed if the AoR holding those contacts is also destroyed (or via
other paths where a contact may be deleted.)

Change-Id: I7feec8b9278cac3c5263a4c0483f4a0f3b62426e

4 years agomain/stasis_endpoints: Fix ContactStatusChange JSON for roundtrip_usec field
Matt Jordan [Tue, 3 Nov 2015 14:15:16 +0000 (08:15 -0600)]
main/stasis_endpoints: Fix ContactStatusChange JSON for roundtrip_usec field

The JSON packing for the ContactStatusChange event forgot to include the
roundtrip_usec field. As a result, the field never showed up in any event,
even when the data was available. This patch corrects that error by properly
packing the JSON blob with the data.

Change-Id: I8df80da659a44010afbd48f645967518ff5daa17

4 years agochan_sip: Allow websockets to be disabled.
Corey Farrell [Tue, 3 Nov 2015 02:24:58 +0000 (21:24 -0500)]
chan_sip: Allow websockets to be disabled.

This patch adds a new setting "websockets_enabled" to sip.conf.
Setting this to false allows chan_sip to be used without causing
conflicts with res_pjsip_transport_websocket.

ASTERISK-24106 #close
Reported by: Andrew Nagy

Change-Id: I04fe8c4f2d57b2d7375e0e25826c91a72e93bea7

4 years agores_pjsip: Set threadpool max size default to 50.
Mark Michelson [Mon, 2 Nov 2015 23:19:21 +0000 (17:19 -0600)]
res_pjsip: Set threadpool max size default to 50.

During a stress test of subscriptions, a huge blast of
subscription-related traffic resulted in the threadpool expanding to a
ridiculous number of threads. The balooning of threads resulted in an
increase of memory, which led to a crash due to being out of memory.

An easy fix for the particular test was to limit the size of the
threadpool, thus reining in the amount of memory that would be used. It
was decided that there really is no downside to having a non-infinite
default value for the maximum size of the threadpool, so this change
introduces 50 threads as the maximum threadpool size for the SIP
threadpool.

ASTERISK-25513 #close
Reported by John Bigelow

Change-Id: If0b9514f1d9b172540ce1a6e2f2ffa1f2b6119be

4 years agoMerge "pjsip_options: Schedule/unschedule qualifies on AoR creation/destruction"
Joshua Colp [Mon, 2 Nov 2015 22:02:10 +0000 (16:02 -0600)]
Merge "pjsip_options: Schedule/unschedule qualifies on AoR creation/destruction"

4 years agoMerge "StatsD: Send stuff to the StatsD server and test"
Joshua Colp [Mon, 2 Nov 2015 21:11:09 +0000 (15:11 -0600)]
Merge "StatsD: Send stuff to the StatsD server and test"

4 years agoStatsD: Send stuff to the StatsD server and test
tcambron [Thu, 29 Oct 2015 20:25:32 +0000 (15:25 -0500)]
StatsD: Send stuff to the StatsD server and test

Added code to allow the StatsD dialplan application to
send data to the server specified in statsd.conf.

ASTERISK-25419

Change-Id: I400db2f37c6ddf61515ff5a019646e36dcd0f922

4 years agopjsip_options: Schedule/unschedule qualifies on AoR creation/destruction
Matt Jordan [Mon, 2 Nov 2015 12:57:22 +0000 (06:57 -0600)]
pjsip_options: Schedule/unschedule qualifies on AoR creation/destruction

When an AoR is created or destroyed dynamically, the scheduled OPTIONS
requests that qualify the contacts on the AoR are not necessarily started
or destroyed, particularly for persistent contacts created for that AoR.
This patch adds create/update/delete sorcery observers for an AoR, which
schedule/unschedule the qualifies as expected.

Change-Id: Ic287ed2e2952a7808ee068776fe966f9554bdf7d

4 years agoMerge "app_queue: Added reason pause of member"
Matt Jordan [Mon, 2 Nov 2015 13:59:18 +0000 (07:59 -0600)]
Merge "app_queue: Added reason pause of member"

4 years agoMakefile: Add a rule 'basic-pbx' that installs the Basic PBX configs
Matt Jordan [Fri, 30 Oct 2015 18:22:23 +0000 (13:22 -0500)]
Makefile: Add a rule 'basic-pbx' that installs the Basic PBX configs

This patch adds a rule for installing the Super Awesome Company based 'Basic
PBX' configuration files. As part of adding this rule, a bit of the content
that makes up installing the configuration files under the 'samples' target
was refactored into a make subroutine for usage by additional later config
make targets.

Change-Id: I6c2e27906f73e2919a2b691da0be20ae70302404

4 years agores_pjsip_pubsub: Fix assertion when UAS dialog creation fails.
Joshua Colp [Thu, 29 Oct 2015 13:28:33 +0000 (10:28 -0300)]
res_pjsip_pubsub: Fix assertion when UAS dialog creation fails.

When compiled with assertions enabled one will occur when destroying
the subscription tree when UAS dialog creation fails. This is because
the code assumes that a dialog will always exist on a subscription
tree when in reality during this specific scenario it won't.

This change makes it so a dialog is not removed from the subscription
tree if it is not present.

ASTERISK-25505 #close

Change-Id: Id5c182b055aacc5e66c80546c64804ce19218dee

4 years agoMerge "chan_sip: Do not send all codecs on INVITE."
Matt Jordan [Thu, 29 Oct 2015 13:26:39 +0000 (08:26 -0500)]
Merge "chan_sip: Do not send all codecs on INVITE."

4 years agoMerge "StatsD: Add user input validation to the application"
Joshua Colp [Wed, 28 Oct 2015 20:42:54 +0000 (15:42 -0500)]
Merge "StatsD: Add user input validation to the application"

4 years agoStatsD: Add user input validation to the application
tcambron [Thu, 8 Oct 2015 16:50:44 +0000 (11:50 -0500)]
StatsD: Add user input validation to the application

Added code to accept user input and validate it before
allowing it to be sent to the StatsD server.

ASTERISK-25419
Reported By: Ashley Sanders

Change-Id: I55c7ce44326a68ad6c5c1514b9575ac50f25bbc3

4 years agoMerge "res_pjsip: Add "like" processing to pjsip list and show commands"
Joshua Colp [Wed, 28 Oct 2015 11:31:01 +0000 (06:31 -0500)]
Merge "res_pjsip:  Add "like" processing to pjsip list and show commands"

4 years agoMerge "install_prereq: Update repositories before install on Debian systems"
Joshua Colp [Mon, 26 Oct 2015 18:51:28 +0000 (13:51 -0500)]
Merge "install_prereq: Update repositories before install on Debian systems"

4 years agochan_sip: Do not send all codecs on INVITE.
Alexander Traud [Mon, 26 Oct 2015 16:42:03 +0000 (17:42 +0100)]
chan_sip: Do not send all codecs on INVITE.

Since version 13, Asterisk sent all allowed codecs as callee, even when the
caller did not request/support them. In case of dynamic RTP payloads, this led
to the same ID for different codecs, which is not allowed by SIP/SDP. Now, the
intersection between the requested and the supported codecs is send again.

ASTERISK-24543 #close

Change-Id: Ie90cb8bf893b0895f8d505e77343de3ba152a287

4 years agoMerge "build: GCC 5.1.x catches some new const, array bounds and missing paren issues"
Joshua Colp [Mon, 26 Oct 2015 16:32:13 +0000 (11:32 -0500)]
Merge "build: GCC 5.1.x catches some new const, array bounds and missing paren issues"

4 years agoMerge "format: Update the maximum packetization time for iLBC 30."
Matt Jordan [Mon, 26 Oct 2015 15:50:10 +0000 (10:50 -0500)]
Merge "format: Update the maximum packetization time for iLBC 30."

4 years agoinstall_prereq: Update repositories before install on Debian systems
Rodrigo Ramírez Norambuena [Mon, 19 Oct 2015 12:11:55 +0000 (09:11 -0300)]
install_prereq: Update repositories before install on Debian systems

When to install packages the indexed local is more old of the
version of software on the repository they have been upgraded by security
update then get the package will give 404 not found.

The patch prevent by update local index to repository for aptitude before
install.

ASTERISK-25495 #close

Reporte by: Rodrigo Ramírez Norambuena

Change-Id: I645959e553aac542805ced394cac2dca964051fa

4 years agoMerge "res_pjsip_pubsub: Prevent sending NOTIFY on destroyed dialog."
Matt Jordan [Sun, 25 Oct 2015 15:15:00 +0000 (10:15 -0500)]
Merge "res_pjsip_pubsub: Prevent sending NOTIFY on destroyed dialog."

4 years agoMerge "res_pjsip_pubsub: Ensure dialog lock balance."
Matt Jordan [Sun, 25 Oct 2015 15:14:36 +0000 (10:14 -0500)]
Merge "res_pjsip_pubsub: Ensure dialog lock balance."

4 years agoMerge "res_pjsip_pubsub: Prevent crashes on final NOTIFY."
Matt Jordan [Sun, 25 Oct 2015 15:13:03 +0000 (10:13 -0500)]
Merge "res_pjsip_pubsub: Prevent crashes on final NOTIFY."

4 years agoMerge "res_pjsip_pubsub: Remove serializer when sending final NOTIFY."
Matt Jordan [Sun, 25 Oct 2015 15:12:43 +0000 (10:12 -0500)]
Merge "res_pjsip_pubsub: Remove serializer when sending final NOTIFY."

4 years agoMerge "res_pjsip_pubsub: Fix crash on destruction of empty subscription tree."
Matt Jordan [Sun, 25 Oct 2015 15:12:10 +0000 (10:12 -0500)]
Merge "res_pjsip_pubsub: Fix crash on destruction of empty subscription tree."

4 years agoMerge "res_pjsip_pubsub: Solidify lifetime and ownership of objects."
Matt Jordan [Sun, 25 Oct 2015 15:11:06 +0000 (10:11 -0500)]
Merge "res_pjsip_pubsub: Solidify lifetime and ownership of objects."

4 years agobuild: GCC 5.1.x catches some new const, array bounds and missing paren issues
George Joseph [Sat, 24 Oct 2015 18:08:41 +0000 (12:08 -0600)]
build: GCC 5.1.x catches some new const, array bounds and missing paren issues

Fixed 1 issue in each of the affected files.

ASTERISK-25494 #close
Reported-by: George Joseph
Tested-by: George Joseph

Change-Id: I818f149cd66a93b062df421e1c73c7942f5a4a77

4 years agores_pjsip: Add "like" processing to pjsip list and show commands
George Joseph [Tue, 20 Oct 2015 21:02:30 +0000 (15:02 -0600)]
res_pjsip:  Add "like" processing to pjsip list and show commands

Add the ability to filter output from pjsip list and show commands
using the "like" predicate like chan_sip.

For endpoints, aors, auths, registrations, identifyies and transports,
the modification was a simple change of an ast_sorcery_retrieve_by_fields
call to ast_sorcery_retrieve_by_regex.  For channels and contacts a
little more work had to be done because neither of those objects are
true sorcery objects.  That was just removing the non-matching object
from the final container.  Of course, a little extra plumbing in the
common pjsip_cli code was needed to parse the "like" and pass the regex
to the get_container callbacks.

Some of the get_container code in res_pjsip_endpoint_identifier was also
refactored for simplicity.

ASTERISK-25477 #close
Reported by: Bryant Zimmerman
Tested by: George Joseph

Change-Id: I646d9326b778aac26bb3e2bcd7fa1346d24434f1

4 years agoMerge "res_pjsip_outbound_registration: registration stops due to fatal 4xx response"
Joshua Colp [Fri, 23 Oct 2015 20:35:34 +0000 (15:35 -0500)]
Merge "res_pjsip_outbound_registration: registration stops due to fatal 4xx response"

4 years agores_pjsip_outbound_registration: registration stops due to fatal 4xx response
Kevin Harwell [Wed, 21 Oct 2015 17:22:19 +0000 (12:22 -0500)]
res_pjsip_outbound_registration: registration stops due to fatal 4xx response

During outbound registration it is possible to receive a fatal (any permanent/
non-temporary 4xx, 5xx, 6xx) response from the registrar that is simply due
to a problem with the registrar itself. Upon receiving the failure response
Asterisk terminates outbound registration for the given endpoint.

This patch adds an option, 'fatal_retry_interval', that when set continues
outbound registration at the given interval up to 'max_retries' upon receiving
a fatal response.

ASTERISK-25485 #close

Change-Id: Ibc2c7b47164ac89cc803433c0bbe7063bfa143a2

4 years agoMerge "chan_sip: Fix autoframing=yes."
Joshua Colp [Fri, 23 Oct 2015 11:51:48 +0000 (06:51 -0500)]
Merge "chan_sip: Fix autoframing=yes."

4 years agoMerge topic 'fix_oom_crash'
Joshua Colp [Fri, 23 Oct 2015 11:51:17 +0000 (06:51 -0500)]
Merge topic 'fix_oom_crash'

* changes:
  strings.c: Fix __ast_str_helper() to always return a terminated string.
  Add missing failure checks to ast_str_set_va() callers.

4 years agoMerge "res_pjsip: Move URI validation to use time."
Joshua Colp [Fri, 23 Oct 2015 11:48:42 +0000 (06:48 -0500)]
Merge "res_pjsip: Move URI validation to use time."

4 years agoformat_cap: Detect vector allocation failures.
Mark Michelson [Thu, 22 Oct 2015 22:07:55 +0000 (17:07 -0500)]
format_cap: Detect vector allocation failures.

A crash was seen on a system that ran out of memory due to Asterisk not
checking for vector allocation failures in format_cap.c. With this
change, if either of the AST_VECTOR_INIT calls fail, we will return a
value indicating failure.

Change-Id: Ieb9c59f39dfde6d11797a92b45e0cf8ac5722bc8

4 years agores_pjsip_pubsub: Prevent sending NOTIFY on destroyed dialog.
Mark Michelson [Fri, 2 Oct 2015 20:32:09 +0000 (15:32 -0500)]
res_pjsip_pubsub: Prevent sending NOTIFY on destroyed dialog.

A certain situation can result in our attempting to send a NOTIFY on a
destroyed dialog. Say we attempt to send a NOTIFY to a subscriber, but
that subscriber has dropped off the network. We end up retransmitting
that NOTIFY until the appropriate SIP timer says to destroy the NOTIFY
transaction. When the pjsip evsub code is told that the transaction has
been terminated, it responds in kind by alerting us that the
subscription has been terminated, destroying the subscription, and then
removing its reference to the dialog, thus destroying the dialog.

The problem is that when we get told that the subscription is being
terminated, we detect that we have not sent a terminating NOTIFY
request, so we queue up such a NOTIFY to be sent out. By the time that
queued NOTIFY gets sent, the dialog has been destroyed, so attempting to
send that NOTIFY can result in a crash.

The fix being introduced here is actually a reintroduction of something
the pubsub code used to employ. We hold a reference to the dialog and
wait to decrement our reference to the dialog until our subscription
tree object is destroyed. This way, we can send messages on the dialog
even if the PJSIP evsub code wants to terminate earlier than we would
like.

In doing this, some NULL checks for subscription tree dialogs have been
removed since NULL dialogs are no longer actually possible.

Change-Id: I013f43cddd9408bb2a31b77f5db87a7972bfe1e5

4 years agores_pjsip_pubsub: Ensure dialog lock balance.
Mark Michelson [Tue, 29 Sep 2015 19:53:22 +0000 (14:53 -0500)]
res_pjsip_pubsub: Ensure dialog lock balance.

When sending a NOTIFY, we lock the dialog and then unlock the dialog
when finished. A recent change made it so that the subscription tree's
dialog pointer will be set NULL when sending the final NOTIFY request
out. This means that when we attempt to unlock the dialog, we pass a
NULL pointer to pjsip_dlg_dec_lock(). The result is that the dialog
remains locked after we think we have unlocked it. When a response to
the NOTIFY arrives, the monitor thread attempts to lock the dialog, but
it cannot because we never released the dialog lock. This results in
Asterisk being unable to process incoming SIP traffic any longer.

The fix in this patch is to use a local pointer to save off the pointer
value of the subscription tree's dialog when locking and unlocking the
dialog. This way, if the subscription tree's dialog pointer is NULLed
out, the local pointer will still have point to the proper place and the
dialog lock will be unlocked as we expect.

Change-Id: I7ddb3eaed7276cceb9a65daca701c3d5e728e63a

4 years agores_pjsip_pubsub: Prevent crashes on final NOTIFY.
Mark Michelson [Mon, 28 Sep 2015 21:36:25 +0000 (16:36 -0500)]
res_pjsip_pubsub: Prevent crashes on final NOTIFY.

The SIP dialog is removed from the subscription tree when the final
NOTIFY is sent. However, after the final NOTIFY is sent, the persistence
update function still attempts to access the cseq from the dialog,
resulting in a crash.

This fix removes the subscription persistence at the same time that the
dialog is removed from the subscription tree. This way, there is no
attempt to update persistence when the subscription is being destroyed.

Change-Id: Ibb46977a6cef9c51dc95f40f43446e3d11eed5bb

4 years agores_pjsip_pubsub: Remove serializer when sending final NOTIFY.
Mark Michelson [Thu, 17 Sep 2015 22:28:30 +0000 (17:28 -0500)]
res_pjsip_pubsub: Remove serializer when sending final NOTIFY.

There have been crashes seen where a taskprocessor's listener is NULL
unexpectedly.

Looking at backtraces, the problem was specifically seen in PJSIP
serializers.

Subscriptions make the mistake of removing a serializer from a dialog
during subscription tree destruction. Since subscription trees are
reference-counted, guaranteeing the circumstances behind the destruction
are not possible. This makes it so that the dialog serializer can be
removed while not holding the dialog lock. This makes it possible for
the distributor to get a pointer to the dialog serializer and have that
serializer get freed out from under it.

The fix for this is to remove the serializer from a subscription dialog
when sending the final NOTIFY. This guarantees that the serializer is
removed with the dialog lock held. By doing this, we guarantee that if
the distributor gains access to the dialog's serializer, it will not be
possible for the serializer to get freed by another thread.

Change-Id: I21f5dac33529f65cec45679bdace60670800ff66

4 years agores_pjsip_pubsub: Fix crash on destruction of empty subscription tree.
Mark Michelson [Wed, 2 Sep 2015 14:14:19 +0000 (09:14 -0500)]
res_pjsip_pubsub: Fix crash on destruction of empty subscription tree.

If an old persistent subscription is recreated but then immediately
destroyed because it is out of date, the subscription tree will have no
leaf subscriptions on it. This was resulting in a crash when attempting
to destroy the subscription tree.

A simple NULL check fixes this problem.

Change-Id: I85570b9e2bcc7260a3fe0ad85904b2a9bf36d2ac

4 years agores_pjsip_pubsub: Solidify lifetime and ownership of objects.
Mark Michelson [Tue, 1 Sep 2015 20:47:19 +0000 (15:47 -0500)]
res_pjsip_pubsub: Solidify lifetime and ownership of objects.

There have been crashes and general instability seen in the pubsub code,
so this patch introduces three changes to increase the stability.

First, the ownership model for subscriptions has been modified. Due to
RLS, subscriptions are stored in memory as a tree structure. Prior to my
patch, the PJSIP subscription was the owner of the subscription tree.
When the PJSIP subscription told us that it was terminating, we started
destroying the subscription tree along with all of the individual leaf
subscriptions that belong to the tree. The problem with this model is
that the two actors in play here, the PJSIP subscription and the
individual leaf subscriptions, need to have joint ownership of the
subscription tree. So now, the PJSIP subscription and the individual
leaf subscriptions each have a reference to the subscription tree. This
way, we will not actually free memory until no players are left that
care. The PJSIP subscription is a bigger stakeholder, in that if the
PJSIP subscription's reference to the subscription tree is removed, the
subscription tree instructs the leaf subscriptions to shut down and drop
their references to the subscription tree when possible. The individual
leaf subscriptions, upon being told to shut down, can drop their stasis
subscriptions or whatever they use to learn of new state, and then drop
their reference to the subscription tree once they are ready to die.

Second, the lifetime of a PJSIP subscription's reference to our
subscription tree has been altered. As I learned from doing a deep dive,
the PJSIP evsub code can tell Asterisk multiple times that the
subscription has been terminated, and not all of these times
are especially helpful. I have altered the message flow that we use for
SIP subscriptions such that we will always drop the PJSIP subscription's
reference to the subscription tree when we send the NOTIFY that
terminates a SIP subscription. This also means that we will now queue
NOTIFY requests to be sent after responding to incoming SUBSCRIBEs so
that we can have predictable state changes from the PJSIP evsub code.

Third, the synchronization of operations has been improved. PJSIP can
call into our code from a serializer thread (e.g. upon receiving an
incoming request) or from the monitor thread (e.g. when a subscription
times out). Because of this, there is the possibility of competing
threads stepping on each other. PJSIP attempts to do some
synchronization on its own by always keeping the dialog lock held when
it calls into us. However, since we end up pushing tasks into the
serializer, the result was that serialized operations were not grabbing
the dialog lock and could, as a result, step on something that was being
attempted by a different thread. Now we ensure that serialized
operations grab the dialog lock, then check for extenuating
circumstances, then proceed with their operation if they can.

Change-Id: Iff2990c40178dad9cc5f6a5c7f76932ec644b2e5

4 years agostrings.c: Fix __ast_str_helper() to always return a terminated string.
Richard Mudgett [Mon, 19 Oct 2015 20:28:46 +0000 (15:28 -0500)]
strings.c: Fix __ast_str_helper() to always return a terminated string.

Users of functions which call __ast_str_helper() such as the ones listed
below are likely to not check the return value for failure so ensuring
that the string is always nil terminated is a good safety measure.

ast_str_set_va()
ast_str_append_va()
ast_str_set()
ast_str_append()

Change-Id: I36ab2d14bb6015868b49329dda8639d70fbcae07

4 years agoAdd missing failure checks to ast_str_set_va() callers.
Richard Mudgett [Mon, 19 Oct 2015 20:27:40 +0000 (15:27 -0500)]
Add missing failure checks to ast_str_set_va() callers.

Change-Id: I0c2cdcd53727bdc6634095c61294807255bd278f

4 years agoMerge "funcs/func_holdintercept: Actually add the HOLD_INTERCEPT function"
Joshua Colp [Wed, 21 Oct 2015 21:47:10 +0000 (16:47 -0500)]
Merge "funcs/func_holdintercept: Actually add the HOLD_INTERCEPT function"

4 years agoMerge "rest-api-templates: Wikify error code response reasons"
Joshua Colp [Wed, 21 Oct 2015 21:44:30 +0000 (16:44 -0500)]
Merge "rest-api-templates: Wikify error code response reasons"

4 years agoMerge "main/cdr: Allow modules to modify CDR fields before dispatching them"
Joshua Colp [Wed, 21 Oct 2015 18:11:44 +0000 (13:11 -0500)]
Merge "main/cdr: Allow modules to modify CDR fields before dispatching them"

4 years agores_pjsip: Move URI validation to use time.
Joshua Colp [Wed, 21 Oct 2015 16:44:17 +0000 (13:44 -0300)]
res_pjsip: Move URI validation to use time.

In a realtime based system with a limited number of threadpool threads
it is possible for a deadlock to occur. This happens when permanent
endpoint state is updated, which will cause database queries to be done.
These queries may result in URI validation being done which is done
synchronously using a PJSIP thread. If all PJSIP threads are in use
processing traffic they themselves may be blocked waiting to get the
permanent endpoint container lock when identifying an endpoint.

This change moves URI validation to occur at use time instead of
configuration time. While this comes at a cost of not seeing a problem
until you use it it does solve the underlying deadlock problem.

ASTERISK-25486 #close

Change-Id: I2d7d167af987d23b3e8199e4a68f3359eba4c76a

4 years agoformat: Update the maximum packetization time for iLBC 30.
Alexander Traud [Wed, 21 Oct 2015 13:08:51 +0000 (15:08 +0200)]
format: Update the maximum packetization time for iLBC 30.

In September 2006, the maximum packetization time (ptime) were set to such a
low value, packetization was disabled for many codecs actually. This was fixed
for many codecs but not for iLBC 30. This enables packetization for iLBC which
can be enabled for example via allow=ilbc:60,gsm,alaw,ulaw in the file sip.conf.

ASTERISK-7803

Change-Id: I2ef90023d35efb7cb8fe96ed74f53f6846ffad12

4 years agochan_sip: Fix autoframing=yes.
Alexander Traud [Wed, 21 Oct 2015 14:51:11 +0000 (16:51 +0200)]
chan_sip: Fix autoframing=yes.

With Asterisk 13, the structures ast_format and ast_codec changed. Because of
that, the paketization timing (framing) of the RTP channel moved away from the
formats/codecs. In the course of that change, the ptime of the callee was not
honored anymore, when the optional autoframing was enabled.

ASTERISK-25484 #close

Change-Id: Ic600ccaa125e705922f89c72212c698215d239b4

4 years agorest-api-templates: Wikify error code response reasons
Matt Jordan [Wed, 21 Oct 2015 03:24:36 +0000 (22:24 -0500)]
rest-api-templates: Wikify error code response reasons

Error response code descriptions may contain wiki markup that need to be
escaped. Without this patch, Confluence will reject the document being sent
and the responsible script will raise an exception.

Change-Id: I21fcb66fee7f6332381f2b99b1b0195dff215ee5

4 years agofuncs/func_holdintercept: Actually add the HOLD_INTERCEPT function
Matt Jordan [Tue, 20 Oct 2015 17:06:52 +0000 (12:06 -0500)]
funcs/func_holdintercept: Actually add the HOLD_INTERCEPT function

When ab803ec342 was committed, it accidentally forgot to actually *add* the
HOLD_INTERCEPT function. This highlights two interesting points:
* Gerrit forces you to put the patch as it is going to into the repo up for
  review, which Review Board did not. Yay Gerrit.
* No one apparently bothered to use this feature, or else they don't know about
  it. I'm going to go with the latter explanation.

ASTERISK-24922

Change-Id: Ida38278f259dd07c334a36f9b7d5475b5db72396