From: Stuart Henderson Subject: Re: mail/dovecot: backport small fix for OpenBSD To: Brad Smith , OpenBSD ports Date: Sat, 2 Nov 2024 17:00:37 +0000 to save digging around in archives, can you send a current diff please? On 2024/11/02 12:34, Kirill A. Korinsky wrote: > On Mon, 21 Oct 2024 13:12:51 +0200, > Stuart Henderson wrote: > > > > I think that change is appropriate for pf. > > > > Brad, Stuart, are you ok with it to -current? > > Special in the case that seems that dovecot-2.3 will be around for a quite > long time dues to removed features at 2.4. > > > -- > > Sent from a phone, apologies for poor formatting. > > > > On 20 October 2024 03:59:33 Brad Smith wrote: > > > On 2024-10-13 5:52 a.m., Kirill A. Korinsky wrote: > > >> ports@, Brad, Here a backport of small fix for OpenBSD from mine PR > > >> to fix 2.4 branch on > > >> OpenBSD: https://github.com/dovecot/core/pull/224 Right now I can > > >> find in logs a errors like: Fatal: connect(...) failed: Address > > >> already in use not often, like a few times per week. With this fix > > >> which extend FreeBSD's condition to OpenBSD as well, such > > >> errors dissapears. The diff: > > > Seeing as the PR is a bunch of fixes, either way put a brief description > > > at the top of the patch. You might want to look at updating the comment > > > at the very top of lib/net.c to include OpenBSD. > > > for (try = 0;;) { fd = net_connect_ip_once(ip, port, my_ip, sock_type, > > > blocking); if (fd != -1 || try++ >= MAX_CONNECT_RETRIES || (errno != > > > EADDRNOTAVAIL > > > #if defined(__FreeBSD__) || defined(__OpenBSD__) /* busy */ && errno > > > != EADDRINUSE /* pf may cause this if another connection used the same > > > port recently */ && errno != EACCES > > > #endif )) I am not sure what is considered normal and appropriate for userland > > > network and PF. I guess I'd be Ok if someone else more familiar with > > > these kinds of bits said this is appropriate. > > >> > > > -- > wbr, Kirill