Memory leak

[eduard.antonyan]

All I'm doing is going through the webpage and analyzing the data,
here's the outline of the code:

in OnDocumentCompl ete(LPCTSTR lpszURL)
{
IDispatch *pDispatch = GetHtmlDocument ();
IHTMLDocument2 *pDoc;
IHTMLElement *pBody;
BSTR str;

pDispatch->QueryInterface (IID_IHTMLDocum ent2,(void**)&p Doc);
pDispatch->Release();

pDoc->get_body((IHTM LElement **)&pBody);
pDoc->Release();

pBody->get_innerHTML( &str);
pBody->Release();

//---------------------------
// Analyze the str using wcsstr, Navigate2 depending on the results.
//---------------------------

SysFreeString(s tr);

CHtmlView::OnDo cumentComplete( lpszURL);
}

The above results in a memory leak so what am I doing wrong?

I've tried calling CHtmlView::OnDo cumentComplete( lpszURL) before I do
everything else and calling SysFreeString before doing Navigate2..
Can't think of anything else..:(

[Jochen Kalmbach [MVP]]

Hi eduard!

All I'm doing is going through the webpage and analyzing the data,
here's the outline of the code:

in OnDocumentCompl ete(LPCTSTR lpszURL)
{
IDispatch *pDispatch = GetHtmlDocument ();
IHTMLDocument2 *pDoc;
IHTMLElement *pBody;
BSTR str;

pDispatch->QueryInterface (IID_IHTMLDocum ent2,(void**)&p Doc);
pDispatch->Release();

pDoc->get_body((IHTM LElement **)&pBody);
pDoc->Release();

pBody->get_innerHTML( &str);
pBody->Release();

//---------------------------
// Analyze the str using wcsstr, Navigate2 depending on the results.
//---------------------------

SysFreeString(s tr);

CHtmlView::OnDo cumentComplete( lpszURL);
}

The above results in a memory leak so what am I doing wrong?

HOW do you detect the memory leak?
Also be aware, that COM uses some "intelligen t string pooling", so
"SysFreeStr ing" does not necessarily release the memory!

Greetings
Jochen

[goharik]

Thanks for reply.

Well, using the obvious way - task manager (it can grow relatively
fast).. Or crt dumping.

So any way of checking whether SysFreeString releases the memory or
not? And also any ways of making it release the memory (or going around
that problem..:-\)?

[eduard.antonyan]

Thanks for reply.

Well, using the obvious way - task manager (it can grow relatively
fast).. Or crt dumping.

So any way of checking whether SysFreeString releases the memory or
not? And also any ways of making it release the memory (or going around
that problem..:-\)?

[Jochen Kalmbach [MVP]]

Hi eduard!

Well, using the obvious way - task manager (it can grow relatively
fast).. Or crt dumping.

CRT-Dumping does not show any COM-Leaks!

And in task manager you have hopefully looked at "VM Size" and *not* at
"Memory usage"!

For leak findung you can use UMDH or my leakfinder:
http://www.codeproject.com/tools/leakfinder.asp

Greetings
Jochen

[eduard.antonyan]

Thanks, I'll check it out.

In task manager - yes, both actually.

Actually it seems to me that it leaks regardless of the code, i.e. just
the CHtmlWindow on its own. After a few hundred clicks it already
increases by ~40K (browsing the same pages btw, IE 7).

Jochen Kalmbach [MVP] wrote:

Hi eduard!

Well, using the obvious way - task manager (it can grow relatively
fast).. Or crt dumping.

CRT-Dumping does not show any COM-Leaks!

And in task manager you have hopefully looked at "VM Size" and *not* at
"Memory usage"!

For leak findung you can use UMDH or my leakfinder:
http://www.codeproject.com/tools/leakfinder.asp

Greetings
Jochen

[eduard.antonyan]

*CHtmlView