> On Dec 17, 2015, at 21:36, luke <luke...@gmail.com> wrote: > >> On Fri, Dec 18, 2015 at 4:42 AM, John Baldwin <j...@freebsd.org> wrote: >> Author: jhb >> Date: Thu Dec 17 20:42:05 2015 >> New Revision: 292410 >> URL: https://svnweb.freebsd.org/changeset/base/292410 >> >> Log: >> Exit cleanly if malloc() fails to allocate a buffer for a copy of the >> current MBR. >> >> PR: 205322 >> Submitted by: Alexander Kuleshov <kuleshovm...@gmail.com> >> MFC after: 1 week >> >> Modified: >> head/usr.sbin/boot0cfg/boot0cfg.c >> >> Modified: head/usr.sbin/boot0cfg/boot0cfg.c >> ============================================================================== >> --- head/usr.sbin/boot0cfg/boot0cfg.c Thu Dec 17 20:33:20 2015 >> (r292409) >> +++ head/usr.sbin/boot0cfg/boot0cfg.c Thu Dec 17 20:42:05 2015 >> (r292410) >> @@ -337,6 +337,8 @@ read_mbr(const char *disk, u_int8_t **mb >> return (mbr_size); >> } >> *mbr = malloc(sizeof(buf)); >> + if (mbr == NULL) >> + errx(1, "%s: unable to allocate MBR buffer", disk); >> memcpy(*mbr, buf, sizeof(buf)); >> close(fd);
> Hi, > > Should the check be against *mbr ? > + if (*mbr == NULL) > + errx(1, "%s: unable to allocate MBR buffer", disk); Yup! _______________________________________________ svn-src-head@freebsd.org mailing list https://lists.freebsd.org/mailman/listinfo/svn-src-head To unsubscribe, send any mail to "svn-src-head-unsubscr...@freebsd.org"