sourcestats: clamp minsamples and maxsamples in initialization

Don't leave the variables set to values outside their effective range.
This has no functional impact, but makes it clear what is the precedence
of the two settings.

diff --git a/sourcestats.c b/sourcestats.c
index 0f2dedb816ab952fd85d2a92c843ac4962ce305a..986956f84a6be0fda764ceb53b2b99977e49eb9e 100644 (file)
--- a/sourcestats.c
+++ b/sourcestats.c
@@ -211,8 +211,8 @@ SST_CreateInstance(uint32_t refid, IPAddr *addr, int min_samples, int max_sample
   SST_Stats inst;
   inst = MallocNew(struct SST_Stats_Record);
 
-  inst->min_samples = min_samples;
-  inst->max_samples = max_samples;
+  inst->max_samples = max_samples > 0 ? CLAMP(1, max_samples, MAX_SAMPLES) : MAX_SAMPLES;
+  inst->min_samples = CLAMP(1, min_samples, inst->max_samples);
   inst->fixed_min_delay = min_delay;
   inst->fixed_asymmetry = asymmetry;