On 24/07/18 04:06 PM, Paolo Abeni wrote:

--- a/net/sched/act_api.c
+++ b/net/sched/act_api.c
@@ -895,6 +895,11 @@ struct tc_action *tcf_action_init_1(struct net *net, 
struct tcf_proto *tp,
                }
        }
+ if (a->tcfa_action == TC_ACT_REDIRECT) {
+               net_warn_ratelimited("TC_ACT_REDIRECT can't be used directly");
+               a->tcfa_action = TC_ACT_UNSPEC;
+       }
+

Why not just reject the rule instead of changing the code underneath the
user?

cheers,
jamal

Reply via email to