[Bug 861132] Re: setenv ("NAME", NULL) corrupts environment

[Robert Ancell]

** Patch added: "Proposed fix"
   https://bugs.launchpad.net/ubuntu/+source/eglibc/+bug/861132/+attachment/2472890/+files/0001-Fix-setenv-NAME-NULL-corrupting-the-environment.patch

** Bug watch added: Sourceware.org Bugzilla #13231
   http://sourceware.org/bugzilla/show_bug.cgi?id=13231

** Also affects: eglibc via
   http://sourceware.org/bugzilla/show_bug.cgi?id=13231
   Importance: Unknown
       Status: Unknown

-- 
You received this bug notification because you are a member of Ubuntu
Foundations Bugs, which is subscribed to eglibc in Ubuntu.
https://bugs.launchpad.net/bugs/861132

Title:
  setenv ("NAME", NULL) corrupts environment

Status in Embedded GLIBC:
  Unknown
Status in “eglibc” package in Ubuntu:
  New

Bug description:
  setenv ("NAME", NULL) corrupts the environment.  It doesn't seem
  specified what the function should do when value is NULL, but the code
  does check for it - it just does the wrong thing:

  stdlib/setenv.c:
  ...
  __add_to_environ (name, value, combined, replace)
  ...
    const size_t vallen = value != NULL ? strlen (value) + 1 : 0;
  ...
        memcpy (new_value, name, namelen);
        new_value[namelen] = '=';
        memcpy (&new_value[namelen + 1], value, vallen);
  ...

  i.e. the new value is set to "NAME=" without the trailing nul
  character.

  Found in bug 861123 where indicator-datetime does a:
  x = g_strdup (getenv ("NAME"));
  unsetenv ("NAME");
  // do something
  setenv ("NAME", x);

To manage notifications about this bug go to:
https://bugs.launchpad.net/eglibc/+bug/861132/+subscriptions