diff options
| author | Sagi Grimberg <sagig@mellanox.com> | 2015-04-14 11:08:13 -0400 |
|---|---|---|
| committer | Doug Ledford <dledford@redhat.com> | 2015-04-15 16:07:12 -0400 |
| commit | a065fe6aa25ba6ba93c02dc13486131bb3c64d5f (patch) | |
| tree | 1f947568457fe28015a016373baedc0212ff315c /drivers | |
| parent | 30bf1d58ae8e87fef20248c7166777deab0d7f9c (diff) | |
IB/iser: Fix wrong calculation of protection buffer length
This length miss-calculation may cause a silent data corruption
in the DIX case and cause the device to reference unmapped area.
Fixes: d77e65350f2d ('libiscsi, iser: Adjust data_length to include protection information')
Signed-off-by: Sagi Grimberg <sagig@mellanox.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/infiniband/ulp/iser/iser_initiator.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/drivers/infiniband/ulp/iser/iser_initiator.c b/drivers/infiniband/ulp/iser/iser_initiator.c index 20e859a6f1a6..76eb57b31a59 100644 --- a/drivers/infiniband/ulp/iser/iser_initiator.c +++ b/drivers/infiniband/ulp/iser/iser_initiator.c | |||
| @@ -409,8 +409,8 @@ int iser_send_command(struct iscsi_conn *conn, | |||
| 409 | if (scsi_prot_sg_count(sc)) { | 409 | if (scsi_prot_sg_count(sc)) { |
| 410 | prot_buf->buf = scsi_prot_sglist(sc); | 410 | prot_buf->buf = scsi_prot_sglist(sc); |
| 411 | prot_buf->size = scsi_prot_sg_count(sc); | 411 | prot_buf->size = scsi_prot_sg_count(sc); |
| 412 | prot_buf->data_len = data_buf->data_len >> | 412 | prot_buf->data_len = (data_buf->data_len >> |
| 413 | ilog2(sc->device->sector_size) * 8; | 413 | ilog2(sc->device->sector_size)) * 8; |
| 414 | } | 414 | } |
| 415 | 415 | ||
| 416 | if (hdr->flags & ISCSI_FLAG_CMD_READ) { | 416 | if (hdr->flags & ISCSI_FLAG_CMD_READ) { |