> On Jul 18, 2015, at 1:52 PM, Mikolaj Kucharski <miko...@kucharski.name> wrote: > > On Sat, Jul 18, 2015 at 11:44:17AM -0600, Brent Cook wrote: >> On Thu, May 28, 2015 at 5:24 PM, Mikolaj Kucharski >> <miko...@kucharski.name> wrote: >>> minimalisic patch as below fixed the issue for me: >>> >>> Index: constraint.c >>> =================================================================== >>> RCS file: /cvs/src/usr.sbin/ntpd/constraint.c,v >>> retrieving revision 1.12 >>> diff -u -p -u -r1.12 constraint.c >>> --- constraint.c 28 May 2015 21:34:36 -0000 1.12 >>> +++ constraint.c 28 May 2015 23:14:47 -0000 >>> @@ -279,7 +279,7 @@ constraint_check_child(void) >>> &cstr->addr->ss), >>> CONSTRAINT_SCAN_INTERVAL); >>> } >>> >>> - if (fail || cstr->state < STATE_REPLY_RECEIVED) { >>> + if (fail || cstr->state < STATE_QUERY_SENT) { >>> cstr->senderrors++; >>> constraint_close(cstr->fd); >>> } >> >> Thank you for the in-depth analysis. I have been running this patch >> the last couple >> of days and have not experienced any issues as well. What say you reyk? > > I can confirm that I run this patch from the end of May (from the time > when I've posted this to the misc mailing list) on amd64 and i386 and I > don't have any issues with it. I did not test portable. >
Thanks, I have committed your simpler patch.