Damjan Jovanovic : mshtml/tests: Fix a HRESULT comparison in ProtocolSink_ReportResult().

Alexandre Julliard julliard at winehq.org
Mon Apr 13 15:05:57 CDT 2020


Module: wine
Branch: master
Commit: 5e75310837e5ec77ebc361d689ea3279fa49ebac
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=5e75310837e5ec77ebc361d689ea3279fa49ebac

Author: Damjan Jovanovic <damjan.jov at gmail.com>
Date:   Fri Apr 10 13:30:22 2020 +0200

mshtml/tests: Fix a HRESULT comparison in ProtocolSink_ReportResult().

ProtocolSink_ReportResult() only checks expect_hrResult is non-zero,
instead of comparing it against hres
like the same code snippet in test_protocol_fail().

Signed-off-by: Damjan Jovanovic <damjan.jov at gmail.com>
Signed-off-by: Jacek Caban <jacek at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/mshtml/tests/protocol.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/dlls/mshtml/tests/protocol.c b/dlls/mshtml/tests/protocol.c
index 8f37d09c55..ba767d21c6 100644
--- a/dlls/mshtml/tests/protocol.c
+++ b/dlls/mshtml/tests/protocol.c
@@ -139,7 +139,7 @@ static HRESULT WINAPI ProtocolSink_ReportResult(IInternetProtocolSink *iface, HR
     CHECK_EXPECT(ReportResult);
 
     if(expect_hr_win32err) {
-        ok((hrResult&0xffff0000) == ((FACILITY_WIN32 << 16)|0x80000000) || expect_hrResult,
+        ok((hrResult&0xffff0000) == ((FACILITY_WIN32 << 16)|0x80000000) || hrResult == expect_hrResult,
                 "expected win32 err or %08x got: %08x\n", expect_hrResult, hrResult);
     }else {
         ok(hrResult == expect_hrResult || (expect_hrResult == E_INVALIDARG && hrResult == MK_E_SYNTAX)




More information about the wine-cvs mailing list