examples/l2fwd-keepalive: fix integer overflow
authorRemy Horton <remy.horton@intel.com>
Thu, 10 Dec 2015 09:50:07 +0000 (09:50 +0000)
committerThomas Monjalon <thomas.monjalon@6wind.com>
Thu, 10 Dec 2015 21:27:20 +0000 (22:27 +0100)
Fix Coverity warning with potential 32-bit integer multiplication overflow
where final result is expected to be 64-bit.

>>> CID 120144 (#1 of 1): Unintentional integer overflow (OVERFLOW_BEFORE_WIDEN)

Fixes: e64833f2273a ("examples/l2fwd-keepalive: add sample application")

Signed-off-by: Remy Horton <remy.horton@intel.com>
Acked-by: John McNamara <john.mcnamara@intel.com>
examples/l2fwd-keepalive/main.c

index 8d7b09e..f4d52f2 100644 (file)
@@ -471,7 +471,7 @@ l2fwd_parse_args(int argc, char **argv)
                /* timer period */
                case 'T':
                        timer_period = l2fwd_parse_timer_period(optarg)
-                               * 1000 * TIMER_MILLISECOND;
+                               * (int64_t)(1000 * TIMER_MILLISECOND);
                        if (timer_period < 0) {
                                printf("invalid timer period\n");
                                l2fwd_usage(prgname);