X-Git-Url: https://gerrit.fd.io/r/gitweb?a=blobdiff_plain;f=src%2Fvppinfra%2Fbihash_template.c;h=d8b97b5f3d08b8319eaf12f2cf4ed426b7d6747e;hb=eab38d91e8db5ad271598a63781a7afa3bd8b5ea;hp=7c817a205890aa2ed0099ef9bd85ac0699ad77c9;hpb=5e6b9580f202188cbe158368bdbe35c3f39973d7;p=vpp.git

diff --git a/src/vppinfra/bihash_template.c b/src/vppinfra/bihash_template.c
index 7c817a20589..d8b97b5f3d0 100644
--- a/src/vppinfra/bihash_template.c
+++ b/src/vppinfra/bihash_template.c
@@ -199,7 +199,7 @@ BV (split_and_rehash_linear)
       /* Find a free slot in the new linear scan bucket */
       for (; j < new_length; j++)
 	{
-	  /* Old value in use? Forget it. */
+	  /* Old value not in use? Forget it. */
 	  if (BV (clib_bihash_is_free) (&(old_values->kvp[i])))
 	    goto doublebreak;
 
@@ -212,11 +212,12 @@ BV (split_and_rehash_linear)
 	      j++;
 	      goto doublebreak;
 	    }
-	  /* This should never happen... */
-	  clib_warning ("BUG: linear rehash failed!");
-	  BV (value_free) (h, new_values);
-	  return 0;
 	}
+      /* This should never happen... */
+      clib_warning ("BUG: linear rehash failed!");
+      BV (value_free) (h, new_values);
+      return 0;
+
     doublebreak:;
     }
   return new_values;