| From a065fe6aa25ba6ba93c02dc13486131bb3c64d5f Mon Sep 17 00:00:00 2001 |
| From: Sagi Grimberg <sagig@mellanox.com> |
| Date: Tue, 14 Apr 2015 18:08:13 +0300 |
| Subject: IB/iser: Fix wrong calculation of protection buffer length |
| |
| From: Sagi Grimberg <sagig@mellanox.com> |
| |
| commit a065fe6aa25ba6ba93c02dc13486131bb3c64d5f upstream. |
| |
| 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> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| |
| --- |
| drivers/infiniband/ulp/iser/iser_initiator.c | 4 ++-- |
| 1 file changed, 2 insertions(+), 2 deletions(-) |
| |
| --- 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 |
| if (scsi_prot_sg_count(sc)) { |
| prot_buf->buf = scsi_prot_sglist(sc); |
| prot_buf->size = scsi_prot_sg_count(sc); |
| - prot_buf->data_len = data_buf->data_len >> |
| - ilog2(sc->device->sector_size) * 8; |
| + prot_buf->data_len = (data_buf->data_len >> |
| + ilog2(sc->device->sector_size)) * 8; |
| } |
| |
| if (hdr->flags & ISCSI_FLAG_CMD_READ) { |
