diff options
author | Yang Yingliang <yangyingliang@huawei.com> | 2013-12-17 02:29:17 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2013-12-19 15:06:55 -0500 |
commit | 79c11f2e3fd3e0e3594d4155821ef426153a773f (patch) | |
tree | a480220039767c512ff141c956b4b6f31d45076f /net/sched | |
parent | 3b69a4c9becde783cd89758075780f9d871ba4b6 (diff) |
sch_cbq: remove unnecessary null pointer check
It already has a NULL pointer check of rtab in qdisc_put_rtab().
Remove the check outside of qdisc_put_rtab().
Signed-off-by: Yang Yingliang <yangyingliang@huawei.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/sched')
-rw-r--r-- | net/sched/sch_cbq.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/net/sched/sch_cbq.c b/net/sched/sch_cbq.c index d5a8a4b2454f..e25183333807 100644 --- a/net/sched/sch_cbq.c +++ b/net/sched/sch_cbq.c | |||
@@ -1783,8 +1783,7 @@ cbq_change_class(struct Qdisc *sch, u32 classid, u32 parentid, struct nlattr **t | |||
1783 | qdisc_root_sleeping_lock(sch), | 1783 | qdisc_root_sleeping_lock(sch), |
1784 | tca[TCA_RATE]); | 1784 | tca[TCA_RATE]); |
1785 | if (err) { | 1785 | if (err) { |
1786 | if (rtab) | 1786 | qdisc_put_rtab(rtab); |
1787 | qdisc_put_rtab(rtab); | ||
1788 | return err; | 1787 | return err; |
1789 | } | 1788 | } |
1790 | } | 1789 | } |