fib: fix debug preprocessor directives 88/27788/2
authorMatthew Smith <mgsmith@netgate.com>
Thu, 2 Jul 2020 22:24:17 +0000 (17:24 -0500)
committerNeale Ranns <nranns@cisco.com>
Mon, 6 Jul 2020 08:35:59 +0000 (08:35 +0000)
commitee167e524c6244b400045907c322862bb4f42c6c
tree7af61a07e64ef66ad69a697d89fee00c6fcfedc2
parentfbb846cfa1f95d57159f9ad6cc429f6d9d0b8696
fib: fix debug preprocessor directives

Type: fix

Some debugging declarations were wrapped in an 'ifdef CLIB_DEBUG'.
This seems to always evaluate true because clib.h defines CLIB_DEBUG
to 0 if its not defined.

The result is that if a route table is added and a route is added to
it and then the table is deleted, VPP exits because the debug function
fib_table_assert_empty() gets called whether VPP was started using a
debug build or a release build.

Change the ifdef to 'if CLIB_DEBUG > 0'.

Change-Id: I357dc2c299e81b95244f2f7efaadb8e0de27627a
Signed-off-by: Matthew Smith <mgsmith@netgate.com>
src/vnet/fib/fib_entry.c
src/vnet/fib/fib_table.h
src/vnet/fib/ip4_fib.c
src/vnet/fib/ip6_fib.c