* [PATCH net-next] fib: fix fib_rules_ops indirect calls wrappers
@ 2020-07-29 18:10 Brian Vazquez
2020-07-29 20:27 ` David Miller
0 siblings, 1 reply; 2+ messages in thread
From: Brian Vazquez @ 2020-07-29 18:10 UTC (permalink / raw)
To: Brian Vazquez, Brian Vazquez, Eric Dumazet, Paolo Abeni,
David S . Miller
Cc: linux-kernel, netdev, Stephen Rothwell
This patch fixes:
commit b9aaec8f0be5 ("fib: use indirect call wrappers in the most common
fib_rules_ops") which didn't consider the case when
CONFIG_IPV6_MULTIPLE_TABLES is not set.
Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Fixes: b9aaec8f0be5 ("fib: use indirect call wrappers in the most common fib_rules_ops")
Signed-off-by: Brian Vazquez <brianvv@google.com>
---
net/core/fib_rules.c | 33 ++++++++++++++++++++-------------
1 file changed, 20 insertions(+), 13 deletions(-)
diff --git a/net/core/fib_rules.c b/net/core/fib_rules.c
index e7a8f87b0bb2b..fce645f6b9b10 100644
--- a/net/core/fib_rules.c
+++ b/net/core/fib_rules.c
@@ -16,6 +16,13 @@
#include <net/ip_tunnels.h>
#include <linux/indirect_call_wrapper.h>
+#ifdef CONFIG_IPV6_MULTIPLE_TABLES
+#define INDIRECT_CALL_MT(f, f2, f1, ...) \
+ INDIRECT_CALL_INET(f, f2, f1, __VA_ARGS__)
+#else
+#define INDIRECT_CALL_MT(f, f2, f1, ...) INDIRECT_CALL_1(f, f1, __VA_ARGS__)
+#endif
+
static const struct fib_kuid_range fib_kuid_range_unset = {
KUIDT_INIT(0),
KUIDT_INIT(~0),
@@ -268,10 +275,10 @@ static int fib_rule_match(struct fib_rule *rule, struct fib_rules_ops *ops,
uid_gt(fl->flowi_uid, rule->uid_range.end))
goto out;
- ret = INDIRECT_CALL_INET(ops->match,
- fib6_rule_match,
- fib4_rule_match,
- rule, fl, flags);
+ ret = INDIRECT_CALL_MT(ops->match,
+ fib6_rule_match,
+ fib4_rule_match,
+ rule, fl, flags);
out:
return (rule->flags & FIB_RULE_INVERT) ? !ret : ret;
}
@@ -302,15 +309,15 @@ int fib_rules_lookup(struct fib_rules_ops *ops, struct flowi *fl,
} else if (rule->action == FR_ACT_NOP)
continue;
else
- err = INDIRECT_CALL_INET(ops->action,
- fib6_rule_action,
- fib4_rule_action,
- rule, fl, flags, arg);
-
- if (!err && ops->suppress && INDIRECT_CALL_INET(ops->suppress,
- fib6_rule_suppress,
- fib4_rule_suppress,
- rule, arg))
+ err = INDIRECT_CALL_MT(ops->action,
+ fib6_rule_action,
+ fib4_rule_action,
+ rule, fl, flags, arg);
+
+ if (!err && ops->suppress && INDIRECT_CALL_MT(ops->suppress,
+ fib6_rule_suppress,
+ fib4_rule_suppress,
+ rule, arg))
continue;
if (err != -EAGAIN) {
--
2.28.0.rc0.142.g3c755180ce-goog
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH net-next] fib: fix fib_rules_ops indirect calls wrappers
2020-07-29 18:10 [PATCH net-next] fib: fix fib_rules_ops indirect calls wrappers Brian Vazquez
@ 2020-07-29 20:27 ` David Miller
0 siblings, 0 replies; 2+ messages in thread
From: David Miller @ 2020-07-29 20:27 UTC (permalink / raw)
To: brianvv; +Cc: brianvv.kernel, edumazet, pabeni, linux-kernel, netdev, sfr
From: Brian Vazquez <brianvv@google.com>
Date: Wed, 29 Jul 2020 11:10:18 -0700
> This patch fixes:
> commit b9aaec8f0be5 ("fib: use indirect call wrappers in the most common
> fib_rules_ops") which didn't consider the case when
> CONFIG_IPV6_MULTIPLE_TABLES is not set.
>
> Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
> Fixes: b9aaec8f0be5 ("fib: use indirect call wrappers in the most common fib_rules_ops")
> Signed-off-by: Brian Vazquez <brianvv@google.com>
Applied, thanks Brian.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-07-29 20:27 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-29 18:10 [PATCH net-next] fib: fix fib_rules_ops indirect calls wrappers Brian Vazquez
2020-07-29 20:27 ` David Miller
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.