Date: Sun, 7 Aug 2016 13:19:20 -0500 From: Justin Hibbits <jhibbits@freebsd.org> To: Mark Millard <markmi@dsl-only.net> Cc: svn-src-head@freebsd.org Subject: Re: svn commit: r303797 - head/contrib/binutils/bfd Message-ID: <20160807131920.402b3cec@zhabar.knownspace> In-Reply-To: <F3CC05FE-FB3E-4E61-8DD1-5CBA27264EB0@dsl-only.net> References: <F3CC05FE-FB3E-4E61-8DD1-5CBA27264EB0@dsl-only.net>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sat, 6 Aug 2016 23:52:58 -0700 Mark Millard <markmi@dsl-only.net> wrote: > Is there to be an MFC for the below? > > > Author: jhibbits > > Date: Sat Aug 6 15:10:14 2016 > > New Revision: 303797 > > URL: > > https://svnweb.freebsd.org/changeset/base/303797 > > > > > > Log: > > Check the first byte of the array for NUL, instead of the array > > as a NULL pointer > > The partition_name field is an array, so can never be NULL > > itself. Check only the first byte instead. > > > > This was found when test building with clang, but I'm not sure > > how it passes gcc's warnings either. > > MFC After: 11-RELEASE - Justin
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20160807131920.402b3cec>