diff options
author | Dan Carpenter <dan.carpenter@oracle.com> | 2018-03-28 07:48:36 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2018-03-29 14:33:29 -0400 |
commit | 99fe29d3a25f813146816b322367b4c6a0fdec84 (patch) | |
tree | 8197459068121adbb98ea603bb421d922f1f4927 | |
parent | 981f1f803516666d3e02f1d707a7f4b0ebf41967 (diff) |
test_bpf: Fix NULL vs IS_ERR() check in test_skb_segment()
The skb_segment() function returns error pointers on error. It never
returns NULL.
Fixes: 76db8087c4c9 ("net: bpf: add a test for skb_segment in test_bpf module")
Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Acked-by: Daniel Borkmann <daniel@iogearbox.net>
Reviewed-by: Yonghong Song <yhs@fb.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | lib/test_bpf.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/lib/test_bpf.c b/lib/test_bpf.c index b2badf6b23cd..8e157806df7a 100644 --- a/lib/test_bpf.c +++ b/lib/test_bpf.c | |||
@@ -6649,7 +6649,7 @@ static __init int test_skb_segment(void) | |||
6649 | } | 6649 | } |
6650 | 6650 | ||
6651 | segs = skb_segment(skb, features); | 6651 | segs = skb_segment(skb, features); |
6652 | if (segs) { | 6652 | if (!IS_ERR(segs)) { |
6653 | kfree_skb_list(segs); | 6653 | kfree_skb_list(segs); |
6654 | ret = 0; | 6654 | ret = 0; |
6655 | pr_info("%s: success in skb_segment!", __func__); | 6655 | pr_info("%s: success in skb_segment!", __func__); |