From 5b3e04c74f188dbe6c4edab57e555ee3a01fb8f7 Mon Sep 17 00:00:00 2001 From: Vladislav Grishenko Date: Mon, 5 Sep 2022 10:32:46 +0500 Subject: nat: fix nat44_ed set_session_limit crash Setting session limit should return error for unknown fib. Optimize max_translations_per_fib expanding and drop unnecessary trailing fib entry. Type: fix Change-Id: Ie7d2b363ade48f53598faa617a49cce7b2db6400 Signed-off-by: Vladislav Grishenko --- test/test_nat44_ed.py | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) (limited to 'test/test_nat44_ed.py') diff --git a/test/test_nat44_ed.py b/test/test_nat44_ed.py index 323249e4424..d4dd4be5d89 100644 --- a/test/test_nat44_ed.py +++ b/test/test_nat44_ed.py @@ -2939,10 +2939,13 @@ class TestNAT44EDMW(TestNAT44ED): limit = 5 - # 2 interfaces pg0, pg1 (vrf10, limit 1 tcp session) - # non existing vrf_id makes process core dump + # 2 interfaces pg0, pg1 (vrf10, limit 5 tcp sessions) self.vapi.nat44_set_session_limit(session_limit=limit, vrf_id=10) + # expect error when bad is specified + with self.vapi.assert_negative_api_retval(): + self.vapi.nat44_set_session_limit(session_limit=limit, vrf_id=20) + self.nat_add_inside_interface(inside) self.nat_add_inside_interface(inside_vrf10) self.nat_add_outside_interface(outside) -- cgit 1.2.3-korg