HEADS DOWN

Andrey Chernov ache at freebsd.org
Thu May 3 16:04:03 UTC 2007


On Wed, May 02, 2007 at 11:08:18PM -0500, Sean C. Farley wrote:
>  On Wed, 2 May 2007, Daniel Eischen wrote:
> 
> > On Wed, 2 May 2007, Sean C. Farley wrote:
> 
>  <snip>
> 
> >> 2. getenv() sets errno to EINVAL and returns NULL if given a bad name
> >>    to find.  setenv() and unsetenv() perform the same check on the
> >>    name; should not getenv() do the same?  The check is easy to
> >>    remove.
> >
> > I don't think getenv() should set errno.  The fact that it
> > returns NULL is sufficient and POSIX doesn't define any errors
> > for it.
> 
>  Fixed for errno.  Also, no value is appropriate for errno when the name
>  does not exist.  How about the feature that getenv() returns a NULL for
>  a bad name instead of allowing a core dump?  Is that acceptable?

Speaking about POSIXed error checking in *env() you can look at my backed 
out implemetation (via cvs diff), you may find it useful for you.

-- 
http://ache.pp.ru/


More information about the freebsd-arch mailing list