commit fbb846cfa1f95d57159f9ad6cc429f6d9d0b8696
author Florin Coras <fcoras@cisco.com> Thu Jul 02 14:52:34 2020 -0700
committer Florin Coras <fcoras@cisco.com> Thu Jul 02 14:52:34 2020 -0700
tree 1cd9371a3701cb0d2a024a7db83d1405d97a5764
parent 3e9ff291da7e76bfa2fe6a15a7842edb9933f8b8

tcp: fix coverity warning

Type: fix

Signed-off-by: Florin Coras <fcoras@cisco.com>
Change-Id: I60828bc084b9adbb925c306d26060412c410c516

src/vnet/tcp/tcp_cli.c

diff --git a/src/vnet/tcp/tcp_cli.c b/src/vnet/tcp/tcp_cli.c
index ada2ac0..b76b404 100644
--- a/src/vnet/tcp/tcp_cli.c
+++ b/src/vnet/tcp/tcp_cli.c
@@ -907,7 +907,7 @@
   int max_index, i;
 
   n_elts = pool_elts (tm->half_open_connections);
-  max_index = pool_len (tm->half_open_connections) - 1;
+  max_index = clib_max (pool_len (tm->half_open_connections), 1) - 1;
   if (verbose && end == ~0 && n_elts > 50)
     {
       vlib_cli_output (vm, "Too many connections, use range <start> <end>");
@@ -966,7 +966,7 @@
     {
       if (unformat (line_input, "range %u %u", &start, &end))
 	;
-      else if (unformat (line_input, "verbose %d"), &verbose)
+      else if (unformat (line_input, "verbose %d", &verbose))
 	;
       else if (unformat (line_input, "verbose"))
 	verbose = 1;