diff options
author | Sarang Radke <sarang.radke@qlogic.com> | 2010-02-18 13:07:23 -0500 |
---|---|---|
committer | James Bottomley <James.Bottomley@suse.de> | 2010-02-19 11:44:40 -0500 |
commit | 236b0249c2274cd24bc98a1fbbb5e78d861d4bd9 (patch) | |
tree | 76e41cd7d7b5d6792b60ea69d0443260031fe0e7 /drivers/scsi | |
parent | ee37e09d81a4acf328f68189af12f116401f8c0f (diff) |
[SCSI] qla2xxx: Initialize payload receive length in failure path of vendor commands
Since bsg_job->reply->reply_payload_rcv_len is uninitialized
in case of failure, fc-transport triggers a warning for each command
failed.
Cc: stable@kernel.org
Signed-off-by: Giridhar Malavali <giridhar.malavali@qlogic.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'drivers/scsi')
-rw-r--r-- | drivers/scsi/qla2xxx/qla_attr.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/scsi/qla2xxx/qla_attr.c b/drivers/scsi/qla2xxx/qla_attr.c index 5a19aea1702..cbce52eb9b9 100644 --- a/drivers/scsi/qla2xxx/qla_attr.c +++ b/drivers/scsi/qla2xxx/qla_attr.c | |||
@@ -2284,6 +2284,7 @@ qla2x00_process_vendor_specific(struct fc_bsg_job *bsg_job) | |||
2284 | "scsi(%ld) Vendor request %s failed\n", vha->host_no, type)); | 2284 | "scsi(%ld) Vendor request %s failed\n", vha->host_no, type)); |
2285 | rval = 0; | 2285 | rval = 0; |
2286 | bsg_job->reply->result = (DID_ERROR << 16); | 2286 | bsg_job->reply->result = (DID_ERROR << 16); |
2287 | bsg_job->reply->reply_payload_rcv_len = 0; | ||
2287 | fw_sts_ptr = ((uint8_t*)bsg_job->req->sense) + sizeof(struct fc_bsg_reply); | 2288 | fw_sts_ptr = ((uint8_t*)bsg_job->req->sense) + sizeof(struct fc_bsg_reply); |
2288 | memcpy( fw_sts_ptr, response, sizeof(response)); | 2289 | memcpy( fw_sts_ptr, response, sizeof(response)); |
2289 | fw_sts_ptr += sizeof(response); | 2290 | fw_sts_ptr += sizeof(response); |