| Message ID | 20211228004542.568277-1-colin.i.king@gmail.com (mailing list archive) |
|---|---|
| State | Accepted |
| Commit | 0f1eae8e565e632f64670a5730894f22819fcaad |
| Delegated to: | Netdev Maintainers |
| Headers | show |
| Series | net: caif: remove redundant assignment to variable expectlen | expand |
Hello: This patch was applied to netdev/net-next.git (master) by David S. Miller <davem@davemloft.net>: On Tue, 28 Dec 2021 00:45:42 +0000 you wrote: > Variable expectlen is being assigned a value that is never read, the > assignment occurs before a return statement. The assignment is > redundant and can be removed. > > Signed-off-by: Colin Ian King <colin.i.king@gmail.com> > --- > net/caif/cfserl.c | 1 - > 1 file changed, 1 deletion(-) Here is the summary with links: - net: caif: remove redundant assignment to variable expectlen https://git.kernel.org/netdev/net-next/c/0f1eae8e565e You are awesome, thank you!
diff --git a/net/caif/cfserl.c b/net/caif/cfserl.c index 40cd57ad0a0f..aee11c74d3c8 100644 --- a/net/caif/cfserl.c +++ b/net/caif/cfserl.c @@ -128,7 +128,6 @@ static int cfserl_receive(struct cflayer *l, struct cfpkt *newpkt) if (pkt != NULL) cfpkt_destroy(pkt); layr->incomplete_frm = NULL; - expectlen = 0; spin_unlock(&layr->sync); return -EPROTO; }
Variable expectlen is being assigned a value that is never read, the assignment occurs before a return statement. The assignment is redundant and can be removed. Signed-off-by: Colin Ian King <colin.i.king@gmail.com> --- net/caif/cfserl.c | 1 - 1 file changed, 1 deletion(-)