net/sched: act_api: move TCA_EXT_WARN_MSG to the correct hierarchy
authorPedro Tammela <pctammela@mojatatu.com>
Fri, 24 Feb 2023 17:56:01 +0000 (14:56 -0300)
committerDavid S. Miller <davem@davemloft.net>
Mon, 27 Feb 2023 12:16:34 +0000 (12:16 +0000)
TCA_EXT_WARN_MSG is currently sitting outside of the expected hierarchy
for the tc actions code. It should sit within TCA_ACT_TAB.

Fixes: 0349b8779cc9 ("sched: add new attr TCA_EXT_WARN_MSG to report tc extact message")
Reviewed-by: Jamal Hadi Salim <jhs@mojatatu.com>
Signed-off-by: Pedro Tammela <pctammela@mojatatu.com>
Reviewed-by: Simon Horman <simon.horman@corigine.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sched/act_api.c

index fce5228860990c80b3c1af306dfcb4ad871505b2..34c50867504160d02d413ea211c075525ad6bf26 100644 (file)
@@ -1596,12 +1596,12 @@ static int tca_get_fill(struct sk_buff *skb, struct tc_action *actions[],
        if (tcf_action_dump(skb, actions, bind, ref, false) < 0)
                goto out_nlmsg_trim;
 
-       nla_nest_end(skb, nest);
-
        if (extack && extack->_msg &&
            nla_put_string(skb, TCA_EXT_WARN_MSG, extack->_msg))
                goto out_nlmsg_trim;
 
+       nla_nest_end(skb, nest);
+
        nlh->nlmsg_len = skb_tail_pointer(skb) - b;
 
        return skb->len;