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 f04f337..012f938 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
 }