summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorNeale Ranns <neale@graphiant.com>2022-03-25 08:34:53 +0000
committerNeale Ranns <neale@graphiant.com>2022-04-08 12:09:45 +0000
commitba2f55154c4ae91a98348ac48a0d11f0de9772db (patch)
treebfed1cb0b249f05dfe856104a6820599a015503d
parent1cbcdcef20b7950e91d63ecf61c9bb1586e1e0b2 (diff)
api: Do not assert of short message lengths
Type: improvement Short message lengths are correctly handled by the code, asserting makes unit tests that verify this behaviour (e.g. test_ip_punt_api_validation) fail/crash with a debug image. Signed-off-by: Neale Ranns <neale@graphiant.com> Change-Id: Ib3a377e5d49fde0eee252b92f0e58a8a0d0d83ec
-rw-r--r--src/vlibapi/api_shared.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/src/vlibapi/api_shared.c b/src/vlibapi/api_shared.c
index 77278431fb7..17560e5de23 100644
--- a/src/vlibapi/api_shared.c
+++ b/src/vlibapi/api_shared.c
@@ -551,7 +551,6 @@ msg_handler_internal (api_main_t *am, void *the_msg, uword msg_len,
if (calc_size_fp)
{
calc_size = (*calc_size_fp) (the_msg);
- ASSERT (calc_size <= msg_len);
if (calc_size > msg_len)
{
clib_warning (