mscoree/tests: Remove the parent argument from create_new_dir() (Coverity).

This commit is contained in:
Sven Baars 2023-05-07 14:23:49 +02:00 committed by Alexandre Julliard
parent 8d11327ecf
commit 1b36e9b2fa

View file

@ -592,18 +592,14 @@ static BOOL compile_cs(const WCHAR *source, const WCHAR *target, const WCHAR *ty
return ret; return ret;
} }
static BOOL create_new_dir(WCHAR newdir[MAX_PATH], const WCHAR* parent, static BOOL create_new_dir(WCHAR newdir[MAX_PATH], const WCHAR* prefix)
const WCHAR* prefix)
{ {
WCHAR path[MAX_PATH]; WCHAR path[MAX_PATH];
BOOL try_tmpdir = TRUE; BOOL try_tmpdir = TRUE;
static unsigned i = 0; static unsigned i = 0;
if (!parent) GetCurrentDirectoryW(ARRAY_SIZE(path), path);
{
GetCurrentDirectoryW(ARRAY_SIZE(path), path);
parent = path;
}
while (1) while (1)
{ {
swprintf(newdir, MAX_PATH, L"%s\\%s%04d", path, prefix, i); swprintf(newdir, MAX_PATH, L"%s\\%s%04d", path, prefix, i);
@ -617,7 +613,6 @@ static BOOL create_new_dir(WCHAR newdir[MAX_PATH], const WCHAR* parent,
try_tmpdir = FALSE; try_tmpdir = FALSE;
GetTempPathW(ARRAY_SIZE(path), path); GetTempPathW(ARRAY_SIZE(path), path);
path[wcslen(path) - 1] = 0; /* redundant trailing backslash */ path[wcslen(path) - 1] = 0; /* redundant trailing backslash */
parent = path;
break; break;
case ERROR_ALREADY_EXISTS: case ERROR_ALREADY_EXISTS:
i++; i++;
@ -638,7 +633,7 @@ static void test_loadpaths_execute(const WCHAR *exe_name, const WCHAR *dll_name,
DWORD exit_code = 0xdeadbeef; DWORD exit_code = 0xdeadbeef;
BOOL ret; BOOL ret;
ok(create_new_dir(tmpdir, NULL, L"loadpaths"), ok(create_new_dir(tmpdir, L"loadpaths"),
"failed to create a new dir %lu\n", GetLastError()); "failed to create a new dir %lu\n", GetLastError());
end = tmpdir + wcslen(tmpdir); end = tmpdir + wcslen(tmpdir);