Unhandled exception at 0x1024be5b (msvcr80d.dll) in TestApp.exe: 0xC0000005: Access violation reading location 0xcdcdcdc1.
ctest.h:
...
unsigned char *data;
...
ctest.cpp:
int CTest::Free()
{
if (!this->data)
return -1;
delete this->data; // this line
return 0;
}
CTest::CTest() : data( NULL ){}foo *pFoo = 0;delete pFoo; // no problems...pFoo = new foo;delete pFoo; // no problemsdelete pFoo; // **BOOM**CTest::CTest() : data(0){}CTest::CTest(std::string filename) : data(0){ this->Load(filename);}CTest::~CTest(){ this->Free();}void CTest::Free(){ if (!this->valid) return; if (!this->data) return; delete this->data; // this line} CTest *Image = new CTest; Image->Load("box.png"); int w = Image->width; int h = Image->height; unsigned char *data= Image->getdata(); delete Image; // this lineDebug Assertion Failed!Program: ...File: dbgdel.cppLine: 52Expression: _BLOCK_TYPE_IS_VALID(pHead->nBlockUse) msvcr80d.dll!operator delete(void * pUserData=0x02184960) Line 52 + 0x51 bytes C++> TestApp.exe!CTest::Free() Line 51 + 0x18 bytes C++ TestApp.exe!main() Line 61 C++ TestApp.exe!mainCRTStartup() Line 558 + 0x19 bytes C kernel32.dll!RegisterWaitForInputIdle() + 0x49 bytes Quote:Original post by TDragon
At a guess, it would seem to mean that you're trying to delete memory that was not allocated by your program. That's all I can tell without seeing more of what's going on...
int CTest::Load(std::string filename){ corona::Image *img = corona::OpenImage(filename.c_str()); if (!img) { printf("failed to load image file\n"); return -1; } this->width = img->getWidth(); this->height = img->getHeight(); //this->data = new unsigned char[(this->width * this->height) * 4]; this->data = (unsigned char *)img->getPixels(); return 0;}Quote:Original post by Lacutis
Perhaps your OpenImage function isn't returning a NULL if it fails to load the image? Post the contents of "OpenImage()" and maybe that will show us where the problem is.
My guess is that openimage returns a pointer to a temporary created inside the function, which is a valid address at that point, and later on you are trying to deallocate a pointer which points to memory that isn't valid anymore.
Also as a side note, I had a little chuckle with the thread title off the main page "trouble deallocating, by storage" ;)
