crypt32/tests: Make the msg tests usable on win9x (avoid crash)
Detlef Riekenberg
wine.dev at web.de
Sun Jul 20 07:37:15 CDT 2008
First of all: crypt32 is cryptic
- Empty lines between test-blocks would make it much more readable
- The ERROR_* codes in the Header are decimal.
Print the value from GetLastError() as hex is a bad idea, but print
the value as hex without an "0x"-prefix makes it stupid, when you are
not familiar with the code.
The current code crashed on win9x:
- pvCtrlPara in CryptMsgControl is referenced without validation (!
NULL)
for the first test (dwCtrlType == CMSG_CTRL_VERIFY_SIGNATURE)
=> I did not care abot other dwCtrlTypes and skip the whole test
- CryptAcquireContextW is not implemented in advapi32 on win9x,
and that results in failed and crashed tests.
=> I used the current code to detect nt/win9x
=> For the tests, i changed them to use CryptAcquireContextA
Is there any special reason to use CryptAcquireContextW in the tests?
CryptAcquireContextA works fine on win98se, w2k and Wine
Changelog:
crypt32/tests: Make the msg tests usable on win9x
--
By by ... Detlef
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-crypt32-tests-Make-the-msg-tests-usable-on-win9x.txt
Type: application/mbox
Size: 0 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-patches/attachments/20080720/c8c4591c/attachment-0001.bin
More information about the wine-patches
mailing list