Skip to content

W
wine-winehq
Commit 1f4c48a0 authored by Robert Wilhelm's avatar Robert Wilhelm Committed by Alexandre Julliard
Browse files
Options

scrrun: Correct string length returned from GetTempName.

You have to pass string length without terminal null to SysAllocStringLen. Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=53569
parent 412e1de2
Hide whitespace changes
Inline Side-by-side
Showing with 2 additions and 2 deletions
dlls/scrrun/filesystem.c
View file @ 1f4c48a0
...@@ -3449,7 +3449,7 @@ static HRESULT WINAPI filesys_GetTempName(IFileSystem3 *iface, BSTR *result) ...@@ -3449,7 +3449,7 @@ static HRESULT WINAPI filesys_GetTempName(IFileSystem3 *iface, BSTR *result)
if (!result) if (!result)
return E_POINTER; return E_POINTER;
if (!(*result = SysAllocStringLen(NULL, 13))) if (!(*result = SysAllocStringLen(NULL, 12)))
return E_OUTOFMEMORY; return E_OUTOFMEMORY;
if(!RtlGenRandom(&random, sizeof(random))) if(!RtlGenRandom(&random, sizeof(random)))
......
dlls/scrrun/tests/filesystem.c
View file @ 1f4c48a0
...@@ -485,7 +485,7 @@ static void test_GetTempName(void) ...@@ -485,7 +485,7 @@ static void test_GetTempName(void)
hr = IFileSystem3_GetTempName(fs3, &result); hr = IFileSystem3_GetTempName(fs3, &result);
ok(hr == S_OK, "Unexpected hr %#lx.\n", hr); ok(hr == S_OK, "Unexpected hr %#lx.\n", hr);
ok(!!wcsstr( result,L".tmp"), "GetTempName returned %s, expected .tmp suffix\n", debugstr_w(result)); ok(!!wcsstr( result,L".tmp"), "GetTempName returned %s, expected .tmp suffix\n", debugstr_w(result));
todo_wine ok(SysStringLen(result) == lstrlenW(result),"GetTempName returned %s, has incorrect string len.\n", debugstr_w(result)); ok(SysStringLen(result) == lstrlenW(result),"GetTempName returned %s, has incorrect string len.\n", debugstr_w(result));
SysFreeString(result); SysFreeString(result);
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment