faraphel/chrony
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

ntp: don't save timestamps if transmit_packet() failed

Browse source 
Don't save server RX and TX timestamp to clientlog if the transmission
or authentication failed (e.g. packet is handled in ntp_signd). They
will not be needed.
This commit is contained in:
Miroslav Lichvar 2021-10-25 10:23:41 +02:00
parent c5dde9b66a
commit 189bf9c536
1 changed files with 5 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
ntp_core.c
View file

@ -2146,10 +2146,11 @@ NCR_ProcessRxUnknown(NTP_Remote_Address *remote_addr, NTP_Local_Address *local_a
version = info.version; version = info.version;
/* Send a reply */ /* Send a reply */
transmit_packet(my_mode, interleaved, poll, version, kod, NULL, if (!transmit_packet(my_mode, interleaved, poll, version, kod, NULL,
&message->receive_ts, &message->transmit_ts, &message->receive_ts, &message->transmit_ts,
rx_ts, tx_ts, local_ntp_rx, NULL, remote_addr, local_addr, rx_ts, tx_ts, local_ntp_rx, NULL, remote_addr, local_addr,
message, &info); message, &info))
return;
if (local_ntp_rx) if (local_ntp_rx)
CLG_SaveNtpTimestamps(local_ntp_rx, tx_ts ? &tx_ts->ts : NULL); CLG_SaveNtpTimestamps(local_ntp_rx, tx_ts ? &tx_ts->ts : NULL);