ATL / WTL / STL

bujji_bec wrote:

char *cp,*cp1;

bujji_bec wrote:

delete []cp; // this one giving problem
delete []cp1; // -----------do----------

delete cp;<br />
delete cp1;

bujji_bec wrote:

int main()
{
char *cp,*cp1;
Config Objcon("../","rrr.ini");
Section objsec = Objcon["server"];
cp=objsec["ip"];
cp1=objsec["port"];
cout<<cp<<"\n"<<"cp1;

delete []cp;="" this="" one="" giving="" problem
delete="" []cp1;="" -----------do----------
}<="" blockquote="">

bujji_bec wrote:

i am Getting Same problem

According to the code in the original post, Section::operator [] (LPCTSTR token) allocates some buffer internally. The implementation inside this operator matters how memory is released (by delete).

Do you mind posting the body of Section::operator [] (LPCTSTR) ??

---

Maxwell Chen

bujji_bec wrote:

int main()
{
char *cp,*cp1;
Config Objcon("../","rrr.ini");
Section objsec = Objcon["server"];
cp=objsec["ip"];
cp1=objsec["port"];
cout<<cp<<"\n"<<"cp1;

delete []cp;="" this="" one="" giving="" problem
delete="" []cp1;="" -----------do----------
}

in="" the="" program="" output="" coming="" correct.="" but="" delete="" its="" memory="" issues.<="" blockquote="">


I'm assuming that your Config object is defined in your DLL? That being the case, your memory is being allocated in the DLL's memoryspace, and you are trying to delete it from the application's memory space. This will cause an exception and will crash your application.

To avoid this, you have 2 options:

1) Add a method to your Config class to clean up memory it allocates. This will basically turn it into a factory for the objects it creates (what is made in the factory is also deleted in the factory).

2) Use Boost's/TR1's shared_ptr class to wrap your pointers. This will prevent this problem completely without you needing to call delete at all. Note that to do this, you should modify your DLL to return shared_ptrs.

If you decide to become a software engineer, you are signing up to have a 1/2" piece of silicon tell you exactly how stupid you really are for 8 hours a day, 5 days a week

Zac