diff options
| author |   Chad Dupuis <chad.dupuis@qlogic.com> | 2011-02-23 15:27:07 -0800 |
|---|---|---|
| committer |   James Bottomley <James.Bottomley@suse.de> | 2011-02-25 12:24:59 -0500 |
| commit | 7c8cf2fee1027caaadf0e96145ff97f931ba4bdd (patch) | |
| tree | 9fb7422358ca3c710a3dcb244a77299dff14478a /drivers/scsi/qla2xxx/qla_nx.c | |
| parent | [SCSI] qla2xxx: Remove host_lock in queuecommand function (diff) | |
| download | linux-dev-7c8cf2fee1027caaadf0e96145ff97f931ba4bdd.tar.xz linux-dev-7c8cf2fee1027caaadf0e96145ff97f931ba4bdd.zip | |
[SCSI] qla2xxx: Fix array subscript is above array bounds in qla2xx_build_scsi_type_6_iocbs().
The additional increment of dsd_seg was causing the compiler to throw an array
out of bounds warning. This patch moves to a direct assignment of
cmd_pkt->fcp_data_dseg_len so that the compiler doesn't generate an array out
bounds warning.
Signed-off-by: Chad Dupuis <chad.dupuis@qlogic.com>
Signed-off-by: Madhuranath Iyengar <Madhu.Iyengar@qlogic.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'drivers/scsi/qla2xxx/qla_nx.c')
| -rw-r--r-- | drivers/scsi/qla2xxx/qla_nx.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/qla2xxx/qla_nx.c b/drivers/scsi/qla2xxx/qla_nx.c index fdb96a3584a5..b37445d4ea2f 100644 --- a/drivers/scsi/qla2xxx/qla_nx.c +++ b/drivers/scsi/qla2xxx/qla_nx.c @@ -2547,7 +2547,7 @@ qla2xx_build_scsi_type_6_iocbs(srb_t *sp, struct cmd_type_6 *cmd_pkt, dsd_seg = (uint32_t *)&cmd_pkt->fcp_data_dseg_address; *dsd_seg++ = cpu_to_le32(LSD(dsd_ptr->dsd_list_dma)); *dsd_seg++ = cpu_to_le32(MSD(dsd_ptr->dsd_list_dma)); - *dsd_seg++ = dsd_list_len; + cmd_pkt->fcp_data_dseg_len = dsd_list_len; } else { *cur_dsd++ = cpu_to_le32(LSD(dsd_ptr->dsd_list_dma)); *cur_dsd++ = cpu_to_le32(MSD(dsd_ptr->dsd_list_dma)); |