Skip to content

Commit 44741e9

Browse files
idoschPaolo Abeni
authored andcommitted
selftests: fib_nexthops: Add test cases for error routes deletion
Add test cases that check that error routes (e.g., blackhole) are deleted when their nexthop is deleted. Output without "ipv4: Fix reference count leak when using error routes with nexthop objects": # ./fib_nexthops.sh -t "ipv4_fcnal ipv6_fcnal" IPv4 functional ---------------------- [...] WARNING: Unexpected route entry TEST: Error route removed on nexthop deletion [FAIL] IPv6 ---------------------- [...] TEST: Error route removed on nexthop deletion [ OK ] Tests passed: 20 Tests failed: 1 Tests skipped: 0 Output with "ipv4: Fix reference count leak when using error routes with nexthop objects": # ./fib_nexthops.sh -t "ipv4_fcnal ipv6_fcnal" IPv4 functional ---------------------- [...] TEST: Error route removed on nexthop deletion [ OK ] IPv6 ---------------------- [...] TEST: Error route removed on nexthop deletion [ OK ] Tests passed: 21 Tests failed: 0 Tests skipped: 0 Signed-off-by: Ido Schimmel <[email protected]> Reviewed-by: David Ahern <[email protected]> Link: https://patch.msgid.link/[email protected] Signed-off-by: Paolo Abeni <[email protected]>
1 parent ac782f4 commit 44741e9

1 file changed

Lines changed: 15 additions & 0 deletions

File tree

tools/testing/selftests/net/fib_nexthops.sh

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -800,6 +800,14 @@ ipv6_fcnal()
800800
set +e
801801
check_nexthop "dev veth1" ""
802802
log_test $? 0 "Nexthops removed on admin down"
803+
804+
# error routes should be deleted when their nexthop is deleted
805+
run_cmd "$IP li set dev veth1 up"
806+
run_cmd "$IP -6 nexthop add id 58 dev veth1"
807+
run_cmd "$IP ro add blackhole 2001:db8:101::1/128 nhid 58"
808+
run_cmd "$IP nexthop del id 58"
809+
check_route6 "2001:db8:101::1" ""
810+
log_test $? 0 "Error route removed on nexthop deletion"
803811
}
804812

805813
ipv6_grp_refs()
@@ -1459,6 +1467,13 @@ ipv4_fcnal()
14591467

14601468
run_cmd "$IP ro del 172.16.102.0/24"
14611469
log_test $? 0 "Delete route when not specifying nexthop attributes"
1470+
1471+
# error routes should be deleted when their nexthop is deleted
1472+
run_cmd "$IP nexthop add id 23 dev veth1"
1473+
run_cmd "$IP ro add blackhole 172.16.102.100/32 nhid 23"
1474+
run_cmd "$IP nexthop del id 23"
1475+
check_route "172.16.102.100" ""
1476+
log_test $? 0 "Error route removed on nexthop deletion"
14621477
}
14631478

14641479
ipv4_grp_fcnal()

0 commit comments

Comments
 (0)