[SCSI] libfc: bug in erroring out upon FCP_RSP_LEN_VAL in fc_fcp_resp
authorYi Zou <yi.zou@intel.com>
Fri, 9 Apr 2010 21:23:05 +0000 (14:23 -0700)
committerJames Bottomley <James.Bottomley@suse.de>
Sun, 11 Apr 2010 19:02:40 +0000 (14:02 -0500)
fc_fcp_resp is assuming when FCP_SNS_LEN_VAL is set, the FCP_RSP_LEN_VAL
is not, which is not true. This leads to not copying the sense data and
error out a valid FCP_RSP.

Signed-off-by: Yi Zou <yi.zou@intel.com>
Signed-off-by: Robert Love <robert.w.love@intel.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
drivers/scsi/libfc/fc_fcp.c

index 220c4bc536ca37dcd2c98f01dbcf8b53e10dbc9c..f99d66fa2ef57eeca33b1af051cc0076e1a797cd 100644 (file)
@@ -844,8 +844,7 @@ static void fc_fcp_resp(struct fc_fcp_pkt *fsp, struct fc_frame *fp)
                                         * exit here
                                         */
                                        return;
-                               } else
-                                       goto err;
+                               }
                        }
                        if (flags & FCP_SNS_LEN_VAL) {
                                snsl = ntohl(rp_ex->fr_sns_len);