]> git.ipfire.org Git - thirdparty/chrony.git/commitdiff
sourcestats: reset instance before loading dump file
authorMiroslav Lichvar <mlichvar@redhat.com>
Thu, 15 Feb 2018 16:10:45 +0000 (17:10 +0100)
committerMiroslav Lichvar <mlichvar@redhat.com>
Fri, 16 Feb 2018 10:09:54 +0000 (11:09 +0100)
Don't rely on the caller to reset the instance and always reset it
before loading data to make sure it can't get to an unexpected state.

sourcestats.c

index 2aec02cac59b66a8b4c1b2bf75aff2c1421b339c..9c1c4dec7a02d19b7fd8bb73566958a0132b1fad 100644 (file)
@@ -884,7 +884,7 @@ SST_LoadFromFile(SST_Stats inst, FILE *in)
   char line[1024];
   double weight;
 
-  assert(!inst->n_samples);
+  SST_ResetInstance(inst);
 
   if (fgets(line, sizeof(line), in) &&
       sscanf(line, "%d", &inst->n_samples) == 1 &&
@@ -933,7 +933,6 @@ SST_LoadFromFile(SST_Stats inst, FILE *in)
     return 1;
 
   inst->last_sample = inst->n_samples - 1;
-  inst->runs_samples = 0;
 
   find_min_delay_sample(inst);
   SST_DoNewRegression(inst);