From nobody Sat Oct 14 00:18:08 2023 X-Original-To: dev-commits-ports-branches@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4S6kVT11gjz4x1Rc; Sat, 14 Oct 2023 00:18:09 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256 client-signature RSA-PSS (4096 bits) client-digest SHA256) (Client CN "mxrelay.nyi.freebsd.org", Issuer "R3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 4S6kVS5hqRz3RwJ; Sat, 14 Oct 2023 00:18:08 +0000 (UTC) (envelope-from git@FreeBSD.org) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1697242688; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=/BBa8OzLrS1ejLie5g/pxCympJCbr509iecIqjrL+2Y=; b=BmJ3t3ZxkLzblcFI8l3nqngvLK5yR4+kUqRJe9fI9siaAaBSgv4YFb4J2+7XWtUyVyrRAi b+C6gWoYYjexu9cOMcc3kwL4EPUnXZ3x7lGz1a3A1jkPfXiulrIxnbsRsqDvehcVhI9d6k qv+FwvsyFtW7nmM2+U064sr9IScl8FqIBF9AYtgbLsHaPIvCSZKaK/1FlcpqDPdMHgcHKo crfuEHJiM1SuJtd0NLn5llMpmhM8q6R94E+fG3+zWMFzm9/L/9sjoYK9eqrbRGPwvcVl4Q OgWVgpzIUa4s0Y76x1XFFRw5b7YmfhM0wb/pITfTA9/GBxbMLD583Z/dkObYKg== ARC-Seal: i=1; s=dkim; d=freebsd.org; t=1697242688; a=rsa-sha256; cv=none; b=OeYNg5BY67XEoItegIdvXu7zfYTPhuW0oXP4eHjIxPgjo7OF4ummYaWKn647D6rI3oTa3s uiT65bBSxzfbVSOC7UXOfBGWEdzcuBgDRZeBbBbf/rS75hzsQzRGLBMzP0rJL32rXtMrxo cdQxxaqdqJfy1wT49+pdsDWHq9k6FWakw7UX7490zKhjkNZvnzzaqpKqGnGJPXNHVnjN5p ASWfT1EeLNgdXzTuQ0ReL1wbbf0Oq9PwyJ4/OrjOfo4DkKE9MmuMvRyEfhxhLfa/3ny7fZ 6CTUawml4AUyRANvvr4yARvC6rU6IDbzn9NkZqgs8/UCI2H2tEeBw3DFS36QpQ== ARC-Authentication-Results: i=1; mx1.freebsd.org; none ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=freebsd.org; s=dkim; t=1697242688; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:content-transfer-encoding; bh=/BBa8OzLrS1ejLie5g/pxCympJCbr509iecIqjrL+2Y=; b=f1tAFRvOGU50P0YZkgwZJwupDxKXhmdptoQFmwe/axNbNwa+mnvImAUsJGbk0USNbewL0J Gb7SByIF4sKN9qDM7LI8AhUvIH8GxBWwPMYl7IXEOFEwpM+IR98GpYdqk1eA9SrapU888q 9uHXd9DgkC4B7IJ4mTEkeNfNo9ruYqoIIU8JvL8ukXC8upnmt4H3COBFN92tguCH8Ft/xD +DOZu2e7p0pWSXUXwokMZM9KjvQ3+iJOBR3MBEvZjDmUJNQTKhz8oFG0wB/2iph5EiNF65 mBnV3tqwdAQcn593piXY4odb+aIe/hwaXGBEGG6rP3Bd39Ng3tD90LRDIAb83A== Received: from gitrepo.freebsd.org (gitrepo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:5]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 4S6kVS4pCYz17qK; Sat, 14 Oct 2023 00:18:08 +0000 (UTC) (envelope-from git@FreeBSD.org) Received: from gitrepo.freebsd.org ([127.0.1.44]) by gitrepo.freebsd.org (8.17.1/8.17.1) with ESMTP id 39E0I8x1067767; Sat, 14 Oct 2023 00:18:08 GMT (envelope-from git@gitrepo.freebsd.org) Received: (from git@localhost) by gitrepo.freebsd.org (8.17.1/8.17.1/Submit) id 39E0I8ZI067764; Sat, 14 Oct 2023 00:18:08 GMT (envelope-from git) Date: Sat, 14 Oct 2023 00:18:08 GMT Message-Id: <202310140018.39E0I8ZI067764@gitrepo.freebsd.org> To: ports-committers@FreeBSD.org, dev-commits-ports-all@FreeBSD.org, dev-commits-ports-branches@FreeBSD.org From: Cy Schubert Subject: git: 0706b7c3f62c - 2023Q4 - dns/unbound: Fix loop when ENOBUFS is returned List-Id: Commits to the quarterly branches of the FreeBSD ports repository List-Archive: https://lists.freebsd.org/archives/dev-commits-ports-branches List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-dev-commits-ports-branches@freebsd.org X-BeenThere: dev-commits-ports-branches@freebsd.org MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 8bit X-Git-Committer: cy X-Git-Repository: ports X-Git-Refname: refs/heads/2023Q4 X-Git-Reftype: branch X-Git-Commit: 0706b7c3f62cdf724ca43d361918932202a91fe3 Auto-Submitted: auto-generated The branch 2023Q4 has been updated by cy: URL: https://cgit.FreeBSD.org/ports/commit/?id=0706b7c3f62cdf724ca43d361918932202a91fe3 commit 0706b7c3f62cdf724ca43d361918932202a91fe3 Author: Cy Schubert AuthorDate: 2023-10-13 02:04:20 +0000 Commit: Cy Schubert CommitDate: 2023-10-13 23:55:41 +0000 dns/unbound: Fix loop when ENOBUFS is returned - Fix send of udp retries when ENOBUFS is returned. It stops looping and also waits for the condition to go away. Reported to upstream by Florian Obser. PR: 274352, 274446 Approved by: jaap@NLnetLabs.nl (maintainer) (cherry picked from commit 05c229e187e0dd8d812db2ebb10f74ca1c423efc) --- dns/unbound/Makefile | 1 + dns/unbound/files/patch-netevent.c | 159 +++++++++++++++++++++++++++++++++++++ 2 files changed, 160 insertions(+) diff --git a/dns/unbound/Makefile b/dns/unbound/Makefile index 7697561bdef2..b3ba87493a1e 100644 --- a/dns/unbound/Makefile +++ b/dns/unbound/Makefile @@ -1,5 +1,6 @@ PORTNAME= unbound DISTVERSION= 1.18.0 +PORTREVISION= 1 CATEGORIES= dns MASTER_SITES= https://www.nlnetlabs.nl/downloads/unbound/ diff --git a/dns/unbound/files/patch-netevent.c b/dns/unbound/files/patch-netevent.c new file mode 100644 index 000000000000..e94ab49defa1 --- /dev/null +++ b/dns/unbound/files/patch-netevent.c @@ -0,0 +1,159 @@ +--- util/netevent.c.orig 2023-08-30 01:01:13.000000000 -0700 ++++ util/netevent.c 2023-10-12 19:00:53.157995000 -0700 +@@ -116,6 +116,8 @@ + + /** timeout in millisec to wait for write to unblock, packets dropped after.*/ + #define SEND_BLOCKED_WAIT_TIMEOUT 200 ++/** max number of times to wait for write to unblock, packets dropped after.*/ ++#define SEND_BLOCKED_MAX_RETRY 5 + + /** Let's make timestamping code cleaner and redefine SO_TIMESTAMP* */ + #ifndef SO_TIMESTAMP +@@ -402,9 +404,10 @@ + WSAGetLastError() == WSAENOBUFS || + WSAGetLastError() == WSAEWOULDBLOCK) { + #endif ++ int retries = 0; + /* if we set the fd blocking, other threads suddenly + * have a blocking fd that they operate on */ +- while(sent == -1 && ( ++ while(sent == -1 && retries < SEND_BLOCKED_MAX_RETRY && ( + #ifndef USE_WINSOCK + errno == EAGAIN || errno == EINTR || + # ifdef EWOULDBLOCK +@@ -419,6 +422,13 @@ + #endif + )) { + #if defined(HAVE_POLL) || defined(USE_WINSOCK) ++ int send_nobufs = ( ++#ifndef USE_WINSOCK ++ errno == ENOBUFS ++#else ++ WSAGetLastError() == WSAENOBUFS ++#endif ++ ); + struct pollfd p; + int pret; + memset(&p, 0, sizeof(p)); +@@ -457,8 +467,48 @@ + log_err("poll udp out failed: %s", + sock_strerror(errno)); + return 0; ++ } else if((pret < 0 && ++#ifndef USE_WINSOCK ++ errno == ENOBUFS ++#else ++ WSAGetLastError() == WSAENOBUFS ++#endif ++ ) || (send_nobufs && retries > 0)) { ++ /* ENOBUFS, and poll returned without ++ * a timeout. Or the retried send call ++ * returned ENOBUFS. It is good to ++ * wait a bit for the error to clear. */ ++ /* The timeout is 20*(2^(retries+1)), ++ * it increases exponentially, starting ++ * at 40 msec. After 5 tries, 1240 msec ++ * have passed in total, when poll ++ * returned the error, and 1200 msec ++ * when send returned the errors. */ ++#ifndef USE_WINSOCK ++ pret = poll(NULL, 0, (SEND_BLOCKED_WAIT_TIMEOUT/10)<<(retries+1)); ++#else ++ pret = WSAPoll(NULL, 0, (SEND_BLOCKED_WAIT_TIMEOUT/10)<<(retries+1)); ++#endif ++ if(pret < 0 && ++#ifndef USE_WINSOCK ++ errno != EAGAIN && errno != EINTR && ++# ifdef EWOULDBLOCK ++ errno != EWOULDBLOCK && ++# endif ++ errno != ENOBUFS ++#else ++ WSAGetLastError() != WSAEINPROGRESS && ++ WSAGetLastError() != WSAEINTR && ++ WSAGetLastError() != WSAENOBUFS && ++ WSAGetLastError() != WSAEWOULDBLOCK ++#endif ++ ) { ++ log_err("poll udp out timer failed: %s", ++ sock_strerror(errno)); ++ } + } + #endif /* defined(HAVE_POLL) || defined(USE_WINSOCK) */ ++ retries++; + if (!is_connected) { + sent = sendto(c->fd, (void*)sldns_buffer_begin(packet), + sldns_buffer_remaining(packet), 0, +@@ -665,7 +715,8 @@ + WSAGetLastError() == WSAENOBUFS || + WSAGetLastError() == WSAEWOULDBLOCK) { + #endif +- while(sent == -1 && ( ++ int retries = 0; ++ while(sent == -1 && retries < SEND_BLOCKED_MAX_RETRY && ( + #ifndef USE_WINSOCK + errno == EAGAIN || errno == EINTR || + # ifdef EWOULDBLOCK +@@ -680,6 +731,13 @@ + #endif + )) { + #if defined(HAVE_POLL) || defined(USE_WINSOCK) ++ int send_nobufs = ( ++#ifndef USE_WINSOCK ++ errno == ENOBUFS ++#else ++ WSAGetLastError() == WSAENOBUFS ++#endif ++ ); + struct pollfd p; + int pret; + memset(&p, 0, sizeof(p)); +@@ -718,8 +776,48 @@ + log_err("poll udp out failed: %s", + sock_strerror(errno)); + return 0; ++ } else if((pret < 0 && ++#ifndef USE_WINSOCK ++ errno == ENOBUFS ++#else ++ WSAGetLastError() == WSAENOBUFS ++#endif ++ ) || (send_nobufs && retries > 0)) { ++ /* ENOBUFS, and poll returned without ++ * a timeout. Or the retried send call ++ * returned ENOBUFS. It is good to ++ * wait a bit for the error to clear. */ ++ /* The timeout is 20*(2^(retries+1)), ++ * it increases exponentially, starting ++ * at 40 msec. After 5 tries, 1240 msec ++ * have passed in total, when poll ++ * returned the error, and 1200 msec ++ * when send returned the errors. */ ++#ifndef USE_WINSOCK ++ pret = poll(NULL, 0, (SEND_BLOCKED_WAIT_TIMEOUT/10)<<(retries+1)); ++#else ++ pret = WSAPoll(NULL, 0, (SEND_BLOCKED_WAIT_TIMEOUT/10)<<(retries+1)); ++#endif ++ if(pret < 0 && ++#ifndef USE_WINSOCK ++ errno != EAGAIN && errno != EINTR && ++# ifdef EWOULDBLOCK ++ errno != EWOULDBLOCK && ++# endif ++ errno != ENOBUFS ++#else ++ WSAGetLastError() != WSAEINPROGRESS && ++ WSAGetLastError() != WSAEINTR && ++ WSAGetLastError() != WSAENOBUFS && ++ WSAGetLastError() != WSAEWOULDBLOCK ++#endif ++ ) { ++ log_err("poll udp out timer failed: %s", ++ sock_strerror(errno)); ++ } + } + #endif /* defined(HAVE_POLL) || defined(USE_WINSOCK) */ ++ retries++; + sent = sendmsg(c->fd, &msg, 0); + } + }