On 12/1/18 12:34 AM, David Disseldorp wrote:
spc5r17.pdf specifies:
   4.3.1 ASCII data field requirements
   ASCII data fields shall contain only ASCII printable characters (i.e.,
   code values 20h to 7Eh) and may be terminated with one or more ASCII
   null (00h) characters.
   ASCII data fields described as being left-aligned shall have any
   unused bytes at the end of the field (i.e., highest offset) and the
   unused bytes shall be filled with ASCII space characters (20h).

LIO currently space-pads the T10 VENDOR IDENTIFICATION and PRODUCT
IDENTIFICATION fields in the standard INQUIRY data. However, the
PRODUCT REVISION LEVEL field in the standard INQUIRY data as well as the
T10 VENDOR IDENTIFICATION field in the INQUIRY Device Identification VPD
Page are zero-terminated/zero-padded.

Fix this inconsistency by using space-padding for all of the above
fields.

Signed-off-by: David Disseldorp <dd...@suse.de>
Reviewed-by: Christoph Hellwig <h...@lst.de>
Reviewed-by: Bryant G. Ly <b...@catalogicsoftware.com>
Reviewed-by: Lee Duncan <ldun...@suse.com>
---
  drivers/target/target_core_spc.c | 17 ++++++++++++-----
  1 file changed, 12 insertions(+), 5 deletions(-)

Reviewed-by: Hannes Reinecke <h...@suse.com>

Cheers,

Hannes

Reply via email to