28166 Commits (012fda29d23bac1d2b06e9a8933980047da30246)

 

All Branches   

Search

Author	SHA1	Message	Date
Mark Michelson	012fda29d2	CDR: Alter destruction pattern for CDR chains. ... CDRs form chains. When the root of the chain is destroyed, it then unreferences the next CDR in the chain. That CDR is destroyed, and it then unreferences the next CDR in the chain. This repeats until the end of the chain is reached. While this typically does not cause any sort of problems, it is possible in strange scenarios for the CDR chain to grow way longer than expected. In such a scenario, the destruction pattern can result in a stack overflow. This patch fixes the problem by switching from a recursive pattern to an iterative pattern for destruction. When the root CDR is destroyed, it is responsible for iterating over the rest of the CDRs and unreferencing each one. Other CDRs in the chain, since they are not the root, will simply destroy themselves and be done. This causes the stack depth not to increase. ASTERISK-26421 #close Reported by Andrew Nagy Change-Id: I3ca90c2b8051f3b7ead2e0e43f60d2c18fb204b8	10 years ago
George Joseph	546ec4b038	pjproject_bundled: Add patch to address SSL crash ... Addresses crashes when an attempt is made to operate on an SSL socket after the socket has been closed. ASTERISK-26477 #close Change-Id: I421305b357558b4f9e690210dc0f4831ef4b3002	10 years ago
Michael Kuron	f1fd873df0	chan_sip: Only send video on outgoing channel if incoming channel supports it ... Previously, the settings videosupport=always and videosupport=yes behaved identically and unconditionally caused a video offer to be sent in the SDP on an outgoing call. This was a regression introduced with commit 5a1d90e1fb in Asterisk 1.6.1. This commit restores correct behavior: videosupport=always causes a video offer to be sent unconditionally, while videosupport=yes will only offer video on an outbound channel if the incoming channel it is bridged to also supports video. That way, the device receiving the outgoing call can display the correct user interface elements for audio or video and will not unnecessarily show a blank video window on an audio-only call. ASTERISK-17470 #close Change-Id: I782f4409d436114dbc97061c3570c0cd24f7c3ae	10 years ago
zuul	d1d446b237	Merge "Fix issues with bundled pjproject cached download." into 13	10 years ago
zuul	3bdd7c0e38	Merge "Audit ast_json_pack() calls for needed UTF-8 checks." into 13	10 years ago
zuul	8d12d6021b	Merge "json: Check party id name, number, subaddresses for UTF-8." into 13	10 years ago
zuul	05c6ab0d8f	Merge "json: Add UTF-8 check call." into 13	10 years ago
zuul	4d1cb6a46d	Merge "res_config_mysql: Fix several issues related to recent table changes" into 13	10 years ago
zuul	0fa6036b97	Merge "aoc.c: Whitespace cleanup" into 13	10 years ago
zuul	1b9ef66c50	Merge "app_queue.c: Fix clearing of pause reason string." into 13	10 years ago
Corey Farrell	ce4cfd2eca	Fix issues with bundled pjproject cached download. ... Previously when testing I had a preexisting makeopts in ASTTOPDIR. The ordering of configure.ac causes --with-externals-cache to be processed after third-party configure. In cases where the Asterisk clone is cleaned it would cause pjproject to be downloaded to /tmp. This moves processing of the externals cache and sounds cache to happen before third-party configure. This also addresses a possible issue with the third-party Makefile. If TMPDIR is set by the environment it would override the path given to --with-externals-cache. ASTERISK-26416 Change-Id: Ifab7f35bfcd5a31a31a3a4353cc26a68c8c6592d	10 years ago
Richard Mudgett	3c54328c57	Audit ast_json_pack() calls for needed UTF-8 checks. ... Added needed UTF-8 checks before constructing json objects in various files for strings obtained outside the system. In this case string values from a channel driver's peer and not from the user setting channel variables. * aoc.c: Fixed type mismatch in s_to_json() for time and granularity json object construction. ASTERISK-26466 Reported by: Richard Mudgett Change-Id: Iac2d867fa598daba5c5dbc619b5464625a7f2096	10 years ago
Richard Mudgett	7f8f125738	json: Check party id name, number, subaddresses for UTF-8. ... * Updated unit test as ast_json_name_number() is now NULL tolerant. ASTERISK-26466 #close Reported by: Richard Mudgett Change-Id: I7d4e14194f8f81f24a1dc34d1b8602c0950265a6	10 years ago
Richard Mudgett	9621c9bcbc	json: Add UTF-8 check call. ... Since the json library does not make the check function public we recreate/copy the function in our interface module. ASTERISK-26466 Reported by: Richard Mudgett Change-Id: I36d3d750b6f5f1a110bc69ea92b435ecdeeb2a99	10 years ago
Richard Mudgett	e4bb9f9a37	aoc.c: Whitespace cleanup ... * In s_to_json() removed unnecessary ast_json_ref() to ast_json_null() when creating the type json object. The ref is a noop. Change-Id: I2be8b836876fc2e34a27c161f8b1c53b58a3889a	10 years ago
Richard Mudgett	bcac905bd3	app_queue.c: Fix clearing of pause reason string. ... The pause reason is not always cleared when it should be cleared. * Made set_queue_member_pause() always clear pause reason if not pausing with a reason string. Change-Id: I993dad19626ec017478a230e980989438b778c53	10 years ago
Richard Mudgett	ee4ae2b648	app_minivm.c: Fix malformed ast_json_pack() call. ... Change-Id: I082b239022fac462666e52a14a44304748908dc0	10 years ago
George Joseph	90ae4e4337	res_config_mysql: Fix several issues related to recent table changes ... Unlike any of the other database drivers, res_config_mysql checks that the table definition matches the requirements for every insert and update statement. Since all requirements are forced to 'char', any column that isn't a char, like ps_contacts' expiration_time, qualify_timeout, etc., will throw a warning. It's kinda harmless but very misleading. Since no other driver does those checks on insert or update, they've been removed from res_config_mysql. Also, all the logic that actually attempted to ALTER the table to fix the issue has been removed. With the move to alembic, the auto-alter functionality is not only unnecessary, it's also dangerous. The other issue is that res_config_mysql calls the mysql_insert_id function inside store_mysql. Presumably the intention was to return the number of rows inserted DESPITE A NOTE IN THE CODE THAT THE VALUE IS NON_PORTABLE AND MAY CHANGE. That value is then returned to config realtime as the number of rows inserted. Guess what? The value changed. It now only returns the number of rows inserted if there's an auto increment column on the table, which ps_contacts doesn't have. Otherwise it returns 0. So now, the insert worked but we tell config realtime and sorcery that no rows were inserted. That call to mysql_insert_id was removed and we now always return 1 if the insert succeeded. We're only inserting 1 row at a time anyway. If the insert fails, we still return -1. ASTERISK-26362 #close Reported-by: Carlos Chavez Change-Id: I83ce633efdb477b03c8399946994ee16fefceaf4	10 years ago
zuul	3633c7926c	Merge "bundled_pjproject: Add tests for programs used by the Makefile, et al." into 13	10 years ago
Torrey Searle	86c15db6a1	res_fax: Fix a tight race condition causing fax to crash in audio fallback ... When T.38 gets rejected and G711 failback occurs there is a period of time where neither AST_FAX_TECH_T38 nor AST_FAX_TECH_AUDIO is set, leading to a crash. Change-Id: Icc3f457b2292d48a9d7843dac0028347420cc982	10 years ago
Joshua Colp	2fbc0531e0	Merge "Add text of cdr directory into README.md for ast-db-manage" into 13	10 years ago
zuul	2971c1b4eb	Merge "audiohooks: Remove redundant codec translations when using audiohooks" into 13	10 years ago
Rodrigo Ramírez Norambuena	29b7a5b00f	Add text of cdr directory into README.md for ast-db-manage ... Change-Id: I68321c4bea50730c39fdb486e5f23aeadd1ad636	10 years ago
zuul	39dbe7aba4	Merge "vector: After remove element recheck index" into 13	10 years ago
Joshua Colp	64b43d4474	Merge "app_dial: Add the "Q" option to set the cause on unanswered channels" into 13	10 years ago
zuul	5e19935395	Merge "astobj2: Add backtrace to log_bad_ao2." into 13	10 years ago
George Joseph	f919edc4e2	app_dial: Add the "Q" option to set the cause on unanswered channels ... The "Q" option will set the cause on the unanswered channels when another channel answers. It overrides the default of ANSWERED_ELSEWHERE. NOTE: chan_sip does not support setting the cause on a CANCEL to anything other than ANSWERED_ELSEWHERE. ASTERISK-26446 #close Change-Id: I71742e0919aaa16784c30a2b2e73fbeed7672e47	10 years ago
zuul	da78600feb	Merge "res_rtp_asterisk: Fix infinite DTMF issue when switching to P2P bridge" into 13	10 years ago
Badalyan Vyacheslav	a884b26392	vector: After remove element recheck index ... Small fix. It is necessary to double-check the index that we just removed because there is a new element. ASTERISK-26453 #close Change-Id: Ib947fa94dc91dcd9341f357f1084782c64434eb7	10 years ago
zuul	d75beaa04a	Merge "cel_odbc: Fix memory leak on module unload" into 13	10 years ago
Torrey Searle	349c34f72a	res_rtp_asterisk: Fix infinite DTMF issue when switching to P2P bridge ... If a bridge switched to P2P when a DTMF was in progress it was possible for the DTMF to continue being sent indefinitely. Change-Id: I7e2a3efe0d59d4b214ed50cd0b5d0317e2d92e29	10 years ago
zuul	f2e392a3d2	Merge "pjproject_bundled: Add MALLOC_DEBUG capability" into 13	10 years ago
Badalyan Vyacheslav	9da3489d24	res_pjsip_config_wizard: Memory leak in module_unload ... Fixed a memory leak. It removes only the first element. Added a useful feature in vector.h to remove all items under the CMP through a callback function / macro. ASTERISK-26453 #close Change-Id: I84508353463456d2495678f125738e20052da950	10 years ago
Badalyan Vyacheslav	fa2885b3ff	cel_odbc: Fix memory leak on module unload ... Change-Id: Ic7a1236eba2408090fdabb5f717b5fa455ead715	10 years ago
zuul	f809850aa0	Merge "alembic: Allow cdr, config and voicemail to exist in the same schema" into 13	10 years ago
George Joseph	e6b0053d75	bundled_pjproject: Add tests for programs used by the Makefile, et al. ... Added tests for bzip2, tar, patch, sed and nm to configure.ac. Set DOWNLOAD_TO_STDOUT to a working command line regardless of whether the download program is wget, curl or fetch. Added a 'configure.m4' file to the third-party directory which takes care of calling any third-party project setup. Had to move some pjproject_bundled stuff up in configure.ac so it was called before the third-party configure macro. The pjproject tarball is now downloaded to the externals_cache_dir if it was specified on the ./configure command line Removed regeneration of the pjproject aconfigure file. It was only needed for an old patch that no longer applies. Converted the tests for symbols to explicit tests since we know that they're now available in the bundled version. Saves a little time during configure. ASTERISK-26416 #close Reported-by: Corey Farrell Change-Id: Id1d94251c0155f8dd41b7de7067f35cfbaafbb9b	10 years ago
George Joseph	0dc0356e39	pjproject_bundled: Add MALLOC_DEBUG capability ... pjproject_bundled will now use the asterisk memory debugging APIs if MALLOC_DEBUG is turned on in menuselect. Because this required stubs for the executable programs and the python bindings, some Makefile reorganization was needed to properly handle the dependencies. As a result, the makefile now individually makes each of the pjproject libraries separately instead of making them all in 1 shot. The only visible change is that there are separate status lines printed for each library instead oif 1 for all libs. Also, the making of the pjproject dependency files was eliminated. They're not needed for building unless you're actively modifying pjproject source files and it makes the build process faster. Finally, any issues with parallel builds should be resolved again making the build faster. Change-Id: Icc5e3d658fbfb00e0a46b44c66dcc2522d5171b0	10 years ago
Corey Farrell	dd873bcada	astobj2: Add backtrace to log_bad_ao2. ... * Compile __ast_assert_failed unconditionally. * Use __ast_assert_failed to log messages from log_bad_ao2 * Remove calls to ast_assert(0) that happen after log_bad_ao2 was run. Change-Id: I48f1af44b2718ad74a421ff75cb6397b924a9751	10 years ago
George Joseph	86550f9c17	alembic: Allow cdr, config and voicemail to exist in the same schema ... cdr, config and voicemail are all separate alembic trees. Because alembic's default is to use a table named 'alembic_version' to store the current tree revision, the 3 trees can't exist in the same schema without stepping on each other. Now each tree uses 'alembic_version_<tree_name>' as the version table. Each tree's env.py script now first checks for 'alembic_version'. If it finds it AND its revision is in the tree's history, the script renames it to 'alembic_version_<tree_name>'. Regardless, the script then continues with the migration using 'alembic_version_<tree_name>' and creates that table if it's not found. The result is that if an existing 'alembic_version' table was found but it didn't belong to this tree, it's left alone and 'alembic_version_<tree_name>' is used or created. WARNING: If multiple trees are using the same schema, they MUST NOT CRU or D any objects with names that might exist in the other trees. An example would be 'yesno_values' type. If two trees perform operations on it, one tree could pull it out from under the other. Thankfully we currently don't share any names among cdr, config and voicemail. NOTE: Since the env.py scripts in each tree were identical, a common env.py has been placed in the ast-db-manage directory and a symlink to it has been placed in each tree directory. ASTERISK-24311 #close Reported-by: Dafi Ni Change-Id: I4d593f000350deb5d21a14fa1e9bc3896844d898	10 years ago
Alexander Traud	f166681c12	chan_sip: Honor support of Symmetric Response (rport) for SIP requests. ... In the SIP channel driver chan_sip, the default is "auto_force_rport". When no NAT was detected, for example in case of IPv6, Asterisk uses the IP address from the headers within the SIP-REGISTER for subsequent SIP signaling. When the remote party specifies support for Symmetric Response (RFC 3581) via the parameter "rport", Asterisk should not extract the port from the SIP headers but reuse the port of the transport. This did not happen because of a typo. ASTERISK-26438 #close Change-Id: If6e7891848aaf96666dee5305695f7c6667cd5a6	10 years ago
Michael Walton	430f6e5388	audiohooks: Remove redundant codec translations when using audiohooks ... The main frame read and write handlers in main/channel.c don't use the optimum placement in the processing flow for calling audiohooks callbacks, as far as codec translation is concerned. This change places the audiohooks callback code: * After the channel read translation if the frame is not linear before the translation, thereby increasing the chance that the frame is linear as required by audiohooks * Before the channel write translation if the frame is linear at this point This prevents the audiohooks code from instantiating additional translation paths to/from linear where a linear frame format is already available, saving valuable CPU cycles ASTERISK-26419 Change-Id: I6edd5771f0740e758e7eb42558b953f046c01f8f	10 years ago
Joshua Colp	b0b106a473	Merge "Remove "format_ogg_opus: New format"" into 13	10 years ago
Kevin Harwell	2449d2877c	Remove "format_ogg_opus: New format" ... This reverts commit 40aa28131b. ASTERISK-26426 #close Change-Id: I81e55c3c512f1dd6f49896f0c6b97a07d74fd8f5	10 years ago
George Joseph	f0a2e628d6	download_externals: Fix issue with re-install ... Needed to ignore an xmlstarlet return code for optional element. Change-Id: I6a96f709b4b38c9a3f3dda4e8b07903787e16873 Reported-by: Dan Jenkins	10 years ago
George Joseph	1e8b3d00ce	Merge "codec_opus: Add download ability to menuselect" into 13	10 years ago
George Joseph	269ee510c9	Merge "codec_opus: Replace res_format_attr_opus with the one from codec_opus" into 13	10 years ago
George Joseph	0674f319b9	Merge "format_ogg_opus: New format" into 13	10 years ago
zuul	1497e29c4f	Merge "chan_sip: Resolve externhost not to IPv6; instead go for IPv4." into 13	10 years ago
George Joseph	5258c067ae	codec_opus: Add download ability to menuselect ... Updated codecs/codecs.xml to add codec_opus to the external download list. ASTERISK-26409 Change-Id: Ia07b36539f30e852125fb2b94147dc9774df31a4	10 years ago
George Joseph	a5af8709c8	codec_opus: Replace res_format_attr_opus with the one from codec_opus ... Preparation ASTERISK-26409 Change-Id: I9f20e7cce00c32464d9a180e81283d49d199d0a3 (cherry picked from commit 59f7662a93)	10 years ago