if (desc->fip_dtype == FIP_DT_FLOGI) {
...
els = (struct fip_encaps *)desc;
fh = (struct fc_frame_header *)(els + 1);
'fh' cannot be NULL here after shifting a valid pointer 'desc'.
Detected using the static analysis tool - Svace.
Fixes: d3c995f1dcf9 ("[SCSI] fnic: FIP VLAN Discovery Feature Support")
Signed-off-by: Alexey Kodanev <aleksei.kodanev@xxxxxxxxxxx>
---
drivers/scsi/fnic/fnic_fcs.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/scsi/fnic/fnic_fcs.c b/drivers/scsi/fnic/fnic_fcs.c
index 55632c67a8f2..ca4214db72f5 100644
--- a/drivers/scsi/fnic/fnic_fcs.c
+++ b/drivers/scsi/fnic/fnic_fcs.c
@@ -338,9 +338,6 @@ static inline int is_fnic_fip_flogi_reject(struct fcoe_ctlr *fip,
els = (struct fip_encaps *)desc;
fh = (struct fc_frame_header *)(els + 1);
- if (!fh)
- return 0;
-
/*
* ELS command code, reason and explanation should be = Reject,
* unsupported command and insufficient resource
--
2.25.1
