关天CHTML VIEW 有内存泄漏的问题及解决办法。

关于CHTMLView有内存泄漏的问题及解决办法

应用 CHtmlView 类，发现过一个问题，CHtmlView创建的页面如果有flash或者是其他背景音乐的时候， 当关闭这个页面，背景音乐还是在响，这就说明CHtmlView关闭的时候，并没有正确的释放加载进来的资源。

在1.17版本的JJIE浏览器中， 就存在着这样的一个问题。我也是偶然发现的。

于是我上网查了一下， 发现CHtmlView确实存在着资源释放的问题：

存在内存泄漏的函数如下：
• Navigate
• GetFullName
• GetType
• GetLocationName
• GetLocationURL
• LoadFromResource(LPCTSTR lpszResource)
• LoadFromResource(UINT nRes)

在前五个函数中， 都有着这样一个函数 SysFreeString， 但是这个函数是需要BSTR支持的。如果我们网页上的资源不是Unicode格式的，我们在使用SysFreeString的时候就不能正确的释放资源，从而导致内存泄露。
后两者导致内存泄漏是因为 CHtmlView::Navigate 在这些函数中调用了两次，所以没有正确的释放资源

正因为有着这些MFC内部设计上的bug，所以才导致了1.17版的CHtmlView不能释放一些网络上的资源。

根据微软网站上的一些技术文章，我找到了解决办法，修改了CHtmlView，现在这个问题已经修复，重载上面的7个函数，并修改：

CString CFixHtmlView::GetFullName() const
{
　　ASSERT(m_pBrowserApp != NULL);

　　BSTR bstr;
　　m_pBrowserApp->get_FullName(&bstr);
　　CString retVal(bstr);

　　SysFreeString(bstr); // Added this line to prevent leak.
　　return retVal;
}

CString CFixHtmlView::GetType() const
{
　　ASSERT(m_pBrowserApp != NULL);

　　BSTR bstr;
　　m_pBrowserApp->get_Type(&bstr);

　　CString retVal(bstr);

　　SysFreeString(bstr); // Added this line to prevent leak.
　　return retVal;
}

CString CFixHtmlView::GetLocationName() const
{
　　ASSERT(m_pBrowserApp != NULL);

　　BSTR bstr;
　　m_pBrowserApp->get_LocationName(&bstr);
　　CString retVal(bstr);

　　SysFreeString(bstr); // Added this line to prevent leak.
　　return retVal;
}

CString CFixHtmlView::GetLocationURL() const
{
　　ASSERT(m_pBrowserApp != NULL);

　　BSTR bstr;
　　m_pBrowserApp->get_LocationURL(&bstr);
　　CString retVal(bstr);

　　SysFreeString(bstr); // Added this line to prevent leak.
　　return retVal;
}

void CFixHtmlView::Navigate(LPCTSTR lpszURL, DWORD dwFlags /* = 0 */,
LPCTSTR lpszTargetFrameName /* = NULL */ ,
LPCTSTR lpszHeaders /* = NULL */, LPVOID lpvPostData /* = NULL */,
DWORD dwPostDataLen /* = 0 */)
{
　　CString strurl(/blog/lpszURL);
　　BSTR bstrURL = strURL.AllocSysString();

　　COleSafeArray vPostData;
　　if (lpvPostData != NULL)
　　{
　　　 if (dwPostDataLen == 0)
　　　　　dwPostDataLen = lstrlen((LPCTSTR) lpvPostData);

　　　 vPostData.CreateOneDim(VT_UI1, dwPostDataLen, lpvPostData);
　　}

　　m_pBrowserApp->Navigate(bstrURL, COleVariant((long) dwFlags, VT_I4), COleVariant(lpszTargetFrameName, VT_BSTR),
　　　　　　　　　　　　　　vPostData, COleVariant(lpszHeaders, VT_BSTR));

　　SysFreeString(bstrURL); // Added this line to prevent leak.
}


BOOL CFixHtmlView::LoadFromResource(LPCTSTR lpszResource)
{
　　HINSTANCE hInstance = AfxGetResourceHandle();
　　ASSERT(hInstance != NULL);

　　CString strResourceURL;
　　BOOL bRetVal = TRUE;
　　LPTSTR lpszModule = new TCHAR[_MAX_PATH];

　　if (GetModuleFileName(hInstance, lpszModule, _MAX_PATH))
　　{
　　　 strResourceURL.Format(_T("res://%s/%s"), lpszModule, lpszResource);
　　　 Navigate(strResourceURL, 0, 0, 0);
　　}
　　else
　　　 bRetVal = FALSE;

　　delete [] lpszModule;
　　return bRetVal;
}

BOOL CFixHtmlView::LoadFromResource(UINT nRes)
{
　　HINSTANCE hInstance = AfxGetResourceHandle();
　　ASSERT(hInstance != NULL);

　　CString strResourceURL;
　　BOOL bRetVal = TRUE;
　　LPTSTR lpszModule = new TCHAR[_MAX_PATH];

　　if (GetModuleFileName(hInstance, lpszModule, _MAX_PATH))
　　{
　　　 strResourceURL.Format(_T("res://%s/%d"), lpszModule, nRes);
　　　 Navigate(strResourceURL, 0, 0, 0);
　　}
　　else
　　　 bRetVal = FALSE;

　　delete [] lpszModule;
　　return bRetVal;
}