diff options
| author | Thomas Graf <tgraf@suug.ch> | 2005-11-05 21:14:14 +0100 |
|---|---|---|
| committer | Thomas Graf <tgr@axs.localdomain> | 2005-11-05 22:02:26 +0100 |
| commit | a8aaa9958eea2420e13d5a00c3fae934e0a3889e (patch) | |
| tree | 1e85c385c7ce81afdf8bb7203df1ce56cd404892 | |
| parent | [PKT_SCHED]: GRED: Use a central table definition change procedure (diff) | |
| download | linux-a8aaa9958eea2420e13d5a00c3fae934e0a3889e.tar.xz linux-a8aaa9958eea2420e13d5a00c3fae934e0a3889e.zip | |
[PKT_SCHED]: GRED: Report out-of-bound DPs as illegal
Signed-off-by: Thomas Graf <tgraf@suug.ch>
Signed-off-by: Arnaldo Carvalho de Melo <acme@mandriva.com>
| -rw-r--r-- | net/sched/sch_gred.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/net/sched/sch_gred.c b/net/sched/sch_gred.c index fdc20ced52ef..b04b07fcc2cf 100644 --- a/net/sched/sch_gred.c +++ b/net/sched/sch_gred.c @@ -451,12 +451,9 @@ static int gred_change(struct Qdisc *sch, struct rtattr *opt) return -EINVAL; ctl = RTA_DATA(tb[TCA_GRED_PARMS-1]); - if (ctl->DP > MAX_DPs-1 ) { - /* misbehaving is punished! Put in the default drop probability */ - DPRINTK("\nGRED: DP %u not in the proper range fixed. New DP " - "set to default at %d\n",ctl->DP,table->def); - ctl->DP=table->def; - } + + if (ctl->DP >= table->DPs) + return -EINVAL; if (table->tab[ctl->DP] == NULL) { table->tab[ctl->DP]=kmalloc(sizeof(struct gred_sched_data), |