Skip site navigation (1)Skip section navigation (2)
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>