Write resource to file

[JoshT]

I think this should work, but I'm getting an Access Violation at the first WriteFile. Anyone know why?

Thanks,

//file names
#define CPL "\\wgpocpl.cpl"
#define DLL "\\wgpoadmn.dll"

	char * pchSystemDir = NULL; //the windows system directory
	HRSRC hCpl = NULL; //handle to control panel resource
	HRSRC hDll = NULL; //handle to DLL resource
	HGLOBAL hCpl2 = NULL;
	HGLOBAL hDll2 = NULL;
	char * pchCpl = NULL; //location of resource in memory
	char * pchDll = NULL; //location of resource in memory
	char * pchCplName = NULL; //full file name of cpl
	char * pchDllName = NULL; //full file name of dll
	HANDLE hCplFile = INVALID_HANDLE_VALUE;
	HANDLE hDllFile = INVALID_HANDLE_VALUE;

		//load the resources into memory
		hCpl = FindResource(NULL, IDR_CPL, "CUSTOM");
		hDll = FindResource(NULL, IDR_DLL, "CUSTOM");
		hCpl2 = LoadResource(NULL, hCpl);
		hDll2 = LoadResource(NULL, hDll);
		pchCpl = (char*)LockResource(hCpl2);
		pchDll = (char*)LockResource(hDll2);

		//create the files
		hCplFile = CreateFile(pchCplName, GENERIC_WRITE, FILE_SHARE_READ, NULL, CREATE_ALWAYS, FILE_ATTRIBUTE_ARCHIVE, NULL);
		hDllFile = CreateFile(pchDllName, GENERIC_WRITE, FILE_SHARE_READ, NULL, CREATE_ALWAYS, FILE_ATTRIBUTE_ARCHIVE, NULL);

		HeapFree(GetProcessHeap(), 0, pchCplName);
		HeapFree(GetProcessHeap(), 0, pchDllName);

		//write to the files
		WriteFile(hCplFile, pchCpl, SizeofResource(hInstance, hCpl), NULL, NULL);
		WriteFile(hDllFile, pchDll, SizeofResource(hInstance, hDll), NULL, NULL);

		//close the files
		CloseHandle(hCplFile);
		CloseHandle(hDllFile);

[Vlatko]

Just a thought
[quote]
BOOL WriteFile(
HANDLE hFile, // handle to file to write to
LPCVOID lpBuffer, // pointer to data to write to file
DWORD nNumberOfBytesToWrite, // number of bytes to write
LPDWORD lpNumberOfBytesWritten, // pointer to number of bytes written
LPOVERLAPPED lpOverlapped // pointer to structure for overlapped I/O
);


Windows NT: If lpOverlapped is NULL, lpNumberOfBytesWritten cannot be NULL.


Are you using NT. If you are than your last parameters are both NULL.

[JoshT]

Thanks! I completely overlooked that - I declared an int and passed it a pointer to it instead of NULL and it works great.