Commit 5777781b authored Sep 12, 2018 by YueHaibing Committed by Greg Kroah-Hartman Jan 08, 2024

scsi: bnx2fc: Remove set but not used variable 'oxid'



[ Upstream commit efcbe99818ac9bd93ac41e8cf954e9aa64dd9971 ]

Fixes gcc '-Wunused-but-set-variable' warning:

drivers/scsi/bnx2fc/bnx2fc_fcoe.c: In function 'bnx2fc_rcv':
drivers/scsi/bnx2fc/bnx2fc_fcoe.c:435:17: warning:
 variable 'oxid' set but not used [-Wunused-but-set-variable]

Signed-off-by: YueHaibing <yuehaibing@huawei.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Stable-dep-of: 08c94d80b2da ("scsi: bnx2fc: Fix skb double free in bnx2fc_rcv()")
Signed-off-by: Sasha Levin <sashal@kernel.org>

parent 51aaa85e

drivers/scsi/bnx2fc/bnx2fc_fcoe.c

+0 −3

Original line number	Diff line number	Diff line
		@@ -436,7 +436,6 @@ static int bnx2fc_rcv(struct sk_buff skb, struct net_device dev,
		struct fcoe_rcv_info *fr;
		struct fcoe_percpu_s *bg;
		struct sk_buff *tmp_skb;
		unsigned short oxid;

		interface = container_of(ptype, struct bnx2fc_interface,
		fcoe_packet_type);
		@@ -470,8 +469,6 @@ static int bnx2fc_rcv(struct sk_buff skb, struct net_device dev,
		skb_set_transport_header(skb, sizeof(struct fcoe_hdr));
		fh = (struct fc_frame_header *) skb_transport_header(skb);

		oxid = ntohs(fh->fh_ox_id);

		fr = fcoe_dev_from_skb(skb);
		fr->fr_dev = lport;