I haven't downloaded your source, but I think I know what your problem is.

Change this:

m_chrString = new char[ length ];

to this:

m_chrString = new char[ length + 1 ];

Remember, you must allocate one more than the length to accomodate the null terminator at the end of the character array.

Hope this helps!

Josh

Of course the alloc with length+1 was right, but here's my problem with that answer. What does that have to do with the delete call? delete [], does NOT in any way care anything about the contents of your string, nor the NULL terminator, it just looks before the address of the array and checks how much was allocated, then it deletes that much. I don't see how fixing your alloc error would fix your delete problem. Perhaps I'm ignoring the fact that maybe your program allocated two strings in a row and therefore the alloc error cause the first string to store it's NULL terminator over the control information stored in front of the second string (which in turn would crash delete). Oh well....guess I'll never know for sure.

Xai: You're probably right. Whenever you write past the allocated space you get, in Microsoft terminology, "undefined" results. As in you could be fine, the program could crash.

Or the whole computer could ignite in flames (only with the Microsoft(tm) Self-Destruct kit, sold seperately)

- Splat