diff options
| author | Jean Sacren <sakiwit@gmail.com> | 2021-12-08 10:20:25 +0300 |
|---|---|---|
| committer | Jakub Kicinski <kuba@kernel.org> | 2021-12-10 05:35:11 +0300 |
| commit | 9745177c948984d61f7853f922d501cc440aae47 (patch) | |
| tree | 0751336ac0274fde4b3a4ed9499e6f4846ef5db1 /net/x25 | |
| parent | 3150a73366b64e3109f0facbc98bcacbc14e81ba (diff) | |
| download | linux-9745177c948984d61f7853f922d501cc440aae47.tar.xz | |
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.c | 2 |
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){ |