3965 Commits (c16000f201520b398cfaaaeeff74da171fa3a2ee)

Author	SHA1	Message	Date
Sean Bright	c16000f201	res_rtp_asterisk: Fix mapping of pjsip's ICE roles to ours ... Change-Id: Ia578ede1a55b21014581793992a429441903278b	9 years ago
Sergej Kasumovic	708cdc0b8e	res_stasis_device_state: Unsubscribe should remove old subscriptions ... Case scenario with Applications ARI: * Once you subscribe to deviceState with Applications REST API, it will be added into subscription pool. * When you unsubscribe it will remove from the device_state_subscription hash table but not from the subscription pool. * When you subscribe again, it will add it to pool again. * Now you will have two subscriptions and you will receive same event twice. This fix should now remove deviceState subscription from pool and it should fix unsubscribe on deviceState. ASTERISK-27130 #close Change-Id: I718b70d770a086e39b4ddba4f69a3c616d4476c4	9 years ago
Joshua Colp	f0558d58f0	Merge "res_rtp_asterisk: Use RTP component for ICE if RTCP-MUX is in use." into 13	9 years ago
Joshua Colp	51761b759d	res_rtp_asterisk: Use RTP component for ICE if RTCP-MUX is in use. ... This change makes it so that if an RTCP packet is being sent the RTP ICE component is used for sending if RTCP-MUX is in use. ASTERISK-27133 Change-Id: I6200f611ede709602ee9b89501720c29545ed68b	9 years ago
Torrey Searle	a4c85309f0	res/res_stasis_snoop: generate silence when audiohook returns null ... Currently when rtp is paused, no packets are written to the recorded audio file, causing the silence to be skipped and recording not properly time aligned. The read handler as been adapted to return a silence frame of the correct size. ASTERISK-27128 #close Change-Id: I2d7f60650457860b9c70907b14426756b058a844	9 years ago
Jenkins2	e29284b131	Merge "res/res_pjsip_t38 ensure t38 requests get rejected quickly" into 13	9 years ago
George Joseph	4e555437dc	res_musiconhold: Add kill_escalation_delay, kill_method to class ... By default, when res_musiconhold reloads or unloads, it sends a HUP signal to custom applications (and all descendants), waits 100ms, then sends a TERM signal, waits 100ms, then finally sends a KILL signal. An application which is interacting with an external device and/or spawns children of its own may not be able to exit cleanly in the default times, expecially if sent a KILL signal, or if it's children are getting signals directly from res_musiconhoild. * To allow extra time, the 'kill_escalation_delay' class option can be used to set the number of milliseconds res_musiconhold waits before escalating kill signals, with the default being the current 100ms. * To control to whom the signals are sent, the "kill_method" class option can be set to "process_group" (the default, existing behavior), which sends signals to the application and its descendants directly, or "process" which sends signals only to the application itself. Change-Id: Iff70a1a9405685a9021a68416830c0db5158603b	9 years ago
Jenkins2	1498d66f69	Merge "res_pjsip: Fix crash with from_user containing invalid characters." into 13	9 years ago
Benjamin Keith Ford	25e18bf514	res_pjsip: Fix crash with from_user containing invalid characters. ... If the from_user field contains certain characters (like @, {, ^, etc.), PJSIP will return a null value for the URI when attempting to parse it. This causes a crash when trying to dial out through a trunk that contains these invalid characters in its from_user field. This change checks the configuration and ensures that an endpoint will not be created if the from_user contains an invalid character. It also adds a null check to the PJSIP URI parsing as a backup. ASTERISK-27036 #close Reported by: Maxim Vasilev Change-Id: I0396fdb5080604e0bdf1277464d5c8a85db913d0	9 years ago
Richard Mudgett	aa514f420b	res_rtp_asterisk.c: Fix TURN deadlock by using ICE session group lock. ... When a message is received on the TURN socket, the code processing the message needs to call into the ICE/STUN session for further processing. This code path locks the TURN group lock then the ICE/STUN group lock. In another thread an ICE/STUN timer can fire off to send a keep alive message over the TURN socket. In this code path, the ICE/STUN group lock is obtained then the TURN group lock is obtained to send the packet. A classic deadlock case if the group locks are not the same. * Made TURN get created using the ICE/STUN session's group lock. NOTE: I was originally concerned that the ICE/STUN session can get recreated by ice_reset_session() for an event like RTCP multiplexing causing a change during SDP negotiation. In this case the TURN group lock would become different. However, TURN is also recreated as part of the ICE/STUN recreation in ice_create() when all known ICE candidates are added to the new ICE session. While the ICE/STUN and TURN sessions are being recreated there is a period where the group locks could be different. ASTERISK-27023 #close Patches: res_rtp_asterisk-turn-deadlock-fix.patch (license #6502) patch uploaded by Michael Walton (modified) Change-Id: Ic870edb99ce4988a8c8eb6e678ca7f19da1432b9	9 years ago
George Joseph	642c597507	Merge "pjsip_distributor.c: Fix deadlock with TCP type transports." into 13	9 years ago
Jenkins2	aadece5664	Merge "pjsip_distributor.c: Fix unidentified_requests hash functions." into 13	9 years ago
George Joseph	40490768cc	Merge "chan_pjsip: Fix ability to send UPDATE on COLP" into 13	9 years ago
Richard Mudgett	0d64cbde57	pjsip_distributor.c: Fix deadlock with TCP type transports. ... When a SIP message comes in on a transport, pjproject obtains the lock on the transport and pulls the data out of the socket. Unlike UDP, the TCP transport does not allow concurrent access. Without concurrency the transport lock is not released when the transport's message complete callback is called. The processing continues and eventually Asterisk starts processing the SIP message. The first thing Asterisk tries to do is determine the associated dialog of the message to determine the associated serializer. To get the associated serializer safely requires us to get the dialog lock. To send a request or response message for a dialog, pjproject obtains the dialog lock and then obtains the transport lock. Deadlock can result because of the opposite order the locks are obtained. * Fix the deadlock by obtaining the serializer associated with the dialog another way that doesn't involve obtaining the dialog lock. In this case, we use an ao2 container to hold the associated endpoint and serializer. The new locks are held a brief time and won't overlap other existing lock times. ASTERISK-27090 #close Change-Id: I9ed63f4da9649e9db6ed4be29c360968917a89bd	9 years ago
Richard Mudgett	905d18e8bf	pjsip_distributor.c: Fix unidentified_requests hash functions. ... The OBJ_SEARCH_xxx defines should not be used as if they were individual bits. They represent a multi-bit enumeration value field. Change-Id: I32abc9a475396dab02402a7014357dd94284e17b	9 years ago
George Joseph	6bd7c0f37c	chan_pjsip: Fix ability to send UPDATE on COLP ... When connected_line_method is "invite", we're supposed to determine if the client can support UPDATE and if it can, send UPDATE instead of INVITE to avoid the SDP renegotiation. Not only was pjproject not setting the PJSIP_INV_SUPPORT_UPDATE flag, we were testing that invite_tsx wasn't NULL which isn't always the case. * Updated chan_pjsip/update_connected_line_information to drop the requirement that invite_tsx isn't NULL. * Submitted patch to pjproject sip_inv.c that sets the PJSIP_INV_SUPPORT_UPDATE flag correctly. * Updated pjsip.conf.sample to clarify what happens when "invite" is specified. ASTERISK-27095 Change-Id: Ic2381b3567b8052c616d96fbe79564c530e81560	9 years ago
Joshua Colp	0426b1d88a	res_rtp_asterisk: Fix issues with ICE renegotiation. ... When re-inviting to add more streams it is possible for the role of existing ICE sessions to be changed to the incorrect value. This results in subsequent refreshes within the sessions getting a role conflict and the ICE session breaking down. This change only sets the role to be the new value if an ICE renegotiation is actually going to happen, otherwise the existing role is preserved. As well if we encounter a situation where a unidirectional ICE negotiation happens and the other side does not send us candidates we will not store any information for sending traffic, even though we know where they are reachable. This change fixes this by using the source of the ICE traffic itself as the target if no candidates are known and we receive some ICE traffic. ASTERISK-27088 Change-Id: I71228181e358917fcefc3100fad21b2fc02a59a9	9 years ago
Torrey Searle	1f59d08924	res/res_pjsip_t38: fix incorrect increment of media_count ... The T38 sdp callback incorrectly has a side effect of incrementing the media_count. This can lead to core dumps. Change-Id: I7bb2f4987de4046ec52cfc34e5ea0662dae32af8	9 years ago
Torrey Searle	9fbc34d2bd	res_pjsip: Add DTMF INFO Failback mode ... The existing auto dtmf mode reverts to inband if 4733 fails to be negotiated. This patch adds a new mode auto_info which will switch to INFO instead of inband if 4733 is not available. ASTERISK-27066 #close Change-Id: Id185b11e84afd9191a2f269e8443019047765e91	9 years ago
Torrey Searle	154d2914fa	res/res_pjsip_t38 ensure t38 requests get rejected quickly ... arm the t38 webhook always, so we can correctly reject a T38 negotiation request when t38 is disabled on a channel Change-Id: Ib1ffe35aee145d4e0fe61dd012580be11aae079d	9 years ago
Richard Mudgett	764d04fa87	res_pjsip_mwi.c: Eliminate RAII_VAR in contact delete observer ... Change-Id: I0bc97c6608de1d1a4228826b3b3be43f162f05f3	9 years ago
Alexei Gradinari	0f6a9617eb	res_pjsip_mwi: update unsolicited MWI subscriptions on updating contact ... Do not need to unsubscribe/subscribe on creating the ednpoint's contact. The modified function create_mwi_subscriptions_for_endpoint adds the subscription only if it does not exist. The subscriptions aren't added for active contacts which are retrieved on startup from realtime if mwi_disable_initial_unsolicited=yes. Because the mwi_contact_added is not called. So the subscriptions also should be created on updating contact. ASTERISK-26230 #close Change-Id: I47e265af9296ca09aa42a316fdacac104148cee4	9 years ago
Jenkins2	9554166182	Merge "res_corosync: Change thread stack size" into 13	9 years ago
Jenkins2	34b3665612	Merge "res_pjsip_mwi: unsubscribe unsolicited MWI on deleting endpoint last contact" into 13	9 years ago
Jenkins2	507ce0aa95	Merge "res_stasis: Plug reference leak on stolen channels" into 13	9 years ago
Jenkins2	707e0e62e6	Merge "res_pjsip: New endpoint option "notify_early_inuse_ringing"" into 13	9 years ago
Alexei Gradinari	8e749c8f51	res_pjsip_mwi: unsubscribe unsolicited MWI on deleting endpoint last contact ... If the endpoint's last contact is deleted unsolicited MWI has to be unsubscribed. ASTERISK-27051 #close Change-Id: I33e174e0b9dba0998927d16d6d100fda5c7254e0	9 years ago
George Joseph	edfdb4dff5	res_stasis: Plug reference leak on stolen channels ... When a stasis channel is stolen by another app, the control structure is unreffed but never unlinked from the app_controls container. This causes the channel reference to leak. Added OBJ_UNLINK to the callback in channel_stolen_cb. Also added some additional channel lifecycle debug messages to channel.c. ASTERISK-27059 #close Repoorted-by: George Joseph Change-Id: Ib820936cd49453f20156971785e7f4f182c56e14	9 years ago
Alexei Gradinari	a6e4899612	res_pjsip: New endpoint option "notify_early_inuse_ringing" ... This option was added to control whether to notify dialog-info state 'early' or 'confirmed' on Ringing when already INUSE. The value "yes" is useful for some SIP phones (Cisco SPA) to be able to indicate and pick up ringing devices. ASTERISK-26919 #close Change-Id: Ie050bc30023543c7dfb4365c5be3ce58c738c711	9 years ago
Jenkins2	0b7a0681a3	Merge "res_ari: Add "module loaded" check to ari stubs" into 13	9 years ago
Jan Friesse	005a4afa6b	res_corosync: Change thread stack size ... In Corosync 2.x libraries were changed to use LibQB IPC. Sadly LibQB IPC doesn't support copy-free access to received buffer, so Corosync libraries were rewritten to use stack as buffer. Mostly the needed stack size is quite small, but for all *_dispatch functions, 1MiB is needed. Asterisk function ast_pthread_create_background set stack size for new thread to much smaller AST_BACKGROUND_STACKSIZE (~500KiB). This results in Asterisk crash when running with Corosync 2.x. Patch solves this issue by creating it's own version of ast_pthread_create_background which sets stack size to much higher value (actually it's AST_BACKGROUND_STACKSIZE + 3MiB). Another problem may appear when "corosync show members" netconsole command is executed. It is also executed in thread and also has only 500KiB stack size. Sadly it calls corosync_cfg_get_node_addrs which again needs at least 1MiB stack. Solution is to use HAVE_COROSYNC_CFG_STATE_TRACK as a discriminator between Corosync 1.x and 2.x. If 1.x is found, nothing changes. If 2.x is found, NodeID is displayed instead of IP address. ASTERISK-25370 #close Reported by: mdu113 Change-Id: Id95b0d21ab6e708e7d74ad8786c587211676fa08	9 years ago
George Joseph	7901b9853e	res_ari: Add "module loaded" check to ari stubs ... The recent change to make the use of LOAD_DECLINE more consistent caused res_ari to unload itself before declining if the ari.conf file wasn't found. The ari stubs though still tried to use the configuration resulting in segfaults. This patch creates a new CHECK_ARI_MODULE_LOADED macro which tests to see if res_ari is actually loaded and causes the stubs to also decline if it isn't. The macro was then added to the mustache template's "load_module" function. ASTERISK-27026 #close Reported-by: Ronald Raikes Change-Id: I263d56efa628ee3c411bdcd16d49af6260c6c91d	9 years ago
Jenkins2	ae6c38db98	Merge "res_pjsip_pubsub: Fix reference to released endpoint" into 13	9 years ago
Jenkins2	320fb81580	Merge "res_pjsip_refer/session: Calls dropped during transfer" into 13	9 years ago
George Joseph	8a858275eb	Merge "res_rtp_asterisk: Fix ssrc change for rtcp srtp" into 13	9 years ago
Jenkins2	ffedf68136	Merge "res_pjsip_session: Correct inverted test in session_outgoing_nat_hook" into 13	9 years ago
George Joseph	c2eea791e4	res_pjsip_pubsub: Fix reference to released endpoint ... destroy_subscription was attempting to get the id of the subscription tree's endpoint after we'd already called ao2_cleanup on it causing a segfault. Moved the cleanup until after the debug statement and since endpoint could also be NULL at this point, check for that as well. ASTERISK-27057 #close Reported-by: Ryan Smith Change-Id: Ice0a7727f560cf204d870a774c6df71e159b1678	9 years ago
George Joseph	2dee95cc7a	res_pjsip_session: Correct inverted test in session_outgoing_nat_hook ... There was a typo introduced in commit 776ffd77 which was preventing the transport's external media address from being used. ASTERISK-27024 #close Reported-by: Christopher van de Sande patches: patch.diff submitted by Florian Floimair (license 6892) Change-Id: I7ec617171eaa2d86d2680b00cf37d5088adafc27	9 years ago
Jørgen H	e16a669c70	res_pjsip_transport_websocket: Add NULL check in get_write_timeout ... Added check for NULL return value when calling ast_sorcery_retrieve_by_id in function get_write_timeout ASTERISK-27046 Change-Id: I9357717278da631c3a1cb502c412693929b0cb41	9 years ago
Jenkins2	44d4b55697	Merge "res_pjsip_mwi: don't create mwi subscriptions if initial unsolicited disabled" into 13	9 years ago
George Joseph	7dafe82751	res_rtp_asterisk: Fix ssrc change for rtcp srtp ... It looks like there was a copy/paste error in ast_rtp_change_source where if there was a rtcp srtp instance, instead of updating its ssrc we were updating the srtp instance ssrc twice. ASTERISK-27022 #close Reported-by: Michael Walton Change-Id: Ic88f3aee7227b401c58745ac265ff92c19620095	9 years ago
Kevin Harwell	6cdf3191d3	res_pjsip_refer/session: Calls dropped during transfer ... When doing an attended transfer it's possible for the transferer, after receiving an accepted response from Asterisk, to send a BYE to Asterisk, which can then be processed before Asterisk has time to start and/or complete the transfer process. This of course causes the transfer to not complete successfully, thus dropping the call. This patch makes it so any BYEs received from the transferer, after the REFER, that initiate a session end are deferred until the transfer is complete. This allows the channel that would have otherwise been hung up by Asterisk to remain available throughout the transfer process. ASTERISK-27053 #close Change-Id: I43586db79079457d92d71f1fd993be9a3b409d5a	9 years ago
Alexei Gradinari	59c9bbe696	res_pjsip_mwi: don't create mwi subscriptions if initial unsolicited disabled ... If sending unsolicited mwi to all endpoints on startup is disabled (mwi_disable_initial_unsolicited=yes) do not need to create subscriptions. If there are many (thousands) realtime endpoints configured with unsolicited mwi and Vociemail Storage configured as ODBC or IMAP there will be huge number of DB/IMAP requests on startup. ASTERISK-26230 #close Change-Id: I50ae909639e3ee298b931a54def4b2b9e0fb86c5	9 years ago
Joshua Colp	996a4791ff	pjsip: Extend 'asymmetric_rtp_codec' option to include us changing. ... PJSIP support in Asterisk differs from chan_sip in that it allows media to be sent as-is without transcoding provided the codecs were negotiated in the SDP. This is allowed according to the RFC. Support for this differs quite a lot though and some endpoints do not handle it well. This change extends the 'asymmetric_rtp_codec' option to also cover this case. When set to no (the default) the code behaves as chan_sip does - the best codec is selected and we will only ever send that, unless we change what we are sending if the remote side changes. When set to yes we will send media as-is without transcoding if the codec has been negotiated in the SDP. ASTERISK-26996 Change-Id: Ib1647f6902a0843e8c435946f831c2159e8d1d51	9 years ago
Jenkins2	812f5b51cb	Merge "res_pjsip: Add support for returning only reachable contacts and use it." into 13	9 years ago
Sean Bright	c093bf8072	res_rtp_multicast: Use consistent timestamps when possible ... When a frame destined for a MulticastRTP channel does not have timing information (such as when an 'originate' is done), we generate the RTP timestamps ourselves without regard to the number of samples we are about to send. Instead, use the same method as res_rtp_asterisk and 'predict' a timestamp given the number of samples. If the difference between the timestamp that we generate and the one we predict is within a specific threshold, use the predicted timestamp so that we end up with timestamps that are consistent with the number of samples we are actually sending. Change-Id: I2bf0db3541b1573043330421cbb114ff0f22ec1f	9 years ago
Joshua Colp	746c2c5745	res_pjsip: Add support for returning only reachable contacts and use it. ... This introduces the ability for PJSIP code to specify filtering flags when retrieving PJSIP contacts. The first flag for use causes the query code to only retrieve contacts that are not unreachable. This change has been leveraged by both the Dial() process and the PJSIP_DIAL_CONTACTS dialplan function so they will now only attempt calls to contacts which are not unreachable. ASTERISK-26281 Change-Id: I8233b4faa21ba3db114f5a42e946e4b191446f6c	9 years ago
Jenkins2	95b2b542f6	Merge "format: Reintroduce smoother flags" into 13	9 years ago
Joshua Colp	3d7e9375fb	Merge "res_srtp: Add support for libsrtp2" into 13	9 years ago
Jenkins2	174a9b5d99	Merge "stasis_recording: Correct ast_asprintf error checking" into 13	9 years ago