emo/busybox
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

ntpd: abs(tmx.offset) was truncating a "long" typed value

Browse Source 
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
Denys Vlasenko 2019-10-25 17:35:58 +02:00
parent b4ad1d86b6
commit d6ff27de15
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
networking/ntpd.c
View File

@ -1876,9 +1876,11 @@ update_local_clock(peer_t *p)
//15:31:53.473 update from:<IP> offset:+0.000007 delay:0.158142 jitter:0.010922 clock drift:+9.343ppm tc:6 //15:31:53.473 update from:<IP> offset:+0.000007 delay:0.158142 jitter:0.010922 clock drift:+9.343ppm tc:6
//15:32:58.902 update from:<IP> offset:-0.000728 delay:0.158222 jitter:0.009454 clock drift:+9.298ppm tc:6 //15:32:58.902 update from:<IP> offset:-0.000728 delay:0.158222 jitter:0.009454 clock drift:+9.298ppm tc:6
/* /*
* This expression would choose MIN_FREQHOLD + 8 in the above example. * This expression would choose MIN_FREQHOLD + 8 in the above example
* (off_065 is +1 for each 0.065536 seconds of offset).
*/ */
G.FREQHOLD_cnt = 1 + MIN_FREQHOLD + ((unsigned)(abs(tmx.offset)) >> 16); unsigned off_065 = abs((int)(tmx.offset >> 16));
G.FREQHOLD_cnt = 1 + MIN_FREQHOLD + off_065;
} }
G.FREQHOLD_cnt--; G.FREQHOLD_cnt--;
tmx.status |= STA_FREQHOLD; tmx.status |= STA_FREQHOLD;