commit 20dcaf5f7044517512a38885f64244b7f24c4a18
author Patrick Rohr <prohr@google.com> Fri Feb 11 13:14:45 2022 +0100
committer Patrick Rohr <prohr@google.com> Fri Feb 11 13:24:51 2022 +0100
tree 1a45c93c9ee913ffd26dc218718a66b453916281
parent e46975857cceb53399395619283de6786522b178

Fix wrong nla_type for matchall

TCA_U32_ACT was missed when converting from the u32 filter to matchall.
Interestingly, cls_matchall does not seem to validate this type as the
kernel happily accepted this configuration (which absolutely should not
happen as TCA_U32_ACT is greater than __TCA_MATCHALL_MAX).

Bug: 218840346
Test: atest CtsNetTestCases:RateLimitTest
Change-Id: Ia24683cbd5fbd10084163db6e6a4415ec03f6f3f

staticlibs/native/tcutils/tcutils.cpp

diff --git a/staticlibs/native/tcutils/tcutils.cpp b/staticlibs/native/tcutils/tcutils.cpp
index 0e17f67..144a4c9 100644
--- a/staticlibs/native/tcutils/tcutils.cpp
+++ b/staticlibs/native/tcutils/tcutils.cpp
@@ -196,7 +196,7 @@
                 .acts = {
                     .attr = {
                         .nla_len = sizeof(mRequest.opt.acts),
-                        .nla_type = TCA_U32_ACT,
+                        .nla_type = TCA_MATCHALL_ACT,
                     },
                     .act1 = {
                         .attr = {