Date: Fri, 11 May 2012 20:04:12 +0300 From: Konstantin Belousov <kostikbel@gmail.com> To: Andrey Chernov <ache@freebsd.org>, Konstantin Belousov <kib@freebsd.org>, src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: Re: svn commit: r235266 - head/lib/libc/stdlib Message-ID: <20120511170412.GR2358@deviant.kiev.zoral.com.ua> In-Reply-To: <20120511145432.GA19173@vniz.net> References: <201205111129.q4BBT85G081093@svn.freebsd.org> <20120511145432.GA19173@vniz.net>
next in thread | previous in thread | raw e-mail | index | archive | help
[-- Attachment #1 --]
On Fri, May 11, 2012 at 06:54:32PM +0400, Andrey Chernov wrote:
> Thanks, but in this two places old errno value should be saved across
> free() because free() may damage it:
I do not agree. free() cannot change errno.
>
> On Fri, May 11, 2012 at 11:29:08AM +0000, Konstantin Belousov wrote:
> ...
> > + if (lstat(resolved, &sb) != 0) {
> > + if (m)
> > + free(resolved);
> > +
> return (NULL);
> ...
> > + errno = ENOTDIR;
> > if (m)
> > free(resolved);
> > return (NULL);
>
[-- Attachment #2 --]
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.12 (FreeBSD)
iEYEARECAAYFAk+tRosACgkQC3+MBN1Mb4j+YQCgrlfXN2Ip4mlwRoeEi4tnIbDv
ujYAoJ71OuDNL42dKPGDPL0+pQlUrHwa
=n1I7
-----END PGP SIGNATURE-----
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20120511170412.GR2358>