Re: [PATCH] net/sched: remove redundant null check on head

2017-05-04 Thread David Miller
From: Colin King Date: Wed, 3 May 2017 14:50:40 +0100 > From: Colin Ian King > > head is previously null checked and so the 2nd null check on head > is redundant and therefore can be removed. > > Detected by CoverityScan, CID#1399505 ("Logically dead code") > > Signed-off-by: Colin Ian King

Re: [PATCH] net/sched: remove redundant null check on head

2017-05-03 Thread Jiri Pirko
Wed, May 03, 2017 at 04:21:15PM CEST, wha...@bfs.de wrote: > > >Am 03.05.2017 15:50, schrieb Colin King: >> From: Colin Ian King >> >> head is previously null checked and so the 2nd null check on head >> is redundant and therefore can be removed. >> >> Detected by CoverityScan, CID#1399505 ("Log

Re: [PATCH] net/sched: remove redundant null check on head

2017-05-03 Thread walter harms
Am 03.05.2017 15:50, schrieb Colin King: > From: Colin Ian King > > head is previously null checked and so the 2nd null check on head > is redundant and therefore can be removed. > > Detected by CoverityScan, CID#1399505 ("Logically dead code") > > Signed-off-by: Colin Ian King > --- > net/

Re: [PATCH] net/sched: remove redundant null check on head

2017-05-03 Thread Jiri Pirko
Wed, May 03, 2017 at 03:50:40PM CEST, colin.k...@canonical.com wrote: >From: Colin Ian King > >head is previously null checked and so the 2nd null check on head >is redundant and therefore can be removed. > >Detected by CoverityScan, CID#1399505 ("Logically dead code") > >Signed-off-by: Colin Ian

[PATCH] net/sched: remove redundant null check on head

2017-05-03 Thread Colin King
From: Colin Ian King head is previously null checked and so the 2nd null check on head is redundant and therefore can be removed. Detected by CoverityScan, CID#1399505 ("Logically dead code") Signed-off-by: Colin Ian King --- net/sched/cls_matchall.c | 3 +-- 1 file changed, 1 insertion(+), 2