diff options
author | Robert Love <robert.w.love@intel.com> | 2010-05-07 18:18:30 -0400 |
---|---|---|
committer | James Bottomley <James.Bottomley@suse.de> | 2010-05-16 22:22:30 -0400 |
commit | d29510a2968f87eaf455c606cd8802b6f8a0774b (patch) | |
tree | 76cebf287eb388597b978ef041ab03a9553377ea /drivers/scsi/fcoe/libfcoe.c | |
parent | 1b80e0f91c41db4633c0a61f727de169a9e9b275 (diff) |
[SCSI] libfc: Remove extra pointer check
The fcf pointer is checked again after this verification
making the first check redundant. Remote the first check.
Signed-off-by: Robert Love <robert.w.love@intel.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'drivers/scsi/fcoe/libfcoe.c')
-rw-r--r-- | drivers/scsi/fcoe/libfcoe.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/scsi/fcoe/libfcoe.c b/drivers/scsi/fcoe/libfcoe.c index aadd24962e92..ec4c88c2d131 100644 --- a/drivers/scsi/fcoe/libfcoe.c +++ b/drivers/scsi/fcoe/libfcoe.c | |||
@@ -948,8 +948,7 @@ static void fcoe_ctlr_recv_clr_vlink(struct fcoe_ctlr *fip, | |||
948 | u32 desc_mask; | 948 | u32 desc_mask; |
949 | 949 | ||
950 | LIBFCOE_FIP_DBG(fip, "Clear Virtual Link received\n"); | 950 | LIBFCOE_FIP_DBG(fip, "Clear Virtual Link received\n"); |
951 | if (!fcf) | 951 | |
952 | return; | ||
953 | if (!fcf || !fc_host_port_id(lport->host)) | 952 | if (!fcf || !fc_host_port_id(lport->host)) |
954 | return; | 953 | return; |
955 | 954 | ||