commit a15dbfac22b4d75af2d54941a549b358ac14780c
author Reece Dunn <msclrhd@gmail.com> Tue Mar 02 21:31:46 2010 +0000
committer Alexandre Julliard <julliard@winehq.org> Wed Mar 03 11:50:04 2010 +0100
tree 1efcb039d28b6c2f847aad5d4e06efae23be077d
parent 1b46c0660d4baad9f4afe0086fb4e44240b18131

crypt32: Fix CryptFreeTls when calling with a freed index.

dlls/crypt32/main.c

diff --git a/dlls/crypt32/main.c b/dlls/crypt32/main.c
index 1a5972f..5844b18 100644
--- a/dlls/crypt32/main.c
+++ b/dlls/crypt32/main.c
@@ -161,8 +161,13 @@
 
 BOOL WINAPI I_CryptFreeTls(DWORD dwTlsIndex, DWORD unknown)
 {
+    BOOL ret;
+
     TRACE("(%d, %d)\n", dwTlsIndex, unknown);
-    return TlsFree(dwTlsIndex);
+
+    ret = TlsFree(dwTlsIndex);
+    if (!ret) SetLastError( E_INVALIDARG );
+    return ret;
 }
 
 BOOL WINAPI I_CryptGetOssGlobal(DWORD x)

dlls/crypt32/tests/main.c

diff --git a/dlls/crypt32/tests/main.c b/dlls/crypt32/tests/main.c
index 9b622bb..7c11e8e 100644
--- a/dlls/crypt32/tests/main.c
+++ b/dlls/crypt32/tests/main.c
@@ -254,10 +254,8 @@
         ret = pI_CryptFreeTls(index, 0);
         ok(ret, "I_CryptFreeTls failed: %08x\n", GetLastError());
         ret = pI_CryptFreeTls(index, 0);
-        /* Not sure if this fails because TlsFree should fail, so leave as
-         * todo for now.
-         */
-        todo_wine ok(!ret && GetLastError() == E_INVALIDARG,
+        ok(!ret, "I_CryptFreeTls succeeded\n");
+        ok(GetLastError() == E_INVALIDARG,
          "Expected E_INVALIDARG, got %08x\n", GetLastError());
     }
     /* Similar pass, check I_CryptDetachTls */