34180 Commits (3d33980c029aa8045950c0e7afa1efd4f26ed557)

 

All Branches   

Search

Author	SHA1	Message	Date
Naveen Albert	3d33980c02	func_frame_drop: Add debug messages for dropped frames. ... Add debug messages in scenarios where frames that are usually processed are dropped or skipped. Resolves: #1371 (cherry picked from commit 1ddf63d83f)	4 weeks ago
Naveen Albert	e2525a4945	test_res_prometheus: Fix compilation failure on Debian 13. ... curl_easy_setopt expects long types, so be explicit. Resolves: #1369 (cherry picked from commit 3036fb7887)	4 weeks ago
Naveen Albert	29efce1117	func_frame_drop: Handle allocation failure properly. ... Handle allocation failure and simplify the allocation using asprintf. Resolves: #1366 (cherry picked from commit 8ca6968317)	4 weeks ago
Alexey Khabulyak	7d04ff2878	pbx_lua.c: segfault when pass null data to term_color function ... This can be reproduced under certain curcomstences. For example: call app.playback from lua with invalid data: app.playback({}). pbx_lua.c will try to get data for this playback using lua_tostring function. This function returs NULL for everything but strings and numbers. Then, it calls term_color with NULL data. term_color function can call(if we don't use vt100 compat term) ast_copy_string with NULL inbuf which cause segfault. bt example: ast_copy_string (size=8192, src=0x0, dst=0x7fe44b4be8b0) at /usr/src/asterisk/asterisk-20.11.0/include/asterisk/strings.h:412 Resolves: https://github.com/asterisk/asterisk/issues/1363 (cherry picked from commit b098f2aa0f)	4 weeks ago
Naveen Albert	2612ba32fc	bridge.c: Obey BRIDGE_NOANSWER variable to skip answering channel. ... If the BRIDGE_NOANSWER variable is set on a channel, it is not supposed to answer when another channel bridges to it using Bridge(), and this is checked when ast_bridge_call* is called. However, another path exists (bridge_exec -> ast_bridge_add_channel) where this variable was not checked and channels would be answered. We now check the variable there. Resolves: #401 Resolves: #1364 (cherry picked from commit a23c467c82)	4 weeks ago
Ben Ford	ca5335d90e	Update version for Asterisk 23	2 months ago
Alexey Khabulyak	b0421fc87c	app_dial.c: Moved channel lock to prevent deadlock ... It's reproducible with pbx_lua, not regular dialplan. deadlock description: 1. asterisk locks a channel 2. calls function onedigit_goto 3. calls ast_goto_if_exists funciton 4. checks ast_exists_extension -> pbx_extension_helper 5. pbx_extension_helper calls pbx_find_extension 6. Then asterisk starts autoservice in a new thread 7. autoservice run tries to lock the channel again Because our channel is locked already, autoservice can't lock. Autoservice can't lock -> autoservice stop is waiting forever. onedigit_goto waits for autoservice stop. Resolves: https://github.com/asterisk/asterisk/issues/1335	2 months ago
Mike Bradeen	dce107234a	res_pjsip_diversion: resolve race condition between Diversion header processing and redirect ... Based on the firing order of the PJSIP call-backs on a redirect, it was possible for the Diversion header to not be included in the outgoing 181 response to the UAC and the INVITE to the UAS. This change moves the Diversion header processing to an earlier PJSIP callback while also preventing the corresponding update that can cause a duplicate 181 response when processing the header at that time. Resolves: #1349	2 months ago
Allan Nathanson	31571a5079	file.c: with "sounds_search_custom_dir = yes", search "custom" directory ... With `sounds_search_custom_dir = yes`, we are supposed to search for sounds in the `AST_DATA_DIR/sounds/custom` directory before searching the normal directories. Unfortunately, a recent change (https://github.com/asterisk/asterisk/pull/1172) had a typo resulting in the "custom" directory not being searched. This change restores this expected behavior. Resolves: #1353	2 months ago
Sperl Viktor	c1f24b74d7	cel: Add STREAM_BEGIN, STREAM_END and DTMF event types. ... Fixes: #1280 UserNote: Enabling the tracking of the STREAM_BEGIN and the STREAM_END event types in cel.conf will log media files and music on hold played to each channel. The STREAM_BEGIN event's extra field will contain a JSON with the file details (path, format and language), or the class name, in case of music on hold is played. The DTMF event's extra field will contain a JSON with the digit and the duration in milliseconds.	2 months ago
Naveen Albert	613d645d04	func_dialplan: Remove deprecated/redundant function. ... Remove VALID_EXTEN, which was deprecated/superseded by DIALPLAN_EXISTS in Asterisk 11 (commit 8017b65bb9), as DIALPLAN_EXISTS does the same thing and is more flexible. Resolves: #1347 UpgradeNote: The deprecated VALID_EXTEN function has been removed. Use DIALPLAN_EXISTS instead.	2 months ago
Naveen Albert	c8f844a900	logger.c: Remove deprecated/redundant configuration option. ... Remove the deprecated 'rotatetimestamp' config option, as this was deprecated by 'rotatestrategy' in 1.6 by commit f5a14167f3. Resolves: #1345 UpgradeNote: The deprecated rotatetimestamp option has been removed. Use rotatestrategy instead.	2 months ago
Naveen Albert	5c3cd44563	cli.c: Remove deprecated and redundant CLI command. ... The "no debug channel" command has been deprecated since 1.6 (commit 691363656f), as it is replaced by "core set debug channel", which also supports tab-completion on channels. Remove the redundant command. Resolves: #1343 UpgradeNote: The deprecated "no debug channel" command has now been removed; use "core set debug channel" instead.	2 months ago
Naveen Albert	aca74d100c	app_queue: Remove redundant/deprecated function. ... QUEUE_MEMBER_COUNT has been deprecated since at least 1.6, for fully duplicating functionality available in the QUEUE_MEMBER function; remove it now. Resolves: #1341 UpgradeNote: The deprecated QUEUE_MEMBER_COUNT function has been removed; use QUEUE_MEMBER(<queue>,logged) instead.	2 months ago
George Joseph	baa73b6b12	channelstorage_cpp_map_name_id.cc: Refactor iterators for thread-safety. ... The fact that deleting an object from a map invalidates any iterator that happens to currently point to that object was overlooked in the initial implementation. Unfortunately, there's no way to detect that an iterator has been invalidated so the result was an occasional SEGV triggered by modules like app_chanspy that opens an iterator and can keep it open for a long period of time. The new implementation doesn't keep the underlying C++ iterator open across calls to ast_channel_iterator_next() and uses a read lock on the map to ensure that, even for the few microseconds we use the iterator, another thread can't delete a channel from under it. Even with this change, the iterators are still WAY faster than the ao2_legacy storage driver. Full details about the new implementation are located in the comments for iterator_next() in channelstorage_cpp_map_name_id.cc. Resolves: #1309	2 months ago
George Joseph	22d405e900	res_srtp: Add menuselect options to enable AES_192, AES_256 and AES_GCM ... UserNote: Options are now available in the menuselect "Resource Modules" category that allow you to enable the AES_192, AES_256 and AES_GCM cipher suites in res_srtp. Of course, libsrtp and OpenSSL must support them but modern versions do. Previously, the only way to enable them was to set the CFLAGS environment variable when running ./configure. The default setting is to disable them preserving existing behavior.	2 months ago
zhou_jiajian	511d22ef5e	cdr: add CANCEL dispostion in CDR ... In the original implementation, both CANCEL and NO ANSWER states were consolidated under the NO ANSWER disposition. This patch introduces a separate CANCEL disposition, with an optional configuration switch to enable this new disposition. Resolves: #1323 UserNote: A new CDR option "canceldispositionenabled" has been added that when set to true, the NO ANSWER disposition will be split into two dispositions: CANCEL and NO ANSWER. The default value is 'no'	2 months ago
Naveen Albert	2ae6cb7c93	func_curl: Allow auth methods to be set. ... Currently the CURL function only supports Basic Authentication, the default auth method in libcurl. Add an option that also allows enabling digest authentication. Resolves: #1332 UserNote: The httpauth field in CURLOPT now allows the authentication methods to be set.	2 months ago
George Joseph	723410e312	res_stir_shaken: Test for missing semicolon in Identity header. ... ast_stir_shaken_vs_verify() now makes sure there's a semicolon in the Identity header to prevent a possible segfault. Resolves: #GHSA-mrq5-74j5-f5cr	2 months ago
ThatTotallyRealMyth	f973619520	safe_asterisk: Add ownership checks for /etc/asterisk/startup.d and its files. ... UpgradeNote: The safe_asterisk script now checks that, if it was run by the root user, the /etc/asterisk/startup.d directory and all the files it contains are owned by root. If the checks fail, safe_asterisk will exit with an error and Asterisk will not be started. Additionally, the default logging destination is now stderr instead of tty "9" which probably won't exist in modern systems. Resolves: #GHSA-v9q8-9j8m-5xwp	2 months ago
George Joseph	43bf8a4ded	options: Change ast_options from ast_flags to ast_flags64. ... DeveloperNote: The 32-bit ast_options has no room left to accomodate new options and so has been converted to an ast_flags64 structure. All internal references to ast_options have been updated to use the 64-bit flag manipulation macros. External module references to the 32-bit ast_options should continue to work on little-endian systems because the least-significant bytes of a 64 bit integer will be in the same location as a 32-bit integer. Because that's not the case on big-endian systems, we've swapped the bytes in the flags manupulation macros on big-endian systems so external modules should still work however you are encouraged to test.	2 months ago
Alexei Gradinari	3e178dcfd6	res_config_odbc: Prevent Realtime fallback on record-not-found (SQL_NO_DATA) ... This patch fixes an issue in the ODBC Realtime engine where Asterisk incorrectly falls back to the next configured backend when the current one returns SQL_NO_DATA (i.e., no record found). This is a logical error and performance risk in multi-backend configurations. Solution: Introduced CONFIG_RT_NOT_FOUND ((void *)-1) as a special return marker. ODBC Realtime backend now return CONFIG_RT_NOT_FOUND when no data is found. Core engine stops iterating on this marker, avoiding unnecessary fallback. Notes: Other Realtime backends (PostgreSQL, LDAP, etc.) can be updated similarly. This patch only covers ODBC. Fixes: #1305	2 months ago
Sven Kube	9820a62263	resource_channels.c: Don't call ast_channel_get_by_name on empty optional arguments ... `ast_ari_channels_create` and `ast_ari_channels_dial` called the `ast_channel_get_by_name` function with optional arguments. Since 8f1982c4d6, this function logs an error for empty channel names. This commit adds checks for empty optional arguments that are used to call `ast_channel_get_by_name` to prevent these error logs.	2 months ago
Naveen Albert	4a3c9ea574	app_agent_pool: Remove documentation for removed option. ... The already-deprecated "password" option for the AGENT function was removed in commit d43b17a872 for Asterisk 12, but the documentation for it wasn't removed then. Resolves: #1321	2 months ago
Tinet-mucw	009e3ef3f5	pbx.c: When the AST_SOFTHANGUP_ASYNCGOTO flag is set, pbx_extension_helper should return directly. ... Under certain circumstances the context/extens/prio are set in the ast_async_goto, for example action Redirect. In the situation that action Redirect is broken by pbx_extension_helper this info is changed. This will cause the current dialplan location to be executed twice. In other words, the Redirect action does not take effect. Resolves: #1315	3 months ago
Sperl Viktor	5f433e2442	res_agi: Increase AGI command buffer size from 2K to 8K ... Fixes: #1317	3 months ago
Naveen Albert	741ad37086	app_voicemail: Remove deprecated options. ... Remove the deprecated maxmessage and minmessage options, which have been superseded by maxsecs and minsecs since 1.6. Also remove the deprecated 'cz' language option (deprecated since 1.8.) Resolves: #1298 UpgradeNote: The deprecated maxmessage and minmessage options have been removed; use maxsecs and minsecs instead. The deprecated 'cz' language has also been removed; use 'cs' instead.	3 months ago
Naveen Albert	16280837d2	ast_tls_cert: Make certificate validity configurable. ... Currently, the ast_tls_cert script is hardcoded to produce certificates with a validity of 365 days, which is not generally desirable for self- signed certificates. Make this parameter configurable. Resolves: #1307	3 months ago
George Joseph	32b8bf0169	cdr.c: Set tenantid from party_a->base instead of chan->base. ... The CDR tenantid was being set in cdr_object_alloc from the channel->base snapshot. Since this happens at channel creation before the dialplan is even reached, calls to `CHANNEL(tenantid)=<something>` in the dialplan were being ignored. Instead we now take tenantid from party_a when cdr_object_create_public_records() is called which is after the call has ended and all channel snapshots rebuilt. This is exactly how accountcode and amaflags, which can also be set in tha dialplpan, are handled. Resolves: #1259	3 months ago
George Joseph	baf729c7f2	.github: Reduce number of inputs to Releaser to 10. ... The max number of inputs supported by GitHub is 10 so is_security and is_hotfix were factored into a single choice entry.	3 months ago
George Joseph	472eb45dc9	.github: Add skip-cherry-pick and skip-test-builds to Releaser.	3 months ago
George Joseph	1f52ae3582	app_mixmonitor: Update the documentation concerning the "D" option. ... When using the "D" option to output interleaved audio, the file extension must be ".raw". That info wasn't being properly rendered in the markdown and HTML on the documentation site. The XML was updated to move the note in the option section to a warning in the description. Resolves: #1269	3 months ago
Naveen Albert	8ef3e4fb3d	sig_analog: Properly handle STP, ST2P, and ST3P for fgccamamf. ... Previously, we were only using # (ST) as a terminator, and not handling A (STP), B (ST2P), or C (ST3P), which erroneously led to it being treated as part of the dialed number. Parse any of these as the start digit. Resolves: #1301	3 months ago
Naveen Albert	5de45323c7	res_musiconhold: Remove options that were deprecated in Asterisk 14. ... Commit 9c1f34c7e9 added dedicated options for random sorting functionality and deprecated older options that now duplicated these capabilities. Remove these deprecated options. Resolves: #1296 UpgradeNote: The deprecated random and application=r options have been removed; use sort=random instead.	3 months ago
Naveen Albert	c12f4a886d	res_agi: Remove deprecated DeadAGI application. ... DeadAGI was deprecated 7 years ago, in Asterisk 15, as it duplicates functionality in the AGI app. This removes the application. Resolves: #258 UpgradeNote: The DeadAGI application, which was deprecated in Asterisk 15, has now been removed. The same functionality is available in the AGI app.	3 months ago
kodokaii	f9219284cb	chan_websocket: Reset frame_queue_length to 0 after FLUSH_MEDIA ... In the WebSocket channel driver, the FLUSH_MEDIA command clears all frames from the queue but does not reset the frame_queue_length counter. As a result, the driver incorrectly thinks the queue is full after flushing, which prevents new multimedia frames from being sent, especially after multiple flush commands. This fix sets frame_queue_length to 0 after flushing, ensuring the queue state is consistent with its actual content. Fixes: #1304	3 months ago
George Joseph	0086f4a0b8	channelstorage_cpp_map_name_id: Fix callback returning non-matching channels. ... When the callback() API was invoked but no channel passed the test, callback would return the last channel tested instead of NULL. It now correctly returns NULL when no channel matches. Resolves: #1288	3 months ago
Martin Tomec	9e7788ad21	chan_pjsip.c: Change SSRC after media source change ... When the RTP media source changes, such as after a blind transfer, the new source introduces a discontinuous timestamp. According to RFC 3550, Section 5.1, an RTP stream's timestamp for a given SSRC must increment monotonically and linearly. To comply with the standard and avoid a large timestamp jump on the existing SSRC, a new SSRC is generated for the new media stream. This change resolves known interoperability issues with certain SBCs (like Sonus/Ribbon) that stop forwarding media when they detect such a timestamp violation. This code uses the existing implementation from chan_sip. Resolves: #927	3 months ago
Naveen Albert	415daae95f	users.conf: Remove deprecated users.conf integration. ... users.conf was deprecated in Asterisk 21 and is now being removed for Asterisk 23, in accordance with the Asterisk deprecation policy. This consists of: * Removing integration with app_directory, app_voicemail, chan_dahdi, chan_iax2, and AMI. * users.conf was also partially used for res_phoneprov, and this remaining functionality is consolidated to a separate phoneprov_users.conf, used only by res_phoneprov. Resolves: #1292 UpgradeNote: users.conf has been removed and all channel drivers must be configured using their specific configuration files. The functionality previously in users.conf for res_phoneprov is now in phoneprov_users.conf.	3 months ago
George Joseph	5963e624e2	Media over Websocket Channel Driver ... * Created chan_websocket which can exchange media over both inbound and outbound websockets which the driver will frame and time. See http://s.asterisk.net/mow for more information. * res_http_websocket: Made defines for max message size public and converted a few nuisance verbose messages to debugs. * main/channel.c: Changed an obsolete nuisance error to a debug. * ARI channels: Updated externalMedia to include chan_websocket as a supported transport. UserNote: A new channel driver "chan_websocket" is now available. It can exchange media over both inbound and outbound websockets and will both frame and re-time the media it receives. See http://s.asterisk.net/mow for more information. UserNote: The ARI channels/externalMedia API now includes support for the WebSocket transport provided by chan_websocket.	3 months ago
George Joseph	d5bd2b3ce9	channelstorage: Rename callbacks that conflict with DEBUG_FD_LEAKS. ... DEBUG_FD_LEAKS replaces calls to "open" and "close" with functions that keep track of file descriptors, even when those calls are actually callbacks defined in structures like ast_channelstorage_instance->open and don't touch file descriptors. This causes compilation failures. Those callbacks have been renamed to "open_instance" and "close_instance" respectively. Resolves: #1287	3 months ago
Stanislav Abramenkov	b158011c4e	bundled_pjproject: Avoid deadlock between transport and transaction ... Backport patch from upstream * Avoid deadlock between transport and transaction https://github.com/pjsip/pjproject/commit/edde06f261ac Issue described in https://github.com/pjsip/pjproject/issues/4442	3 months ago
Michal Hajek	4522eb1222	audiohook.c: Improve frame pairing logic to avoid MixMonitor breakage with mixed codecs ... This patch adjusts the read/write synchronization logic in audiohook_read_frame_both() to better handle calls where participants use different codecs or sample sizes (e.g., alaw vs G.722). The previous hard threshold of 2 * samples caused MixMonitor recordings to break or stutter when frames were not aligned between both directions. The new logic uses a more tolerant limit (1.5 * samples), which prevents audio tearing without causing excessive buffer overruns. This fix specifically addresses issues with MixMonitor when recording directly on a channel in a bridge using mixed codecs. Reported-by: Michal Hajek <michal.hajek@daktela.com> Resolves: #1276 Resolves: #1279	3 months ago
Sean Bright	574ddb9eae	channelstorage_makeopts.xml: Remove errant XML character. ... Resolves: #1282	3 months ago
mkmer	d5ef09ea0c	utils.h: Add rounding to float conversion to int. ... Quote from an audio engineer NR9V: There is a minor issue of a small amount of crossover distortion though as a result of `ast_slinear_saturated_multiply_float()` not rounding the float. This could result in some quiet but potentially audible distortion artifacts in lower volume parts of the signal. If you have for example a sign wave function with a max amplitude of just a few samples, all samples between -1 and 1 will be truncated to zero, resulting in the waveform no longer being a sine wave and in harmonic distortion. Resolves: #1176	3 months ago
Tinet-mucw	8149554e90	pbx.c: when set flag AST_SOFTHANGUP_ASYNCGOTO, ast_explicit_goto should return -1. ... Under certain circumstances the context/extens/prio are set in the ast_async_goto, for example action Redirect. In the situation that action Redirect is broken by GotoIf this info is changed. that will causes confusion in dialplan execution. Resolves: #1273	3 months ago
Sean Bright	fa50490b13	res_musiconhold.c: Ensure we're always locked around music state access.	3 months ago
Sean Bright	0e9e381e0d	res_musiconhold.c: Annotate when the channel is locked.	3 months ago
Jaco Kroon	014da68306	res_musiconhold: Appropriately lock channel during start. ... This relates to #829 This doesn't sully solve the Ops issue, but it solves the specific crash there. Further PRs to follow. In the specific crash the generator was still under construction when moh was being stopped, which then proceeded to close the stream whilst it was still in use. Signed-off-by: Jaco Kroon <jaco@uls.co.za>	3 months ago
George Joseph	8b8a8c1475	res_stir_shaken.so: Handle X5U certificate chains. ... The verification process will now load a full certificate chain retrieved via the X5U URL instead of loading only the end user cert. * Renamed crypto_load_cert_from_file() and crypto_load_cert_from_memory() to crypto_load_cert_chain_from_file() and crypto_load_cert_chain_from_memory() respectively. * The two load functions now continue to load certs from the file or memory PEMs and store them in a separate stack of untrusted certs specific to the current verification context. * crypto_is_cert_trusted() now uses the stack of untrusted certs that were extracted from the PEM in addition to any untrusted certs that were passed in from the configuration (and any CA certs passed in from the config of course). Resolves: #1272 UserNote: The STIR/SHAKEN verification process will now load a full certificate chain retrieved via the X5U URL instead of loading only the end user cert.	3 months ago