[PATCH 0/1] MR3507: win32u: Copy an incoming CB_GETCOMBOBOXINFO when packing it for the client.

[Tim Clem (＠tclem)]

The only important field at this point is cbSize, but if it's zero, the
message will not be processed.

---

The other option would be to zero the buffer and then do the equivalent of `((COMBOBOXINFO *)buffer)->cbSize = sizeof(COMBOBOXINFO)`, rather than copying the incoming lparam. That doesn't seem better to me, but I'm no expert.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/3507

[PATCH 1/1] win32u: Copy an incoming CB_GETCOMBOBOXINFO when packing it for the client.

[Tim Clem]

From: Tim Clem <tclem(a)codeweavers.com>

The only important field at this point is cbSize, but if it's zero, the
message will not be processed.
---
 dlls/win32u/message.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/dlls/win32u/message.c b/dlls/win32u/message.c
index 71c1017a3f0..095b3694e3d 100644
--- a/dlls/win32u/message.c
+++ b/dlls/win32u/message.c
@@ -1842,9 +1842,6 @@ void pack_user_message( void *buffer, size_t size, UINT message,
         }
         return;
     }
-    case CB_GETCOMBOBOXINFO:
-        memset( buffer, 0, size );
-        return;
     }
 
     if (size) memcpy( buffer, lparam_ptr, size );
-- 
GitLab

https://gitlab.winehq.org/wine/wine/-/merge_requests/3507