diff options
author | Patrick McHardy <kaber@trash.net> | 2008-07-01 22:52:38 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2008-07-01 22:52:38 -0400 |
commit | ff31ab56c0e900235f653e375fc3b01ba2d8d6a3 (patch) | |
tree | e35bab914ce853527e1531dc7681a13a08720ed6 /net/sched/sch_atm.c | |
parent | 77a538d5aa25a8866606a1faa4300c9aa2a59dfc (diff) |
net-sched: change tcf_destroy_chain() to clear start of filter list
Pass double tcf_proto pointers to tcf_destroy_chain() to make it
clear the start of the filter list for more consistency.
Signed-off-by: Patrick McHardy <kaber@trash.net>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sched/sch_atm.c')
-rw-r--r-- | net/sched/sch_atm.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/net/sched/sch_atm.c b/net/sched/sch_atm.c index 335273416384..8e5f70ba3a15 100644 --- a/net/sched/sch_atm.c +++ b/net/sched/sch_atm.c | |||
@@ -160,7 +160,7 @@ static void atm_tc_put(struct Qdisc *sch, unsigned long cl) | |||
160 | *prev = flow->next; | 160 | *prev = flow->next; |
161 | pr_debug("atm_tc_put: qdisc %p\n", flow->q); | 161 | pr_debug("atm_tc_put: qdisc %p\n", flow->q); |
162 | qdisc_destroy(flow->q); | 162 | qdisc_destroy(flow->q); |
163 | tcf_destroy_chain(flow->filter_list); | 163 | tcf_destroy_chain(&flow->filter_list); |
164 | if (flow->sock) { | 164 | if (flow->sock) { |
165 | pr_debug("atm_tc_put: f_count %d\n", | 165 | pr_debug("atm_tc_put: f_count %d\n", |
166 | file_count(flow->sock->file)); | 166 | file_count(flow->sock->file)); |
@@ -588,8 +588,7 @@ static void atm_tc_destroy(struct Qdisc *sch) | |||
588 | pr_debug("atm_tc_destroy(sch %p,[qdisc %p])\n", sch, p); | 588 | pr_debug("atm_tc_destroy(sch %p,[qdisc %p])\n", sch, p); |
589 | /* races ? */ | 589 | /* races ? */ |
590 | while ((flow = p->flows)) { | 590 | while ((flow = p->flows)) { |
591 | tcf_destroy_chain(flow->filter_list); | 591 | tcf_destroy_chain(&flow->filter_list); |
592 | flow->filter_list = NULL; | ||
593 | if (flow->ref > 1) | 592 | if (flow->ref > 1) |
594 | printk(KERN_ERR "atm_destroy: %p->ref = %d\n", flow, | 593 | printk(KERN_ERR "atm_destroy: %p->ref = %d\n", flow, |
595 | flow->ref); | 594 | flow->ref); |