ReceiveTempSMS
/
Sweden-Number
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

infosoft/tests: Fix the infinite loop and crash on Windows 10 1607+. 

The test only has expected data for the first four words. So abort if
more are found.

Signed-off-by: Francois Gouget <fgouget@codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard@winehq.org>
This commit is contained in:
Francois Gouget 2021-05-10 15:53:19 +02:00 committed by Alexandre Julliard
parent 21e002aa7e
commit 86edbc8b14
1 changed files with 13 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
dlls/infosoft/tests/infosoft.c
View File

@ -70,11 +70,20 @@ static ULONG WINAPI ws_Release(IWordSink *iface)
static HRESULT WINAPI ws_PutWord(IWordSink *iface, ULONG cwc, const WCHAR *pwcInBuf, static HRESULT WINAPI ws_PutWord(IWordSink *iface, ULONG cwc, const WCHAR *pwcInBuf,
ULONG cwcSrcLen, ULONG cwcSrcPos) ULONG cwcSrcLen, ULONG cwcSrcPos)
{ {
ok(testres[wordnum].len == cwcSrcLen, "wrong length\n"); HRESULT rc = S_OK;
ok(!cwcSrcPos ||(testres[wordnum].ofs == cwcSrcPos), "wrong offset\n"); if (wordnum < ARRAY_SIZE(testres))
ok(!memcmp(testres[wordnum].data, pwcInBuf, cwcSrcLen), "wrong data\n"); {
ok(testres[wordnum].len == cwcSrcLen, "wrong length\n");
ok(!cwcSrcPos ||(testres[wordnum].ofs == cwcSrcPos), "wrong offset\n");
ok(!memcmp(testres[wordnum].data, pwcInBuf, cwcSrcLen), "wrong data\n");
}
else
{
ok(0, "found too many words: %d\n", wordnum + 1);
rc = E_FAIL;
}
wordnum++; wordnum++;
return S_OK; return rc;
} }
static HRESULT WINAPI ws_PutAltWord(IWordSink *iface, ULONG cwc, const WCHAR *pwcInBuf, static HRESULT WINAPI ws_PutAltWord(IWordSink *iface, ULONG cwc, const WCHAR *pwcInBuf,