Add some test cases for _wputenv, fix deletion of non-existent

environment variables, return -1 on error.
This commit is contained in:
Mike Hearn 2004-07-19 21:23:02 +00:00 committed by Alexandre Julliard
parent 46bdee35c3
commit ad4a00d35a
4 changed files with 47 additions and 2 deletions

View file

@ -91,7 +91,11 @@ int _putenv(const char *str)
*dst++ = *str++;
*dst = '\0';
ret = !SetEnvironmentVariableA(name, value[0] ? value : NULL);
ret = SetEnvironmentVariableA(name, value[0] ? value : NULL) ? 0 : -1;
/* _putenv returns success on deletion of non-existent variable, unlike [Rtl]SetEnvironmentVariable */
if ((ret == -1) && (GetLastError() == ERROR_ENVVAR_NOT_FOUND)) ret = 0;
/* Update the __p__environ array only when already initialized */
if (_environ)
_environ = msvcrt_SnapshotOfEnvironmentA(_environ);
@ -123,7 +127,11 @@ int _wputenv(const MSVCRT_wchar_t *str)
*dst++ = *str++;
*dst = 0;
ret = !SetEnvironmentVariableW(name, value[0] ? value : NULL);
ret = SetEnvironmentVariableW(name, value[0] ? value : NULL) ? 0 : -1;
/* _putenv returns success on deletion of non-existent variable, unlike [Rtl]SetEnvironmentVariable */
if ((ret == -1) && (GetLastError() == ERROR_ENVVAR_NOT_FOUND)) ret = 0;
/* Update the __p__environ array only when already initialized */
if (_environ)
_environ = msvcrt_SnapshotOfEnvironmentA(_environ);

View file

@ -1,5 +1,6 @@
Makefile
cpp.ok
environ.ok
file.ok
headers.ok
heap.ok

View file

@ -8,6 +8,7 @@ EXTRAINCL = -I$(TOPSRCDIR)/include/msvcrt -I$(SRCDIR)/..
CTESTS = \
cpp.c \
environ.c \
file.c \
headers.c \
heap.c \

View file

@ -0,0 +1,35 @@
/*
* Unit tests for C library environment routines
*
* Copyright 2004 Mike Hearn <mh@codeweavers.com>
*
* This library is free software; you can redistribute it and/or
* modify it under the terms of the GNU Lesser General Public
* License as published by the Free Software Foundation; either
* version 2.1 of the License, or (at your option) any later version.
*
* This library is distributed in the hope that it will be useful,
* but WITHOUT ANY WARRANTY; without even the implied warranty of
* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
* Lesser General Public License for more details.
*
* You should have received a copy of the GNU Lesser General Public
* License along with this library; if not, write to the Free Software
* Foundation, Inc., 59 Temple Place, Suite 330, Boston, MA 02111-1307 USA
*/
#include "wine/test.h"
#include <stdlib.h>
START_TEST(environ)
{
ok( _putenv("cat=") == 0, "_putenv failed on deletion of non-existent environment variable\n" );
ok( _putenv("cat=dog") == 0, "failed setting cat=dog\n" );
ok( strcmp(getenv("cat"), "dog") == 0, "getenv did not return 'dog'\n" );
ok( _putenv("cat=") == 0, "failed deleting cat\n" );
ok( _putenv("=") == -1, "should not accept '=' as input\n" );
ok( _putenv("=dog") == -1, "should not accept '=dog' as input\n" );
ok( getenv("nonexistent") == NULL, "getenv should fail with nonexistent var name\n" );
}