Re: [Quickfix-developers] Leak at DataDictionary::readMSXML ?

[<OM...@th...>]

Thanks Hugo, this does look like a forgotten pointer.  We will add this
change to the repository.  Although there is actually very little heap
allocation in QuickFIX,  we have begun using auto pointers for those
situations.  This should prevent future occurences like this.

--oren



|---------+----------------------------------------------->
|         |           "Hugo Leote"                        |
|         |           <Hug...@in...>         |
|         |           Sent by:                            |
|         |           qui...@li...|
|         |           ceforge.net                         |
|         |                                               |
|         |                                               |
|         |           11/26/2002 10:01 AM                 |
|         |                                               |
|---------+----------------------------------------------->
  >----------------------------------------------------------------------------------------------|
  |                                                                                              |
  |       To:       qui...@li...                                    |
  |       cc:                                                                                    |
  |       Subject:  [Quickfix-developers] Leak at DataDictionary::readMSXML ?                    |
  >----------------------------------------------------------------------------------------------|




I'm using quickfix 1.3.2 on Visual C++ 6.0 and I'm getting some memory
leaks after creating an Acceptor.

After tracking down the problem, it came to DataDictionary::readMSXML.
If we look at the code (DataDictionary.cpp, line 323(?)):

(...)
  FIXContent* pContent = new FIXContent( *this );
(...)

When is this deleted ?
Seems like a forgotten pointer.

If I delete it at the end of readMSXML :

(...)
  pRdr->Release();
  CoUninitialize();
  delete pContent;
(...)

the memory leaks go away.


Thanks in advance,
Hugo Leote



-------------------------------------------------------
This SF.net email is sponsored by: Get the new Palm Tungsten T
handheld. Power & Color in a compact size!
http://ads.sourceforge.net/cgi-bin/redirect.pl?palm0002en
_______________________________________________
Quickfix-developers mailing list
Qui...@li...
https://lists.sourceforge.net/lists/listinfo/quickfix-developers