diff options
| author | John Levon <john.levon@joyent.com> | 2019-12-18 06:37:16 -0800 |
|---|---|---|
| committer | John Levon <john.levon@joyent.com> | 2019-12-20 03:35:52 -0800 |
| commit | d0972c645cf979856ed66c1c6a1f082383c90b0c (patch) | |
| tree | 13e31343e255ff7ac0232479da75969d6862ce8c /usr/src | |
| parent | 028b5df8ad1713f1c0c8ba060ec660fe0a20261d (diff) | |
| download | illumos-joyent-d0972c645cf979856ed66c1c6a1f082383c90b0c.tar.gz | |
12093 devid_scsi_encode() should use a static format string
Reviewed by: Jason King <jason.brian.king@gmail.com>
Reviewed by: Toomas Soome <tsoome@me.com>
Approved by: Dan McDonald <danmcd@joyent.com>
Diffstat (limited to 'usr/src')
| -rw-r--r-- | usr/src/common/devid/devid_scsi.c | 10 |
1 files changed, 7 insertions, 3 deletions
diff --git a/usr/src/common/devid/devid_scsi.c b/usr/src/common/devid/devid_scsi.c index 0a97d15029..752bfea07f 100644 --- a/usr/src/common/devid/devid_scsi.c +++ b/usr/src/common/devid/devid_scsi.c @@ -25,6 +25,10 @@ */ /* + * Copyright 2019 Joyent, Inc. + */ + +/* * These functions are used to encode SCSI INQUIRY data into * Solaris devid / guid values. */ @@ -232,7 +236,7 @@ devid_scsi_encode( (void) strncat(msg, inq_std->inq_revision, sizeof (inq_std->inq_revision)); (void) strcat(msg, "\n"); - cmn_err(CE_WARN, msg); + cmn_err(CE_WARN, "%s", msg); kmem_free(msg, MSG_NOT_STANDARDS_COMPLIANT_SIZE); } @@ -301,7 +305,7 @@ static int is_page83_data_valid(uchar_t *inq83, size_t inq83_len) { - int covered_desc_len = 0; + int covered_desc_len = 0; int dlen = 0; uchar_t *dblk = NULL; @@ -1336,7 +1340,7 @@ scsi_wwnstr_hexcase(char *wwnstr, int upper_case_hex) * Function: scsi_wwnstr_skip_ua_prefix * * Description: This routine removes the leading 'w' in wwnstr, - * if its in unit-address form. + * if its in unit-address form. * * Arguments: wwnstr - the string wwn to be transformed * |