diff options
author | WANG Cong <xiyou.wangcong@gmail.com> | 2013-12-20 03:08:51 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2013-12-20 17:06:27 -0500 |
commit | 568a153a22d8f338a5ebda70e6bd139f6d8bb2c3 (patch) | |
tree | 944006c1defc9335807ae15c30c466b5fb3f681f /net/sched/act_pedit.c | |
parent | 852ad5e631967ae2203cb08c5b6b42c26011ed63 (diff) |
net_sched: fix a regression in tc actions
This patch fixes:
1) pass mask rather than size to tcf_hashinfo_init()
2) the cleanup should be in reversed order in mirred_cleanup_module()
Reported-by: Eric Dumazet <eric.dumazet@gmail.com>
Fixes: 369ba56787d7469c0afd ("net_sched: init struct tcf_hashinfo at register time")
Cc: Eric Dumazet <eric.dumazet@gmail.com>
Cc: David S. Miller <davem@davemloft.net>
Signed-off-by: Cong Wang <xiyou.wangcong@gmail.com>
Signed-off-by: Jamal Hadi Salim <jhs@mojatatu.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sched/act_pedit.c')
-rw-r--r-- | net/sched/act_pedit.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/net/sched/act_pedit.c b/net/sched/act_pedit.c index aa5347c1b9f1..e2520e90a10d 100644 --- a/net/sched/act_pedit.c +++ b/net/sched/act_pedit.c | |||
@@ -246,7 +246,7 @@ MODULE_LICENSE("GPL"); | |||
246 | 246 | ||
247 | static int __init pedit_init_module(void) | 247 | static int __init pedit_init_module(void) |
248 | { | 248 | { |
249 | int err = tcf_hashinfo_init(&pedit_hash_info, PEDIT_TAB_MASK+1); | 249 | int err = tcf_hashinfo_init(&pedit_hash_info, PEDIT_TAB_MASK); |
250 | if (err) | 250 | if (err) |
251 | return err; | 251 | return err; |
252 | return tcf_register_action(&act_pedit_ops); | 252 | return tcf_register_action(&act_pedit_ops); |