Thread: Function and pointer troubles

For malloc to work, I need to know how much memory is needed. For convenience, since it's done a lot, I've put this into a function as a parameter. The function is to set the global variable for the memory usage then malloc the amount of memory needed. Trouble is, I'm always getting 0. Using the debugger, I'm seeing that it is reaching the line, but it's not being set, and thus malloc is allocating 0 bytes, causing problems.

The main essentials are these:

Code:

unsigned char *SkyBlueData; // data to malloc
unsigned int SkyBlueMemoryUsage; // memory used
LPVOID SkyBlueDataPointer;
BITMAPINFO SkyBlueMainInfo;
BITMAPINFOHEADER SkyBlueInfo;
LPBITMAPINFOHEADER SkyBlueInfoPointer;
HBITMAP SkyBlueBMPHandle;

The function declaration is this:

Code:

int LoadImageFile(const char FileName[64], int FileType, int LoadType, BITMAPINFOHEADER *BMPInfo, unsigned char *BMPData, int FogUsed, double ObjectScaling, unsigned int *BMPMemoryUsage, unsigned int Special)

This is the key area:

Code:

	BMPMemoryUsage = BMPInfo->biSizeImage; // sets the memory usage
	BMPData = malloc(BMPMemoryUsage);
	fread(BMPData, 1, BMPMemoryUsage, FileHandle);

This is how the function is called, with other essentials:

Code:

	SkyBlueInfoPointer = &SkyBlueInfo;
	LoadImageFile("scenery\\SkyBlue.bmp", 1, 0, &SkyBlueInfo, &SkyBlueData, 0, 0.0, &SkyBlueMemoryUsage, 0);
	SkyBlueDataPointer = &SkyBlueData;

I can't just set the specific variable in the function - it'll be the only one set. Basically, I want to pass the SkyBlueMemoryUsage variable to the function so it can get set int the function for malloc to work properly and to keep a log of the memory usage. In other words, I have a global variable, A, that want a function, B, to set within it with the variable to be changed, A, referenced in the function's parameters.

I'm not getting any compiler errors or any warnings relevant to this function call (how else could I have used the debugger to see that I'm getting zero?). I've got the info structure filled out, including biSizeImage (and the many others), done by reading from the BMP image. How do you dereference a pointer? I'm confused. The other areas, the BMPInfo and BMPData elements work fine as is, but not this single variable that doesn't use an array or is part of a struct.

Edit: By using biSizeImage instead, malloc is working... somewhat. I have new problem now. Before using malloc, BMPData filled in the global variable correctly and I could see the image just fine. With using malloc, nothing shows up, as if it's allocating a bunch of local pointers because what is referenced to by the parameter is zero - a pointer that points to nothing.

Here's the updated function declaration:

Code:

int LoadImageFile(const char FileName[64], int FileType, int LoadType, BITMAPINFOHEADER *BMPInfo, unsigned char *BMPData, int FogUsed, double ObjectScaling, unsigned int Special)

This is the area for malloc (it's supposedly allocating an array of pointers instead of actual data but I don't know how to get it to reference the data I passed onto the function instead of this parameter):

Code:

BMPData = malloc(BMPInfo->biSizeImage);
			fread(BMPData, 1, BMPInfo->biSizeImage, FileHandle); // read the main image data // reads in BGR format

Here's the updated call to the function:

Code:

LoadImageFile("scenery\\SkyBlue.bmp", 1, 0, &SkyBlueInfo, SkyBlueData, 0, 0.0, 0); // current version
// LoadImageFile("scenery\\SkyBlue.bmp", 1, 0, &SkyBlueInfo, &SkyBlueData, 0, 0.0, 0); // tried this, but doesn't work

It's probably the dereferencing of pointers (whatever that means) that's getting at me. Why it works for BMPInfo, that I don't know. I also don't know why it works for BMPData without malloc, but doesn't with it.

It is allowed for biSizeImage to be zero (and obviously it is, since you are passing 0 to malloc -- there is no magic here). You must calculate it yourself from other information in the header.

biSizeImage is not 0, I checked it. Punching numbers into calculator, I see that it exactly matches what the calculation shows. If you want the full code, here:

Code:

int LoadImageFile(const char FileName[64], int FileType, int LoadType, BITMAPINFOHEADER *BMPInfo, unsigned char *BMPData, int FogUsed, double ObjectScaling, unsigned int Special)
{
	char BasePath[192]; // the base path, that of which started from
	char FullFileName[256]; // the full path combining the base path and the parameter
	unsigned int ArrayIndex = 0;
	float NewColor = 0;
	const float FogIntensity = (float)ObjectScaling;
	const float FogRange = (float)visibility;
	const float FogColorRed = (float)(FogColor[2]);
	const float FogColorGreen = (float)(FogColor[1]);
	const float FogColorBlue = (float)(FogColor[0]);
	unsigned int CurrentRow;
	unsigned int CurrentPixel;
	float BaseScaling;
	
	GetCurrentDirectory(MAX_PATH, BasePath);
	sprintf(FullFileName, "%s\\images\\%s", BasePath, FileName); // combine the base path and file name
	FileHandle = fopen(FullFileName, "rb"); // read the source file to display, binary mode
	
	if (FileHandle == 0) // if the file can't be found
	{
		sprintf(WarningString, "Error:  This file cannot be found:\n%s", FullFileName); // an error since the file can't be found
		MessageBox(WindowHandle, WarningString, "File not found", MB_OK | MB_ICONHAND);
		return 1; // indicates an error occurred
	}
	
	if (FileType == 1) // reading a BMP file
	{
		if (LoadType == 0) // normal full load
		{
			fseek(FileHandle, 14, SEEK_SET); // skip the basic file header data but read from the info header
			fread(&BMPInfo->biSize, 4, 1, FileHandle); // bytes 0E through 11
			fread(&BMPInfo->biWidth, 4, 1, FileHandle); // bytes 12 through 15
			fread(&BMPInfo->biHeight, 4, 1, FileHandle); // bytes 16 through 19
			fread(&BMPInfo->biPlanes, 2, 1, FileHandle); // bytes 1A and 1B
			fread(&BMPInfo->biBitCount, 2, 1, FileHandle); // bytes 1C and 1D
			fread(&BMPInfo->biCompression, 4, 1, FileHandle); // bytes 1E through 21
			fread(&BMPInfo->biSizeImage, 4, 1, FileHandle); // bytes 22 through 25
			fread(&BMPInfo->biXPelsPerMeter, 4, 1, FileHandle); // bytes 26 through 29
			fread(&BMPInfo->biYPelsPerMeter, 4, 1, FileHandle); // bytes 2A through 2D
			fread(&BMPInfo->biClrUsed, 4, 1, FileHandle); // bytes 2E through 31
			fread(&BMPInfo->biClrImportant, 4, 1, FileHandle); // bytes 32 through 35
			
			*BMPData = malloc(BMPInfo->biSizeImage);
			fread(BMPData, 1, BMPInfo->biSizeImage, FileHandle); // read the main image data // reads in BGR format
		}
		
		if (LoadType == 1) // just a reload
		{
			fseek(FileHandle, 54, SEEK_SET); // skip to the main image data
			fread(BMPData, 1, BMPInfo->biSizeImage, FileHandle); // read the main image data // reads in BGR format
		}
	}
... // similar case for TGA images, then fog processing and premultiplication
}

... // in WinMain:
	SkyBlueInfoPointer = &SkyBlueInfo;
	LoadImageFile("scenery\\SkyBlue.bmp", 1, 0, &SkyBlueInfo, &SkyBlueData, 0, 0.0, 0);
	SkyBlueDataPointer = &SkyBlueData; // copies the pointer

If I keep the & in the function call for this part, I can't see the warnings list - it's empty. Removing it and I do. With it removed, I get this as the only warning, involving that line with malloc. As is, I get the top warning. By having only the top one have the ampersand, I get the two warnings at the bottom:

Code:

1>.\Platform Masters.c(383) : warning C4047: '=' : 'unsigned char' differs in levels of indirection from 'void *'

1>.\Platform Masters.c(2784) : warning C4047: 'function' : 'unsigned char *' differs in levels of indirection from 'unsigned char **'
1>.\Platform Masters.c(2784) : warning C4024: 'LoadImageFile' : different types for formal and actual parameter 5

The only other warnings are from mmsystem (a Windows element) and unreferenced local variables and parameters.

When I run it, I get an unhandled exception error. With the ampersand, it happens with "free". Without the ampersand, it happens at malloc.

In addition, I didn't know you could have the * in front of something when an = is to follow. If you want to look at the bitmap image so you can see that biSizeImage is not zero, I can zip it up and give a link to it.

Code:

*BMPData = malloc(BMPInfo->biSizeImage);

You are assigning the result of malloc() to wherever BMPData is pointing, i.e. nowhere. You should have:

Code:

BMPData = malloc(BMPInfo->biSizeImage);

I've made this change, and still nothing happens and it's still the same case - a bad pointer. I have 2 calls to the function. One uses the & for the data, the other doesn't. I put a breakpoint on this line:

Code:

SkyBlueDataPointer = &SkyBlueData;

I see that SkyBlueDataPointer is nonzero and SkyBlueData (which contains the image data) is zero when I put the mouse over them. Remove the ampersand and both are zero. If it helps, these are the two function calls:

Code:

	SkyBlueInfoPointer = &SkyBlueInfo;
	LoadImageFile("scenery\\SkyBlue.bmp", 1, 0, &SkyBlueInfo, &SkyBlueData, 0, 0.0, 0);
	SkyBlueDataPointer = &SkyBlueData;
	FullFogDayInfoPointer = &FullFogDayInfo;
	LoadImageFile("scenery\\FullFogDay.bmp", 1, 0, &FullFogDayInfo, FullFogDayData, 0, 0.0, 0);
	FullFogDayDataPointer = &FullFogDayData;

The top one, line 2784 is the one with the warnings, but the bottom one doesn't have it. Both of which have bad pointers. It's for this reason that I thought malloc was allocating to BMPData instead of the real data.