Skip site navigation (1)Skip section navigation (2)
Date:      Thu, 16 Aug 2018 12:29:32 -0700 (PDT)
From:      "Rodney W. Grimes" <freebsd@pdx.rh.CN85.dnsmgr.net>
To:        Warner Losh <imp@bsdimp.com>
Cc:        John-Mark Gurney <jmg@funkthat.com>, Marcelo Araujo <araujo@freebsd.org>,  src-committers <src-committers@freebsd.org>, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   Re: svn commit: r337887 - head/usr.sbin/bhyve
Message-ID:  <201808161929.w7GJTWfS055411@pdx.rh.CN85.dnsmgr.net>
In-Reply-To: <CANCZdfqQea=vrJPJ1mafjuX8ZTyx6bFW=WpawdVfyq45v40hAw@mail.gmail.com>

next in thread | previous in thread | raw e-mail | index | archive | help
> On Thu, Aug 16, 2018 at 11:06 AM, John-Mark Gurney <jmg@funkthat.com> wrote:
> 
> > Marcelo Araujo wrote this message on Thu, Aug 16, 2018 at 06:31 +0000:
> > > Author: araujo
> > > Date: Thu Aug 16 06:31:54 2018
> > > New Revision: 337887
> > > URL: https://svnweb.freebsd.org/changeset/base/337887
> > >
> > > Log:
> > >   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.
> > >
> > > Modified:
> > >   head/usr.sbin/bhyve/pci_nvme.c
> > >
> > > Modified: head/usr.sbin/bhyve/pci_nvme.c
> > > ============================================================
> > ==================
> > > --- head/usr.sbin/bhyve/pci_nvme.c    Thu Aug 16 06:20:25 2018
> > (r337886)
> > > +++ head/usr.sbin/bhyve/pci_nvme.c    Thu Aug 16 06:31:54 2018
> > (r337887)
> > > @@ -1714,6 +1714,11 @@ pci_nvme_parse_opts(struct pci_nvme_softc *sc,
> > char *o
> >
> > [...]
> >
> > >                       memset(sc->ctrldata.sn, 0, sizeof(sc->ctrldata.sn
> > ));
> > >                       strncpy(sc->ctrldata.sn, config,
> > >                               sizeof(sc->ctrldata.sn));
> >
> > This memset is unneeded, as strncpy will write NUL bytes to fill out
> > the buffer:
> > If src is less than len characters long, the remainder of
> >      dst is filled with `\0' characters.
> >
> 
> It also looks like the comment was wrong. The newest NVMe standards say
> these fields should be 7-bit and space-padded.

lol, which is what the vendor that caused me grief with
ata serial numbers did decades ago.

-- 
Rod Grimes                                                 rgrimes@freebsd.org



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201808161929.w7GJTWfS055411>