summaryrefslogtreecommitdiffstats
path: root/net
diff options
context:
space:
mode:
authorPatrick McHardy <kaber@trash.net>2006-03-23 01:16:48 -0800
committerDavid S. Miller <davem@davemloft.net>2006-03-23 01:16:48 -0800
commit1ae39a430b692552e5aceb63fa35ce95fcbadc6a (patch)
tree598cfbcff6a1e84f6668d75bc8c92a28f73a381a /net
parenta5cdc030030ef5a16e48aebeb65067bdc3120899 (diff)
downloadop-kernel-dev-1ae39a430b692552e5aceb63fa35ce95fcbadc6a.zip
op-kernel-dev-1ae39a430b692552e5aceb63fa35ce95fcbadc6a.tar.gz
[NET_SCHED]: cls_u32: remove unnecessary NULL-ptr check
In both cases n can't be NULL without crashing anyway. Coverity #78 Signed-off-by: Patrick McHardy <kaber@trash.net> Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net')
-rw-r--r--net/sched/cls_u32.c6
1 files changed, 2 insertions, 4 deletions
diff --git a/net/sched/cls_u32.c b/net/sched/cls_u32.c
index 2b67047..78e0525 100644
--- a/net/sched/cls_u32.c
+++ b/net/sched/cls_u32.c
@@ -347,8 +347,7 @@ static int u32_destroy_key(struct tcf_proto *tp, struct tc_u_knode *n)
if (n->ht_down)
n->ht_down->refcnt--;
#ifdef CONFIG_CLS_U32_PERF
- if (n)
- kfree(n->pf);
+ kfree(n->pf);
#endif
kfree(n);
return 0;
@@ -680,8 +679,7 @@ static int u32_change(struct tcf_proto *tp, unsigned long base, u32 handle,
return 0;
}
#ifdef CONFIG_CLS_U32_PERF
- if (n)
- kfree(n->pf);
+ kfree(n->pf);
#endif
kfree(n);
return err;
OpenPOWER on IntegriCloud