diff options
| author | Cong Wang <xiyou.wangcong@gmail.com> | 2018-04-06 17:19:41 -0700 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2018-04-07 12:36:45 -0400 |
| commit | f12c643209db0626f2f54780d86bb93bfa7a9c2d (patch) | |
| tree | c67e80ab3ed11ee28606b96475c72e8ac0e3b4d1 /net | |
| parent | dd9a122ae99ae471beed4d4f8073d71e8d31ffa6 (diff) | |
| download | lwn-f12c643209db0626f2f54780d86bb93bfa7a9c2d.tar.gz lwn-f12c643209db0626f2f54780d86bb93bfa7a9c2d.zip | |
net_sched: fix a missing idr_remove() in u32_delete_key()
When we delete a u32 key via u32_delete_key(), we forget to
call idr_remove() to remove its handle from IDR.
Fixes: e7614370d6f0 ("net_sched: use idr to allocate u32 filter handles")
Reported-by: Marcin Kabiesz <admin@hostcenter.eu>
Tested-by: Marcin Kabiesz <admin@hostcenter.eu>
Signed-off-by: Cong Wang <xiyou.wangcong@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
| -rw-r--r-- | net/sched/cls_u32.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/net/sched/cls_u32.c b/net/sched/cls_u32.c index ed8b6a24b9e9..bac47b5d18fd 100644 --- a/net/sched/cls_u32.c +++ b/net/sched/cls_u32.c @@ -489,6 +489,7 @@ static int u32_delete_key(struct tcf_proto *tp, struct tc_u_knode *key) RCU_INIT_POINTER(*kp, key->next); tcf_unbind_filter(tp, &key->res); + idr_remove(&ht->handle_idr, key->handle); tcf_exts_get_net(&key->exts); call_rcu(&key->rcu, u32_delete_key_freepf_rcu); return 0; |