summaryrefslogtreecommitdiffstats
path: root/net/x25
diff options
context:
space:
mode:
authorJean Sacren <sakiwit@gmail.com>2021-12-08 08:20:25 +0100
committerJakub Kicinski <kuba@kernel.org>2021-12-10 03:35:11 +0100
commit9745177c948984d61f7853f922d501cc440aae47 (patch)
tree0751336ac0274fde4b3a4ed9499e6f4846ef5db1 /net/x25
parentMerge git://git.kernel.org/pub/scm/linux/kernel/git/netdev/net (diff)
downloadlinux-9745177c948984d61f7853f922d501cc440aae47.tar.xz
linux-9745177c948984d61f7853f922d501cc440aae47.zip
net: x25: drop harmless check of !more
'more' is checked first. When !more is checked immediately after that, it is always true. We should drop this check. Signed-off-by: Jean Sacren <sakiwit@gmail.com> Acked-by: Martin Schiller <ms@dev.tdt.de> Link: https://lore.kernel.org/r/20211208024732.142541-5-sakiwit@gmail.com Signed-off-by: Jakub Kicinski <kuba@kernel.org>
Diffstat (limited to 'net/x25')
-rw-r--r--net/x25/x25_in.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/net/x25/x25_in.c b/net/x25/x25_in.c
index e1c4197af468..b981a4828d08 100644
--- a/net/x25/x25_in.c
+++ b/net/x25/x25_in.c
@@ -41,7 +41,7 @@ static int x25_queue_rx_frame(struct sock *sk, struct sk_buff *skb, int more)
return 0;
}
- if (!more && x25->fraglen > 0) { /* End of fragment */
+ if (x25->fraglen > 0) { /* End of fragment */
int len = x25->fraglen + skb->len;
if ((skbn = alloc_skb(len, GFP_ATOMIC)) == NULL){