Hello,

I found a small (copy/paste) mistake in this patch:

-    /* w2k3,XP, newer w2k: CRYPT_E_NO_MATCH */
-    ok(!ret && (GetLastError() == CRYPT_E_NO_MATCH),
+    /* w2k3,XP, newer w2k: CRYPT_E_NO_MATCH, older w2k: CRYPT_E_ASN1_BADTAG */
+    ok(!ret && (GetLastError() == CRYPT_E_NO_MATCH ||
+                broken(GetLastError() == CRYPT_E_NO_MATCH)),

The comment indicates the alternative error, but the actual code doesn't check 
that one

HTH,
Joris


      


Reply via email to