February 2016 - Wine-commits

Andrew Eikum : winmm: Reset dwUser in mixerGetLineInfo.
by Alexandre Julliard 11 Feb '16

11 Feb '16

Module: wine Branch: master Commit: 992e3c5dc8b1c5d52cffef96e7452cb17300e472 URL: http://source.winehq.org/git/wine.git/?a=commit;h=992e3c5dc8b1c5d52cffef96e… Author: Andrew Eikum <aeikum(a)codeweavers.com> Date: Wed Feb 10 12:33:23 2016 -0600 winmm: Reset dwUser in mixerGetLineInfo. Signed-off-by: Andrew Eikum <aeikum(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/winmm/tests/mixer.c | 4 ++++ dlls/winmm/waveform.c | 2 ++ 2 files changed, 6 insertions(+) diff --git a/dlls/winmm/tests/mixer.c b/dlls/winmm/tests/mixer.c index 82eb338..f03a71e 100644 --- a/dlls/winmm/tests/mixer.c +++ b/dlls/winmm/tests/mixer.c @@ -438,11 +438,13 @@ static void mixer_test_deviceA(int device) mixerlineA.cbStruct = sizeof(mixerlineA); mixerlineA.dwDestination=d; + mixerlineA.dwUser = (ULONG_PTR)0xdeadbeef; rc = mixerGetLineInfoA(mix, &mixerlineA, MIXER_GETLINEINFOF_DESTINATION); ok(rc==MMSYSERR_NOERROR||rc==MMSYSERR_NODRIVER, "mixerGetLineInfoA(MIXER_GETLINEINFOF_DESTINATION): " "MMSYSERR_NOERROR expected, got %s\n", mmsys_error(rc)); + ok(mixerlineA.dwUser == 0, "dwUser was not reset\n"); if (rc==MMSYSERR_NODRIVER) trace(" No Driver\n"); else if (rc==MMSYSERR_NOERROR) { @@ -822,11 +824,13 @@ static void mixer_test_deviceW(int device) mixerlineW.cbStruct = sizeof(mixerlineW); mixerlineW.dwDestination=d; + mixerlineW.dwUser = (ULONG_PTR)0xdeadbeef; rc = mixerGetLineInfoW(mix, &mixerlineW, MIXER_GETLINEINFOF_DESTINATION); ok(rc==MMSYSERR_NOERROR||rc==MMSYSERR_NODRIVER, "mixerGetLineInfoW(MIXER_GETLINEINFOF_DESTINATION): " "MMSYSERR_NOERROR expected, got %s\n", mmsys_error(rc)); + ok(mixerlineW.dwUser == 0, "dwUser was not reset\n"); if (rc==MMSYSERR_NODRIVER) trace(" No Driver\n"); else if (rc==MMSYSERR_NOERROR && winetest_interactive) { diff --git a/dlls/winmm/waveform.c b/dlls/winmm/waveform.c index 93a5004..a4917a2 100644 --- a/dlls/winmm/waveform.c +++ b/dlls/winmm/waveform.c @@ -4287,6 +4287,8 @@ UINT WINAPI mixerGetLineInfoW(HMIXEROBJ hmix, LPMIXERLINEW lpmliW, DWORD fdwInfo if(!mmdevice) return MMSYSERR_INVALHANDLE; + lpmliW->dwUser = 0; + switch(fdwInfo & MIXER_GETLINEINFOF_QUERYMASK){ case MIXER_GETLINEINFOF_DESTINATION: return WINMM_GetDestinationLineInfo(mmdevice, mmdev_index, lpmliW,

1 0

Andrew Eikum : ntdll: Initialize variable (Coverity).
by Alexandre Julliard 11 Feb '16

11 Feb '16

Module: wine Branch: master Commit: 0df38064c2940275c110274c9880ec4e7b0e8e3d URL: http://source.winehq.org/git/wine.git/?a=commit;h=0df38064c2940275c110274c9… Author: Andrew Eikum <aeikum(a)codeweavers.com> Date: Wed Feb 10 12:53:42 2016 -0600 ntdll: Initialize variable (Coverity). Signed-off-by: Andrew Eikum <aeikum(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/ntdll/nt.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/ntdll/nt.c b/dlls/ntdll/nt.c index 5c8b7fd..a18dfae 100644 --- a/dlls/ntdll/nt.c +++ b/dlls/ntdll/nt.c @@ -2340,7 +2340,7 @@ NTSTATUS WINAPI NtQuerySystemInformation( NTSTATUS WINAPI NtQuerySystemInformationEx(SYSTEM_INFORMATION_CLASS SystemInformationClass, void *Query, ULONG QueryLength, void *SystemInformation, ULONG Length, ULONG *ResultLength) { - ULONG len; + ULONG len = 0; NTSTATUS ret = STATUS_NOT_IMPLEMENTED; TRACE("(0x%08x,%p,%u,%p,%u,%p) stub\n", SystemInformationClass, Query, QueryLength, SystemInformation,

1 0

Andrew Eikum : dsound: Avoid NULL pointer dereference (Coverity).
by Alexandre Julliard 11 Feb '16

11 Feb '16

Module: wine Branch: master Commit: 468477b69410f3f9731b4f9fec568945acb8e1ea URL: http://source.winehq.org/git/wine.git/?a=commit;h=468477b69410f3f9731b4f9fe… Author: Andrew Eikum <aeikum(a)codeweavers.com> Date: Wed Feb 10 12:54:04 2016 -0600 dsound: Avoid NULL pointer dereference (Coverity). Signed-off-by: Andrew Eikum <aeikum(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/dsound/buffer.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/dsound/buffer.c b/dlls/dsound/buffer.c index 2a80c3f..e85a5d1 100644 --- a/dlls/dsound/buffer.c +++ b/dlls/dsound/buffer.c @@ -1205,9 +1205,9 @@ HRESULT IDirectSoundBufferImpl_Duplicate( HeapFree(GetProcessHeap(),0,dsb->pwfx); HeapFree(GetProcessHeap(),0,dsb); dsb = NULL; - } + }else + IDirectSoundBuffer8_AddRef(&dsb->IDirectSoundBuffer8_iface); - IDirectSoundBuffer8_AddRef(&dsb->IDirectSoundBuffer8_iface); *ppdsb = dsb; return hres; }

1 0

Andrew Eikum : dsound: Handle default case in dsound3d mode switch ( Coverity).
by Alexandre Julliard 11 Feb '16

11 Feb '16

Module: wine Branch: master Commit: f461610e6c93f74e7eb07567f0740c8f357eb951 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f461610e6c93f74e7eb07567f… Author: Andrew Eikum <aeikum(a)codeweavers.com> Date: Wed Feb 10 12:54:15 2016 -0600 dsound: Handle default case in dsound3d mode switch (Coverity). Signed-off-by: Andrew Eikum <aeikum(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/dsound/sound3d.c | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/dlls/dsound/sound3d.c b/dlls/dsound/sound3d.c index e428053..840ab93 100644 --- a/dlls/dsound/sound3d.c +++ b/dlls/dsound/sound3d.c @@ -170,11 +170,6 @@ void DSOUND_Calc3DBuffer(IDirectSoundBufferImpl *dsb) switch (dsb->ds3db_ds3db.dwMode) { - case DS3DMODE_DISABLE: - TRACE("3D processing disabled\n"); - /* this one is here only to eliminate annoying warning message */ - DSOUND_RecalcVolPan (&dsb->volpan); - return; case DS3DMODE_NORMAL: TRACE("Normal 3D processing mode\n"); /* we need to calculate distance between buffer and listener*/ @@ -187,6 +182,11 @@ void DSOUND_Calc3DBuffer(IDirectSoundBufferImpl *dsb) vDistance = dsb->ds3db_ds3db.vPosition; flDistance = VectorMagnitude (&vDistance); break; + default: + TRACE("3D processing disabled\n"); + /* this one is here only to eliminate annoying warning message */ + DSOUND_RecalcVolPan (&dsb->volpan); + return; } if (flDistance > dsb->ds3db_ds3db.flMaxDistance)

1 0

Andrew Eikum : xaudio2: Protect access to source voice during creation ( Coverity).
by Alexandre Julliard 11 Feb '16

11 Feb '16

Module: wine Branch: master Commit: 2c82e8b477fe476b2eb3ad4452225619ca2e7a58 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2c82e8b477fe476b2eb3ad445… Author: Andrew Eikum <aeikum(a)codeweavers.com> Date: Wed Feb 10 12:54:25 2016 -0600 xaudio2: Protect access to source voice during creation (Coverity). Signed-off-by: Andrew Eikum <aeikum(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/xaudio2_7/xaudio_dll.c | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/dlls/xaudio2_7/xaudio_dll.c b/dlls/xaudio2_7/xaudio_dll.c index 6d766a1..0c39354 100644 --- a/dlls/xaudio2_7/xaudio_dll.c +++ b/dlls/xaudio2_7/xaudio_dll.c @@ -1367,8 +1367,10 @@ static HRESULT WINAPI IXAudio2Impl_CreateSourceVoice(IXAudio2 *iface, EnterCriticalSection(&This->lock); LIST_FOR_EACH_ENTRY(src, &This->source_voices, XA2SourceImpl, entry){ + EnterCriticalSection(&src->lock); if(!src->in_use) break; + LeaveCriticalSection(&src->lock); } if(&src->entry == &This->source_voices){ @@ -1394,6 +1396,8 @@ static HRESULT WINAPI IXAudio2Impl_CreateSourceVoice(IXAudio2 *iface, src->lock.DebugInfo->Spare[0] = (DWORD_PTR)(__FILE__ ": XA2SourceImpl.lock"); src->xa2 = This; + + EnterCriticalSection(&src->lock); } src->in_use = TRUE; @@ -1406,6 +1410,7 @@ static HRESULT WINAPI IXAudio2Impl_CreateSourceVoice(IXAudio2 *iface, src->al_fmt = get_al_format(pSourceFormat); if(!src->al_fmt){ src->in_use = FALSE; + LeaveCriticalSection(&src->lock); WARN("OpenAL can't convert this format!\n"); return AUDCLNT_E_UNSUPPORTED_FORMAT; } @@ -1418,6 +1423,7 @@ static HRESULT WINAPI IXAudio2Impl_CreateSourceVoice(IXAudio2 *iface, if(FAILED(hr)){ HeapFree(GetProcessHeap(), 0, src->fmt); src->in_use = FALSE; + LeaveCriticalSection(&src->lock); return hr; } @@ -1428,6 +1434,7 @@ static HRESULT WINAPI IXAudio2Impl_CreateSourceVoice(IXAudio2 *iface, ERR_(winediag)("OpenAL ran out of sources, consider increasing its source limit.\n"); HeapFree(GetProcessHeap(), 0, src->fmt); src->in_use = FALSE; + LeaveCriticalSection(&src->lock); return E_OUTOFMEMORY; } @@ -1435,6 +1442,8 @@ static HRESULT WINAPI IXAudio2Impl_CreateSourceVoice(IXAudio2 *iface, alSourcePlay(src->al_src); + LeaveCriticalSection(&src->lock); + #if XAUDIO2_VER == 0 *ppSourceVoice = (IXAudio2SourceVoice*)&src->IXAudio20SourceVoice_iface; #elif XAUDIO2_VER <= 3

1 0

Andrew Eikum : xaudio2: Protect access to submix voice during creation ( Coverity).
by Alexandre Julliard 11 Feb '16

11 Feb '16

Module: wine Branch: master Commit: 7bb885a3d3ac4fdc435fc79fc5632b6b9e70ffa7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=7bb885a3d3ac4fdc435fc79fc… Author: Andrew Eikum <aeikum(a)codeweavers.com> Date: Wed Feb 10 12:54:40 2016 -0600 xaudio2: Protect access to submix voice during creation (Coverity). Signed-off-by: Andrew Eikum <aeikum(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/xaudio2_7/xaudio_dll.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/dlls/xaudio2_7/xaudio_dll.c b/dlls/xaudio2_7/xaudio_dll.c index 0c39354..503c974 100644 --- a/dlls/xaudio2_7/xaudio_dll.c +++ b/dlls/xaudio2_7/xaudio_dll.c @@ -1475,8 +1475,10 @@ static HRESULT WINAPI IXAudio2Impl_CreateSubmixVoice(IXAudio2 *iface, EnterCriticalSection(&This->lock); LIST_FOR_EACH_ENTRY(sub, &This->submix_voices, XA2SubmixImpl, entry){ + EnterCriticalSection(&sub->lock); if(!sub->in_use) break; + LeaveCriticalSection(&sub->lock); } if(&sub->entry == &This->submix_voices){ @@ -1498,11 +1500,14 @@ static HRESULT WINAPI IXAudio2Impl_CreateSubmixVoice(IXAudio2 *iface, InitializeCriticalSection(&sub->lock); sub->lock.DebugInfo->Spare[0] = (DWORD_PTR)(__FILE__ ": XA2SubmixImpl.lock"); + + EnterCriticalSection(&sub->lock); } sub->in_use = TRUE; LeaveCriticalSection(&This->lock); + LeaveCriticalSection(&sub->lock); #if XAUDIO2_VER == 0 *ppSubmixVoice = (IXAudio2SubmixVoice*)&sub->IXAudio20SubmixVoice_iface;

1 0

Andrew Eikum : wineoss.drv: Avoid potential overflow (Coverity).
by Alexandre Julliard 11 Feb '16

11 Feb '16

Module: wine Branch: master Commit: b2caf7513a3851289766d09d3fa9da9a2814c940 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b2caf7513a3851289766d09d3… Author: Andrew Eikum <aeikum(a)codeweavers.com> Date: Wed Feb 10 12:54:51 2016 -0600 wineoss.drv: Avoid potential overflow (Coverity). Signed-off-by: Andrew Eikum <aeikum(a)codeweavers.com> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/wineoss.drv/mmdevdrv.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/wineoss.drv/mmdevdrv.c b/dlls/wineoss.drv/mmdevdrv.c index d2a0c86..bbac32e 100644 --- a/dlls/wineoss.drv/mmdevdrv.c +++ b/dlls/wineoss.drv/mmdevdrv.c @@ -1203,7 +1203,7 @@ static HRESULT WINAPI AudioClient_GetStreamLatency(IAudioClient *iface, /* pretend we process audio in Period chunks, so max latency includes * the period time. Some native machines add .6666ms in shared mode. */ - *latency = This->period_us * 10 + 6666; + *latency = (REFERENCE_TIME)This->period_us * 10 + 6666; LeaveCriticalSection(&This->lock);

1 0

Sebastian Lackner : combase: Implement WindowsCompareStringOrdinal.
by Alexandre Julliard 11 Feb '16

11 Feb '16

Module: wine Branch: master Commit: e39405d44db8fa59a6f46b904fdb80ff29dc1866 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e39405d44db8fa59a6f46b904… Author: Sebastian Lackner <sebastian(a)fds-team.de> Date: Wed Feb 10 23:25:23 2016 +0100 combase: Implement WindowsCompareStringOrdinal. Signed-off-by: Sebastian Lackner <sebastian(a)fds-team.de> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- .../api-ms-win-core-winrt-string-l1-1-0.spec | 2 +- dlls/combase/combase.spec | 2 +- dlls/combase/string.c | 33 ++++++++++++++++++++++ include/winnls.h | 1 + 4 files changed, 36 insertions(+), 2 deletions(-) diff --git a/dlls/api-ms-win-core-winrt-string-l1-1-0/api-ms-win-core-winrt-string-l1-1-0.spec b/dlls/api-ms-win-core-winrt-string-l1-1-0/api-ms-win-core-winrt-string-l1-1-0.spec index fa048d8..1b661aa 100644 --- a/dlls/api-ms-win-core-winrt-string-l1-1-0/api-ms-win-core-winrt-string-l1-1-0.spec +++ b/dlls/api-ms-win-core-winrt-string-l1-1-0/api-ms-win-core-winrt-string-l1-1-0.spec @@ -6,7 +6,7 @@ @ stub HSTRING_UserSize64 @ stub HSTRING_UserUnmarshal @ stub HSTRING_UserUnmarshal64 -@ stub WindowsCompareStringOrdinal +@ stdcall WindowsCompareStringOrdinal(ptr ptr ptr) combase.WindowsCompareStringOrdinal @ stdcall WindowsConcatString(ptr ptr ptr) combase.WindowsConcatString @ stdcall WindowsCreateString(wstr long ptr) combase.WindowsCreateString @ stdcall WindowsCreateStringReference(wstr long ptr ptr) combase.WindowsCreateStringReference diff --git a/dlls/combase/combase.spec b/dlls/combase/combase.spec index 5082f39..430ca95 100644 --- a/dlls/combase/combase.spec +++ b/dlls/combase/combase.spec @@ -287,7 +287,7 @@ @ stub WdtpInterfacePointer_UserSize64 @ stdcall WdtpInterfacePointer_UserUnmarshal(ptr ptr ptr ptr) ole32.WdtpInterfacePointer_UserUnmarshal @ stub WdtpInterfacePointer_UserUnmarshal64 -@ stub WindowsCompareStringOrdinal +@ stdcall WindowsCompareStringOrdinal(ptr ptr ptr) @ stdcall WindowsConcatString(ptr ptr ptr) @ stdcall WindowsCreateString(wstr long ptr) @ stdcall WindowsCreateStringReference(wstr long ptr ptr) diff --git a/dlls/combase/string.c b/dlls/combase/string.c index dd7c8e9..bd18a73 100644 --- a/dlls/combase/string.c +++ b/dlls/combase/string.c @@ -372,3 +372,36 @@ BOOL WINAPI WindowsIsStringEmpty(HSTRING str) return TRUE; return priv->length == 0; } + +/*********************************************************************** + * WindowsCompareStringOrdinal (combase.@) + */ +HRESULT WINAPI WindowsCompareStringOrdinal(HSTRING str1, HSTRING str2, INT32 *res) +{ + struct hstring_private *priv1 = impl_from_HSTRING(str1); + struct hstring_private *priv2 = impl_from_HSTRING(str2); + const WCHAR *buf1 = empty, *buf2 = empty; + UINT32 len1 = 0, len2 = 0; + + TRACE("(%p, %p, %p)\n", str1, str2, res); + + if (res == NULL) + return E_INVALIDARG; + if (str1 == str2) + { + *res = 0; + return S_OK; + } + if (str1) + { + buf1 = priv1->buffer; + len1 = priv1->length; + } + if (str2) + { + buf2 = priv2->buffer; + len2 = priv2->length; + } + *res = CompareStringOrdinal(buf1, len1, buf2, len2, FALSE) - CSTR_EQUAL; + return S_OK; +} diff --git a/include/winnls.h b/include/winnls.h index 603c408..6e5897b 100644 --- a/include/winnls.h +++ b/include/winnls.h @@ -831,6 +831,7 @@ WINBASEAPI INT WINAPI CompareStringA(LCID,DWORD,LPCSTR,INT,LPCSTR,INT); WINBASEAPI INT WINAPI CompareStringW(LCID,DWORD,LPCWSTR,INT,LPCWSTR,INT); #define CompareString WINELIB_NAME_AW(CompareString) WINBASEAPI INT WINAPI CompareStringEx(LPCWSTR,DWORD,LPCWSTR,INT,LPCWSTR,INT,LPNLSVERSIONINFO,LPVOID,LPARAM); +WINBASEAPI INT WINAPI CompareStringOrdinal(const WCHAR *,INT,const WCHAR *,INT,BOOL); WINBASEAPI LCID WINAPI ConvertDefaultLocale(LCID); WINBASEAPI BOOL WINAPI EnumCalendarInfoA(CALINFO_ENUMPROCA,LCID,CALID,CALTYPE); WINBASEAPI BOOL WINAPI EnumCalendarInfoW(CALINFO_ENUMPROCW,LCID,CALID,CALTYPE);

1 0

Sebastian Lackner : combase/tests: Add tests for WindowsCompareStringOrdinal.
by Alexandre Julliard 11 Feb '16

11 Feb '16

Module: wine Branch: master Commit: f3827dab8f2ed5f678d6f6369ca917e8fb0ae1ff URL: http://source.winehq.org/git/wine.git/?a=commit;h=f3827dab8f2ed5f678d6f6369… Author: Sebastian Lackner <sebastian(a)fds-team.de> Date: Wed Feb 10 23:25:48 2016 +0100 combase/tests: Add tests for WindowsCompareStringOrdinal. Signed-off-by: Sebastian Lackner <sebastian(a)fds-team.de> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- dlls/combase/tests/string.c | 63 +++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 63 insertions(+) diff --git a/dlls/combase/tests/string.c b/dlls/combase/tests/string.c index 8e112c3..db6c8f5 100644 --- a/dlls/combase/tests/string.c +++ b/dlls/combase/tests/string.c @@ -27,6 +27,7 @@ #include "wine/test.h" +static HRESULT (WINAPI *pWindowsCompareStringOrdinal)(HSTRING, HSTRING, INT32 *); static HRESULT (WINAPI *pWindowsConcatString)(HSTRING, HSTRING, HSTRING *); static HRESULT (WINAPI *pWindowsCreateString)(LPCWSTR, UINT32, HSTRING *); static HRESULT (WINAPI *pWindowsCreateStringReference)(LPCWSTR, UINT32, HSTRING_HEADER *, HSTRING *); @@ -52,6 +53,7 @@ static BOOL init_functions(void) win_skip("Failed to load combase.dll, skipping tests\n"); return FALSE; } + SET(WindowsCompareStringOrdinal); SET(WindowsConcatString); SET(WindowsCreateString); SET(WindowsCreateStringReference); @@ -394,6 +396,66 @@ static void test_concat(void) ok(concat == NULL, "Concatenate created new string\n"); } +static void test_compare(void) +{ + HSTRING str1, str2; + HSTRING_HEADER header1, header2; + INT32 res; + + /* Test comparison of string buffers */ + ok(pWindowsCreateString(input_string1, 3, &str1) == S_OK, "Failed to create string\n"); + ok(pWindowsCreateString(input_string2, 3, &str2) == S_OK, "Failed to create string\n"); + + ok(pWindowsCompareStringOrdinal(str1, str1, &res) == S_OK, "Failed to compare string\n"); + ok(res == 0, "Expected 0, got %d\n", res); + ok(pWindowsCompareStringOrdinal(str1, str2, &res) == S_OK, "Failed to compare string\n"); + ok(res == -1, "Expected -1, got %d\n", res); + ok(pWindowsCompareStringOrdinal(str2, str1, &res) == S_OK, "Failed to compare string\n"); + ok(res == 1, "Expected 1, got %d\n", res); + ok(pWindowsCompareStringOrdinal(str2, str2, &res) == S_OK, "Failed to compare string\n"); + ok(res == 0, "Expected 0, got %d\n", res); + ok(pWindowsCompareStringOrdinal(str1, NULL, &res) == S_OK, "Failed to compare string\n"); + ok(res == 1, "Expected 1, got %d\n", res); + ok(pWindowsCompareStringOrdinal(NULL, str1, &res) == S_OK, "Failed to compare string\n"); + ok(res == -1, "Expected -1, got %d\n", res); + ok(pWindowsCompareStringOrdinal(str2, NULL, &res) == S_OK, "Failed to compare string\n"); + ok(res == 1, "Expected 1, got %d\n", res); + ok(pWindowsCompareStringOrdinal(NULL, str2, &res) == S_OK, "Failed to compare string\n"); + ok(res == -1, "Expected -1, got %d\n", res); + + ok(pWindowsDeleteString(str2) == S_OK, "Failed to delete string\n"); + ok(pWindowsDeleteString(str1) == S_OK, "Failed to delete string\n"); + + /* Test comparison of string references */ + ok(pWindowsCreateStringReference(input_string1, 3, &header1, &str1) == S_OK, "Failed to create string ref\n"); + ok(pWindowsCreateStringReference(input_string2, 3, &header2, &str2) == S_OK, "Failed to create string ref\n"); + + ok(pWindowsCompareStringOrdinal(str1, str1, &res) == S_OK, "Failed to compare string\n"); + ok(res == 0, "Expected 0, got %d\n", res); + ok(pWindowsCompareStringOrdinal(str1, str2, &res) == S_OK, "Failed to compare string\n"); + ok(res == -1, "Expected -1, got %d\n", res); + ok(pWindowsCompareStringOrdinal(str2, str1, &res) == S_OK, "Failed to compare string\n"); + ok(res == 1, "Expected 1, got %d\n", res); + ok(pWindowsCompareStringOrdinal(str2, str2, &res) == S_OK, "Failed to compare string\n"); + ok(res == 0, "Expected 0, got %d\n", res); + ok(pWindowsCompareStringOrdinal(str1, NULL, &res) == S_OK, "Failed to compare string\n"); + ok(res == 1, "Expected 1, got %d\n", res); + ok(pWindowsCompareStringOrdinal(NULL, str1, &res) == S_OK, "Failed to compare string\n"); + ok(res == -1, "Expected -1, got %d\n", res); + ok(pWindowsCompareStringOrdinal(str2, NULL, &res) == S_OK, "Failed to compare string\n"); + ok(res == 1, "Expected 1, got %d\n", res); + ok(pWindowsCompareStringOrdinal(NULL, str2, &res) == S_OK, "Failed to compare string\n"); + ok(res == -1, "Expected -1, got %d\n", res); + + ok(pWindowsDeleteString(str2) == S_OK, "Failed to delete string ref\n"); + ok(pWindowsDeleteString(str1) == S_OK, "Failed to delete string ref\n"); + + /* Test comparison of two empty strings */ + ok(pWindowsCompareStringOrdinal(NULL, NULL, NULL) == E_INVALIDARG, "Incorrect error handling\n"); + ok(pWindowsCompareStringOrdinal(NULL, NULL, &res) == S_OK, "Failed to compare NULL string\n"); + ok(res == 0, "Expected 0, got %d\n", res); +} + START_TEST(string) { if (!init_functions()) @@ -404,4 +466,5 @@ START_TEST(string) test_string_buffer(); test_substring(); test_concat(); + test_compare(); }

1 0

Sebastian Lackner : combase: Implement WindowsTrimStringStart.
by Alexandre Julliard 11 Feb '16

11 Feb '16

Module: wine Branch: master Commit: 211b8b5f61e19acc62e4d42cbf765c43b4879d73 URL: http://source.winehq.org/git/wine.git/?a=commit;h=211b8b5f61e19acc62e4d42cb… Author: Sebastian Lackner <sebastian(a)fds-team.de> Date: Wed Feb 10 23:27:15 2016 +0100 combase: Implement WindowsTrimStringStart. Signed-off-by: Sebastian Lackner <sebastian(a)fds-team.de> Signed-off-by: Alexandre Julliard <julliard(a)winehq.org> --- .../api-ms-win-core-winrt-string-l1-1-0.spec | 2 +- dlls/combase/combase.spec | 2 +- dlls/combase/string.c | 28 ++++++++++++++++++++++ 3 files changed, 30 insertions(+), 2 deletions(-) diff --git a/dlls/api-ms-win-core-winrt-string-l1-1-0/api-ms-win-core-winrt-string-l1-1-0.spec b/dlls/api-ms-win-core-winrt-string-l1-1-0/api-ms-win-core-winrt-string-l1-1-0.spec index 1b661aa..fe63810 100644 --- a/dlls/api-ms-win-core-winrt-string-l1-1-0/api-ms-win-core-winrt-string-l1-1-0.spec +++ b/dlls/api-ms-win-core-winrt-string-l1-1-0/api-ms-win-core-winrt-string-l1-1-0.spec @@ -24,4 +24,4 @@ @ stdcall WindowsSubstring(ptr long ptr) combase.WindowsSubstring @ stdcall WindowsSubstringWithSpecifiedLength(ptr long long ptr) combase.WindowsSubstringWithSpecifiedLength @ stub WindowsTrimStringEnd -@ stub WindowsTrimStringStart +@ stdcall WindowsTrimStringStart(ptr ptr ptr) combase.WindowsTrimStringStart diff --git a/dlls/combase/combase.spec b/dlls/combase/combase.spec index 430ca95..69f47d0 100644 --- a/dlls/combase/combase.spec +++ b/dlls/combase/combase.spec @@ -305,4 +305,4 @@ @ stdcall WindowsSubstring(ptr long ptr) @ stdcall WindowsSubstringWithSpecifiedLength(ptr long long ptr) @ stub WindowsTrimStringEnd -@ stub WindowsTrimStringStart +@ stdcall WindowsTrimStringStart(ptr ptr ptr) diff --git a/dlls/combase/string.c b/dlls/combase/string.c index bd18a73..2d04f15 100644 --- a/dlls/combase/string.c +++ b/dlls/combase/string.c @@ -21,6 +21,7 @@ #include "windows.h" #include "winerror.h" #include "hstring.h" +#include "wine/unicode.h" #include "wine/debug.h" WINE_DEFAULT_DEBUG_CHANNEL(winstring); @@ -405,3 +406,30 @@ HRESULT WINAPI WindowsCompareStringOrdinal(HSTRING str1, HSTRING str2, INT32 *re *res = CompareStringOrdinal(buf1, len1, buf2, len2, FALSE) - CSTR_EQUAL; return S_OK; } + +/*********************************************************************** + * WindowsTrimStringStart (combase.@) + */ +HRESULT WINAPI WindowsTrimStringStart(HSTRING str1, HSTRING str2, HSTRING *out) +{ + struct hstring_private *priv1 = impl_from_HSTRING(str1); + struct hstring_private *priv2 = impl_from_HSTRING(str2); + UINT32 start; + + TRACE("(%p, %p, %p)\n", str1, str2, out); + + if (!out || !str2 || !priv2->length) + return E_INVALIDARG; + if (!str1) + { + *out = NULL; + return S_OK; + } + for (start = 0; start < priv1->length; start++) + { + if (!memchrW(priv2->buffer, priv1->buffer[start], priv2->length)) + break; + } + return start ? WindowsCreateString(&priv1->buffer[start], priv1->length - start, out) : + WindowsDuplicateString(str1, out); +}

1 0