HEADS DOWN

[Sean C. Farley]

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?

Sean
-- 
sean-freebsd at farley.org