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_multiq.c b/net/sched/sch_multiq.c
index a0ffe7158ff3..069f81c97277 100644
--- a/net/sched/sch_multiq.c
+++ b/net/sched/sch_multiq.c
@@ -298,9 +298,6 @@ static int multiq_graft(struct Qdisc *sch, unsigned long arg, struct Qdisc *new,
         struct multiq_sched_data *q = qdisc_priv(sch);
         unsigned long band = arg - 1;
 
-        if (band >= q->bands)
-                return -EINVAL;
-
         if (new == NULL)
                 new = &noop_qdisc;
 
@@ -320,9 +317,6 @@ multiq_leaf(struct Qdisc *sch, unsigned long arg)
         struct multiq_sched_data *q = qdisc_priv(sch);
         unsigned long band = arg - 1;
 
-        if (band >= q->bands)
-                return NULL;
-
         return q->queues[band];
 }
 
@@ -353,11 +347,8 @@ static int multiq_dump_class(struct Qdisc *sch, unsigned long cl,
 {
         struct multiq_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;
 }