summaryrefslogtreecommitdiff
AgeCommit message (Collapse)Author
2018-01-10Merge "res_pjsip.c: Fix ident_to_str() and refactor ident_handler()." into 1313Jenkins2
2018-01-10Merge "app_confbridge: Fix NULL check in action_kick_last." into 13Joshua Colp
2018-01-10Merge "res_pjsip_endpoint_identifier_ip.c: Allow multiple IdentifyDetail AMI ↵Joshua Colp
events." into 13
2018-01-10Merge "res_stasis: Reduce RAII_VAR usage." into 13Joshua Colp
2018-01-09Merge "Revert "codec_opus: Make libcurl a dependency in menuselect"" into 13Jenkins2
2018-01-09res_pjsip.c: Fix ident_to_str() and refactor ident_handler().Richard Mudgett
* Extracted sip_endpoint_identifier_type2str() and sip_endpoint_identifier_str2type() to simplify the calling functions. * Fixed pjsip_configuration.c:ident_to_str() building the endpoint's identify_by value string. Change-Id: Ide876768a8d5d828b12052e2a75008b0563fc509
2018-01-09res_pjsip_endpoint_identifier_ip.c: Allow multiple IdentifyDetail AMI events.Richard Mudgett
The AMI PJSIPShowEndpoint action could only list one IdentifyDetail AMI event per endpoint. However, there is no reason that multiple type=identify sections cannot identify the same endpoint. * Reworked format_ami_endpoint_identify() to generate as many IdentifyDetail AMI events as there are matching identifiers. Change-Id: Ie146792aef72d78e05416ab5b27bc552a30399db
2018-01-09translate: Avoid absolute value on unsigned substraction.Alexander Traud
ast_format_get_sample_rate(.) returns an unsigned type. The difference of a substraction between two unsigned types does not get implicitly converted to a signed type. Therefore, using abs(.) did not make sense. ASTERISK-27549 Change-Id: Ib904d9ee0d46b6fdd1476fbc464fbbf813304017
2018-01-09Revert "codec_opus: Make libcurl a dependency in menuselect"Sean Bright
This reverts commit 028f4320de60a204e457ad606ab0a3318493b431. Change-Id: Ieb91f825cb55202a937f5361c01d356e7662b70c
2018-01-09Merge "res_pjsip.c: Fix endpoint identifier registration name search." into 13Jenkins2
2018-01-09Merge "BuildSystem: Really do not pass unknown-warning options to the ↵Jenkins2
compiler." into 13
2018-01-09Merge "codec_gsm: Avoid shifting a negative signed value." into 13Jenkins2
2018-01-09Merge "res_pjsip_endpoint_identifier_ip.c: Fix apply identify validation." ↵Jenkins2
into 13
2018-01-09Merge "BuildSystem: Find ptlib-config on Debian/Ubuntu." into 13Jenkins2
2018-01-08app_confbridge: Fix NULL check in action_kick_last.Corey Farrell
The check for last_user == NULL needs to happen before we dereference the variable, previously it was possible for us to check flags of a NULL last_user. Change-Id: I274f737aa8af9d2d53e4a78cdd7ad57561003945
2018-01-08res_stasis: Reduce RAII_VAR usage.Corey Farrell
In addition to being a micro-optimization (RAII_VAR has overhead), this change improves output of REF_DEBUG. Unfortunately when RAII_VAR calls ao2_cleanup it does so from a generated _dtor_varname function. For example this caused _dtor_app to release a reference instead of __stasis_app_unregister. Change-Id: I4ce67120583a446babf9adeec678b71d37fcd9e5
2018-01-08Merge "res_stasis: Fix app_is_subscribed_bridge_id." into 13Jenkins2
2018-01-08Merge "General: Avoid implicit conversion to char when changes value to ↵Jenkins2
negative." into 13
2018-01-08Merge "editline: Avoid comparison between pointer and zero character ↵Jenkins2
constant." into 13
2018-01-08Merge "pbx: Prevent execution of NULL pointer." into 13Jenkins2
2018-01-07res_stasis: Fix app_is_subscribed_bridge_id.Corey Farrell
Instead of searching for bridge_id provided in an argument this function always searched for BRIDGE_ALL first. Rewrite this function to work like the similar functions for channel and endpoint functions. Change-Id: Ib5caca69e11727c5c8a7284a1d00621f40f1e60a
2018-01-07Merge "General: Silence modules on (un)load." into 13Jenkins2
2018-01-06Merge "chan_ooh323: Limit outgoinglimit to positive values as intended." into 13Jenkins2
2018-01-06Merge "ooh323cDriver: Fix typo in header guard." into 13Richard Mudgett
2018-01-06General: Silence modules on (un)load.Alexander Traud
Some (normally optional) modules created notices, warnings, and even errors in normal situations like (un)load. This cluttered the command-line interface (CLI) on start and while stopping gracefully. However, when an user went for the script './contrib/scripts/install_prereq', those modules get compiled-in because their prerequisites were met at compile time. Furthermore, because of ASTERISK_27475, the former talkative module 'res_curl' is built as side-effect. ASTERISK-27553 Change-Id: I9f105f46d72553994e820679bfde3478a551b281
2018-01-06BuildSystem: Really do not pass unknown-warning options to the compiler.Alexander Traud
When an older GCC version is called with a too new warning option, GCC exited with an error and Asterisk was not built. Therefore, the configure script tests the installed compiler whether it supports that warning option. If not, Asterisk does not pass it to the installed compiler. However, some compilers (like clang) do not exit (error) but give just a warning in such a case. Because the compiler did not exit, Asterisk passed the unknown-warning option. ASTERISK-27560 Change-Id: Ia9b7747f649b27ff5e9f75c3db3fee4fe7a29621
2018-01-06General: Avoid implicit conversion to char when changes value to negative.Alexander Traud
clang 5.0 warned about this. ASTERISK-27557 Change-Id: I7cceaa88e147cbdf81a3a7beec5c1c20210fa41e
2018-01-06editline: Avoid comparison between pointer and zero character constant.Alexander Traud
gcc 7.2 warned about this. ASTERISK-27559 Change-Id: I48960dda9cf0a11b6a9426f775e632363f8caa74
2018-01-06codec_gsm: Avoid shifting a negative signed value.Alexander Traud
clang 5.0 warned about this. ASTERISK-27558 Change-Id: Icc452ecb0d86bbeba78dae768cc472ec540699df
2018-01-05res_pjsip_endpoint_identifier_ip.c: Fix apply identify validation.Richard Mudgett
The ip_identify_apply() did not validate the configuration for simple static configuration errors or deal well with address resolution errors. * Added missing configuration validation checks. * Fixed address resolution error handling. * Demoted an error message to a warning since it does not fail applying the identify object configuration. Change-Id: I8b519607263fe88e8ce964f526a45359fd362b6e
2018-01-05res_pjsip.c: Fix endpoint identifier registration name search.Richard Mudgett
If an endpoint identifier name in the endpoint_identifier_order list is a prefix to the identifier we are registering, we could install it in the wrong position of the list. Assuming endpoint_identifier_order=username,ip,anonymous then registering the "ip_only" identifier would put the identifier in the wrong position of the priority list. * Fix incorrect strncmp() string prefix matching. Change-Id: Ib8819ec4b811da8a27419fd93528c54d34f01484
2018-01-05BuildSystem: Find ptlib-config on Debian/Ubuntu.Alexander Traud
The current configure script requires that tool when libpt-dev is installed. libpt-dev was installed by libopenh323-dev, bacause you wanted to go for H.323 based channel drivers. ASTERISK-25329 Change-Id: I9c6ab78b7246c21536e1d252dcbffe682f63f83d
2018-01-05chan_ooh323: Limit outgoinglimit to positive values as intended.Alexander Traud
ASTERISK-27552 Change-Id: Ifbf9d51e7374ca2e8b27ec568f6770050fc1a854
2018-01-05ooh323cDriver: Fix typo in header guard.Alexander Traud
ASTERISK-27551 Change-Id: I39ff66031e3373e895e2bc47b23a5e860ea4e012
2018-01-05BuildSystem: Avoid obsolete warning with HELP_STRING on autoconf.Alexander Traud
ASTERISK-26046 Change-Id: I48f05698c235f709225b92bec5aa260fb57d69d1
2018-01-04pbx: Prevent execution of NULL pointer.Corey Farrell
pbx_extension_helper has a check for q->swo.exec == NULL but it doesn't actually return so we would still run the function. Fix the return. Move the 'int res' variable into the only scope which uses it. Change-Id: I0693af921fdc7f56b6a72a21fb816ed08b960a69
2018-01-04Merge "res_pjsip_history: Add missing unlock to CLI command." into 13Jenkins2
2018-01-04Merge "aco: Fix NULL dereference in error path." into 13Jenkins2
2018-01-04res_pjsip_history: Add missing unlock to CLI command.Corey Farrell
Change-Id: I872060a30543776a176a316309602d924a23eb29
2018-01-04aco: Fix NULL dereference in error path.Corey Farrell
Change-Id: Id505167cf0f9414a3c144fa2c1e181a2cf288694
2018-01-04func_odbc: Add missing unlock's to acf_odbc_read.Corey Farrell
Change-Id: I828329ecbd252ae8f27a369a046d2b03102b07c6
2018-01-03res_pjsip_session: Check if sequence header is missingKevin Harwell
The pjsip_msg_find_hdr function can return NULL. This patch adds a check when searching for the sequence header to make sure a NULL pointer is never de-referenced. Change-Id: I19af23aeeded65be016be92360e8cb7ffe51fad2
2018-01-03Merge "core: Use macros to generate ao2_container callbacks where possible." ↵Jenkins2
into 13
2018-01-03Merge "astobj2: Create case-insensitive variants of container function ↵Jenkins2
macros." into 13
2018-01-03Merge "aco: Add missing aco_option_type_string for OPT_TIMELEN_T." into 13Joshua Colp
2018-01-02cdr: submit: fix logic of test for batch modeTzafrir Cohen
ASTERISK-27539 #close Change-Id: I33cdf329d2bb4486dcae975c450f6aae94c515f7
2018-01-02Merge "stasis_channels.c: Misc cleanup." into 13Jenkins2
2018-01-02Merge "cdr_mysql: Make sure connection charset is always set" into 13Jenkins2
2018-01-02aco: Add missing aco_option_type_string for OPT_TIMELEN_T.Corey Farrell
ASTERISK-27117 Change-Id: I8f6c34bb30830be9f7a40823723eb4dcaaa91c61
2018-01-01core: Use macros to generate ao2_container callbacks where possible.Corey Farrell
This uses AO2_STRING_FIELD_HASH_FN and AO2_STRING_FIELD_CMP_FN where possible in the Asterisk core. This removes CMP_STOP from the result of CMP_FN callbacks for the following structure types: * ast_bucket_metadata * ast_bucket_scheme * generic_monitor_instance_list (ccss.c) * named_acl Change-Id: Ide4c1449a894bce70dea1fef664dade9b57578f1