CDHtmlDialog::DisconnectDHtmlElementEvents的怪异问题导致内存访问错误

13妖 2014-08-05 05:35:16

今天在用CDHtmlDialog的时候遇到一个怪异问题，当Navigate打开一个本地htm文件时报内存访问冲突，至今不知道怎么解决，或者说不知道问题出在哪里。
工程中有2个类使用到，其中A类是正常的，而B类100%出现这个问题。

debug下报错位置：

void CDHtmlDialog::DisconnectDHtmlElementEvents()

{

	const DHtmlEventMapEntry* pEventMap = GetDHtmlEventMap();	//此处报错，内存访问冲突



	if (!pEventMap)

		return;



	int i;



	// disconnect from element events

	for (i=0; i<m_SinkedElements.GetSize(); i++)

	{

		CDHtmlElementEventSink *pSink = m_SinkedElements[i];

		AtlUnadvise(pSink->m_spunkElem, __uuidof(IDispatch), pSink->m_dwCookie);

		delete pSink;

	}

	m_SinkedElements.RemoveAll();



	// disconnect from control events

	for (i=0; i<m_ControlSinks.GetSize(); i++)

	{

		DisconnectFromConnectionPoint(m_ControlSinks[i]->m_spunkObj, 

				m_ControlSinks[i]->m_iid, m_ControlSinks[i]->m_dwCookie);

		delete m_ControlSinks[i];

	}

	m_ControlSinks.RemoveAll();

	return;

}

查看this指针，发现此时this已经被删了！~~~oxfreefeee之类的，hwnd都是0x00000005什么的了。

因CDHtmlDialog是COM，所以调用堆栈前部分都不可见，且是消息触发，跟玩完。
查看调用堆栈：
1、CDHtmlDialog::_OnBeforeNavigate2()
2、CDHtmlDialog::OnBeforeNavigate()
3、CDHtmlDialog::DisconnectDHtmlEvents()
4、CDHtmlEventSink::DisconnectFromConnectionPoint()
5、CDHtmlDialog::DisconnectDHtmlElementEvents()

在CDHtmlDialog::_OnBeforeNavigate2()加断点，证明了A类、B类进入次数、数据的格式是一样的（格式一样，其实是废话，内存地址、Navigate的url当然不一样）

此处贴出CDHtmlDialog::DisconnectDHtmlEvents()

void CDHtmlDialog::DisconnectDHtmlEvents()

{

	CComPtr<IHTMLDocument2> sphtmlDoc;

	GetDHtmlDocument(&sphtmlDoc);



	if (sphtmlDoc == NULL)

		return;

	//先调用DisconnectFromConnectionPoint

	DisconnectFromConnectionPoint(sphtmlDoc, __uuidof(HTMLDocumentEvents), m_dwDHtmlEventSinkCookie);

	//再调用DisconnectDHtmlElementEvents

	DisconnectDHtmlElementEvents();

}

跟踪到DisconnectFromConnectionPoint，执行前，查看this指针数据正常，执行后this指针被删除了！想必元凶在此，跟进去。

void CDHtmlEventSink::DisconnectFromConnectionPoint(IUnknown *punkObj, REFIID riid, DWORD& dwCookie)

{

	AtlUnadvise(punkObj, riid, dwCookie);

}

//再跟进去

ATLINLINE ATLAPI AtlUnadvise(

	_Inout_ IUnknown* pUnkCP,

	_In_ const IID& iid,

	_In_ DWORD dw)

{

	if(pUnkCP == NULL)

		return E_INVALIDARG;



	CComPtr<IConnectionPointContainer> pCPC;

	CComPtr<IConnectionPoint> pCP;

	HRESULT hRes = pUnkCP->QueryInterface(__uuidof(IConnectionPointContainer), (void**)&pCPC);

	if (SUCCEEDED(hRes))

		hRes = pCPC->FindConnectionPoint(iid, &pCP);

	if (SUCCEEDED(hRes))

		hRes = pCP->Unadvise(dw);//到此为止了，vs进不去了

	return hRes;

}

A类执行DisconnectFromConnectionPoint函数，返回S_OK，this指针未被修改。
B类执行DisconnectFromConnectionPoint函数，返回S_OK，this指针被修改，地址被删。导致在CDHtmlDialog::DisconnectDHtmlElementEvents()中调用GetDHtmlEventMap()报内存访问错误。

所以，问题出在Unadvise，确切的说是CComPtr<IHTMLDocument2>::Unadvise。
搜了下资料，Unadvise只是将源端、接收端连接断开

请问各位大大，Unadvise内部到底发生了什么事情导致this指针被更改了？

...全文