diff options
author | Guy Harris <guy@alum.mit.edu> | 2016-05-09 16:51:29 -0700 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2016-05-09 23:52:33 +0000 |
commit | ea1ba62aecc7116ab5764d5f0f1c33d4333a3bdf (patch) | |
tree | ed59ee16c5c6dad24632bbe1b7ffb08c89c1e15a /wsutil/nstime.c | |
parent | 9a8a454b0a06ffac377663161e272609adc84c46 (diff) |
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 <guy@alum.mit.edu>
Diffstat (limited to 'wsutil/nstime.c')
-rw-r--r-- | wsutil/nstime.c | 2 |
1 files changed, 1 insertions, 1 deletions
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. |