Quote:Original post by Paulius Maruska
This is probably not that important, but for portability sake you should change the code like this:

// allocate a temporary buffer
int bufferSize = bitmapPtr->infoHeader.biSizeImage / sizeof(unsigned int);// <= not 4
unsigned int *tempBuffer = new unsigned int[ bufferSize ];

// read in the buffer, bitmapFile is an ifstream
bitmapFile.read( (char *)tempBuffer, bitmapPtr->infoHeader.biSizeImage );

// this doesn't work
delete [] tempBuffer;

struct Bitmap32{	BITMAPFILEHEADER	fileHeader;	BITMAPINFOHEADER	infoHeader;	unsigned int		*buffer;};int MyLoadBitmap( char *fileName, Bitmap32 *bitmapPtr ){	// - loads a bitmap with the specified filename and saves the image in a 32-bit buffer	// R [1] success	// R [0] failure	ifstream bitmapFile;	bitmapFile.open( fileName, ios::binary | ios::in );	if( bitmapFile.is_open() )	{		// read in the file header		bitmapFile.read( (char *)&(bitmapPtr->fileHeader), sizeof(BITMAPFILEHEADER) );		// make sure it's a bitmap file		if( bitmapPtr->fileHeader.bfType != 0x4D42 )		{			bitmapFile.close();			return 0;	// file was not a bitmap		}		// read in the info header		bitmapFile.read( (char *)&(bitmapPtr->infoHeader), sizeof(BITMAPINFOHEADER) );		// make sure the bitmap is 32-bit		if( bitmapPtr->infoHeader.biBitCount != 32 )		{			bitmapFile.close();			return 0;	// bitmap was not 32-bit		}		// allocate a temporary buffer		int				bufferSize = bitmapPtr->infoHeader.biSizeImage / 4;		unsigned int	*tempBuffer = new unsigned int[ bufferSize ];		// read in the buffer		bitmapFile.read( (char *)tempBuffer, bitmapPtr->infoHeader.biSizeImage );		delete [] tempBuffer; // this is just to test the delete problem		// flip the buffer because windows stores the colors are BGRA and opengl wants RGBA		unsigned char red	= 0;		unsigned char green = 0;		unsigned char blue	= 0;		unsigned char alpha	= 0;		unsigned int *curPos = tempBuffer;		for( int i = 0; i < bufferSize; i++ )		{			blue	= *curPos;			green	= *curPos >> 8;			red		= *curPos >> 16;			alpha	= *curPos >> 24;			*curPos = ((alpha << 24) + ((blue) << 16) + ((green) << 8) + (red));			curPos++;		}		// delete any previous bitmap buffer data		if( bitmapPtr->buffer != NULL )		{			delete [] bitmapPtr->buffer;			bitmapPtr->buffer = NULL;		}		// assign the buffer to point to the new buffer		bitmapPtr->buffer = tempBuffer;		return 1;	}	else		return 0;	// file could not be opened} // end of MyLoadBitmap

Quote:Original post by Enigma
Additionally the bitmap format is stored as BGR, not BGRA, so it's only three bytes per pixel. Note that each scanline is padded to a multiple of four bytes though.

Quote:Original post by sammyjojo
I really need to not assume things while programming because it looks like that's what got me trouble this time. As you probably already know, I was reading in more then I allocated memory for.

Quote:Original post by sammyjojo
I honestly don't know why my image has 2 extra bytes, because it's 80x80 and 32-bit so it should take up 25600 bytes, but instead it takes up 25602 according to biSizeImage in my BITMAPINFOHEADER.