Don't use GetDC() at all!
Use CClientDC class. It makes all Get/Release for you so you shouldn't care!

Besides, what is the purpose of TimerProc? Why not just handle WM_TIMER message in the CDlg...