diff options
| author | Yang Yingliang <yangyingliang@huawei.com> | 2013-12-17 08:29:17 +0100 |
|---|---|---|
| committer | David S. Miller <davem@davemloft.net> | 2013-12-19 21:06:55 +0100 |
| commit | 79c11f2e3fd3e0e3594d4155821ef426153a773f (patch) | |
| tree | a480220039767c512ff141c956b4b6f31d45076f /net | |
| parent | act_police: remove unnecessary null pointer check (diff) | |
| download | linux-79c11f2e3fd3e0e3594d4155821ef426153a773f.tar.xz linux-79c11f2e3fd3e0e3594d4155821ef426153a773f.zip | |
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')
| -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 qdisc_root_sleeping_lock(sch), tca[TCA_RATE]); if (err) { - if (rtab) - qdisc_put_rtab(rtab); + qdisc_put_rtab(rtab); return err; } } |