[PATCH 1/2] mshtml/tests: fix a HRESULT comparison in ProtocolSink_ReportResult()

Damjan Jovanovic damjan.jov at gmail.com
Fri Apr 10 06:30:22 CDT 2020


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>
---
 dlls/mshtml/tests/protocol.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
-------------- next part --------------
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-devel mailing list