CVS commit: pkgsrc/net/libcares

["Thomas Klausner" <wiz%netbsd.org@localhost>]

Module Name:    pkgsrc
Committed By:   wiz
Date:           Thu Oct 19 14:49:23 UTC 2023

Modified Files:
        pkgsrc/net/libcares: Makefile PLIST distinfo

Log Message:
libcares: update to 1.20.1.

Version 1.20.1 (8 Oct 2023)

GitHub (8 Oct 2023)
- [Daniel Stenberg brought this change]

  ares-test:  silence warning (#564)

  warning: comparison of integer expressions of different signedness

  Fix By: Daniel Stenberg (@bagder)

Brad House (8 Oct 2023)
- fix README.md

GitHub (8 Oct 2023)
- [Brad House brought this change]

  1.20.1 release (#563)

- [Brad House brought this change]

  fix reference to freed memory (#562)

  Issue #561 shows free'd memory could be accessed in some error conditions.

  Fixes Issue #561
  Fix By: Brad House (@bradh352)

Brad House (8 Oct 2023)
- reported build/test systems may timeout on intensive tests. reduce test case to still be relevant but to reduce false positive errors

GitHub (8 Oct 2023)
- [Gregor Jasny brought this change]

  Regression: Fix typo in fuzzcheck target name (#559)

  This seems to be a vim'esque typo introduced with c1b00c41.

  Fix By: Gregor Jasny (@gjasny)

Version 1.20.0 (6 Oct 2023)

Brad House (6 Oct 2023)
- fix slist search off by 1

GitHub (6 Oct 2023)
- [Brad House brought this change]

  1.20.0 release prep (#557)

- [Brad House brought this change]

  ares__buf should return standard error codes.  more helpers implemented. (#558)

  The purpose of this PR is to hopefully make the private API of this set of routines less likely to need to be changed in a future release.  While this is not a public API, it could become harder in 
the future to change usage as it becomes more widely used within c-ares.

  Fix By: Brad House (@bradh352)

- [Brad House brought this change]

  Update from 1989 MIT license text to modern MIT license text (#556)

  ares (and thus c-ares) was originally licensed under the 1989 MIT license text:
  https://fedoraproject.org/wiki/Licensing:MIT#Old_Style_(no_advertising_without_permission)

  This change updates the license to the modern MIT license as recognized here:
  https://opensource.org/license/mit/

  care has been taken to ensure correct attributions remain for the authors contained within the copyright headers, and all authors with attributions in the headers have been contacted for approval 
regarding the change.  Any authors which were not able to be contacted, the original copyright maintains, luckily that exists in only a single file `ares_parse_caa_reply.c` at this time.

  Please see PR #556 for the documented approvals by each contributor.

  Fix By: Brad House (@bradh352)

- [Brad House brought this change]

  Test Harness: use ares_timeout() to calculate the value to pass to select() these days. (#555)

  The test framework was using 100ms timeout passed to select(), and not using ares_timeout() to calculate the actual recommended value based on the queries in queue. Using ares_timeout() tests the 
functionality of ares_timeout() itself and will provide more responsive results.

  Fix By: Brad House (@bradh352)

- [Brad House brought this change]

  Fix for TCP back to back queries (#552)

  As per #266, TCP queries are basically broken. If we get a partial reply, things just don't work, but unlike UDP, TCP may get fragmented and we need to properly handle that.

  I've started creating a basic parser/buffer framework for c-ares for memory safety reasons, but it also helps for things like this where we shouldn't be manually tracking positions and fetching 
only a couple of bytes at a time from a socket. This parser/buffer will be expanded and used more in the future.

  This also resolves #206 by allowing NULL to be specified for some socket callbacks so they will auto-route to the built-in c-ares functions.

  Fixes: #206, #266
  Fix By: Brad House (@bradh352)

- [Brad House brought this change]

  remove acountry from built tools as nerd.dk is gone (#554)

  The acountry utility required a third party DNSBL service from nerd.dk in order to operate. That service has been offline for about a year and there is no other comparable service offering. We are 
keeping the code in the repository as an example, but no longer building it.

  Fixes: #537
  Fix By: Brad House (@bradh352)

- [Brad House brought this change]

  Don't requeue any queries for getaddrinfo() during destruction. (#553)

  During ares_destroy(), any outstanding queries are terminated, however ares_getaddrinfo() had an ordering issue with status codes which in some circumstances could lead to a new query being 
enqueued rather than honoring the termination.

  Fixes #532
  Fix By: @Chilledheart and Brad House (@bradh352)

- [Brad House brought this change]

  ares_getaddrinfo(): Fail faster on AF_UNSPEC if we've already received one address class  (#551)

  As per #541, when using AF_UNSPEC with ares_getaddrinfo() (and in turn with ares_gethostbynam()) if we receive a successful response for one address class, we should not allow the other address 
class to continue on with retries, just return the address class we have.

  This will limit the overall query time to whatever timeout remains for the pending query for the other address class, it will not, however, terminate the other query as it may still prove to be 
successful (possibly coming in less than a millisecond later) and we'd want that result still. It just turns off additional error processing to get the result back quicker.

  Fixes Bug: #541
  Fix By: Brad House (@bradh352)

- [Sam Morris brought this change]

  Avoid producing an ill-formed result when qualifying a name with the root domain (#546)

  This prevents the result of qualifying "name" with "." being "name.." which is ill-formed.

  Fixes Bug: #545
  Fix By: Sam Morris (@yrro)

- [Brad House brought this change]

  Configuration option to limit number of UDP queries per ephemeral port (#549)

  Add a new ARES_OPT_UDP_MAX_QUERIES option with udp_max_queries parameter that can be passed to ares_init_options(). This value defaults to 0 (unlimited) to maintain existing compatibility, any 
positive number will cause new UDP ephemeral ports to be created once the threshold is reached, we'll call these 'connections' even though its technically wrong for UDP.

  Implementation Details:
  * Each server entry in a channel now has a linked-list of connections/ports for udp and tcp. The first connection in the list is the one most likely to be eligible to accept new queries.
  * Queries are now tracked by connection rather than by server.
  * Every time a query is detached from a connection, the connection that it was attached to will be checked to see if it needs to be cleaned up.
  * Insertion, lookup, and searching for connections has been implemented as O(1) complexity so the number of connections will not impact performance.
  * Remove is_broken from the server, it appears it would be set and immediately unset, so must have been invalidated via a prior patch. A future patch should probably track consecutive server errors 
and de-prioritize such servers. The code right now will always try servers in the order of configuration, so a bad server in the list will always be tried and may rely on timeout logic to try the 
next.
  * Various other cleanups to remove code duplication and for clarification.

  Fixes Bug: #444
  Fix By: Brad House (@bradh352)

- [Brad House brought this change]

  its not 1991 anymore, lower default timeout and retry count (#542)

  A lot of time has passed since the original timeouts and retry counts were chosen. We have on and off issues reported due to this. Even on geostationary satellite links, latency is worst case 
around 1.5s. This PR changes the per-server timeout to 2s and the retry count lowered from 4 to 3.

  Fix By: Brad House (@bradh352)

- [Brad House brought this change]

  Modernization: Implement base data-structures and replace usage (#540)

  c-ares currently lacks modern data structures that can make coding easier and more efficient. This PR implements a new linked list, skip list (sorted linked list), and hashtable implementation that 
are easy to use and hard to misuse. Though these implementations use more memory allocations than the prior implementation, the ability to more rapidly iterate on the codebase is a bigger win than 
any marginal performance difference (which is unlikely to be visible, modern systems are much more powerful than when c-ares was initially created).

  The data structure implementation favors readability and audit-ability over performance, however using the algorithmically correct data type for the purpose should offset any perceived losses.

  The primary motivation for this PR is to facilitate future implementation for Issues #444, #135, #458, and possibly #301

  A couple additional notes:

  The ares_timeout() function is now O(1) complexity instead of O(n) due to the use of a skiplist.
  Some obscure bugs were uncovered which were actually being incorrectly validated in the test cases. These have been addressed in this PR but are not explicitly discussed.
  Fixed some dead code warnings in ares_rand for systems that don't need rc4

  Fix By: Brad House (@bradh352)

- [Jérôme Duval brought this change]

  fix missing prefix for CMake generated libcares.pc (#530)

  'pkg-config grpc --cflags' complains with:
  Variable 'prefix' not defined in libcares.pc

  Fix By: Jérôme Duval (@korli)

bradh352 (11 Jul 2023)
- windows get_DNS_Windows port fix for ipv6

- windows get_DNS_Windows port is in network byte order

- backoff to debian 11 due to coverage check failure

- extend on PR #534, windows should also honor a port

GitHub (11 Jul 2023)
- [Brad House brought this change]

  Support configuration of DNS server ports (#534)

  As per https://man.openbsd.org/OpenBSD-5.1/resolv.conf.5 we should
  support bracketed syntax for resolv.conf entries to contain an optional
  port number.

  We also need to utilize this format for configuration of MacOS
  DNS servers as seen when using the Viscosity OpenVPN client, where
  it starts a private DNS server listening on localhost on a non-standard
  port.

  Fix By: Brad House (@bradh352)

Daniel Stenberg (9 Jun 2023)
- provide SPDX identifiers and a REUSE CI job to verify

  All files have their licence and copyright information clearly
  identifiable. If not in the file header, they are set separately in
  .reuse/dep5.

  All used license texts are provided in LICENSES/

GitHub (30 May 2023)
- [Alexey A Tikhonov brought this change]

  Remove unreachable code as reported by Coverity (#527)

  Coverity reported some code as unreachable.  A manual inspection confirmed the reports.

  Fix By: Alexey A Tikhonov (@alexey-tikhonov)

- [Ben Noordhuis brought this change]

  rand: add support for getrandom() (#526)

  glibc provides arc4random_buf() but musl does not and /dev/urandom is
  not always available.

- [Tim Wojtulewicz brought this change]

  Replace uses of sprintf with snprintf (#525)

  sprintf isn't safe even if you think you are using it right.  Switch to snprintf().

  Fix By: Tim Wojtulewicz (@timwoj)

bradh352 (23 May 2023)
- update version and release procedure

GitHub (22 May 2023)
- [Douglas R. Reno brought this change]

  INSTALL.md: Add Watcom instructions and update Windows documentation URLs (#524)

  This commit adds instructions on how to use the WATCOM compiler to build c-ares. This was just tested on c-ares-1.19.1 and works well.

  While going through the links for the C Runtime documentation for Windows systems, I discovered that all three of the KB articles that were linked are now nonexistent. This commit replaces KB94248 
with the current replacement available on Microsoft's website, which also makes the other two KB articles obsolete.

  Fix By: Douglas R. Reno (@renodr)


To generate a diff of this commit:
cvs rdiff -u -r1.34 -r1.35 pkgsrc/net/libcares/Makefile
cvs rdiff -u -r1.18 -r1.19 pkgsrc/net/libcares/PLIST
cvs rdiff -u -r1.28 -r1.29 pkgsrc/net/libcares/distinfo

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: pkgsrc/net/libcares/Makefile
diff -u pkgsrc/net/libcares/Makefile:1.34 pkgsrc/net/libcares/Makefile:1.35
--- pkgsrc/net/libcares/Makefile:1.34   Mon May 22 13:00:41 2023
+++ pkgsrc/net/libcares/Makefile        Thu Oct 19 14:49:23 2023
@@ -1,6 +1,6 @@
-# $NetBSD: Makefile,v 1.34 2023/05/22 13:00:41 wiz Exp $
+# $NetBSD: Makefile,v 1.35 2023/10/19 14:49:23 wiz Exp $
 
-DISTNAME=      c-ares-1.19.1
+DISTNAME=      c-ares-1.20.1
 PKGNAME=       ${DISTNAME:S/c-/libc/1}
 CATEGORIES=    net
 MASTER_SITES=  https://c-ares.haxx.se/download/

Index: pkgsrc/net/libcares/PLIST
diff -u pkgsrc/net/libcares/PLIST:1.18 pkgsrc/net/libcares/PLIST:1.19
--- pkgsrc/net/libcares/PLIST:1.18      Mon May 22 13:00:41 2023
+++ pkgsrc/net/libcares/PLIST   Thu Oct 19 14:49:23 2023
@@ -1,5 +1,4 @@
-@comment $NetBSD: PLIST,v 1.18 2023/05/22 13:00:41 wiz Exp $
-bin/acountry
+@comment $NetBSD: PLIST,v 1.19 2023/10/19 14:49:23 wiz Exp $
 bin/adig
 bin/ahost
 include/ares.h
@@ -14,10 +13,9 @@ lib/cmake/c-ares/c-ares-targets-noconfig
 lib/cmake/c-ares/c-ares-targets.cmake
 lib/libcares.so
 lib/libcares.so.2
-lib/libcares.so.2.6.1
+lib/libcares.so.2.7.1
 lib/libcares_static.a
 lib/pkgconfig/libcares.pc
-man/man1/acountry.1
 man/man1/adig.1
 man/man1/ahost.1
 man/man3/ares_cancel.3

Index: pkgsrc/net/libcares/distinfo
diff -u pkgsrc/net/libcares/distinfo:1.28 pkgsrc/net/libcares/distinfo:1.29
--- pkgsrc/net/libcares/distinfo:1.28   Mon May 22 13:00:41 2023
+++ pkgsrc/net/libcares/distinfo        Thu Oct 19 14:49:23 2023
@@ -1,5 +1,5 @@
-$NetBSD: distinfo,v 1.28 2023/05/22 13:00:41 wiz Exp $
+$NetBSD: distinfo,v 1.29 2023/10/19 14:49:23 wiz Exp $
 
-BLAKE2s (c-ares-1.19.1.tar.gz) = f2913818b5bfdb341924d0f99ef0f626f7c71c9d83a3b26b09a55416b7581dc1
-SHA512 (c-ares-1.19.1.tar.gz) = 466a94efda626e815a6ef7a890637056339f883d549ea6055e289fd8cd2391130e5682c905c0fb3bd7e955af7f6deb793562c170eb0ee066a4a62085a82ba470
-Size (c-ares-1.19.1.tar.gz) = 1579100 bytes
+BLAKE2s (c-ares-1.20.1.tar.gz) = 00de5b55ed137c62298fda011032a9ab4ef7ac4bca54c22778fc427191c99b0a
+SHA512 (c-ares-1.20.1.tar.gz) = 83400fb276ebcf16dfe6f43d56ca87839d132b5a0544420eda9fa148eb85468b3f215593fcefc2a7a3a8ed8b0d4ef093ed99616a4e466b01f6913934240539e4
+Size (c-ares-1.20.1.tar.gz) = 1599175 bytes