dotnet / winforms

Windows Forms is a .NET UI framework for building Windows desktop applications.
MIT License
4.42k stars 985 forks source link

Out of Memory Exception: `TabControl.WmReflectDrawItem()` #12157

Closed samtrion closed 1 month ago

samtrion commented 2 months ago

.NET version

.NET 8.0 SDK: 8.0.204/8.0.401 RUNTIME: 8.0.8

Did it work in .NET Framework?

Not tested/verified

Did it work in any of the earlier releases of .NET Core or .NET 5+?

No, it's related to #10534

Issue description

In our application, we receive a OutOfMemoryException on a daily base. Roughly 4000 times in the last 90 days.

Error: Out of memory.

   at System.Windows.Forms.TabControl.WmReflectDrawItem(Message& m)
   at System.Windows.Forms.TabControl.WndProc(Message& m)
   at System.Windows.Forms.NativeWindow.Callback(HWND hWnd, MessageId msg, WPARAM wparam, LPARAM lparam)

Please merge #10724, which addresses the same problem as before for ComboBox and ListBox.

Steps to reproduce


merriemcgaw commented 2 months ago

@JeremyKuhne we should bring this back to shiproom given then additional reports.

merriemcgaw commented 2 months ago

@lonitra can you take this to Tactics next week? Or just do an email bar check with the new reports. I'm confident we should get approved.

samtrion commented 1 month ago

@lonitra Is there already a planned delivery date for this?

guyvaio commented 1 month ago

+1

lonitra commented 1 month ago

Is there already a planned delivery date for this?

This will be shipped as part of 8.0.11 release, which I believe will be November 5