diff options
| author | FUJITA Tomonori <tomof@acm.org> | 2008-01-26 22:41:50 -0500 |
|---|---|---|
| committer | James Bottomley <James.Bottomley@HansenPartnership.com> | 2008-01-30 14:03:39 -0500 |
| commit | 9d058ecfd444d247b7448e0ef44647514d91a4f2 (patch) | |
| tree | 0095e213d843f5713ed29060ba60e11027111331 /drivers/s390/scsi | |
| parent | 149d6bafc4f19784798cf5154bd1f5ff7e407194 (diff) | |
[SCSI] zfcp: fix sense_buffer access bug
The commit de25deb18016f66dcdede165d07654559bb332bc changed
scsi_cmnd.sense_buffer from a static array to a dynamically allocated
buffer. We can't access to sense_buffer in '&cmd->sense_buffer' way.
Signed-off-by: FUJITA Tomonori <fujita.tomonori@lab.ntt.co.jp>
Acked-by: Christof Schmitt <christof.schmitt@de.ibm.com>
Signed-off-by: James Bottomley <James.Bottomley@HansenPartnership.com>
Diffstat (limited to 'drivers/s390/scsi')
| -rw-r--r-- | drivers/s390/scsi/zfcp_fsf.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/s390/scsi/zfcp_fsf.c b/drivers/s390/scsi/zfcp_fsf.c index e45f85f7c7ed..0dff05840ee2 100644 --- a/drivers/s390/scsi/zfcp_fsf.c +++ b/drivers/s390/scsi/zfcp_fsf.c | |||
| @@ -4224,10 +4224,10 @@ zfcp_fsf_send_fcp_command_task_handler(struct zfcp_fsf_req *fsf_req) | |||
| 4224 | 4224 | ||
| 4225 | ZFCP_LOG_TRACE("%i bytes sense data provided by FCP\n", | 4225 | ZFCP_LOG_TRACE("%i bytes sense data provided by FCP\n", |
| 4226 | fcp_rsp_iu->fcp_sns_len); | 4226 | fcp_rsp_iu->fcp_sns_len); |
| 4227 | memcpy(&scpnt->sense_buffer, | 4227 | memcpy(scpnt->sense_buffer, |
| 4228 | zfcp_get_fcp_sns_info_ptr(fcp_rsp_iu), sns_len); | 4228 | zfcp_get_fcp_sns_info_ptr(fcp_rsp_iu), sns_len); |
| 4229 | ZFCP_HEX_DUMP(ZFCP_LOG_LEVEL_TRACE, | 4229 | ZFCP_HEX_DUMP(ZFCP_LOG_LEVEL_TRACE, |
| 4230 | (void *) &scpnt->sense_buffer, sns_len); | 4230 | (void *)scpnt->sense_buffer, sns_len); |
| 4231 | } | 4231 | } |
| 4232 | 4232 | ||
| 4233 | /* check for overrun */ | 4233 | /* check for overrun */ |
