Date: Sat, 4 Apr 2015 11:16:30 +0200 From: Ed Schouten <ed@80386.nl> To: Alexander Motin <mav@freebsd.org> Cc: svn-src-stable@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org, svn-src-stable-10@freebsd.org Subject: Re: svn commit: r279877 - stable/10/sys/boot/fdt Message-ID: <CAJOYFBBSn7GxzZS0hXzD03M-u_MQnjXFf9wDxxWO4TZWHSnynw@mail.gmail.com> In-Reply-To: <201503110949.t2B9n7Yk000610@svn.freebsd.org> References: <201503110949.t2B9n7Yk000610@svn.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
Hi Alexander,
Quick question:
2015-03-11 10:49 GMT+01:00 Alexander Motin <mav@freebsd.org>:
>         buf = (char *)malloc(sizeof(char) * sz);
> -       bzero(buf, sizeof(char) * sz);
> -
>         if (buf == NULL) {
>                 sprintf(command_errbuf, "could not allocate space "
>                     "for string");
>                 return (1);
>         }
> +       bzero(buf, sizeof(char) * sz);
>
>         idx = 0;
>         for (i = start, idx = 0; i < argc; i++) {
>
Couldn't this use calloc() instead?
Thanks,
-- 
Ed Schouten <ed@80386.nl>
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAJOYFBBSn7GxzZS0hXzD03M-u_MQnjXFf9wDxxWO4TZWHSnynw>
