scsi: target: remove hardcoded T10 Vendor ID in INQUIRY response
authorDavid Disseldorp <ddiss@suse.de>
Wed, 5 Dec 2018 12:18:37 +0000 (13:18 +0100)
committerMartin K. Petersen <martin.petersen@oracle.com>
Sat, 8 Dec 2018 02:54:33 +0000 (21:54 -0500)
Use the value stored in t10_wwn.vendor, which defaults to "LIO-ORG", but
can be reconfigured via the vendor_id ConfigFS attribute.

Signed-off-by: David Disseldorp <ddiss@suse.de>
Reviewed-by: Bryant G. Ly <bly@catalogicsoftware.com>
Reviewed-by: Lee Duncan <lduncan@suse.com>
Reviewed-by: Hannes Reinecke <hare@suse.com>
Reviewed-by: Roman Bolshakov <r.bolshakov@yadro.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
drivers/target/target_core_spc.c

index 5d2993db345ba01cba061603681ad6ce00d5e0de..47094ae01c040cb6d39051244b041c666a8011b0 100644 (file)
@@ -115,7 +115,8 @@ spc_emulate_inquiry_std(struct se_cmd *cmd, unsigned char *buf)
         */
        memset(&buf[8], 0x20,
               INQUIRY_VENDOR_LEN + INQUIRY_MODEL_LEN + INQUIRY_REVISION_LEN);
-       memcpy(&buf[8], "LIO-ORG", sizeof("LIO-ORG") - 1);
+       memcpy(&buf[8], dev->t10_wwn.vendor,
+              strnlen(dev->t10_wwn.vendor, INQUIRY_VENDOR_LEN));
        memcpy(&buf[16], dev->t10_wwn.model,
               strnlen(dev->t10_wwn.model, INQUIRY_MODEL_LEN));
        memcpy(&buf[32], dev->t10_wwn.revision,
@@ -258,8 +259,9 @@ check_t10_vend_desc:
        buf[off+1] = 0x1; /* T10 Vendor ID */
        buf[off+2] = 0x0;
        /* left align Vendor ID and pad with spaces */
-       memset(&buf[off+4], 0x20, 8);
-       memcpy(&buf[off+4], "LIO-ORG", sizeof("LIO-ORG") - 1);
+       memset(&buf[off+4], 0x20, INQUIRY_VENDOR_LEN);
+       memcpy(&buf[off+4], dev->t10_wwn.vendor,
+              strnlen(dev->t10_wwn.vendor, INQUIRY_VENDOR_LEN));
        /* Extra Byte for NULL Terminator */
        id_len++;
        /* Identifier Length */