James Hawkins : msi: Wine fails the test and passes us a Session object back, so free it in this case.

Alexandre Julliard julliard at winehq.org
Fri Dec 11 11:04:47 CST 2009


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

Author: James Hawkins <truiken at gmail.com>
Date:   Thu Dec 10 17:01:24 2009 -0800

msi: Wine fails the test and passes us a Session object back, so free it in this case.

---

 dlls/msi/tests/automation.c |    2 ++
 1 files changed, 2 insertions(+), 0 deletions(-)

diff --git a/dlls/msi/tests/automation.c b/dlls/msi/tests/automation.c
index bf3f942..bb498c0 100644
--- a/dlls/msi/tests/automation.c
+++ b/dlls/msi/tests/automation.c
@@ -721,6 +721,8 @@ static void test_dispatch(void)
     todo_wine ok(hr == DISP_E_EXCEPTION, "IDispatch::Invoke returned 0x%08x\n", hr);
     ok_exception(hr, szOpenPackageException);
     VariantClear(&vararg[0]);
+    if (hr != DISP_E_EXCEPTION)
+        VariantClear(&varresult);
 
     /* Try one parameter, the required BSTR.  Second parameter is optional.
      * Proper installer package exists.  Path to the package is absolute.




More information about the wine-cvs mailing list