net_sched: Remove unnecessary checks for act->ops
authorJamal Hadi Salim <jhs@mojatatu.com>
Mon, 23 Dec 2013 13:02:12 +0000 (08:02 -0500)
committerDavid S. Miller <davem@davemloft.net>
Mon, 6 Jan 2014 21:46:32 +0000 (16:46 -0500)
Remove unnecessary checks for act->ops
(suggested by Eric Dumazet).

Signed-off-by: Jamal Hadi Salim <jhs@mojatatu.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/sched/act_api.c
net/sched/cls_api.c

index 6f103fd76c17c3d8592dca82d50fa38b6075766a..2b69e537baa94a29a3723b2d22769f032cca3262 100644 (file)
@@ -352,18 +352,16 @@ int tcf_action_exec(struct sk_buff *skb, const struct list_head *actions,
        }
        list_for_each_entry(a, actions, list) {
 repeat:
-               if (a->ops) {
-                       ret = a->ops->act(skb, a, res);
-                       if (TC_MUNGED & skb->tc_verd) {
-                               /* copied already, allow trampling */
-                               skb->tc_verd = SET_TC_OK2MUNGE(skb->tc_verd);
-                               skb->tc_verd = CLR_TC_MUNGED(skb->tc_verd);
-                       }
-                       if (ret == TC_ACT_REPEAT)
-                               goto repeat;    /* we need a ttl - JHS */
-                       if (ret != TC_ACT_PIPE)
-                               goto exec_done;
+               ret = a->ops->act(skb, a, res);
+               if (TC_MUNGED & skb->tc_verd) {
+                       /* copied already, allow trampling */
+                       skb->tc_verd = SET_TC_OK2MUNGE(skb->tc_verd);
+                       skb->tc_verd = CLR_TC_MUNGED(skb->tc_verd);
                }
+               if (ret == TC_ACT_REPEAT)
+                       goto repeat;    /* we need a ttl - JHS */
+               if (ret != TC_ACT_PIPE)
+                       goto exec_done;
        }
 exec_done:
        return ret;
@@ -375,27 +373,16 @@ void tcf_action_destroy(struct list_head *actions, int bind)
        struct tc_action *a, *tmp;
 
        list_for_each_entry_safe(a, tmp, actions, list) {
-               if (a->ops) {
-                       if (a->ops->cleanup(a, bind) == ACT_P_DELETED)
-                               module_put(a->ops->owner);
-                       list_del(&a->list);
-                       kfree(a);
-               } else {
-                       /*FIXME: Remove later - catch insertion bugs*/
-                       WARN(1, "tcf_action_destroy: BUG? destroying NULL ops\n");
-                       list_del(&a->list);
-                       kfree(a);
-               }
+               if (a->ops->cleanup(a, bind) == ACT_P_DELETED)
+                       module_put(a->ops->owner);
+               list_del(&a->list);
+               kfree(a);
        }
 }
 
 int
 tcf_action_dump_old(struct sk_buff *skb, struct tc_action *a, int bind, int ref)
 {
-       int err = -EINVAL;
-
-       if (a->ops == NULL)
-               return err;
        return a->ops->dump(skb, a, bind, ref);
 }
 
@@ -406,9 +393,6 @@ tcf_action_dump_1(struct sk_buff *skb, struct tc_action *a, int bind, int ref)
        unsigned char *b = skb_tail_pointer(skb);
        struct nlattr *nest;
 
-       if (a->ops == NULL)
-               return err;
-
        if (nla_put_string(skb, TCA_KIND, a->ops->kind))
                goto nla_put_failure;
        if (tcf_action_copy_stats(skb, a, 0))
@@ -684,7 +668,7 @@ tcf_action_get_1(struct nlattr *nla, struct nlmsghdr *n, u32 portid)
        INIT_LIST_HEAD(&a->list);
        err = -EINVAL;
        a->ops = tc_lookup_action(tb[TCA_ACT_KIND]);
-       if (a->ops == NULL)
+       if (a->ops == NULL) /* could happen in batch of actions */
                goto err_free;
        err = -ENOENT;
        if (a->ops->lookup(a, index) == 0)
@@ -760,7 +744,7 @@ static int tca_action_flush(struct net *net, struct nlattr *nla,
        err = -EINVAL;
        kind = tb[TCA_ACT_KIND];
        a->ops = tc_lookup_action(kind);
-       if (a->ops == NULL)
+       if (a->ops == NULL) /*some idjot trying to flush unknown action */
                goto err_out;
 
        nlh = nlmsg_put(skb, portid, n->nlmsg_seq, RTM_DELACTION, sizeof(*t), 0);
index 12e882ef596bb986b1a9ab9b8abf12f8f1121954..d8c42b1b88e5d3f8ca06f929f2b4ec6287c609e0 100644 (file)
@@ -577,7 +577,7 @@ int tcf_exts_dump(struct sk_buff *skb, struct tcf_exts *exts)
                } else if (exts->police) {
                        struct tc_action *act = tcf_exts_first_act(exts);
                        nest = nla_nest_start(skb, exts->police);
-                       if (nest == NULL)
+                       if (nest == NULL || !act)
                                goto nla_put_failure;
                        if (tcf_action_dump_old(skb, act, 0, 0) < 0)
                                goto nla_put_failure;