ntp_refclock.c, ChangeLog:

  [Bug 1141] refclock_report missing braces cause spurious 'peer event: clock clk_unspec' log entries

bk: 49cdb7aez9k9B9C5T_Rryb4-qPjWtw

diff --git a/ChangeLog b/ChangeLog
index 410d730328edaa18fd3e4d569d16516c5c1370cf..73be22ea7a9bb974cfc7d498d8e4467e761e7d68 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,10 +1,13 @@
+* [Bug 1141] refclock_report missing braces cause spurious "peer event:
+  clock clk_unspec" log entries
 * [Bug 873] Windows serial refclock proper TTY line discipline emulation
-* [Bug 1117] Deferred interface binding under Windows works only correctly
-  if FORCE_DNSRETRY is defined
+* [Bug 1117] Deferred interface binding under Windows works only 
+  correctly if FORCE_DNSRETRY is defined
 * [BUG 1124] Lock QueryPerformanceCounter() client threads to same CPU
-* [Bug 1014] Enable building with VC9 (in Visual Studio 2008, Visual C++ 2008, or SDK)
-* DPRINTF macro made safer, always evaluates to a statement and will not
-  misassociate an else which follows the macro
+* [Bug 1014] Enable building with VC9 (in Visual Studio 2008, Visual
+  C++ 2008, or SDK)
+* DPRINTF macro made safer, always evaluates to a statement and will
+  not misassociate an else which follows the macro
 (4.2.5p158) 2009/01/30 Released by Harlan Stenn <stenn@ntp.org>
 * Fix [CID 72], a typo introduced at the latest fix to prettydate.c.
 (4.2.5p157) 2009/01/26 Released by Harlan Stenn <stenn@ntp.org>

diff --git a/ntpd/ntp_refclock.c b/ntpd/ntp_refclock.c
index 252711e428306f12109c7f987c23c523b2962a25..7785f440471846e8c65dd002066670f3cdcc01a8 100644 (file)
--- a/ntpd/ntp_refclock.c
+++ b/ntpd/ntp_refclock.c
@@ -136,9 +136,10 @@ refclock_report(
        }
        if (pp->lastevent < 15)
                pp->lastevent++;
-       if (pp->currentstatus != code)
+       if (pp->currentstatus != code) {
                pp->currentstatus = code;
                report_event(PEVNT_CLOCK, peer, ceventstr(code));
+       }
 }