Memory leak problem in MFC C++

THe problem is obvious. You never delete them. If they are used throughout the program, and this dialog is always visible, then it's not really a problem, but you should still get in the practice of cleaning up all member variables in your destructor.

you are using
pDC = new CDC; and Bitmap = new CBitmap;

use keyword "delete" to free them.

Add an event handler for the WM_DESTROY windows message, then add the lines below inside it:

if (pDC != NULL) delete pDC;
if (Bitmap != NULL) delete Bitmap;

Be aware that if you select your bitmap onto the device context, i.e. pDC->SelectObject(Bitmap), you should save the pointer to the CBitmap returned by the method call and restore it onto the device context before deleting it, as shown below:

1. Add the following members to your class:
   
   C++

   CDC *pDC;
   CBitmap *pBitmap;
   CBitmap *pOldBitmap;

   
   
2. Add the following lines to the OnInitDialog method of your class:
   
   C++

   pOldBitmap = NULL;
   pDC = new CDC;
   if (pDC != NULL && pDC->CreateCompatibleDC(GetDC()))
   {
      pBitmap = new CBitmap;
      if (pBitmap != NULL && pBitmap->CreateCompatibleBitmap(pDC, 640, 480))
      {
         pOldBitmap = pDC->SelectObject(pBitmap);
      }
   }

   
   
3. Add the following lines to the OnDestroy method of your class:
   
   C++

   if (pOldBitmap != NULL) pDC->SelectObject(pOldBitmap);
   if (pDC != NULL) delete pDC;
   if (pBitmap != NULL) delete pBitmap;

   
   

So, i have a dialog window that doesn't have destructor by default. The only way is to add one. For example if my class is called Game in Game.h I have to type virtual ~Game() and in CPP file - Game::~Game(). Is that true?