From: Vadim Zeitlin <vadim@wxwidgets.org>
Date: Thu, 23 Aug 2001 23:27:06 +0000 (+0000)
Subject: always pass WM_DESTROY to DefWindowProc() - this fixes memory/resource leak for wxListBox
X-Git-Url: https://git.saurik.com/wxWidgets.git/commitdiff_plain/ad4297f34e21cea85219f3866d96c2622d5bb699

always pass WM_DESTROY to DefWindowProc() - this fixes memory/resource leak for wxListBox


git-svn-id: https://svn.wxwidgets.org/svn/wx/wxWidgets/trunk@11447 c3d73ce0-8a6f-49c7-b76d-6d57e0e08775
---

diff --git a/src/msw/window.cpp b/src/msw/window.cpp
index 0ba56334b2..6be083a03d 100644
--- a/src/msw/window.cpp
+++ b/src/msw/window.cpp
@@ -2125,7 +2125,11 @@ long wxWindowMSW::MSWWindowProc(WXUINT message, WXWPARAM wParam, WXLPARAM lParam
             break;
 
         case WM_DESTROY:
-            processed = HandleDestroy();
+            // never set processed to TRUE and *always* pass WM_DESTROY to
+            // DefWindowProc() as Windows may do some internal cleanup when
+            // processing it and failing to pass the message along may cause
+            // memory and resource leaks!
+            (void)HandleDestroy();
             break;
 
         case WM_MOVE: