Yo Hal! On Tue, 27 Sep 2016 12:59:44 -0700 Hal Murray <[email protected]> wrote:
> [email protected] said: > > It it called newpeer() there would be no problem. That function > > tests for NTP_MAXPOLL_UNK and handles the case. > > Yes, but your code is buggy. > > Here is your code (with a few lines omitted): > > if (NTP_MAXPOLL_UNK == maxpoll) > /* not set yet, set to default */ > peer->maxpoll = NTP_MAXDPOLL; > peer->maxpoll = max(maxpoll, NTP_MINPOLL); Yes, I know there is a bug, and I know that is from you patch (mostly), and it still looks good to me. So, got any patch ideas? RGDS GARY --------------------------------------------------------------------------- Gary E. Miller Rellim 109 NW Wilmington Ave., Suite E, Bend, OR 97703 [email protected] Tel:+1 541 382 8588
pgp0SKZf8YCQY.pgp
Description: OpenPGP digital signature
_______________________________________________ devel mailing list [email protected] http://lists.ntpsec.org/mailman/listinfo/devel
