Date: Sat, 6 Aug 2016 23:52:58 -0700 From: Mark Millard <markmi@dsl-only.net> To: svn-src-head@freebsd.org, jhibbits@FreeBSD.org Subject: Re: svn commit: r303797 - head/contrib/binutils/bfd Message-ID: <F3CC05FE-FB3E-4E61-8DD1-5CBA27264EB0@dsl-only.net>
next in thread | raw e-mail | index | archive | help
Is there to be an MFC for the below? > Author: jhibbits > Date: Sat Aug 6 15:10:14 2016 > New Revision: 303797 > URL:=20 > https://svnweb.freebsd.org/changeset/base/303797 >=20 >=20 > Log: > Check the first byte of the array for NUL, instead of the array as a = NULL pointer > =20 > The partition_name field is an array, so can never be NULL itself. = Check only > the first byte instead. > =20 > This was found when test building with clang, but I'm not sure how = it passes > gcc's warnings either. >=20 > Modified: > head/contrib/binutils/bfd/ppcboot.c >=20 > Modified: head/contrib/binutils/bfd/ppcboot.c > = =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D > --- head/contrib/binutils/bfd/ppcboot.c Sat Aug 6 15:06:19 2016 = (r303796) > +++ head/contrib/binutils/bfd/ppcboot.c Sat Aug 6 15:10:14 2016 = (r303797) > @@ -419,7 +419,7 @@ ppcboot_bfd_print_private_bfd_data (abfd > if (tdata->header.os_id) > fprintf (f, "OS_ID =3D 0x%.2x\n", = tdata->header.os_id); > =20 > - if (tdata->header.partition_name) > + if (tdata->header.partition_name[0]) > fprintf (f, _("Partition name =3D \"%s\"\n"), = tdata->header.partition_name); > =20 > for (i =3D 0; i < 4; i++) =3D=3D=3D Mark Millard markmi at dsl-only.net
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?F3CC05FE-FB3E-4E61-8DD1-5CBA27264EB0>