net_sched: remove some unnecessary checks in classful schedulers

The class argument to the ->graft(), ->leaf(), ->dump(), ->dump_stats() all
originate from either ->get() or ->walk() and are always valid.

Remove unnecessary checks.

Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>

1 files changed, 1 insertions, 10 deletions

diff --git a/net/sched/sch_prio.c b/net/sched/sch_prio.c
index 209a4ca4b98d..0f73c412d04b 100644
--- a/net/sched/sch_prio.c
+++ b/net/sched/sch_prio.c
@@ -262,9 +262,6 @@ static int prio_graft(struct Qdisc *sch, unsigned long arg, struct Qdisc *new,
         struct prio_sched_data *q = qdisc_priv(sch);
         unsigned long band = arg - 1;
 
-        if (band >= q->bands)
-                return -EINVAL;
-
         if (new == NULL)
                 new = &noop_qdisc;
 
@@ -284,9 +281,6 @@ prio_leaf(struct Qdisc *sch, unsigned long arg)
         struct prio_sched_data *q = qdisc_priv(sch);
         unsigned long band = arg - 1;
 
-        if (band >= q->bands)
-                return NULL;
-
         return q->queues[band];
 }
 
@@ -316,11 +310,8 @@ static int prio_dump_class(struct Qdisc *sch, unsigned long cl, struct sk_buff *
 {
         struct prio_sched_data *q = qdisc_priv(sch);
 
-        if (cl - 1 > q->bands)
-                return -ENOENT;
         tcm->tcm_handle |= TC_H_MIN(cl);
-        if (q->queues[cl-1])
-                tcm->tcm_info = q->queues[cl-1]->handle;
+        tcm->tcm_info = q->queues[cl-1]->handle;
         return 0;
 }