Dan Kegel : secur32/tests: Don't free if not allocated.

Alexandre Julliard julliard at winehq.org
Wed Jun 4 06:35:42 CDT 2008


Module: wine
Branch: master
Commit: 06abaaf218f89a0e7add7289c42390e677ff77ea
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=06abaaf218f89a0e7add7289c42390e677ff77ea

Author: Dan Kegel <dank at kegel.com>
Date:   Tue Jun  3 07:17:48 2008 -0700

secur32/tests: Don't free if not allocated.

---

 dlls/secur32/tests/main.c |   17 +++++------------
 1 files changed, 5 insertions(+), 12 deletions(-)

diff --git a/dlls/secur32/tests/main.c b/dlls/secur32/tests/main.c
index a6daa28..bc6bbe6 100644
--- a/dlls/secur32/tests/main.c
+++ b/dlls/secur32/tests/main.c
@@ -210,13 +210,12 @@ static void testQuerySecurityPackageInfo(void)
         /* there is no point in testing pkg_info->cbMaxToken since it varies
          * between implementations.
          */
-    }
-
-    sec_status = pFreeContextBuffer(pkg_info);
 
-    ok( sec_status == SEC_E_OK,
-        "Return value of FreeContextBuffer() shouldn't be %s\n",
-        getSecError(sec_status) );
+        sec_status = pFreeContextBuffer(pkg_info);
+        ok( sec_status == SEC_E_OK,
+            "Return value of FreeContextBuffer() shouldn't be %s\n",
+            getSecError(sec_status) );
+    }
 
     /* Test with a nonexistent package, test should fail */
 
@@ -228,12 +227,6 @@ static void testQuerySecurityPackageInfo(void)
         getSecError(SEC_E_SECPKG_NOT_FOUND));
 
     ok(pkg_info == (void *)0xdeadbeef, "wrong pkg_info address %p\n", pkg_info);
-
-    sec_status = pFreeContextBuffer(pkg_info);
-
-    ok( sec_status == SEC_E_OK,
-        "Return value of FreeContextBuffer() shouldn't be %s\n",
-        getSecError(sec_status) );
 }
 
 START_TEST(main)




More information about the wine-cvs mailing list