<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN" "http://www.w3.org/TR/REC-html40/loose.dtd">
<html lang="en">
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
<title>
GitLab
</title>
</head>
<body>
<style type="text/css">
img {
max-width: 100%; height: auto;
}
</style>
<div class="content">
<h3>
Eric S. Raymond pushed to branch master
at <a href="https://gitlab.com/NTPsec/ntpsec">NTPsec / ntpsec</a>
</h3>
<h4>
Commits:
</h4>
<ul>
<li>
<strong><a href="https://gitlab.com/NTPsec/ntpsec/commit/7a485cebc05af1befbbdc6b3d715184f4e9edf54">7a485ceb</a></strong>
<div>
<span>by Eric S. Raymond</span>
<i>at 2016-06-25T07:51:22-04:00</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap">Refactor to isolate network-packet test.</pre>
</li>
<li>
<strong><a href="https://gitlab.com/NTPsec/ntpsec/commit/5fe690248cd499da9969e5434c37095ed6a822d1">5fe69024</a></strong>
<div>
<span>by Eric S. Raymond</span>
<i>at 2016-06-25T08:06:07-04:00</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap">Mark every packet with a network-origin flag - replaces magic-address test.
Getting this wrong was where my previous series of commits went astray.</pre>
</li>
<li>
<strong><a href="https://gitlab.com/NTPsec/ntpsec/commit/af96f908064df5a6539be5df536351a505b3585e">af96f908</a></strong>
<div>
<span>by Eric S. Raymond</span>
<i>at 2016-06-25T08:50:50-04:00</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap">Use is_network_packet() to avoid magic-address matching on restrictions.</pre>
</li>
<li>
<strong><a href="https://gitlab.com/NTPsec/ntpsec/commit/ccf80ca034ff09e3974657e3bdcf2d47fd5704f7">ccf80ca0</a></strong>
<div>
<span>by Eric S. Raymond</span>
<i>at 2016-06-25T09:08:35-04:00</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap">Push ISREFCLOCKADDR() calls up to the config interpreter...
...to isolate them from the core logic. Yes, I got this wrong in the
reverted patch series.</pre>
</li>
<li>
<strong><a href="https://gitlab.com/NTPsec/ntpsec/commit/6eee19adff16c98a27e08b46e18149bffb588b11">6eee19ad</a></strong>
<div>
<span>by Eric S. Raymond</span>
<i>at 2016-06-25T09:14:27-04:00</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap">Document removal of most checking for magic address formats.
In the past, we told reference clocks from real peers by giving the
reference clocks an address of the form 127.127.t.u, where t is the
type and u is the unit number. In ntpd itself, the filtering that
used to be done based on this magic address prefix is now done
using the is_refclock_packet() test on incoming packets. The
remaining instances of magic-address testing are in the
configuration-language interpreter only and are used to prevent
inapropriate configuration commands from being applied to
refclock entries. They'll go away when the configuration syntax
is redesigned.
In theory, therefore, it would now be possible for ntpd to use a
server with an address in the 127.127.t.u range. In practice this
is probably a bad idea as it would confuse ntpq, which keeps some
of these prefix checks in order to be able to recognize clock packets
by address only (that being all it has to work with).
De-confusing ntpq will require some modifications to mode 6
response formats so that the response to a peer query conveys
*explicitly* whether it's a refclock. Even so, legacy ntpq
instances will still be confused.</pre>
</li>
<li>
<strong><a href="https://gitlab.com/NTPsec/ntpsec/commit/87f2bfb892de4215f62700e30aea99038f7d7fcd">87f2bfb8</a></strong>
<div>
<span>by Eric S. Raymond</span>
<i>at 2016-06-25T09:19:50-04:00</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap">In ntpq, don't assume the magic-clock-address convention...
...when reporting refids. This affects the rv report.</pre>
</li>
<li>
<strong><a href="https://gitlab.com/NTPsec/ntpsec/commit/b39ee109d30feccea448eabf1343eb3b7b9a047e">b39ee109</a></strong>
<div>
<span>by Eric S. Raymond</span>
<i>at 2016-06-25T09:21:25-04:00</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap">In ntpq, cooked version of the refid variable no longer...
...assumes the magic-refclock-address convention.</pre>
</li>
<li>
<strong><a href="https://gitlab.com/NTPsec/ntpsec/commit/783d211b94af58d8ca9bfb830c6fddae8de96249">783d211b</a></strong>
<div>
<span>by Eric S. Raymond</span>
<i>at 2016-06-25T12:22:23-04:00</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap">Consistently export driver type name as a clock variable...
...and don't export numeric type any more, as we're trying to make that
strictly an internal convention.</pre>
</li>
<li>
<strong><a href="https://gitlab.com/NTPsec/ntpsec/commit/29f26ba8e95c09c51cde358c43dab243c7428b7f">29f26ba8</a></strong>
<div>
<span>by Eric S. Raymond</span>
<i>at 2016-06-25T13:08:49-04:00</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap">In ntp_control, ship a 'displayname' variable for peers that are clocks.
This replaces the filtering on magic address that used to happen in ntpq.
It has the minor downside that a new ntpq displaying a peer spreadsheet
for an old ntpd will show refclocks by 127.127.t.u address.</pre>
</li>
<li>
<strong><a href="https://gitlab.com/NTPsec/ntpsec/commit/dbda243e2cc6d56eb43ef7f226082541a70e9097">dbda243e</a></strong>
<div>
<span>by Eric S. Raymond</span>
<i>at 2016-06-25T13:35:18-04:00</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap">Remove refnumtoa() and its tests.
This function is no longer needed, because shortname information for
clock drivers is now kept in the driver method table and exported
to ntpq via the displayname peer variable. This change eliminates
the last magic-address-mask test outside the configuration-file
interpreter.</pre>
</li>
<li>
<strong><a href="https://gitlab.com/NTPsec/ntpsec/commit/55331e6e4bb36371ec92de6c39aa9572de32a78a">55331e6e</a></strong>
<div>
<span>by Eric S. Raymond</span>
<i>at 2016-06-25T13:50:47-04:00</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap">Reduce the scope over which IS_REFCLOCKADR() is known and used.
Knowledge of the magic 127.127.t.u address convention for refclocks is
now confined to ntp_config.c and used only for interpreting
configuration syntax.
In theory we could now assign a refclock any unique IP address we
like; neither ntpd nor ntpq would care any more. In practice it's
probably best to continue assigning only 127.127.t.u addresses so
that (at least for a while) we refrain from confusing legacy ntpq
versions.
Together with the elimination of refnumtoa(), this sets the stage
for entirely removing the external visibility of numeric driver
types, and a better clock-configuration syntax.</pre>
</li>
<li>
<strong><a href="https://gitlab.com/NTPsec/ntpsec/commit/65e31229089e01426391834f9cd45caf621d2574">65e31229</a></strong>
<div>
<span>by Eric S. Raymond</span>
<i>at 2016-06-26T00:08:27-04:00</i>
</div>
<pre class="commit-message" style="white-space: pre-wrap">Fix glitches detected by Coverity.</pre>
</li>
</ul>
<h4>30 changed files:</h4>
<ul>
<li class="file-stats">
<a href="#diff-0" style="text-decoration: none">
devel/ChangeLog
</a>
</li>
<li class="file-stats">
<a href="#diff-1" style="text-decoration: none">
docs/includes/ntpq-body.txt
</a>
</li>
<li class="file-stats">
<a href="#diff-2" style="text-decoration: none">
include/ntp_net.h
</a>
</li>
<li class="file-stats">
<a href="#diff-3" style="text-decoration: none">
include/ntp_refclock.h
</a>
</li>
<li class="file-stats">
<a href="#diff-4" style="text-decoration: none">
include/ntp_stdlib.h
</a>
</li>
<li class="file-stats">
<a href="#diff-5" style="text-decoration: none">
include/ntpd.h
</a>
</li>
<li class="file-stats">
<a href="#diff-6" style="text-decoration: none">
include/recvbuff.h
</a>
</li>
<li class="file-stats">
<a href="#diff-7" style="text-decoration: none">
<span class="deleted-file" style="color: #b00">
−
libntp/refnumtoa.c
</span>
</a>
</li>
<li class="file-stats">
<a href="#diff-8" style="text-decoration: none">
libntp/wscript
</a>
</li>
<li class="file-stats">
<a href="#diff-9" style="text-decoration: none">
ntpd/ntp_config.c
</a>
</li>
<li class="file-stats">
<a href="#diff-10" style="text-decoration: none">
ntpd/ntp_control.c
</a>
</li>
<li class="file-stats">
<a href="#diff-11" style="text-decoration: none">
ntpd/ntp_io.c
</a>
</li>
<li class="file-stats">
<a href="#diff-12" style="text-decoration: none">
ntpd/ntp_peer.c
</a>
</li>
<li class="file-stats">
<a href="#diff-13" style="text-decoration: none">
ntpd/ntp_proto.c
</a>
</li>
<li class="file-stats">
<a href="#diff-14" style="text-decoration: none">
ntpd/ntp_refclock.c
</a>
</li>
<li class="file-stats">
<a href="#diff-15" style="text-decoration: none">
ntpd/ntp_restrict.c
</a>
</li>
<li class="file-stats">
<a href="#diff-16" style="text-decoration: none">
ntpd/refclock_jjy.c
</a>
</li>
<li class="file-stats">
<a href="#diff-17" style="text-decoration: none">
ntpd/refclock_neoclock4x.c
</a>
</li>
<li class="file-stats">
<a href="#diff-18" style="text-decoration: none">
ntpd/refclock_parse.c
</a>
</li>
<li class="file-stats">
<a href="#diff-19" style="text-decoration: none">
ntpd/refclock_shm.c
</a>
</li>
<li class="file-stats">
<a href="#diff-20" style="text-decoration: none">
ntpd/refclock_true.c
</a>
</li>
<li class="file-stats">
<a href="#diff-21" style="text-decoration: none">
ntpd/refclock_zyfer.c
</a>
</li>
<li class="file-stats">
<a href="#diff-22" style="text-decoration: none">
ntpq/ntpq-subs.c
</a>
</li>
<li class="file-stats">
<a href="#diff-23" style="text-decoration: none">
ntpq/ntpq.c
</a>
</li>
<li class="file-stats">
<a href="#diff-24" style="text-decoration: none">
ports/winnt/vs2005/libntp.vcproj
</a>
</li>
<li class="file-stats">
<a href="#diff-25" style="text-decoration: none">
ports/winnt/vs2008/libntp/libntp.vcproj
</a>
</li>
<li class="file-stats">
<a href="#diff-26" style="text-decoration: none">
ports/winnt/vs2013/libntp/libntp.vcxproj
</a>
</li>
<li class="file-stats">
<a href="#diff-27" style="text-decoration: none">
ports/winnt/vs2013/libntp/libntp.vcxproj.filters
</a>
</li>
<li class="file-stats">
<a href="#diff-28" style="text-decoration: none">
tests/common/tests_main.c
</a>
</li>
<li class="file-stats">
<a href="#diff-29" style="text-decoration: none">
<span class="deleted-file" style="color: #b00">
−
tests/libntp/refnumtoa.c
</span>
</a>
</li>
</ul>
<h5>The diff was not included because it is too large.</h5>
</div>
<div class="footer" style="margin-top: 10px">
<p style="color: #777; font-size: small">
—
<br>
<a href="https://gitlab.com/NTPsec/ntpsec/compare/920df5b80b52dd21a764460fda9ea24d1bf7aa60...65e31229089e01426391834f9cd45caf621d2574">View it on GitLab</a>.
<br>
You're receiving this email because of your account on gitlab.com.
If you'd like to receive fewer emails, you can
adjust your notification settings.
</p>
</div>
</body>
</html>