]> git.baikalelectronics.ru Git - kernel.git/commitdiff
selftests: fib rule: Drop erroneous TABLE variable
authorGuillaume Nault <gnault@redhat.com>
Mon, 31 Jan 2022 15:42:04 +0000 (16:42 +0100)
committerDavid S. Miller <davem@davemloft.net>
Tue, 1 Feb 2022 14:14:47 +0000 (14:14 +0000)
The fib_rule6_del_by_pref() and fib_rule4_del_by_pref() functions use
an uninitialised $TABLE variable. They should use $RTABLE instead.
This doesn't alter the result of the test, as it just makes the grep
command less specific (but since the script always uses the same table
number, that doesn't really matter).

Let's fix it anyway and, while there, specify the filtering parameters
directly in 'ip -X rule show' to avoid the extra grep command entirely.

Signed-off-by: Guillaume Nault <gnault@redhat.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
tools/testing/selftests/net/fib_rule_tests.sh

index f04f337dd7c6daf915918dc02966b1a1d65f0a31..012f9385d68cd1ac8a4f69f63bf47a9ba2327597 100755 (executable)
@@ -96,7 +96,7 @@ fib_rule6_del()
 
 fib_rule6_del_by_pref()
 {
-       pref=$($IP -6 rule show | grep "$1 lookup $TABLE" | cut -d ":" -f 1)
+       pref=$($IP -6 rule show $1 table $RTABLE | cut -d ":" -f 1)
        $IP -6 rule del pref $pref
 }
 
@@ -168,7 +168,7 @@ fib_rule4_del()
 
 fib_rule4_del_by_pref()
 {
-       pref=$($IP rule show | grep "$1 lookup $TABLE" | cut -d ":" -f 1)
+       pref=$($IP rule show $1 table $RTABLE | cut -d ":" -f 1)
        $IP rule del pref $pref
 }