Couldn't we have a better pattern for this?

    if (__environ == NULL || name[0] == '\0')
      return NULL;