From 0403751060a5445b7c62566265d596a5cb49f158 Mon Sep 17 00:00:00 2001 From: dmiller Date: Sun, 29 Dec 2019 15:52:59 +0000 Subject: [PATCH] Be more clear for static analysis (LGTM) that this does not overflow. #1834 --- nping/common_modified.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/nping/common_modified.cc b/nping/common_modified.cc index 1fb1c0de2..54a772e10 100644 --- a/nping/common_modified.cc +++ b/nping/common_modified.cc @@ -201,7 +201,7 @@ int TargetGroup::rewind() { * get_next_host should be used for skipping the last octet :-) * returns: number of hosts skipped */ int TargetGroup::skip_range(_octet_nums octet) { - unsigned long hosts_skipped = 0, /* number of hosts skipped */ + u32 hosts_skipped = 0, /* number of hosts skipped */ oct = 0; /* octect number */ int i = 0; /* simple lcv */ @@ -212,11 +212,11 @@ int TargetGroup::skip_range(_octet_nums octet) { switch (octet) { case FIRST_OCTET: oct = 0; - hosts_skipped = (last[1] + 1) * (last[2] + 1) * (last[3] + 1); + hosts_skipped = (u32)(last[1] + 1) * (last[2] + 1) * (last[3] + 1); break; case SECOND_OCTET: oct = 1; - hosts_skipped = (last[2] + 1) * (last[3] + 1); + hosts_skipped = (u32)(last[2] + 1) * (last[3] + 1); break; case THIRD_OCTET: oct = 2;