From a8e39c66f798e9c056b5ff26e3bdcc9eba2b9230 Mon Sep 17 00:00:00 2001 From: Andrew Eikum Date: Mon, 1 Nov 2010 12:06:08 -0500 Subject: [PATCH] hlink: Site data should only be set if the hlink has an HlinkSite. --- dlls/hlink/link.c | 5 +++-- dlls/hlink/tests/hlink.c | 2 +- dlls/mshtml/tests/htmldoc.c | 1 - 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/dlls/hlink/link.c b/dlls/hlink/link.c index 3e22c476d47..b0d99aaf4e9 100644 --- a/dlls/hlink/link.c +++ b/dlls/hlink/link.c @@ -211,10 +211,11 @@ static HRESULT WINAPI IHlink_fnGetHlinkSite( IHlink* iface, TRACE("(%p)->(%p %p)\n", This, ppihlSite, pdwSiteData); *ppihlSite = This->Site; - *pdwSiteData = This->SiteData; - if (This->Site) + if (This->Site) { IHlinkSite_AddRef(This->Site); + *pdwSiteData = This->SiteData; + } return S_OK; } diff --git a/dlls/hlink/tests/hlink.c b/dlls/hlink/tests/hlink.c index 21422bc5c4f..3cb6db44a6a 100644 --- a/dlls/hlink/tests/hlink.c +++ b/dlls/hlink/tests/hlink.c @@ -1788,7 +1788,7 @@ static void test_HlinkClone(void) hres = IHlink_GetHlinkSite(cloned, &fnd_site, &fnd_data); ok(hres == S_OK, "GetHlinkSite failed: %08x\n", hres); ok(fnd_site == NULL, "Expected NULL site\n"); - todo_wine ok(fnd_data == 4, "Expected site data to be 4, was: %d\n", fnd_data); + ok(fnd_data == 4, "Expected site data to be 4, was: %d\n", fnd_data); IHlink_Release(cloned); IHlink_Release(hl); diff --git a/dlls/mshtml/tests/htmldoc.c b/dlls/mshtml/tests/htmldoc.c index 641153baba7..80f137b9abd 100644 --- a/dlls/mshtml/tests/htmldoc.c +++ b/dlls/mshtml/tests/htmldoc.c @@ -708,7 +708,6 @@ static HRESULT WINAPI HlinkFrame_Navigate(IHlinkFrame *iface, DWORD grfHLNF, LPB hres = IHlink_GetHlinkSite(pihlNavigate, &site, &site_data); ok(hres == S_OK, "GetHlinkSite failed: %08x\n", hres); ok(site == NULL, "site = %p\n, expected NULL\n", site); - todo_wine ok(site_data == 0xdeadbeef, "site_data = %x\n", site_data); } -- 2.11.4.GIT