c-ares

Commit Graph

Author	SHA1	Message	Date
bradh352	e3cb402bbe	clean up a couple of compiler warnings	3 years ago
bradh352	778d7cd9e7	use helper function for addrinfo to simplify code	3 years ago
bradh352	43d8946dd1	INSTANTIATE_TEST_CASE_P -> INSTANTIATE_TEST_SUITE_P as new convention in googletest	3 years ago
bradh352	3933719395	gmock: update from 1.8.0 to 1.11.0	3 years ago
bradh352	039130583c	Cirrus-CI: fix debian arm build	3 years ago
bradh352	afb20c339f	Cirrus-CI: more updates for proper testing	3 years ago
bradh352	b0106fc7f8	install proper packages for asan and analyze	3 years ago
bradh352	4c3f71bb30	fix crash in tests	3 years ago
bradh352	54fdbfc93b	try to disable container tests	3 years ago
bradh352	a7427d06d6	need g++ for tests on debian	3 years ago
bradh352	e7645eebf8	try cirrus-ci again	3 years ago
bradh352	8f99aab9a1	whitespace	3 years ago
bradh352	001ed0aaf0	start bringing up cirrus-ci	3 years ago
bradh352	abed6353d4	prep for adding new ci	3 years ago
bradh352	2aa85cf5f6	fix cut and paste error	3 years ago
Brad House	29c4ecdc16	RFC6761: special case "localhost" (#430 ) As per RFC6761 Section 6.3, "localhost" lookups need to be special cased to return loopback addresses, and not forward queries to recursive dns servers. We first look up via files (/etc/hosts or equivalent), and if that fails, we then attempt a system-specific address enumeration for loopback addresses (currently Windows-only), and finally fallback to ::1 and 127.0.0.1. Fix By: Brad House (@bradh352) Fixes Bug: #399	3 years ago
Brad House	c642b9fbb1	Reimplement ares_gethostbyname() by wrapping ares_getaddrinfo() (#428 ) ares_gethostbyname() and ares_getaddrinfo() do a lot of similar things, however ares_getaddrinfo() has some desirable behaviors that should be imported into ares_gethostbyname(). For one, it sorts the address lists for the most likely to succeed based on the current system routes. Next, when AF_UNSPEC is specified, it properly handles search lists instead of first searching all of AF_INET6 then AF_INET, since ares_gethostbyname() searches in parallel. Therefore, this PR should also resolve the issues attempted in #94. A few things this PR does: 1. ares_parse_a_reply() and ares_parse_aaaa_reply() had very similar code to translate struct ares_addrinfo into a struct hostent as well as into struct ares_addrttl/ares_addr6ttl this has been split out into helper functions of ares__addrinfo2hostent() and ares__addrinfo2addrttl() to prevent this duplicative code. 2. ares_getaddrinfo() was apparently never honoring HOSTALIASES, and this was discovered once ares_gethostbyname() was turned into a wrapper, the affected test cases started failing. 3. A slight API modification to save the query hostname into struct ares_addrinfo as the last element of name. Since this is the last element, and all user-level instances of struct ares_addrinfo are allocated internally by c-ares, this is not an ABI-breaking change nor would it impact any API compatibility. This was needed since struct hostent has an h_name element. 4. Test Framework: MockServer tests via TCP would fail if more than 1 request was received at a time which is common when ares_getaddrinfo() queries for both A and AAAA records simultaneously. Infact, this was a long standing issue in which the ares_getaddrinfo() test were bypassing TCP alltogether. This has been corrected, the message is now processed in a loop. 5. Some tests had to be updated for overall correctness as they were invalid but somehow passing prior to this change. Change By: Brad House (@bradh352)	3 years ago
bradh352	9aacffe563	ares_getaddrinfo() missing sanity check to fix #426	3 years ago
bradh352	ea68b1bf3d	ares_getaddrinfo(): continue to next domain in search if query returns ARES_ENODATA Some DNS servers may behave badly and return a valid response with no data, in this case, continue on to the next search domain, but cache the result. Fixes Bug: #426 Fix By: Brad House (@bradh352)	3 years ago
bradh352	b035ed8a5e	Allow '/' as a valid character for a returned name As of c-ares 1.17.2, a CNAME an in-addr.arpa delegation broke due to not allowing '/'. This needs to be allowed to not break valid functionality. Fixes Bug: #427 Reported By: Adrian (@leftshift) Fix By: Brad House (@bradh352)	3 years ago
Daniel Stenberg	2227f6982b	libcares.pc.in: update the URL	3 years ago
bradh352	5c995d50b0	ares_expand_name should allow underscores (_) as SRV records legitimately use them c-ares 1.17.2 introduced response validation to prevent a security issue, however it did not have (_) listed as a valid character for domain name responses which caused issues when a CNAME referenced a SRV record which contained underscores. While RFC2181 section 11 does explicitly state not to do validation, that applies to servers not clients. Fixes: #424 Fix By: Brad House (@bradh352)	3 years ago
Daniel Stenberg	04d2f06011	domain: update to use c-ares.org Closes #423	3 years ago
Daniel Stenberg	509f2a00f1	mailing list: moved to lists.haxx.se	3 years ago
Biswapriyo Nath	db5dabc74a	CMake: Fix build in cygwin (#422 ) As cygwin environment has both socket.h and winsock2.h headers check WIN32 not to include the later one here Fix By: Biswapriyo Nath (@Biswa96)	3 years ago
bradh352	6ce842ff93	make building more verbose	3 years ago
bradh352	62003c990b	add appveyor cmake/mingw static-only build	3 years ago
Sinan Kaya	4418939478	CMake: lower case advapi32 for cross-building with mingw (#420 ) When cross compiling with yocto's meta-mingw layer, getting a dependency error. This is caused by the fact that advapi32 is lower case in mingw builds. Fix By: Sinan Kaya <sinan.kaya@microsoft.com>	3 years ago
bradh352	6299d7be38	autotools: add ax_check_gnu_make.m4	3 years ago
bradh352	8e484b2499	autotools: add ax_require_defined.m4	3 years ago
bradh352	3883d99b05	autotools: dont use newer AC_CHECK_INCLUDES_DEFAULT, don't quote AC_ERROR_MSG	3 years ago
bradh352	8afaa9c1b2	import more files needed by newer ax_code_coverage.m4	3 years ago
bradh352	c54ce6c0b9	import more files needed by newer ax_code_coverage.m4	3 years ago
bradh352	e73fb47266	work around autoreconf -fiv first call returning 'error: too many loops'	3 years ago
bradh352	bd368ad615	restore zz40-xc-ovr.m4	3 years ago
bradh352	bad8bde4d4	autotools: processed configure.ac through autoupdate	3 years ago
bradh352	da6a63bba8	autotools. update ax_code_coverage.m4 to latest. don't use deprecated AC_HELP_STRING	3 years ago
bradh352	5ea4616d2c	pull out some old autotools cruft	3 years ago
Felix Yan	7712fcd178	Provide ares_nameser.h as a public interface (#417 ) NodeJS needs ares_nameser.h as a pubic header. Fixes: #415 Fix By: Felix Yan (@felixonmars)	3 years ago
Felix Yan	9c4c483f06	Fix building when latest ax_code_coverage.m4 is imported (#418 ) ax_code_coverage.m4 dropped the @CODE_COVERAGE_RULES@ macro, so we need to switch to the latest recommendation from the m4 file. This requires updates to Makefile.am. Fix By: Felix Yan (@felixonmars)	3 years ago
bradh352	8eccad160f	bump version to match current release	3 years ago
dhrumilrana	76e7d028a2	z/OS minor update, add missing semicolon in ares_init.c (#414 ) Build fix for z/OS Fix by: Dhrumil Rana (@dhrumilrana)	3 years ago
Daniel Bevenius	de042a65f4	add build to .gitignore (#410 ) This commit adds the build directory to be ignored by git. The motivation for adding this to .gitignore as opposed to .git/info/exclude is that the CMake example in INSTALL.md uses build as the name of the directory to be used by CMake. This will cause git to report build as an untracked file. Fix By: Daniel Bevenius (@danbev)	3 years ago
Martin Holeš	9ac7efc382	Add support for URI(Uniform Resource Identifier) records. (#411 ) Add ares_parse_uri_reply() for parsing URI DNS replies. Fix By: Martin Holeš (@martin-256)	3 years ago
Daniel Stenberg	ccebec8303	ares_getaddrinfo.3: available since 1.16.0	3 years ago
Daniel Stenberg	b039ecc87d	README.md: use https:// links	3 years ago
bradh352	6654436a30	fix typo	3 years ago
bradh352	fe282cf172	prep for 1.17.2 release	3 years ago
jeanpierrecartal	11a2bf8efd	Replace strdup() with ares_strdup() (#408 ) strdup() is used in src/lib/ares_parse_a_reply.c and src/lib/ares_parse_aaaa_reply.c whereas allocated memory is freed using ares_free(). Bug: 407 Fix By: Jean-pierre Cartal (@jeanpierrecartal)	3 years ago
Brad House	c9b6c605b5	Validate hostnames in DNS responses and discard from malicious servers (#406 ) To prevent possible users having XSS issues due to intentionally malformed DNS replies, validate hostnames returned in responses and return EBADRESP if they are not valid. It is not clear what legitimate issues this may cause at this point. Bug Reported By: philipp.jeitner@sit.fraunhofer.de Fix By: Brad House (@bradh352)	3 years ago

1 2 3

119 Commits (e3cb402bbeb26abaf9a3459611cb5ba91826795d) All Branches Search

119 Commits (e3cb402bbeb26abaf9a3459611cb5ba91826795d)

All Branches