From cacb4a428715b6a77ddee50d87d5dffe1f83557c Mon Sep 17 00:00:00 2001 From: Uli Heilmeier Date: Sat, 24 Sep 2016 11:17:17 +0200 Subject: Editcap: Fix comparsion between signed and unsigned int secs_per_block is a unsigned integer. Therefore we can't compare a signed integer with it. Change-Id: Ic965a88e0e704e296c4af9193ca4a3504173ae36 Reviewed-on: https://code.wireshark.org/review/17901 Reviewed-by: Peter Wu Petri-Dish: Peter Wu Tested-by: Petri Dish Buildbot Reviewed-by: Michael Mann --- editcap.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'editcap.c') diff --git a/editcap.c b/editcap.c index fb7532d9d3..40a6a81746 100644 --- a/editcap.c +++ b/editcap.c @@ -1399,8 +1399,8 @@ main(int argc, char *argv[]) block_start = phdr->ts; } if (secs_per_block != 0) { - while ((phdr->ts.secs - block_start.secs > secs_per_block) - || (phdr->ts.secs - block_start.secs == secs_per_block + while (((guint32)(phdr->ts.secs - block_start.secs) > secs_per_block) + || ((guint32)(phdr->ts.secs - block_start.secs) == secs_per_block && phdr->ts.nsecs >= block_start.nsecs )) { /* time for the next file */ if (!wtap_dump_close(pdh, &write_err)) { -- cgit v1.2.1