4387 Commits (c4b979a84172cf14d0c00a24906b8639fd798327)

Author	SHA1	Message	Date
Richard Mudgett	d55ded340f	res_smdi.c: Fix module ref counting and inverted test. ... I think this module is so screwed up that it doesn't work anymore. Even with these attempts to fix things it still won't gracefully shut down. The module refs will not go to zero to allow unloading the module. * Fix module ref counting dealing with the SMDI interface object. There were several off-nominal paths that unbalanced the module ref count. Also the destructor freed the ao2 object itself which is bad. Made the smdi_read thread not hold its own ref to the SMDI interface object so when all refs go away the destructor will stop the listener thread. * Fixed the smdi_load() return code of 1 concerning the number of listeners. The test was inverted. Change-Id: Ic288db51b58e395d6a2fc3847f77176c16988784	7 years ago
Joshua Colp	a02522c140	Merge "res_pjsip: improve realtime performance on CLI 'pjsip show contacts'" into 13	7 years ago
Joshua Colp	dc03526380	Merge "res_stasis: Fix stale data in ARI bridges" into 13	7 years ago
Kevin Harwell	fa76297e66	Merge "res_odbc: fix missing SQL error diagnostic" into 13	7 years ago
Ben Ford	aa31657e28	res_rtp_asterisk.c: Add "seqno" strictrtp option ... When networks experience disruptions, there can be large gaps of time between receiving packets. When strictrtp is enabled, this created issues where a flood of packets could come in and be seen as an attack. Another option - seqno - has been added to the strictrtp option that ignores the time interval and goes strictly by sequence number for validity. Change-Id: I8a42b8d193673899c8fc22fe7f98ea87df89be71	7 years ago
Alexei Gradinari	7d79805d54	res_pjsip: improve realtime performance on CLI 'pjsip show contacts' ... CLI command 'pjsip show contacts' inefficiently make a lot of DB requests. For example if there are 10k aors then asterisk requests these 10k records of aor and then does 10k requests of contact - one request per aor. Even if use 'like <pattern>' the asterisk requests all aor's and contact's records and then filters them by itself. This patch gathers contact's container by - retrieving all dynamic contacts by regex (filtered by reg_server) - retrieving all aors with permanent contacts - finally filters container by regex ASTERISK-28077 #close Change-Id: Id0ad65d14952a02fb213273a90f3f680a8149618	7 years ago
George Joseph	efadf89131	Merge "res_rtp_asterisk: Raise event when RTP port is allocated" into 13	7 years ago
Moritz Fain	27c46b8ff3	res_stasis: Fix stale data in ARI bridges ... Fixed an issue that resulted in "Allocation failed" each time an ARI request was made to start playing MOH on a bridge. In bridge_moh_create() we were attaching the after bridge callbacks to chan which is the ;1 channel of the unreal channel pair. We should have attached them to the ;2 channel which is pushed into the bridge by ast_unreal_channel_push_to_bridge(). The callbacks are called when the specific channel leaves the bridging system. Since the ;1 channel is never put into a bridge the callbacks never get called. The callbacks then never remove the moh_wrapper from the app_bridges_moh container. As a result we cannot find the channel associated with the wrapper to start MOH because it has hungup. This is the reason causing the reported issue. * Rather than using after bridge callbacks to cleanup, we now have moh_channel_thread() doing the cleanup when the channel hangs up. * Fixed moh_channel_thread() accumulating control frames on the stasis bridge MOH channel until MOH is stopped. Control frames are no longer accumulated while MOH is playing. * Fixed channel ref counting issue. stasis_app_bridge_moh_channel() may or may not return a channel ref. As a result ast_ari_bridges_start_moh() wouldn't know it may have a channel ref to release. stasis_app_bridge_moh_channel() will now return a ref with the channel it returns. * Eliminated RAII_VAR in bridge_moh_create(). ASTERISK-26094 #close Change-Id: Ibff479e167b3320c68aaabfada7e1d0ef7bd548c	7 years ago
Joshua Colp	155ff8e174	res_rtp_asterisk: Raise event when RTP port is allocated ... This change raises a testsuite event to provide what port Asterisk has actually allocated for RTP. This ensures that testsuite tests can remove any assumption of ports and instead use the actual port in use. ASTERISK-28070 Change-Id: I91bd45782e84284e01c89acf4b2da352e14ae044	7 years ago
Alexei Gradinari	37c1f35922	res_odbc: fix missing SQL error diagnostic ... On SQL error there is not diagnostic information about this error. There is only WARNING res_odbc.c: SQL Execute error -1! The function ast_odbc_print_errors calls a SQLGetDiagField to get the number of available diagnostic records, but the SQLGetDiagField returns 0. However SQLGetDiagRec could return one diagnostic records in this case. Looking at many example of getting diagnostics error information I found out that the best way it's to use only SQLGetDiagRec while it returns SQL_SUCCESS. Also this patch adds calls of ast_odbc_print_errors on SQL_ERROR to res_config_odbc. ASTERISK-28065 #close Change-Id: Iba5ae5470ac49ecd911dd084effbe9efac68ccc1	7 years ago
Sean Bright	3033242a7b	res_rtp_asterisk: Reset all settings on module reload ... 'rtpchecksums' and 'rtcpinterval' are not being reset to their defaults if they are not present in the updated configuration file. Change-Id: I1162e40199314d46cf3225d5e1271c4c81176670	7 years ago
Sean Bright	33ca3664ca	AST-2018-009: Fix crash processing websocket HTTP Upgrade requests ... The HTTP request processing in res_http_websocket allocates additional space on the stack for various headers received during an Upgrade request. An attacker could send a specially crafted request that causes this code to overflow the stack, resulting in a crash. * No longer allocate memory from the stack in a loop to parse the header values. NOTE: There is a slight API change when using the passed in strings as is. We now require the passed in strings to no longer have leading or trailing whitespace. This isn't a problem as the only callers have already done this before passing the strings to the affected function. ASTERISK-28013 #close Change-Id: Ia564825a8a95e085fd17e658cb777fe1afa8091a	7 years ago
George Joseph	c4f1adf78e	Merge "pjproject: Upgrade to 2.8." into 13	7 years ago
Joshua Colp	cd2deadb69	pjproject: Upgrade to 2.8. ... This change brings in PJSIP 2.8, removes all the patches that were merged upstream, and makes a minor change to support a breaking change that was done. ASTERISK-28059 Change-Id: I5097772b11b0f95c3c1f52df6400158666f0a189	7 years ago
Sean Bright	99b2e0c2ff	autoconf: Check for srtp_get_version_string() before using it ... Change-Id: Id2a916ff9448706090e72ff2c7fb3f5ba24a05df	7 years ago
George Joseph	926ac196af	Merge "res_srtp.c: Show linked version of libsrtp on module init" into 13	7 years ago
George Joseph	3a09d9c74c	Merge "res_pjsip: Log IPv6 addresses correctly" into 13	7 years ago
Sean Bright	44375c0616	res_srtp.c: Show linked version of libsrtp on module init ... Change-Id: Ib0a645d6985de5757cc4399ed2524b2d02c4f342	7 years ago
Sean Bright	d3c869c736	res_pjsip: Log IPv6 addresses correctly ... Both pjsip_tx_data.tp_info.dst_name and pjsip_rx_data.pkt_info.src_name store IPv6 addresses without enclosing brackets. This causes some log output to be confusing because it is difficult to separate the IPv6 address from a port specification. * Use pj_sockaddr_print() along with pjsip_tx_data.tp_info.dst_addr and pjsip_rx_data.pkt_info.src_addr where possible for consistent IPv6 output. * When a pj_sockaddr is not available, explicitly wrap IPv6 addresses in brackets. * When assigning pjsip_rx_data.pkt_info.src_name ourselves, make sure to also set pjsip_rx_data.pkt_info.src_addr. Change-Id: I5cfe997ced7883862a12b9c7d8551d76ae02fcf8	7 years ago
Jenkins2	d4becda8ec	Merge "res_musiconhold.c: Restart MOH if previous hold just reached end-of-file" into 13	7 years ago
Jenkins2	641e5b7e63	Merge "optional_api: Remove unused nonoptreq fields" into 13	7 years ago
Walter Doekes	d226458c5b	optional_api: Remove unused nonoptreq fields ... As they're not actively used, they only grow stale. The moduleinfo field itself is kept in Asterisk 13/15 for ABI compatibility. ASTERISK-28046 #close Change-Id: I8df66a7007f807840414bb348511a8c14c05a9fc	7 years ago
Sean Bright	931b68d3e6	res_pjproject: Fix sockaddr conversion routines for non-bundled PJSIP ... The bundled version of pjproject has a patch for Solaris compatability that changes the definition of various socket structures which we need to account for when compiling against a non-bundled version. ASTERISK-28049 #close Change-Id: Ia1ea47c433fc2d915115193ee889a752373925f0	7 years ago
Frederic LE FOLL	a5f87a3a36	res_musiconhold.c: Restart MOH if previous hold just reached end-of-file ... On MOH activation, moh_files_readframe() is called while the current stream attached to the channel is NULL and it calls ast_moh_files_next() immediately. However, it won't call ast_moh_files_next() again if sample reading fails. The failure may occur because res_musiconhold retains the last sample reading position in the channel data and MOH during the previous hold/retrieve just reached EOF. Obviously, a bit of bad luck is required here. * Restructured moh_files_readframe() to try a second time to start MOH if there was no stream setup and the saved position was at EOF. Also added comments describing what is going on for each step. ASTERISK-28029 Change-Id: I1508cf2c094f8feca22d6f76deaa9fdfa9944860	7 years ago
Sean Bright	b0ef54af71	res_pjproject: Add utility functions to convert between socket structures ... Currently, to convert from a pj_sockaddr to an ast_sockaddr, the address needs to be rendered to a string and then parsed into the correct structure. This also involves a call to getaddrinfo(3). The same is true for the inverse operation. Instead, because we know the internal structure of both ast_sockaddr and pj_sockaddr, we can translate directly between the two without the need for an intermediate string. Change-Id: If0fc4bba9643f755604c6ffbb0d7cc46020bc761	7 years ago
George Joseph	5d4cf16c17	Merge "res_pjsip: Fix mwi_subscribe_replaces_unsolicited type mismatch" into 13	7 years ago
Joshua Colp	d0da68bfc9	Merge "res_pjsip_transport_websocket: Properly set src_name for IPv6" into 13	7 years ago
Joshua Colp	6bf0943ecb	Merge "res_fax: Handle fax gateway being started more than once." into 13	7 years ago
Richard Mudgett	f2266602a1	res_pjsip: Fix mwi_subscribe_replaces_unsolicited type mismatch ... ASTERISK-27988 Change-Id: Iccafdd0552ea8aaed647620fb14499f1bf341843	7 years ago
Joshua Colp	cfe8ef917d	Merge "Create --disable-binary-modules option." into 13	7 years ago
Joshua Colp	fa71f498ce	res_fax: Handle fax gateway being started more than once. ... The T.38 fax gateway state machine can cause the fax gateway to be started more than once on a channel depending on the responses of the remote endpoint. This would previously leak the channel name, channel unique id, and underlying fax engine state. This change instead makes it so that if the fax gateway session is already present and not reserved the fax gateway is not started again. ASTERISK-27981 Change-Id: I552d95086860cb18f2522ee40ef47b13b6da2e0e	7 years ago
Sean Bright	f3188a634d	res_pjsip_transport_websocket: Properly set src_name for IPv6 ... SIP responses over WebSockets when the client is using IPv6 have invalid Via headers according to RFC 3261. The 'received' header parameter should not be wrapped in brackets if it is an IPv6 address. When src_name is populated by the built-in PJSIP transports, the code uses pj_sockaddr_print() with 'flags' set to 0, meaning that the brackets are not rendered around IPv6 addresses. This may be related to ASTERISK~27101. See also: https://github.com/onsip/SIP.js/pull/594 ASTERISK-28020 #close Change-Id: I8ea9d289901b837512bee2ca2535e3dc14f04d77	7 years ago
Corey Farrell	abb04622f7	Create --disable-binary-modules option. ... This new option can be passed for ./configure or ./tests/CI/buildAsterisk.sh to prevent download/install of binary modules. Normally enabling the categories MENUSELECT_CODECS or MENUSELECT_RES will result in binary modules being enabled even if the build target is incompatible with those modules. This includes CI scripts which enable categories before disabling specific modules. If more binary modules are offered in the future this will help avoid accidentally downloading them if unwanted or incompatible. Adding a binary module will only require creating a new menuselect entry similar to the existing ones, it will not be necessary to modify the CI scripts. Change-Id: I6b1bd1c75a2e48f05b8b8a45b7a7a2d00a079166	7 years ago
neutrino88	49d388bfa8	res/res_rtp_asterisk: remove debug traces generated by an empty frame ... The realtime text timer pops regularly and sends text frames even if the buffer is empty. This causes a lot of unecessary debug logging. * Made red_write() test if we need to send a frame before calling ast_rtp_write() ASTERISK-28002 Reported by: Emmanuel BUU Tested by: Emmanuel BUU Change-Id: Icf81310c3b8080b615a42060afc02ab41f9523dd	7 years ago
Richard Mudgett	a66fa4db24	res_sorcery_realtime.c: Fix unqualified fetch warning. ... The allow_unqualified_fetch option for the sorcery realtime backend blocked actually fetching all rows when the option is set to warn. * Made issue a warning and actually do the request when allow_unqualified_fetch=warn is set. Change-Id: I74456c80a03a62dce66fc3dc3cb0cf2351ac4312	7 years ago
Joshua Colp	40d2d317ae	Merge "res_pjsip: Resolve transport management leak at shutdown." into 13	7 years ago
George Joseph	0c4d49c5a9	Merge "res/res_pjsip_sdp_rtp: put rtcp-mux in answer only if offered" into 13	7 years ago
Corey Farrell	db367ddbbf	res_pjsip: Resolve transport management leak at shutdown. ... Cleanup idle check scheduled events at shutdown. Change-Id: I61bfbb56bac69fe840c3242927d31ff3593be461	7 years ago
Corey Farrell	7df97d0a00	res_pjsip: Fix leak in pjsip_options. ... sip_options_get_endpoint_state_compositor_state leaked a reference to the first available endpoint state compositor that was found. Change-Id: Idb6be19f7219b6eed1dfb19c1e740dd40cb3fdc7	7 years ago
Joshua Colp	555516233d	res_pjsip_caller_id: Add "party" parameter to RPID header. ... This change adds the "party" parameter to the Remote-Party-ID header which indicates which party the header information is applicable to. In Asterisk this is determined on whether we are the calling or called party. This is added to improve interoperability with some implementations. ASTERISK-28006 Change-Id: I1eec3e377ffff8633b5c1dd59a05e9533122cfca	7 years ago
Torrey Searle	a1b0db826a	res/res_pjsip_sdp_rtp: put rtcp-mux in answer only if offered ... If in the initial sdp the caller doesn't include the line a=rtcp-mux Then asterisk shoud not include rtcp-mux in the response regardless of rtcp-mux being enabled on the endpoint ASTERISK-28007 #close Change-Id: I58e9b9f40a139afc0da5de41906cc608fb62adc7	7 years ago
Joshua Colp	673d509911	Merge "res_pjsip_registrar: Improve performance on inbound handling." into 13	7 years ago
Richard Mudgett	1f97ea7e2c	res_pjsip: Make pjlib.h consistently included. ... * Don't include pjlib.h twice in res_pjsip.h * Consistently use #include <> form for pjproject includes. (pjsip.h and pjlib.h) Change-Id: I3f7b42044840de64edf7e9d7695cb60c45990dc7	7 years ago
Kevin Harwell	4dee592608	Merge "res_pjsip_endpoint_identifier_ip.c: Added regex support to match_header" into 13	7 years ago
Joshua Colp	a5ce9b6c4b	res_pjsip_registrar: Improve performance on inbound handling. ... This change removes a sorcery lookup for retrieving all contacts at the end of the registration process by keeping track of the contacts that are added/updated/deleted. This ensures at the end of the process the container of contacts we have is the current state. Pool usage has also been reduced by allocating one for usage throughout the handling of a REGISTER and resetting it to a clean state. This ensures that in most cases we allocate once and just reuse it. ASTERISK-28001 Change-Id: I1a78b2d46f9a2045dbbff1a3fd6dba84b612b3cb	7 years ago
Joshua Colp	904095833e	Merge "res_rtp_asterisk: In Developer Mode, do not require OpenSSL." into 13	7 years ago
Joshua Colp	edddcb3dc6	res_pjsip_pubsub: Use ast_true for "prune_on_boot". ... Change-Id: Iedec4e7390b3e821987681da24d0298632b9873d	7 years ago
Richard Mudgett	8baedc2204	res_pjsip_endpoint_identifier_ip.c: Added regex support to match_header ... This patch adds regular expression support to make the identify section's match_header option more useful when attempting to match complex headers like the 'To' or 'From' headers. The 'From' header has variable components such as the tag parameter that you cannot predict. To specify a regular expression put slashes around the regular expression in place of the header value. [identify-alice] type=identify endpoint=alice match_header=From: /<sip:alice@127\\.0\\.0\\.1>/ * Added regex support to match_header so you could match a 'To' header among other complex headers. Fixed reported crashes when trying to match special headers like 'Contact'. The identify section's match_header method used code that assumed you were matching a generic header. Any other type of header could cause a crash if the header structure variant did not match the generic header enough. * Made use code that will work for any header type instead of code specific to generic headers. Other fixes while in the area: * Made check all headers of the requested name. * Added some more sanity checks to the configured identify matching options when applying the configuration. ASTERISK-27548 Change-Id: I27dfd4ff5e2259b906640e3c330681b76b4ed1f1	7 years ago
Joshua Colp	d33ed08afc	res_pjsip_pubsub: Treat "prune_on_boot" as a yes / no. ... The alembic for the PJSIP subscription persistence table has the "prune_on_boot" field as a boolean. While in Asterisk we are tolerant of many different definitions of true and false in the database we only accept "yes" and "no". This change makes the field treated as a yes/no instead of an integer, thus storing "yes" and "no" instead of "1" and "0". Change-Id: Ic8b9211b36babefe78f70def6828a135a6ae7ab6	7 years ago
Alexander Traud	a4b105c99b	res_rtp_asterisk: In Developer Mode, do not require OpenSSL. ... OpenSSL is an optional external library and should stay optional even when Developer Mode is configured. ASTERISK-27990 Change-Id: Ia68a4cd5474b26d45e0f43b04032ad598022853b	7 years ago