have a nice day!

As krmed already stated, make sure your allocation routine isn't being called multiple times. However, I wouldn't make krmed's change, since that might just be fixing the symptoms and not the real issue. It is the memory allocation version of the "off by one" error where someone fixes the problem by just adding one without tracking down the reason why the value is off by one. Another problem might be that your closedown code isn't being invoked. I often see people spend endless amounts of time trying to figure out why are routine isn't working when in fact, the routine isn't even being invoked. Both of those problems are the same class of problem, improper execution path. Of course, this is all assuming that it is CallSumBin that is leaking. If you haven't verified that, then you might be looking at the wrong issue. As to what Christian said, he is right. Constructing the filename like that is akin to running with scissors. As long as you are careful, things are "ok". But a slight mistake can cause memory corruption (which can manifest itself as a leak in some instances) and security issues. I won't go so far as saying C style is crap, mainly because I have seen far too many string processing systems created with managed strings that are just dogs. But you are playing with fire and really need to understand what you are doing when you do work with C strings. Tim Smith I'm going to patent thought. I have yet to see any prior art. -- modified at 9:56 Thursday 3rd November, 2005

The message means you're trying to create a file on disk, when that file already exists. You can set your streams to overwrite files, perhaps that's what you need to do ? Christian Graus - Microsoft MVP - C++

According to what you said: valerie99 wrote: because the "NumberofGroups = this->GetContributoryAmounts (mainPos); " will call the "Groups = new GROUPSTRUCT[NumberofGroups]; so, if this is true, when you do this: valerie99 wrote: NumberofGroups = this->GetContributoryAmounts (mainPos); Groups = new GROUPSTRUCT[NumberofGroups]; the first line allocates memory with new and assigns the pointer to Groups, but then you also do your own new and assign it to Groups. Therefore the previous assignment from NumberofGroups = this->GetContributoryAmounts (mainPos); is lost, and a memory leak. Hope that helps. Karl - WK5M PP-ASEL-IA (N43CS) PGP Key: 0xDB02E193 PGP Key Fingerprint: 8F06 5A2E 2735 892B 821C 871A 0411 94EA DB02 E193

I think this shouldn't been done as long since it's for display not conculation, leading zero is meaningful at this point. I could left pad zero in front of string, long city = ( party.getCityCode() == 0 ? Long.parseLong( t.toString() ) : party.getCityCode() ); String szcity = utils.leftPadZeros( String.valueOf( city ), t.length()); long token = cintCity.find2( szCountry, szcity ); but the whole app is using long for citycode, it might cause problem later on party.setCityCode( city ); so I was wondering if I could pad zero in front of long....seems like impossible... thanks, have a good weekend

depends on your design. if there's an Open method, maybe there's a Close method also that will close the database and delete the object. if you're not certain about that, put the delete in the destructor. Maximilien Lincourt Your Head A Splode - Strong Bad

valerie99 wrote: could anybody share some good assembly language links, I just need to understand the basic assembly commend Hope this Work :- http://win32assembly.online.fr[^] "Opinions are neither right nor wrong. I cannot change your opinion. I can, however, change what influences your opinion." - David Crow cheers, Alok Gupta VC Forum Q&A :- I/ IV

valerie99 wrote: everything goes well in debug, but it crashes when it called onCancle() in release mode it's out of my control.... :( let me know also if u overcome this << >>

That's the answer I was looking for hours ago! Thank you kind Sir! After looking into the ressource.h-File I can confirm that it's because of the ID of the Control which lead to the end of the thread. For example: I was kinda terrified of the unknown thread-exit code 32780. Cannot look up what's this about because usually the exit codes are not as far specified. After launching my exit procedere mindlessly with my overloaded OnCancel with ID 32780 the weird exit messages kept coming. But I never thought that far. Hopefully someone will spare some time with this information. best regards

beffekt deivsec wrote: C++ exception handling was used but /EHsc was not selected. That's C4530, not C4503... -- jlr http://jlamas.blogspot.com/[^]

If you are multithreading you cant call thouse funtions from another thread. I dont know if I guessed it rite. -prakash

Thank you for the reply here is part of the dialog window: BOOL CSurchrgApp::InitInstance() { ..... CSurchrgDlg dlg; m_pMainWnd = &dlg; dlg.DoModal(); } when I set breakpoint at dlg.DoModal(); I would get the &dlg, dlg and m_pMainWnd with {CSurchrgDlghWnd=0x0000000}, but this seems like the standard coding, not sure why it's getting all the 00000000......

Is there any multithread involved.? -prakash

probably u may forgot to add the required namespace check it

thanks for your info, here is another function that from same file, somehow, it's done deifferently..... CFavNatDbf::CFavNatDbf(int exclusive) : CB_DBF_NTX("DATA\\FAVNAT.DBF", exclusive) { if (validDBF()) { char* ptrIdxName = m_csIdxFile.GetBuffer( L_tmpnam + 1 ); GetTempFileName( ".", "favnat", 0, ptrIdxName ); _strupr( ptrIdxName ); memmove( ptrIdxName, ptrIdxName + 2, strlen( ptrIdxName ) - 2 ); *strstr( ptrIdxName, ".TMP" ) = '\0'; index_on( ptrIdxName, "CUST_NUM + LOCATION" ); m_csIdxFile.ReleaseBuffer(); top(); BindFieldOrdinals(); } } CFavNatDbf::~CFavNatDbf() { } void CFavNatDbf::BindFieldOrdinals() { }

Johann Gerell wrote: That's not the way to use a std::auto_ptr There can be a good reason for this. Suppose the auto_ptr was declared as a class member and not initialized in the constructor of the class. I have done this from time to time. John

You're welcome. Both of us who answered told you the same thing, so I'm not sure if you're trying to insult him, or insult me. Either way, the question was impossible to answer in any way beyond what we told you. Christian Graus - Microsoft MVP - C++

Well, CTL3D32 was used on NT 3.51 (which still had the 3.1 UI) and I guess in 1998 when VC6 came out, people were still writing apps for 3.51. --Mike-- Visual C++ MVP :cool: LINKS~! Ericahist | 1ClickPicGrabber | NEW~! CP SearchBar v3.0 | C++ Forum FAQ

thanks for the offer........but this old app is linking to lot of library files and some of them are from mapping network drive.....:(