Yo Hal! On Tue, 27 Sep 2016 15:59:21 -0700 Hal Murray <[email protected]> wrote:
> > So, got any patch ideas?
>
> Sorry, it will be obvious after you see it.
>
> You are fixing up the default, and then immediately overwriting that
> store with a new value.
>
> > if (NTP_MAXPOLL_UNK == maxpoll)
> > /* not set yet, set to default */
> > peer->maxpoll = NTP_MAXDPOLL;
> > peer->maxpoll = max(maxpoll, NTP_MINPOLL);
>
> Change the peer->maxpoll = NTP_MAXDPOLL
> to maxpoll = NTP_MAXDPOLL
>
> I haven't tried it yet, but that's clearly a bug.
Duh. Thanks for seeing that, and pointing it out. Fix pushed.
RGDS
GARY
---------------------------------------------------------------------------
Gary E. Miller Rellim 109 NW Wilmington Ave., Suite E, Bend, OR 97703
[email protected] Tel:+1 541 382 8588
pgpZR_fN2BChb.pgp
Description: OpenPGP digital signature
_______________________________________________ devel mailing list [email protected] http://lists.ntpsec.org/mailman/listinfo/devel
