diff options
| author |   Alexander Aring <aring@mojatatu.com> | 2018-02-15 10:54:59 -0500 |
|---|---|---|
| committer |   David S. Miller <davem@davemloft.net> | 2018-02-16 16:05:50 -0500 |
| commit | b36201455aa0749e8708ef97ed9c1c9ece29a113 (patch) | |
| tree | 9574879f74b7aad2cf3616a99a041f58916c6294 /net/sched/act_ipt.c | |
| parent | net: sched: act: add extack for walk callback (diff) | |
| download | linux-dev-b36201455aa0749e8708ef97ed9c1c9ece29a113.tar.xz linux-dev-b36201455aa0749e8708ef97ed9c1c9ece29a113.zip | |
net: sched: act: handle extack in tcf_generic_walker
This patch adds extack handling for a common used TC act function
"tcf_generic_walker()" to add an extack message on failures.
The tcf_generic_walker() function can fail if get a invalid command
different than DEL and GET. The naming "action" here is wrong, the
correct naming would be command.
Cc: David Ahern <dsahern@gmail.com>
Signed-off-by: Alexander Aring <aring@mojatatu.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to '')
| -rw-r--r-- | net/sched/act_ipt.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/net/sched/act_ipt.c b/net/sched/act_ipt.c index f33a8cc5dee6..9784629090ad 100644 --- a/net/sched/act_ipt.c +++ b/net/sched/act_ipt.c @@ -308,7 +308,7 @@ static int tcf_ipt_walker(struct net *net, struct sk_buff *skb, { struct tc_action_net *tn = net_generic(net, ipt_net_id); - return tcf_generic_walker(tn, skb, cb, type, ops); + return tcf_generic_walker(tn, skb, cb, type, ops, extack); } static int tcf_ipt_search(struct net *net, struct tc_action **a, u32 index, @@ -358,7 +358,7 @@ static int tcf_xt_walker(struct net *net, struct sk_buff *skb, { struct tc_action_net *tn = net_generic(net, xt_net_id); - return tcf_generic_walker(tn, skb, cb, type, ops); + return tcf_generic_walker(tn, skb, cb, type, ops, extack); } static int tcf_xt_search(struct net *net, struct tc_action **a, u32 index, |