From ea1ba62aecc7116ab5764d5f0f1c33d4333a3bdf Mon Sep 17 00:00:00 2001 From: Guy Harris Date: Mon, 9 May 2016 16:51:29 -0700 Subject: [PATCH] Fix up the compare chain in nstime_delta(). The first case handles the two time stamps having the same seconds value, so, in the subsequent cases, they're guaranteed not to have the same seconds value; check for b->secs < a->secs, not for b->secs <= a->secs (the two tests will always get the same value, as b->secs != a->secs), to make it clearer what's being done. Change-Id: I6d3806237dae0ea12af92ea0344a31a2c5322b12 Reviewed-on: https://code.wireshark.org/review/15325 Reviewed-by: Guy Harris --- wsutil/nstime.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/wsutil/nstime.c b/wsutil/nstime.c index e7ef252dda..e4b2cab53e 100644 --- a/wsutil/nstime.c +++ b/wsutil/nstime.c @@ -96,7 +96,7 @@ void nstime_delta(nstime_t *delta, const nstime_t *b, const nstime_t *a ) can never result. */ delta->secs = 0; delta->nsecs = b->nsecs - a->nsecs; - } else if (b->secs <= a->secs) { + } else if (b->secs < a->secs) { /* The seconds part of b is less than the seconds part of a, so b is before a. -- 2.34.1