29688 Commits (00b0c67144dabaa8788cb6ac8ecb4f173925a85b)

 

All Branches   

Search

Author	SHA1	Message	Date
Tzafrir Cohen	00b0c67144	cdr: submit: fix logic of test for batch mode ... ASTERISK-27539 #close Change-Id: I33cdf329d2bb4486dcae975c450f6aae94c515f7	8 years ago
Jenkins2	8e5167a0b5	Merge "stasis_channels.c: Misc cleanup." into 13	8 years ago
Jenkins2	3edaa955ba	Merge "cdr_mysql: Make sure connection charset is always set" into 13	8 years ago
Sean Bright	d2c836d24a	ice: Increase foundation buffer size ... Per RFC 5245, the foundation specified with an ICE candidate can be up to 32 characters but we are only allowing for 31. ASTERISK-27498 #close Reported by: Michele Prà Change-Id: I05ce7a5952721a76a2b4c90366168022558dc7cf	8 years ago
Jenkins2	b47882df72	Merge "func_channel.c: Update MASTER_CHANNEL documentation" into 13	8 years ago
Richard Mudgett	751fd9b628	stasis_channels.c: Misc cleanup. ... * Use current OBJ_SEARCH_xxx defines instead of the deprecated versions. * Fix hash_cb and cmp_cb container functions to correctly use the OBJ_SEARCH_xxx values. * Remove incorrect usage of CMP_STOP. Most uses in the system have no effect. This allows the collapse of channel_role_single_cmp_cb() and channel_role_multi_cmp_cb() into channel_role_cmp_cb(). * Remove unnecessary usage of RAII_VAR(). Change-Id: I02c405518cab22aa2a082b61e2353bf7cd629a70	8 years ago
Jenkins2	e4d536b64d	Merge "manager.c: Update AMI Status event documentation" into 13	8 years ago
George Joseph	712450c78a	Merge "menuselect: Fix check for running configure." into 13	8 years ago
Jenkins2	9792bb0aa9	Merge "cdr: Missing NULL check and unlock." into 13	8 years ago
Sean Bright	bae301c18c	cdr_mysql: Make sure connection charset is always set ... When the MYSQL_OPT_RECONNECT option is enabled, the MySQL client API will transparently reconnect when it needs to. Ideally this simplifies our code, but when this reconnection occurs all connection state is lost. Because we are not notified that this has happened, we don't know to set our character set again (with "SET NAMES 'xyz'"). Rather than calling SET NAMES, we instead set the MYSQL_SET_CHARSET_NAME option which will do it for us under the hood on each connect. This option has been present in the MySQL C API for at least 15 years, so it should be safe for most installations. I also snuck a few other changes into this patch: * Default the MySQL port to MYSQL_PORT (3306) instead of 0 if it's not defined. * Fix some erroneous and/or silly checks on the contents of the configuration ast_str values. ASTERISK-27366 #close Reported by: Halil İbrahim YILDIZ Change-Id: I36bf8dc5d5f83584e803b3b1a151dea9396ab8f5	8 years ago
Richard Mudgett	5de8f49294	manager.c: Update AMI Status event documentation ... The AMI Status event had linkedid listed twice and was missing the effective connected line name and number headers. NOTE: The linkedid and other standard channel snapshot fields in the XML documentation are part of the <channel_snapshot/> XML template defined in doc/appdocsxml.xslt. * Cached the effective connected line party id so it doesn't get calculated four times. Change-Id: I004c4c4f9e7b40ef55035c831702721bec82496c	8 years ago
Richard Mudgett	9f1cfbafca	bridge_native_rtp.c: Fix reentrancy framehook crash. ... If two channels enter different native rtp bridges at the same time it is possible that the framehook interface data pointer can be corrupted because the struct variable was declared static. * Fixed the reentrancy corruption by changing the framehook interface struct static variable to a stack local variable. * Moved the hook.data assignment outside of the channel lock. It did not need the lock's protection. It probably was giving a false sense of security. The testsuite channels/pjsip/basic_calls/two_parties/nominal/alice_initiated/bob_hangs_up test caught this with MALLOC_DEBUG and DO_CRASH enabled. Change-Id: If9e35b97d19209b0f984941c1d8eb5f7c55eea91	8 years ago
Richard Mudgett	eead5d0c30	func_channel.c: Update MASTER_CHANNEL documentation ... Be more explicit in what is meant by the master channel to eliminate misunderstanding. ASTERISK-23133 Change-Id: I453bcaf4b99404a5a3e345dbf093ac6c1afcfc72	8 years ago
Corey Farrell	0bdddbe526	menuselect: Fix check for running configure. ... menuselect/Makefile checks that autoconfig.h and makeopts were newer than the '.in' files. Unfortunately running ./configure does not touch autoconfig.h unless the contents will change. Instead of looking at autoconfig.h we just need to ensure that makeopts is newer than configure. Also make change to configure.ac so bootstrap.sh doesn't re-add the extra trailing line-feed. Change-Id: Ief1f831d6717007f9cebb668c14e92782cd2b794	8 years ago
Corey Farrell	d62c87bb8d	cdr: Missing NULL check and unlock. ... * handle_dial_message: Missing a check for NULL peer. * ast_cdr_register: Missing unlock on allocation failure. ast_cdr_register is fixed by reordering so the new structure is allocated and initialized before locking the list. Change-Id: I5799b99270d1a7a716a555c31ac85f4b00ce8686	8 years ago
Corey Farrell	92fb393cab	loader: Add volatile to resource_being_loaded. ... Some compiler optimizers seem to assume that dlopen will not use __attribute__((constructor)) functions to call back to the program. This was causing resource_being_loaded to be optimized away completely. ASTERISK-27531 #close Tested By: abelbeck Change-Id: If17a3b889e06811a0e7119f0539d052494d6ece9	8 years ago
Kevin Harwell	53799318bc	AST-2017-014: res_pjsip - Missing contact header can cause crash ... Those SIP messages that create dialogs require a contact header to be present. If the contact header was missing from the message it could cause Asterisk to crash. This patch checks to make sure SIP messages that create a dialog contain the contact header. If the message does not and it is required Asterisk now returns a "400 Missing Contact header" response. Also added NULL checks when retrieving the contact header that were missing as a "just in case". ASTERISK-27480 #close Change-Id: I1810db87683fc637a9e3e1384a746037fec20afe	8 years ago
Sean Bright	ce3d56920b	Remove as much trailing whitespace as possible. ... Change-Id: I873c1c6d00f447269bd841494459efccdd2c19c0	8 years ago
Jenkins2	35a2e09c65	Merge "Fix some invalid Unicode characters" into 13	8 years ago
Jenkins2	29b10b42bb	Merge "app_voicemail: Fix file copy error handling." into 13	8 years ago
Sean Bright	b4ae112e3a	Fix some invalid Unicode characters ... configs/samples/minivm.conf.sample contains invalid UTF-8, but that appears to be intentional. Change-Id: I7b1e0d332f3380fd0425962a3c9c55f9b200c8cc	8 years ago
Jenkins2	d85290ca48	Merge "Fix Common Typo's." into 13	8 years ago
Joshua Colp	e94962ce63	Merge "docs: Remove old API changes documentation" into 13	8 years ago
Jenkins2	2ca8095691	Merge "app_festival: Fix fd leak on connection failure." into 13	8 years ago
Jenkins2	0097ad91ef	Merge "bridge: Old channel video source not set to NULL after unref." into 13	8 years ago
Joshua Colp	8030309a2f	Merge "core: Fix unused variable error in handle_show_sysinfo." into 13	8 years ago
Corey Farrell	719e8eee03	app_voicemail: Fix file copy error handling. ... Fix error where input/output file descriptors would be closed multiple times. Change-Id: Iba5140b60cb7de79e3d5d92be3c256947aa99da9	8 years ago
Sean Bright	6892c13a2c	docs: Remove old API changes documentation ... Change-Id: I1bc7957121cc7ae27dca04acc3613f4e1858476a	8 years ago
Corey Farrell	82b6ba976f	Fix Common Typo's. ... Fix instances of: * Retreive * Recieve * other then * different then * Repeated words ("the the", "an an", "and and", etc). * othterwise, teh ASTERISK-24198 #close Change-Id: I3809a9c113b92fd9d0d9f9bac98e9c66dc8b2d31	8 years ago
Richard Mudgett	4f45748f52	manager.h: Bump AMI version ... Change-Id: I62e6ddeb261ef012687e1fb6734c554e2499b6bf	8 years ago
Corey Farrell	6b3188fb8c	app_festival: Fix fd leak on connection failure. ... Change-Id: If5efaddcf735ff7d17e55c36cc1388946cee9e0f	8 years ago
Corey Farrell	be3800c49d	bridge: Old channel video source not set to NULL after unref. ... The bridge holds onto the old channel video source after it's been released. This can lead to use after free errors. ASTERISK-27229 #close Change-Id: Ib2dab61677dd8a21f7ad53cdc9b8ca93297838b3	8 years ago
Corey Farrell	d05ec48145	core: Fix unused variable error in handle_show_sysinfo. ... Apparently in OSX it's possible for OSX to HAVE_SYSCTL but not HAVE_SYSINFO or HAVE_SWAPCTL. In this case freeswap caused an unused variable error. ASTERISK-26563 #close Change-Id: I8ec5b1897b786cc1abaf62264aa75039eea05510	8 years ago
Joshua Colp	616659d620	Merge "CLI: Address multiple issues." into 13	8 years ago
Jenkins2	b100233801	Merge "bridge: Stop music on hold on adding an arbitrary channel to a bridge" into 13	8 years ago
Jenkins2	2072057657	Merge "res_rtp_asterisk: Avoid close the rtp/rtcp fd twice." into 13	8 years ago
Joshua Colp	f1d50c0fb0	Merge "core: Fix multiple trivial issues in the core." into 13	8 years ago
Jenkins2	eda4f59656	Merge "chan_sip: Fix memory leaks." into 13	8 years ago
Jenkins2	b539ea2a38	Merge "Remove constant conditionals (dead-code)." into 13	8 years ago
George Joseph	ee62652e2d	Merge "chan_console: Use correct parameter for 'set active'" into 13	8 years ago
Corey Farrell	9adffca9c7	CLI: Address multiple issues. ... * listen uses the variable `s` for the result from ast_poll() then overwrites it with the result of accept(). Create a separate variable poll_result to avoid confusion since ast_poll does not return a file descriptor. * Resolve fd leak that would occur if setsockopt failed in listen. * Reserve an extra byte while processing completion results from remote daemon. This fixes a bug where completion processing used strstr() on a string that was not '\0' terminated. This was no risk to the Asterisk daemon, the bug was only reachable the remote console process. * Resolve leak in handle_showchan when the channel is not found. * Multiple leaks and a deadlock in pbx_config CLI completion. * Fix leaks in "manager show command". Change-Id: I8f633ceb1714867ae30ef4e421858f77c14485a9	8 years ago
Aaron An	c38b750810	res_rtp_asterisk: Avoid close the rtp/rtcp fd twice. ... When RTCP-MUX enabled. rtp->s is the same as rtcp->s, check this before close the file descriptor. Close the FD twice will hangs the asterisk under heavy load. ASTERISK-27299 #close Reported-by: Aaron An Tested-by: AaronAn Change-Id: I870a072d73fd207463ac116ef97100addbc0820a	8 years ago
Jenkins2	dba037d422	Merge "main/app: Fix leaks." into 13	8 years ago
Ivan Poddubny	fef23297b7	bridge: Stop music on hold on adding an arbitrary channel to a bridge ... When a channel that is on hold gets added to a bridge by the Bridge AMI action or the dialplan application of the same name, music continues to play, causing "robotic sound". This commit adds a call to ast_moh_stop to stop the music. Also, it makes the AMI Park action use the right MOH class when the channel gets parked. Reported by: Zane Conkle ASTERISK-25079 #close Change-Id: I4b129c5a20c15e63968842460ac5a1a85903cf9f	8 years ago
Corey Farrell	0e5d8ad09b	chan_sip: Fix memory leaks. ... In change_redirecting_information variables we use ast_strlen_zero to see if a value should be saved. In the case where the value is not NULL but is a zero length string we leaked. handle_response_subscribe leaked a reference to the ccss monitor instance. Change-Id: Ib11444de69c3d5b2360a88ba2feb54d2c2e9f05f	8 years ago
Jenkins2	909f41a2e0	Merge "func_callerid: Initialize app argument structures." into 13	8 years ago
Jenkins2	bde7c0420a	Merge "rtp_engine.c: Eliminate rtcp_report_to_json() RAII_VAR usage." into 13	8 years ago
Oron Peled	fc86e58a5a	chan_console: Use correct parameter for 'set active' ... chan_console supports multiple devices but the CLI only works on a single device. 'console set active' selects this device. Sadly that CLI picks the wrong command-line parameter and will only work for a device called 'active'. ASTERISK-27490 #close Change-Id: I2f0e5fe63db19845bee862575b739360797dc73d	8 years ago
Jenkins2	c7bdffb281	Merge "ast_json_pack(): Use safer json ref mechanism." into 13	8 years ago
Corey Farrell	d6b2f457d9	Remove constant conditionals (dead-code). ... Some variables are set and never changed, making them constant. This means that code in the 'false' block of the conditional is unreachable. In chan_skinny and res_config_ldap I used preprocessor directive `#if 0` as I'm unsure if the unreachable code could be enabled in the future. Change-Id: I62e2aac353d739fb3c983cf768933120f5fba059	8 years ago