Re: Problem with heap and malloc()/free()
"Zapanaz" <http://joecosby.com/code/mail.pl> ha scritto nel messaggio
news:NJCdnRDx8KsD-AvanZ2dnUVZ_qKgnZ2d@zhonka.net...
The current problem, I have class StringHandler, which has a public
member
wchar_t * wc;
I would not expose those kind of public members; I prefer get/set methods.
They offer better encapsulation, information hiding and more robust code,
IMHO.
In my application class, I call a method of StringHandler which
assigns a string to this pointer,
wc = (wchar_t *)malloc(requiredBytes);
int error = MultiByteToWideChar(CP_UTF8, 0, kpcIn, -1, wc,
requiredBytes);
As Norbert wrote, you must pay attention to ::MultiByteToWideChar
parameters, and character count.
(However, to better understand this part of your code, also some sorrounding
code is needed, to better grasp the "context".)
Then in the destructor for StringHandler, I try to free this memory
free(wc);
And I would suggest to set wc=NULL after the freeing, so you are sure that
you don't have dangling references.
However, consider also that you may have copied your class instances, using
operator= or copy constructor.
Did you properly implement the operator= and copy constructor for your
class?
Did you do deep-copy in these "special" methods? (like allocating a new copy
of the buffer, and copy the content from source to destination?)
A first step could be to stop using those raw C-like arrays, both malloc and
new[], and start using a robust C++ container class, like std::vector.
If you add an std::vector instance as a data member (instead of the raw
wchar_t * pointer), std::vector will take care of proper copy and cleanup.
Giovanni