cvs commit: src/usr.sbin/iostat iostat.c
Maxim Konovalov
maxim at macomnet.ru
Sat Jan 12 13:26:11 PST 2008
On Sat, 12 Jan 2008, 15:52-0500, David Schultz wrote:
> On Sat, Jan 12, 2008, Maxim Konovalov wrote:
> > - asprintf(&devname, "%s%d",
> > + if (asprintf(&devname, "%s%d",
> > cur.dinfo->devices[di].device_name,
> > - cur.dinfo->devices[di].unit_number);
> > + cur.dinfo->devices[di].unit_number) == 1)
> > + errx(1, "asprintf() failed (out of memory?)");
>
> err(1, "asprintf") will produce a more meaningful error message,
> by the way. Also, I think you wanted to compare the return value
> of asprintf to -1 instead of 1.
>
err(1, "asprintf") was my first version, yes. But I failed to find in
the man page it sets errno. Thanks for the review, I'll fix the typo
shortly.
--
Maxim Konovalov
More information about the cvs-src
mailing list