]> git.proxmox.com Git - mirror_ubuntu-artful-kernel.git/commit
target: Re-add check to reject control WRITEs with overflow data
authorNicholas Bellinger <nab@linux-iscsi.org>
Thu, 11 May 2017 08:07:24 +0000 (01:07 -0700)
committerThadeu Lima de Souza Cascardo <cascardo@canonical.com>
Tue, 27 Jun 2017 13:16:27 +0000 (10:16 -0300)
commitd8b2236b587532de4f0fdf0ff070488acd121b36
tree977be1f5b14022c304c801aad615e4f10b04c2d6
parent95a2e77bf7bcbd269edfe76b07fc76b3c4f2b074
target: Re-add check to reject control WRITEs with overflow data

BugLink: http://bugs.launchpad.net/bugs/1698799
commit 4ff83daa0200affe1894bd33d17bac404e3d78d4 upstream.

During v4.3 when the overflow/underflow check was relaxed by
commit c72c525022:

  commit c72c5250224d475614a00c1d7e54a67f77cd3410
  Author: Roland Dreier <roland@purestorage.com>
  Date:   Wed Jul 22 15:08:18 2015 -0700

       target: allow underflow/overflow for PR OUT etc. commands

to allow underflow/overflow for Windows compliance + FCP, a
consequence was to allow control CDBs to process overflow
data for iscsi-target with immediate data as well.

As per Roland's original change, continue to allow underflow
cases for control CDBs to make Windows compliance + FCP happy,
but until overflow for control CDBs is supported tree-wide,
explicitly reject all control WRITEs with overflow following
pre v4.3.y logic.

Reported-by: Bart Van Assche <bart.vanassche@sandisk.com>
Cc: Roland Dreier <roland@purestorage.com>
Signed-off-by: Nicholas Bellinger <nab@linux-iscsi.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Stefan Bader <stefan.bader@canonical.com>
Signed-off-by: Thadeu Lima de Souza Cascardo <cascardo@canonical.com>
drivers/target/target_core_transport.c