Memory leak

[gunnar66]

Hi It seems like my exe prog. have a memory leak problem. The program read from the lpCmdLine, 6 different files, merges this to one output file and then close. I close all handle before i Exit.Is there something i have to do, to free memory? I call the exe file 15 times per minute. By the way the program is programmed in Embedded C++, and runs on a CE panel. Some code: #include "stdafx.h" #include "Afx.h" #include "Winuser.h" #include "stdlib.h" int WINAPI WinMain( HINSTANCE hInstance, HINSTANCE hPrevInstance, LPTSTR lpCmdLine, int nCmdShow) { DWORD dwBytesRead, dwBytesWritten, dwPos; char buff[4096]; TCHAR* filnavn1; TCHAR* filnavn2; TCHAR* filnavn3; TCHAR* filnavn4; TCHAR* filnavn5; TCHAR* filnavn6; TCHAR* filnavnhtml; DWORD dwBytesRead, dwBytesWritten, dwPos; HANDLE hFile1, hFile2, hFile3, hFile4, hFile5, hFile6, hAppend; hFile1 = CreateFile (filnavn1, // Open .txt GENERIC_READ, // Open for reading 0, // Do not share NULL, // No security OPEN_EXISTING, // Existing file only FILE_ATTRIBUTE_NORMAL, // Normal file NULL); // No template file //I open 6 files hAppend = CreateFile (filnavnhtml, // Create output file. GENERIC_WRITE, // Open for writing 0, // Do not share NULL, // No security CREATE_ALWAYS, // Open or create FILE_ATTRIBUTE_NORMAL, // Normal file NULL); // No template file dwPos = SetFilePointer (hAppend, 0, NULL, FILE_END); do { if (ReadFile (hFile1, buff, 4096, &dwBytesRead, NULL)) { WriteFile (hAppend, buff, dwBytesRead, &dwBytesWritten, NULL); } } while (dwBytesRead == 4096); // I do this with all 6 files //close all files CloseHandle (hAppend); CloseHandle (hFile1); Return 0; } Thanks

[David Crow]

gunnar66 wrote: It seems like my exe prog. have a memory leak problem. How do you know this? Are you allocating memory for filnavn1 and filnavnhtml?

---

"Opinions are neither right nor wrong. I cannot change your opinion. I can, however, change what influences your opinion." - David Crow

[gunnar66]

Hi In the CE panel the used memory is growing when my exe prog is running. I am not allocating any memory for filnavn1 and filnavnhtml

[David Crow]

gunnar66 wrote: I am not allocating any memory for filnavn1 and filnavnhtml They are declared as pointers. Shouldn't you be allocating memory for them like:

TCHAR *filnavn1 = new TCHAR[some_size];
...
delete [] filnavn1;

Or since they are used for filename purposes, you could just use a stack-based buffer of size MAX_PATH like:

TCHAR filnavn1[MAX_PATH];

---

"Opinions are neither right nor wrong. I cannot change your opinion. I can, however, change what influences your opinion." - David Crow