]> git.ipfire.org Git - thirdparty/chrony.git/commitdiff
sources: don't save SST_GetSelectionData() status
authorMiroslav Lichvar <mlichvar@redhat.com>
Mon, 3 Nov 2025 11:51:00 +0000 (12:51 +0100)
committerMiroslav Lichvar <mlichvar@redhat.com>
Tue, 4 Nov 2025 13:47:35 +0000 (14:47 +0100)
There is no need to save the SST_GetSelectionData() "select_ok" status
as the source is immediately marked as SRC_BAD_STATS if it is not ok.
Nothing else is using this information.

sources.c

index c840e1b9fb4fa1002fd0b178d64669e853d1bd35..7f6002ecc0b882edf1bb6ae627b360060770b512 100644 (file)
--- a/sources.c
+++ b/sources.c
@@ -54,7 +54,6 @@ static int initialised = 0;
 /* ================================================== */
 /* Structure used to hold info for selecting between sources */
 struct SelectInfo {
-  int select_ok;
   double std_dev;
   double root_distance;
   double lo_limit;
@@ -1010,12 +1009,10 @@ SRC_SelectSource(SRC_Instance updated_inst)
       n_unreach_sources++;
 
     si = &sources[i]->sel_info;
-    si->select_ok = SST_GetSelectionData(sources[i]->stats, &now,
-                                         &si->lo_limit, &si->hi_limit, &si->root_distance,
-                                         &si->std_dev, &first_sample_ago,
-                                         &si->last_sample_ago);
 
-    if (!si->select_ok) {
+    if (!SST_GetSelectionData(sources[i]->stats, &now, &si->lo_limit, &si->hi_limit,
+                              &si->root_distance, &si->std_dev, &first_sample_ago,
+                              &si->last_sample_ago)) {
       ++n_badstats_sources;
       mark_source(sources[i], SRC_BAD_STATS);
       if (max_badstat_reach < sources[i]->reachability)