1
0
mirror of https://git.FreeBSD.org/src.git synced 2024-12-17 10:26:15 +00:00

Add a comment explaining how the PSN works and why there is no need for

a null terminator. Also mark CID 1394825 as intentional.

Reported by:	Coverity
CID:		1394825
MFC after:	1 week
Sponsored by:	iXsystems Inc.
This commit is contained in:
Marcelo Araujo 2018-08-16 06:31:54 +00:00
parent e30993c2a6
commit 9e59a2e8ce
Notes: svn2git 2020-12-20 02:59:44 +00:00
svn path=/head/; revision=337887

View File

@ -1714,6 +1714,11 @@ pci_nvme_parse_opts(struct pci_nvme_softc *sc, char *opts)
} else if (!strcmp("sectsz", xopts)) { } else if (!strcmp("sectsz", xopts)) {
sectsz = atoi(config); sectsz = atoi(config);
} else if (!strcmp("ser", xopts)) { } else if (!strcmp("ser", xopts)) {
/*
* This field indicates the Product Serial Number in
* 8-bit ASCII, unused bytes should be NULL characters.
* Ref: NVM Express Management Interface 1.0a.
*/
memset(sc->ctrldata.sn, 0, sizeof(sc->ctrldata.sn)); memset(sc->ctrldata.sn, 0, sizeof(sc->ctrldata.sn));
strncpy(sc->ctrldata.sn, config, strncpy(sc->ctrldata.sn, config,
sizeof(sc->ctrldata.sn)); sizeof(sc->ctrldata.sn));