c-ares

Commit Graph

Author	SHA1	Message	Date
Brad House	68b37e7145	release keys	8 months ago
Brad House	8106efb0a0	kill old wording	8 months ago
Brad House	91fa52c41e	1.30.0 release prep (#774 )	8 months ago
Brad House	dc5a03dcc6	ConfigChange: Linux slight cleanup tweak	9 months ago
Brad House	a16560761e	coverage: apparently coveralls doesn't support LCOV_EXCL_BR_*	9 months ago
Brad House	9cfc468bec	coverage: attempt to use annotations to ignore defensive coding lines	9 months ago
Brad House	01d9c4c9c9	grammar	9 months ago
Brad House	5b1030b038	reuse compliance	9 months ago
Brad House	3429263560	update base README	9 months ago
Brad House	ce4c20afcb	force preservation of newline in license	9 months ago
Brad House	6129d9b79f	Basic support for SIG RR record (RFC 2931 / RFC 2535) (#773 ) With the current c-ares parser, as per PR #765 parsing was broken due to validation that didn't understand the `SIG` record class. This PR adds basic, non validating, and incomplete support for the `SIG` record type. The additional `KEY` and `NXT` which would be required for additional verification of the records is not implemented. It also does not store the raw unprocessed RR data that would be required for the validation. The primary purpose of this PR is to be able to recognize the record and handle some periphery aspects such as validation of the class associated with the RR and to not honor the TTL in the RR in the c-ares query cache since it will always be 0. Fixes #765 Fix By: Brad House (@bradh352)	9 months ago
Brad House	25ad4ca231	clang-format	9 months ago
Brad House	75f6a6fab4	update sonarcloud build commands as per docs	9 months ago
Brad House	ac7ab82633	Sonarcloud: fix minor codesmells, mostly related to marking more parameters and variables as const	9 months ago
Brad House	a6c8fe685d	QueryCache: Fix flushing on server list changes The query cache wasn't properly flushing on server list changes and was attempting to flush even when the server list didn't actually change. Fix By: Brad House (@bradh352)	9 months ago
Brad House	1dff8f6576	Windows: use GetTickCount64() for a monotonic timer that does not wrap	9 months ago
Brad House	f70f09f01c	Fix windows y2k38 issue by creating our own timeval datatype (#772 ) As per Issue #760, the use of `struct timeval` is meant for only time differentials, however it could be used to denote an exact timeout. This could lead to y2k38 issues on some platforms. Fixes Issue #760 Fix By: Brad House (@bradh352)	9 months ago
Brad House	819cda99a3	MacOS UBSAN: yet another location	9 months ago
Brad House	1595518c29	MacOS UBSAN: another hit	9 months ago
Brad House	48dd3e5346	MacOS UBSAN: another attempt	9 months ago
Brad House	c40bdac234	MacOS UBSAN: try another workaround	9 months ago
Brad House	b40daa2c5b	MacOS UBSAN: another workaround	9 months ago
Brad House	4b37f22ff2	MacOS: another UBSAN warning	9 months ago
Brad House	7bd4c4805e	CI: MacOS fix	9 months ago
Brad House	ac508af7fd	CI: run UBSAN on MacOS, update to Sonoma	9 months ago
Brad House	fa6a2370c7	MacOS UBSAN alignment for config reading	9 months ago
Brad House	40fb125849	URI parser needs to do printable ascii validation for string result	9 months ago
Brad House	75de16cf03	Validate `ares_dns_rr_get_str()` can't return non-valid ASCII strings The DNS message protocol when using non-binary strings needs to be in the ASCII printable range. The function prototype does elude to this but it was not actually validating the string was in anyway valid and could be used. DNS parsing will now fail if an expected string isn't an ASCII string. Fixes Issue: #769 Fix By: Brad House (@bradh352)	9 months ago
David Hotham	f06a414f41	correct deprecation (#768 ) `ares_get_servers_ports_csv` does not exist, should be `ares_get_servers_csv` Fix By: David Hotham (@dimbleby)	9 months ago
Brad House	d1722e6e8a	Android: Config Change detection needs to be a no-op There is no way from the NDK to detect changes, must be a no-op to prevent compiler issues. Fixes Issue: #767 Fix By: Brad House (@bradh352)	9 months ago
Brad House	6bbdcf766e	update to checkout@v4 for newer NodeJS	9 months ago
Brad House	55afad6903	packaging issue: thirdparty/apple/dnsinfo.h Missing header reference in Makefile.inc prevents necessary header file from being packaged in official distribution. Fixes Issue: #763 Fix By: Brad House (@bradh352)	9 months ago
Brad House	bf4c5fd223	1.29.0 release prep (#762 )	9 months ago
Brad House	65e3fb4e09	coverity: try to silence warning	9 months ago
Brad House	0fbb5f6b36	branch main	9 months ago
Brad House	7497991ae5	clang-format	9 months ago
Brad House	8d80486e04	Auto reload config on changes (requires EventThread) (#759 ) Automatically detect configuration changes and reload. On systems which provide notification mechanisms, use those, otherwise fallback to polling. When a system configuration change is detected, it asynchronously applies the configuration in order to ensure it is a non-blocking operation for any queries which may still be being processed. On Windows, however, changes aren't detected if a user manually sets/changes the DNS servers on an interface, it doesn't appear there is any mechanism capable of this. We are relying on `NotifyIpInterfaceChange()` for notifications. Fixes Issue: #613 Fix By: Brad House (@bradh352)	9 months ago
David Hotham	2f200b9170	const is fine on ares__channel_[un]lock (#758 ) at https://github.com/c-ares/c-ares/pull/601#issuecomment-1801935063 you chose not to scatter `const` on the public interface because of the plan - now realised - to add threading to c-ares, and in the expectation that even read operations would need to lock the mutex. But the threading implementation has a _pointer_ to a mutex inside the ares channel and as I understand it, that means that it is just fine to mark `ares__channel_lock` (and `ares__channel_unlock`) as taking a `const` channel. It is the pointed-to mutex that is not constant, but C does not propagate `const`-ness through pointers. This PR sprinkles const where appropriate on public interfaces. Fix By: David Hotham (@dimbleby)	9 months ago
Brad House	85fe3d5a73	remove accidentally committed in-development file	9 months ago
Brad House	18a8bcc29e	dns name compression write failure Due to a logic flaw dns name compression writing was not properly implemented which would result in the name prefix not being written for a partial match. Fixes Bug: #757 Fix By: Brad House (@bradh352)	9 months ago
Brad House	fd620e0308	CI: Add Netbsd workflow (#756 ) Add NetBSD as a test target for CI. Fix By: Brad House (@bradh352)	9 months ago
Brad House	93656ff16f	CI: OpenBSD (#755 ) Github actions appears to support building on a few "unsupported" platforms these days: https://github.com/cross-platform-actions/action Lets go ahead and add OpenBSD to try to prevent issues like #754 Fix By: Brad House (@bradh352)	9 months ago
Volker Schlecht	daf00b98a2	Revert PR #659 (#754 ) As described in https://github.com/c-ares/c-ares/issues/753 the change merged with https://github.com/c-ares/c-ares/pull/659 is ill-advised and leads to breakage in applications using c-ares on OpenBSD. See also https://github.com/nodejs/node/issues/52439	9 months ago
Brad House	abff625408	static analyzer warning from `48e8cd2` Fixes new issue from last commit.	9 months ago
Brad House	48e8cd2ed7	ares_getnameinfo(): loosen validation on salen salen validation should be greater than or equal to the required storage size. Its not uncommon to use `struct sockaddr_storage` in modern code which is definitely larger than `struct sockaddr_in` and on some systems even larger than `struct sockaddr_in6`. Fixes Issue: #752 Fix By: Brad House (@bradh352)	9 months ago
Brad House	6bad5a4ba3	ares_reinit(): reduce locked duration	10 months ago
Brad House	26361885be	add missing copyrights	10 months ago
Brad House	a2efab6c75	manpage: remove AUTHOR section The current best practices consider the AUTHOR section to be deprecated and recommend removing such a section.	10 months ago
Brad House	909c56c604	fix endianness from PR #750	10 months ago
Brad House	ce9aa4807c	Apple: reimplement DNS configuration reading (#750 ) The DNS configuration for apple is stored in the system configuration database. Apple does provide an emulated `/etc/resolv.conf` on MacOS (but not iOS), it cannot, however, represent the entirety of the DNS configuration. Alternatively, libresolv could be used to also retrieve some system configuration, but it too is not capable of retrieving the entirety of the DNS configuration. Attempts to use the preferred public API of `SCDynamicStoreCreate()` and friends yielded incomplete DNS information. Instead, that leaves some apple "internal" symbols from `configd` that we need to access in order to get the entire configuration. We can see that we're not the only ones to do this as Google Chrome also does: https://chromium.googlesource.com/chromium/src/+/HEAD/net/dns/dns_config_watcher_mac.cc These internal functions are what what`libresolv` and `scutil` use to retrieve the dns configuration. Since these symbols are not publicly available, we will dynamically load the symbols from `libSystem` and import the `dnsinfo.h` private header extracted from: https://opensource.apple.com/source/configd/configd-1109.140.1/dnsinfo/dnsinfo.h Fix By: Brad House (@bradh352)	10 months ago

1 2 3 4 5 ...

633 Commits (b417a79b9bb314acb9e97ca022603109b5870a9e) All Branches Search

633 Commits (b417a79b9bb314acb9e97ca022603109b5870a9e)

All Branches