Date: Fri, 27 Aug 1999 00:44:37 +0100 From: Ben Smithurst <ben@scientia.demon.co.uk> To: "Mark J. Taylor" <mtaylor@cybernet.com> Cc: hackers@freebsd.org Subject: Re: Request for review: vnconfig.c Message-ID: <19990827004437.B3921@lithium.scientia.demon.co.uk> In-Reply-To: <XFMail.990826155714.mtaylor@cybernet.com> References: <XFMail.990826155714.mtaylor@cybernet.com>
next in thread | previous in thread | raw e-mail | index | archive | help
Mark J. Taylor wrote:
> + dev = malloc(strlen(vnp->dev)+6);
> + (void)sprintf(dev, "/dev/%s", vnp->dev);
You should be checking that malloc() doesn't return NULL, before trying
to write into the allocated space.
--
Ben Smithurst | PGP: 0x99392F7D
ben@scientia.demon.co.uk | key available from keyservers and
| ben+pgp@scientia.demon.co.uk
To Unsubscribe: send mail to majordomo@FreeBSD.org
with "unsubscribe freebsd-hackers" in the body of the message
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?19990827004437.B3921>