July 2023 - Wine-Commits - list.winehq.org

Jacek Caban : win32u: Use user message packing for scroll info messages.
by Alexandre Julliard July 31, 2023

July 31, 2023

Module: wine Branch: master Commit: 10254e326f89003706805d9a64def5415875e891 URL: https://gitlab.winehq.org/wine/wine/-/commit/10254e326f89003706805d9a64def5… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Jul 18 13:43:25 2023 +0200 win32u: Use user message packing for scroll info messages. --- dlls/user32/winproc.c | 15 ++++++--------- dlls/win32u/message.c | 29 +++++++++++++++++++++++------ dlls/win32u/tests/win32u.c | 19 +++++++++++++++++++ dlls/wow64win/user.c | 3 +++ 4 files changed, 51 insertions(+), 15 deletions(-) diff --git a/dlls/user32/winproc.c b/dlls/user32/winproc.c index 6fd35691e68..66974e053ed 100644 --- a/dlls/user32/winproc.c +++ b/dlls/user32/winproc.c @@ -857,19 +857,13 @@ BOOL unpack_message( HWND hwnd, UINT message, WPARAM *wparam, LPARAM *lparam, case WM_STYLECHANGING: case WM_STYLECHANGED: case WM_GETDLGCODE: - break; - case WM_NOTIFY: - /* WM_NOTIFY cannot be sent across processes (MSDN) */ - return FALSE; case SBM_SETSCROLLINFO: - minsize = sizeof(SCROLLINFO); - break; case SBM_GETSCROLLINFO: - if (!get_buffer_space( buffer, sizeof(SCROLLINFO), size )) return FALSE; - break; case SBM_GETSCROLLBARINFO: - if (!get_buffer_space( buffer, sizeof(SCROLLBARINFO), size )) return FALSE; break; + case WM_NOTIFY: + /* WM_NOTIFY cannot be sent across processes (MSDN) */ + return FALSE; case EM_GETSEL: case SBM_GETRANGE: case CB_GETEDITSEL: @@ -1090,6 +1084,9 @@ BOOL WINAPI User32CallWindowProc( struct win_proc_params *params, ULONG size ) case WM_STYLECHANGING: case WM_STYLECHANGED: case WM_GETDLGCODE: + case SBM_SETSCROLLINFO: + case SBM_GETSCROLLINFO: + case SBM_GETSCROLLBARINFO: { LRESULT *result_ptr = (LRESULT *)buffer - 1; *result_ptr = result; diff --git a/dlls/win32u/message.c b/dlls/win32u/message.c index 9f8bf863b55..e2967446372 100644 --- a/dlls/win32u/message.c +++ b/dlls/win32u/message.c @@ -555,6 +555,15 @@ static BOOL unpack_message( HWND hwnd, UINT message, WPARAM *wparam, LPARAM *lpa break; } return TRUE; + case SBM_SETSCROLLINFO: + minsize = sizeof(SCROLLINFO); + break; + case SBM_GETSCROLLINFO: + if (!get_buffer_space( buffer, sizeof(SCROLLINFO), size )) return FALSE; + break; + case SBM_GETSCROLLBARINFO: + if (!get_buffer_space( buffer, sizeof(SCROLLBARINFO), size )) return FALSE; + break; case WM_WINE_SETWINDOWPOS: { WINDOWPOS wp; @@ -1349,6 +1358,13 @@ size_t user_message_size( UINT message, WPARAM wparam, LPARAM lparam, BOOL other case WM_GETDLGCODE: size = sizeof(MSG); break; + case SBM_SETSCROLLINFO: + case SBM_GETSCROLLINFO: + size = sizeof(SCROLLINFO); + break; + case SBM_GETSCROLLBARINFO: + size = sizeof(SCROLLBARINFO); + break; } return size; @@ -1480,6 +1496,13 @@ static void copy_user_result( void *buffer, size_t size, LRESULT result, UINT me case WM_STYLECHANGING: copy_size = sizeof(STYLESTRUCT); break; + case SBM_SETSCROLLINFO: + case SBM_GETSCROLLINFO: + copy_size = sizeof(SCROLLINFO); + break; + case SBM_GETSCROLLBARINFO: + copy_size = sizeof(SCROLLBARINFO); + break; default: return; } @@ -1507,12 +1530,6 @@ static void copy_reply( LRESULT result, HWND hwnd, UINT message, WPARAM wparam, case CB_GETCOMBOBOXINFO: copy_size = sizeof(COMBOBOXINFO); break; - case SBM_GETSCROLLINFO: - copy_size = sizeof(SCROLLINFO); - break; - case SBM_GETSCROLLBARINFO: - copy_size = sizeof(SCROLLBARINFO); - break; case EM_GETRECT: case LB_GETITEMRECT: case CB_GETDROPPEDCONTROLRECT: diff --git a/dlls/win32u/tests/win32u.c b/dlls/win32u/tests/win32u.c index d6cecc778d9..c6ed563d444 100644 --- a/dlls/win32u/tests/win32u.c +++ b/dlls/win32u/tests/win32u.c @@ -1659,6 +1659,10 @@ static void test_wndproc_hook(void) static const STYLESTRUCT style_in = { .styleOld = 1, .styleNew = 2 }; static const STYLESTRUCT style_out = { .styleOld = 10, .styleNew = 20 }; static const MSG msg_in = { .wParam = 1, .lParam = 2 }; + static const SCROLLINFO si_in = { .cbSize = sizeof(si_in), .nPos = 6 }; + static const SCROLLINFO si_out = { .cbSize = sizeof(si_in), .nPos = 60 }; + static const SCROLLBARINFO sbi_in = { .xyThumbTop = 6 }; + static const SCROLLBARINFO sbi_out = { .xyThumbTop = 60 }; static const struct lparam_hook_test lparam_hook_tests[] = { @@ -1813,6 +1817,21 @@ static void test_wndproc_hook(void) .lparam_size = sizeof(msg_in), .lparam = &msg_in, .poison_lparam = TRUE, .check_size = sizeof(msg_in), }, + { + "SBM_SETSCROLLINFO", SBM_SETSCROLLINFO, + .lparam_size = sizeof(si_in), .lparam = &si_in, .change_lparam = &si_out, + .check_size = sizeof(si_in), + }, + { + "SBM_GETSCROLLINFO", SBM_GETSCROLLINFO, + .lparam_size = sizeof(si_in), .lparam = &si_in, .change_lparam = &si_out, + .check_size = sizeof(si_in), + }, + { + "SBM_GETSCROLLBARINFO", SBM_GETSCROLLBARINFO, + .lparam_size = sizeof(sbi_in), .lparam = &sbi_in, .change_lparam = &sbi_out, + .check_size = sizeof(sbi_in), + }, /* messages that don't change lparam */ { "WM_USER", WM_USER }, { "WM_NOTIFY", WM_NOTIFY }, diff --git a/dlls/wow64win/user.c b/dlls/wow64win/user.c index c90dc22e31f..47c3ae6849d 100644 --- a/dlls/wow64win/user.c +++ b/dlls/wow64win/user.c @@ -832,6 +832,9 @@ static size_t packed_result_32to64( UINT message, WPARAM wparam, const void *par case WM_ASKCBFORMATNAME: case WM_GETMINMAXINFO: case WM_STYLECHANGING: + case SBM_SETSCROLLINFO: + case SBM_GETSCROLLINFO: + case SBM_GETSCROLLBARINFO: break; default:

1 0

Jacek Caban : win32u: Use user message packing for EM_GETSEL.
by Alexandre Julliard July 31, 2023

July 31, 2023

Module: wine Branch: master Commit: 0dbafa849612441c1b0bba2781ba17a7f6fa75d3 URL: https://gitlab.winehq.org/wine/wine/-/commit/0dbafa849612441c1b0bba2781ba17… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Jul 18 13:58:02 2023 +0200 win32u: Use user message packing for EM_GETSEL. --- dlls/user32/winproc.c | 22 ++++++++++++---------- dlls/win32u/message.c | 34 ++++++++++++++++++++++++++++------ dlls/win32u/tests/win32u.c | 27 +++++++++++++++++++++++++-- dlls/wow64win/user.c | 3 +++ 4 files changed, 68 insertions(+), 18 deletions(-) diff --git a/dlls/user32/winproc.c b/dlls/user32/winproc.c index 66974e053ed..9f073fabb70 100644 --- a/dlls/user32/winproc.c +++ b/dlls/user32/winproc.c @@ -837,6 +837,15 @@ BOOL unpack_message( HWND hwnd, UINT message, WPARAM *wparam, LPARAM *lparam, if (cds->lpData) cds->lpData = cds + 1; break; } + case EM_GETSEL: + case SBM_GETRANGE: + case CB_GETEDITSEL: + { + DWORD *ptr = *buffer; + *wparam = (WPARAM)ptr++; + *lparam = (LPARAM)ptr; + return TRUE; + } case WM_GETTEXT: case WM_ASKCBFORMATNAME: case WM_WININICHANGE: @@ -864,16 +873,6 @@ BOOL unpack_message( HWND hwnd, UINT message, WPARAM *wparam, LPARAM *lparam, case WM_NOTIFY: /* WM_NOTIFY cannot be sent across processes (MSDN) */ return FALSE; - case EM_GETSEL: - case SBM_GETRANGE: - case CB_GETEDITSEL: - if (*wparam || *lparam) - { - if (!get_buffer_space( buffer, 2*sizeof(DWORD), size )) return FALSE; - if (*wparam) *wparam = (WPARAM)*buffer; - if (*lparam) *lparam = (LPARAM)((DWORD *)*buffer + 1); - } - return TRUE; case EM_GETRECT: case LB_GETITEMRECT: case CB_GETDROPPEDCONTROLRECT: @@ -1087,6 +1086,9 @@ BOOL WINAPI User32CallWindowProc( struct win_proc_params *params, ULONG size ) case SBM_SETSCROLLINFO: case SBM_GETSCROLLINFO: case SBM_GETSCROLLBARINFO: + case EM_GETSEL: + case SBM_GETRANGE: + case CB_GETEDITSEL: { LRESULT *result_ptr = (LRESULT *)buffer - 1; *result_ptr = result; diff --git a/dlls/win32u/message.c b/dlls/win32u/message.c index e2967446372..540c58797b9 100644 --- a/dlls/win32u/message.c +++ b/dlls/win32u/message.c @@ -564,6 +564,16 @@ static BOOL unpack_message( HWND hwnd, UINT message, WPARAM *wparam, LPARAM *lpa case SBM_GETSCROLLBARINFO: if (!get_buffer_space( buffer, sizeof(SCROLLBARINFO), size )) return FALSE; break; + case EM_GETSEL: + case SBM_GETRANGE: + case CB_GETEDITSEL: + if (*wparam || *lparam) + { + if (!get_buffer_space( buffer, 2 * sizeof(DWORD), size )) return FALSE; + if (*wparam) *wparam = (WPARAM)*buffer; + if (*lparam) *lparam = (LPARAM)((DWORD *)*buffer + 1); + } + return TRUE; case WM_WINE_SETWINDOWPOS: { WINDOWPOS wp; @@ -1365,6 +1375,11 @@ size_t user_message_size( UINT message, WPARAM wparam, LPARAM lparam, BOOL other case SBM_GETSCROLLBARINFO: size = sizeof(SCROLLBARINFO); break; + case EM_GETSEL: + case SBM_GETRANGE: + case CB_GETEDITSEL: + size = 2 * sizeof(DWORD); + break; } return size; @@ -1424,6 +1439,10 @@ void pack_user_message( void *buffer, size_t size, UINT message, size = sizeof(*cds); break; } + case EM_GETSEL: + case SBM_GETRANGE: + case CB_GETEDITSEL: + return; } if (size) memcpy( buffer, lparam_ptr, size ); @@ -1503,6 +1522,15 @@ static void copy_user_result( void *buffer, size_t size, LRESULT result, UINT me case SBM_GETSCROLLBARINFO: copy_size = sizeof(SCROLLBARINFO); break; + case EM_GETSEL: + case SBM_GETRANGE: + case CB_GETEDITSEL: + { + DWORD *ptr = buffer; + if (wparam) *(DWORD *)wparam = ptr[0]; + if (lparam) *(DWORD *)lparam = ptr[1]; + break; + } default: return; } @@ -1549,12 +1577,6 @@ static void copy_reply( LRESULT result, HWND hwnd, UINT message, WPARAM wparam, case WM_MDIGETACTIVE: if (lparam) copy_size = sizeof(BOOL); break; - case EM_GETSEL: - case SBM_GETRANGE: - case CB_GETEDITSEL: - if (wparam) *(DWORD *)wparam = *(DWORD *)wparam_src; - if (lparam) copy_size = sizeof(DWORD); - break; case WM_NEXTMENU: copy_size = sizeof(MDINEXTMENU); break; diff --git a/dlls/win32u/tests/win32u.c b/dlls/win32u/tests/win32u.c index c6ed563d444..d2a298436d9 100644 --- a/dlls/win32u/tests/win32u.c +++ b/dlls/win32u/tests/win32u.c @@ -1355,6 +1355,7 @@ struct lparam_hook_test const char *name; UINT message; WPARAM wparam; + BOOL no_wparam_check; LRESULT msg_result; LRESULT check_result; BOOL todo_result; @@ -1378,7 +1379,7 @@ static char lparam_buffer[521]; static void check_params( const struct lparam_hook_test *test, UINT message, WPARAM wparam, LPARAM lparam, BOOL is_ret ) { - if (test->message != WM_MDIGETACTIVE) + if (!test->no_wparam_check) ok( wparam == test->wparam, "got wparam %Ix, expected %Ix\n", wparam, test->wparam ); if (lparam == (LPARAM)lparam_buffer) return; @@ -1406,6 +1407,12 @@ static void check_params( const struct lparam_hook_test *test, UINT message, } break; + case EM_GETSEL: + case SBM_GETRANGE: + case CB_GETEDITSEL: + ok( wparam, "wparam = 0\n" ); + break; + default: if (test->check_size) { const void *expected = is_ret && test->change_lparam ? test->change_lparam : test->lparam; @@ -1663,6 +1670,7 @@ static void test_wndproc_hook(void) static const SCROLLINFO si_out = { .cbSize = sizeof(si_in), .nPos = 60 }; static const SCROLLBARINFO sbi_in = { .xyThumbTop = 6 }; static const SCROLLBARINFO sbi_out = { .xyThumbTop = 60 }; + static const DWORD dw_in = 1, dw_out = 2; static const struct lparam_hook_test lparam_hook_tests[] = { @@ -1743,7 +1751,7 @@ static void test_wndproc_hook(void) .todo = TRUE }, { - "WM_MDIGETACTIVE", WM_MDIGETACTIVE, + "WM_MDIGETACTIVE", WM_MDIGETACTIVE, .no_wparam_check = TRUE, .lparam_size = sizeof(BOOL), .change_lparam = &false_lparam, .todo = TRUE }, @@ -1832,6 +1840,21 @@ static void test_wndproc_hook(void) .lparam_size = sizeof(sbi_in), .lparam = &sbi_in, .change_lparam = &sbi_out, .check_size = sizeof(sbi_in), }, + { + "EM_GETSEL", EM_GETSEL, .no_wparam_check = TRUE, + .lparam_size = sizeof(DWORD), .lparam = &dw_in, .change_lparam = &dw_out, + .check_size = sizeof(DWORD), + }, + { + "SBM_GETRANGE", SBM_GETRANGE, .no_wparam_check = TRUE, + .lparam_size = sizeof(DWORD), .lparam = &dw_in, .change_lparam = &dw_out, + .check_size = sizeof(DWORD), + }, + { + "CB_GETEDITSEL", CB_GETEDITSEL, .no_wparam_check = TRUE, + .lparam_size = sizeof(DWORD), .lparam = &dw_in, .change_lparam = &dw_out, + .check_size = sizeof(DWORD), + }, /* messages that don't change lparam */ { "WM_USER", WM_USER }, { "WM_NOTIFY", WM_NOTIFY }, diff --git a/dlls/wow64win/user.c b/dlls/wow64win/user.c index 47c3ae6849d..9685a51029d 100644 --- a/dlls/wow64win/user.c +++ b/dlls/wow64win/user.c @@ -835,6 +835,9 @@ static size_t packed_result_32to64( UINT message, WPARAM wparam, const void *par case SBM_SETSCROLLINFO: case SBM_GETSCROLLINFO: case SBM_GETSCROLLBARINFO: + case EM_GETSEL: + case SBM_GETRANGE: + case CB_GETEDITSEL: break; default:

1 0

Jacek Caban : win32u: Use user message packing for WM_STYLECHANGING.
by Alexandre Julliard July 31, 2023

July 31, 2023

Module: wine Branch: master Commit: 61e0190a52608580b3cb0116eeb3658da60d339f URL: https://gitlab.winehq.org/wine/wine/-/commit/61e0190a52608580b3cb0116eeb365… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Jul 18 12:47:45 2023 +0200 win32u: Use user message packing for WM_STYLECHANGING. --- dlls/user32/winproc.c | 8 ++++---- dlls/win32u/message.c | 14 +++++++++++--- dlls/win32u/tests/win32u.c | 12 ++++++++++++ dlls/wow64win/user.c | 1 + 4 files changed, 28 insertions(+), 7 deletions(-) diff --git a/dlls/user32/winproc.c b/dlls/user32/winproc.c index 8d6f89eaa6f..e759c5565b7 100644 --- a/dlls/user32/winproc.c +++ b/dlls/user32/winproc.c @@ -854,14 +854,12 @@ BOOL unpack_message( HWND hwnd, UINT message, WPARAM *wparam, LPARAM *lparam, case WM_WINDOWPOSCHANGING: case WM_WINDOWPOSCHANGED: case WM_HELP: + case WM_STYLECHANGING: + case WM_STYLECHANGED: break; case WM_NOTIFY: /* WM_NOTIFY cannot be sent across processes (MSDN) */ return FALSE; - case WM_STYLECHANGING: - case WM_STYLECHANGED: - minsize = sizeof(STYLESTRUCT); - break; case WM_GETDLGCODE: if (*lparam) { @@ -1103,6 +1101,8 @@ BOOL WINAPI User32CallWindowProc( struct win_proc_params *params, ULONG size ) case WM_WINDOWPOSCHANGED: case WM_COPYDATA: case WM_HELP: + case WM_STYLECHANGING: + case WM_STYLECHANGED: { LRESULT *result_ptr = (LRESULT *)buffer - 1; *result_ptr = result; diff --git a/dlls/win32u/message.c b/dlls/win32u/message.c index 7a3acf13ba5..f3068a9c618 100644 --- a/dlls/win32u/message.c +++ b/dlls/win32u/message.c @@ -536,6 +536,10 @@ static BOOL unpack_message( HWND hwnd, UINT message, WPARAM *wparam, LPARAM *lpa memcpy( &ps->hi, &hi, sizeof(hi) ); break; } + case WM_STYLECHANGING: + case WM_STYLECHANGED: + minsize = sizeof(STYLESTRUCT); + break; case WM_WINE_SETWINDOWPOS: { WINDOWPOS wp; @@ -1348,6 +1352,10 @@ size_t user_message_size( UINT message, WPARAM wparam, LPARAM lparam, BOOL other case WM_HELP: size = sizeof(HELPINFO); break; + case WM_STYLECHANGING: + case WM_STYLECHANGED: + size = sizeof(STYLESTRUCT); + break; } return size; @@ -1476,6 +1484,9 @@ static void copy_user_result( void *buffer, size_t size, LRESULT result, UINT me case WM_WINDOWPOSCHANGING: copy_size = sizeof(WINDOWPOS); break; + case WM_STYLECHANGING: + copy_size = sizeof(STYLESTRUCT); + break; default: return; } @@ -1503,9 +1514,6 @@ static void copy_reply( LRESULT result, HWND hwnd, UINT message, WPARAM wparam, case CB_GETCOMBOBOXINFO: copy_size = sizeof(COMBOBOXINFO); break; - case WM_STYLECHANGING: - copy_size = sizeof(STYLESTRUCT); - break; case WM_GETDLGCODE: if (lparam) copy_size = sizeof(MSG); break; diff --git a/dlls/win32u/tests/win32u.c b/dlls/win32u/tests/win32u.c index 501938cd039..fd526bcb36b 100644 --- a/dlls/win32u/tests/win32u.c +++ b/dlls/win32u/tests/win32u.c @@ -1656,6 +1656,8 @@ static void test_wndproc_hook(void) static const COPYDATASTRUCT cds3_in = { .dwData = 2, .lpData = (void *)0xdeadbeef }; static const COPYDATASTRUCT cds4_in = { .cbData = 2 }; static const COPYDATASTRUCT cds5_in = { .lpData = (void *)0xdeadbeef }; + static const STYLESTRUCT style_in = { .styleOld = 1, .styleNew = 2 }; + static const STYLESTRUCT style_out = { .styleOld = 10, .styleNew = 20 }; static const struct lparam_hook_test lparam_hook_tests[] = { @@ -1795,6 +1797,16 @@ static void test_wndproc_hook(void) .lparam_size = sizeof(cds5_in), .lparam = &cds5_in, .poison_lparam = TRUE, .check_size = sizeof(cds5_in), }, + { + "WM_STYLECHANGING", WM_STYLECHANGING, + .lparam_size = sizeof(style_in), .lparam = &style_in, .change_lparam = &style_out, + .check_size = sizeof(style_in) + }, + { + "WM_STYLECHANGED", WM_STYLECHANGED, + .lparam_size = sizeof(style_in), .lparam = &style_in, .poison_lparam = TRUE, + .check_size = sizeof(style_in), + }, /* messages that don't change lparam */ { "WM_USER", WM_USER }, { "WM_NOTIFY", WM_NOTIFY }, diff --git a/dlls/wow64win/user.c b/dlls/wow64win/user.c index 5914a8d6d4e..d5d017a086f 100644 --- a/dlls/wow64win/user.c +++ b/dlls/wow64win/user.c @@ -827,6 +827,7 @@ static size_t packed_result_32to64( UINT message, WPARAM wparam, const void *par case WM_GETTEXT: case WM_ASKCBFORMATNAME: case WM_GETMINMAXINFO: + case WM_STYLECHANGING: break; default:

1 0

Jacek Caban : win32u: Use user message packing for WM_GETDLGCODE.
by Alexandre Julliard July 31, 2023

July 31, 2023

Module: wine Branch: master Commit: a883b3c5c7bf5313e4831fa2da9cdf06f99aa769 URL: https://gitlab.winehq.org/wine/wine/-/commit/a883b3c5c7bf5313e4831fa2da9cdf… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Jul 18 13:39:04 2023 +0200 win32u: Use user message packing for WM_GETDLGCODE. --- dlls/user32/winproc.c | 17 ++--------------- dlls/win32u/message.c | 46 ++++++++++++++++++---------------------------- dlls/win32u/tests/win32u.c | 6 ++++++ dlls/wow64win/user.c | 4 ++++ 4 files changed, 30 insertions(+), 43 deletions(-) diff --git a/dlls/user32/winproc.c b/dlls/user32/winproc.c index e759c5565b7..6fd35691e68 100644 --- a/dlls/user32/winproc.c +++ b/dlls/user32/winproc.c @@ -856,25 +856,11 @@ BOOL unpack_message( HWND hwnd, UINT message, WPARAM *wparam, LPARAM *lparam, case WM_HELP: case WM_STYLECHANGING: case WM_STYLECHANGED: + case WM_GETDLGCODE: break; case WM_NOTIFY: /* WM_NOTIFY cannot be sent across processes (MSDN) */ return FALSE; - case WM_GETDLGCODE: - if (*lparam) - { - MSG msg; - if (size < sizeof(ps->msg)) return FALSE; - msg.hwnd = unpack_handle( ps->msg.hwnd ); - msg.message = ps->msg.message; - msg.wParam = (ULONG_PTR)unpack_ptr( ps->msg.wParam ); - msg.lParam = (ULONG_PTR)unpack_ptr( ps->msg.lParam ); - msg.time = ps->msg.time; - msg.pt = ps->msg.pt; - memcpy( &ps->msg, &msg, sizeof(msg) ); - break; - } - return TRUE; case SBM_SETSCROLLINFO: minsize = sizeof(SCROLLINFO); break; @@ -1103,6 +1089,7 @@ BOOL WINAPI User32CallWindowProc( struct win_proc_params *params, ULONG size ) case WM_HELP: case WM_STYLECHANGING: case WM_STYLECHANGED: + case WM_GETDLGCODE: { LRESULT *result_ptr = (LRESULT *)buffer - 1; *result_ptr = result; diff --git a/dlls/win32u/message.c b/dlls/win32u/message.c index f3068a9c618..9f8bf863b55 100644 --- a/dlls/win32u/message.c +++ b/dlls/win32u/message.c @@ -540,6 +540,21 @@ static BOOL unpack_message( HWND hwnd, UINT message, WPARAM *wparam, LPARAM *lpa case WM_STYLECHANGED: minsize = sizeof(STYLESTRUCT); break; + case WM_GETDLGCODE: + if (*lparam) + { + MSG msg; + if (size < sizeof(ps->msg)) return FALSE; + msg.hwnd = wine_server_ptr_handle( ps->msg.hwnd ); + msg.message = ps->msg.message; + msg.wParam = (ULONG_PTR)unpack_ptr( ps->msg.wParam ); + msg.lParam = (ULONG_PTR)unpack_ptr( ps->msg.lParam ); + msg.time = ps->msg.time; + msg.pt = ps->msg.pt; + memcpy( &ps->msg, &msg, sizeof(msg) ); + break; + } + return TRUE; case WM_WINE_SETWINDOWPOS: { WINDOWPOS wp; @@ -1006,19 +1021,6 @@ static void pack_reply( HWND hwnd, UINT message, WPARAM wparam, LPARAM lparam, push_data( data, &data->ps.wp, sizeof(data->ps.wp) ); break; } - case WM_GETDLGCODE: - if (lparam) - { - MSG *msg = (MSG *)lparam; - data->ps.msg.hwnd = wine_server_user_handle( msg->hwnd ); - data->ps.msg.message = msg->message; - data->ps.msg.wParam = msg->wParam; - data->ps.msg.lParam = msg->lParam; - data->ps.msg.time = msg->time; - data->ps.msg.pt = msg->pt; - push_data( data, &data->ps.msg, sizeof(data->ps.msg) ); - } - break; case SBM_GETSCROLLINFO: push_data( data, (SCROLLINFO *)lparam, sizeof(SCROLLINFO) ); break; @@ -1159,18 +1161,6 @@ static void unpack_reply( HWND hwnd, UINT message, WPARAM wparam, LPARAM lparam, wp->flags = ps->wp.flags; } break; - case WM_GETDLGCODE: - if (lparam && size >= sizeof(ps->msg)) - { - MSG *msg = (MSG *)lparam; - msg->hwnd = wine_server_ptr_handle( ps->msg.hwnd ); - msg->message = ps->msg.message; - msg->wParam = (ULONG_PTR)unpack_ptr( ps->msg.wParam ); - msg->lParam = (ULONG_PTR)unpack_ptr( ps->msg.lParam ); - msg->time = ps->msg.time; - msg->pt = ps->msg.pt; - } - break; case SBM_GETSCROLLINFO: memcpy( (SCROLLINFO *)lparam, buffer, min( sizeof(SCROLLINFO), size )); break; @@ -1356,6 +1346,9 @@ size_t user_message_size( UINT message, WPARAM wparam, LPARAM lparam, BOOL other case WM_STYLECHANGED: size = sizeof(STYLESTRUCT); break; + case WM_GETDLGCODE: + size = sizeof(MSG); + break; } return size; @@ -1514,9 +1507,6 @@ static void copy_reply( LRESULT result, HWND hwnd, UINT message, WPARAM wparam, case CB_GETCOMBOBOXINFO: copy_size = sizeof(COMBOBOXINFO); break; - case WM_GETDLGCODE: - if (lparam) copy_size = sizeof(MSG); - break; case SBM_GETSCROLLINFO: copy_size = sizeof(SCROLLINFO); break; diff --git a/dlls/win32u/tests/win32u.c b/dlls/win32u/tests/win32u.c index fd526bcb36b..d6cecc778d9 100644 --- a/dlls/win32u/tests/win32u.c +++ b/dlls/win32u/tests/win32u.c @@ -1658,6 +1658,7 @@ static void test_wndproc_hook(void) static const COPYDATASTRUCT cds5_in = { .lpData = (void *)0xdeadbeef }; static const STYLESTRUCT style_in = { .styleOld = 1, .styleNew = 2 }; static const STYLESTRUCT style_out = { .styleOld = 10, .styleNew = 20 }; + static const MSG msg_in = { .wParam = 1, .lParam = 2 }; static const struct lparam_hook_test lparam_hook_tests[] = { @@ -1807,6 +1808,11 @@ static void test_wndproc_hook(void) .lparam_size = sizeof(style_in), .lparam = &style_in, .poison_lparam = TRUE, .check_size = sizeof(style_in), }, + { + "WM_GETDLGCODE", WM_GETDLGCODE, + .lparam_size = sizeof(msg_in), .lparam = &msg_in, .poison_lparam = TRUE, + .check_size = sizeof(msg_in), + }, /* messages that don't change lparam */ { "WM_USER", WM_USER }, { "WM_NOTIFY", WM_NOTIFY }, diff --git a/dlls/wow64win/user.c b/dlls/wow64win/user.c index d5d017a086f..c90dc22e31f 100644 --- a/dlls/wow64win/user.c +++ b/dlls/wow64win/user.c @@ -758,6 +758,10 @@ static size_t packed_message_64to32( UINT message, WPARAM wparam, memcpy( params32, &hi32, sizeof(hi32) ); return sizeof(hi32); } + + case WM_GETDLGCODE: + msg_64to32( params64, params32 ); + return sizeof(MSG32); } memmove( params32, params64, size );

1 0

Jacek Caban : win32u: Use user message packing for WM_COPYDATA.
by Alexandre Julliard July 31, 2023

July 31, 2023

Module: wine Branch: master Commit: 69834a3095985f0fdc882ca36f38061347642f4a URL: https://gitlab.winehq.org/wine/wine/-/commit/69834a3095985f0fdc882ca36f3806… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Jul 18 12:24:04 2023 +0200 win32u: Use user message packing for WM_COPYDATA. --- dlls/user32/winproc.c | 26 ++++++-------------- dlls/win32u/message.c | 33 ++++++++++++++++++++++++++ dlls/win32u/tests/win32u.c | 59 +++++++++++++++++++++++++++++++++++++++++++--- dlls/wow64win/user.c | 14 +++++++++++ 4 files changed, 110 insertions(+), 22 deletions(-) diff --git a/dlls/user32/winproc.c b/dlls/user32/winproc.c index fb41d444a45..1363aad3340 100644 --- a/dlls/user32/winproc.c +++ b/dlls/user32/winproc.c @@ -831,6 +831,12 @@ BOOL unpack_message( HWND hwnd, UINT message, WPARAM *wparam, LPARAM *lparam, ncp->lppos = (WINDOWPOS *)((NCCALCSIZE_PARAMS *)ncp + 1); } break; + case WM_COPYDATA: + { + COPYDATASTRUCT *cds = *buffer; + if (cds->lpData) cds->lpData = cds + 1; + break; + } case WM_GETTEXT: case WM_ASKCBFORMATNAME: case WM_WININICHANGE: @@ -848,25 +854,6 @@ BOOL unpack_message( HWND hwnd, UINT message, WPARAM *wparam, LPARAM *lparam, case WM_WINDOWPOSCHANGING: case WM_WINDOWPOSCHANGED: break; - case WM_COPYDATA: - { - COPYDATASTRUCT cds; - if (size < sizeof(ps->cds)) return FALSE; - cds.dwData = (ULONG_PTR)unpack_ptr( ps->cds.dwData ); - if (ps->cds.lpData) - { - cds.cbData = ps->cds.cbData; - cds.lpData = &ps->cds + 1; - minsize = sizeof(ps->cds) + cds.cbData; - } - else - { - cds.cbData = 0; - cds.lpData = 0; - } - memcpy( &ps->cds, &cds, sizeof(cds) ); - break; - } case WM_NOTIFY: /* WM_NOTIFY cannot be sent across processes (MSDN) */ return FALSE; @@ -1126,6 +1113,7 @@ BOOL WINAPI User32CallWindowProc( struct win_proc_params *params, ULONG size ) case WM_COMPAREITEM: case WM_WINDOWPOSCHANGING: case WM_WINDOWPOSCHANGED: + case WM_COPYDATA: { LRESULT *result_ptr = (LRESULT *)buffer - 1; *result_ptr = result; diff --git a/dlls/win32u/message.c b/dlls/win32u/message.c index 641fa905af3..5cadaa9e75b 100644 --- a/dlls/win32u/message.c +++ b/dlls/win32u/message.c @@ -504,6 +504,25 @@ static BOOL unpack_message( HWND hwnd, UINT message, WPARAM *wparam, LPARAM *lpa memcpy( *buffer, &wp, sizeof(wp) ); break; } + case WM_COPYDATA: + { + COPYDATASTRUCT cds; + if (size < sizeof(ps->cds)) return FALSE; + cds.dwData = (ULONG_PTR)unpack_ptr( ps->cds.dwData ); + if (ps->cds.lpData) + { + cds.cbData = ps->cds.cbData; + cds.lpData = &ps->cds + 1; + minsize = sizeof(ps->cds) + cds.cbData; + } + else + { + cds.cbData = 0; + cds.lpData = 0; + } + memcpy( &ps->cds, &cds, sizeof(cds) ); + break; + } case WM_WINE_SETWINDOWPOS: { WINDOWPOS wp; @@ -1307,6 +1326,12 @@ size_t user_message_size( UINT message, WPARAM wparam, LPARAM lparam, BOOL other case WM_WINDOWPOSCHANGED: size = sizeof(WINDOWPOS); break; + case WM_COPYDATA: + { + const COPYDATASTRUCT *cds = lparam_ptr; + size = sizeof(*cds) + cds->cbData; + break; + } } return size; @@ -1358,6 +1383,14 @@ void pack_user_message( void *buffer, size_t size, UINT message, case WM_ASKCBFORMATNAME: if (wparam) memset( buffer, 0, char_size( ansi )); return; + case WM_COPYDATA: + { + const COPYDATASTRUCT *cds = lparam_ptr; + if (cds->lpData && cds->cbData) + memcpy( (char *)buffer + sizeof(*cds), cds->lpData, cds->cbData ); + size = sizeof(*cds); + break; + } } if (size) memcpy( buffer, lparam_ptr, size ); diff --git a/dlls/win32u/tests/win32u.c b/dlls/win32u/tests/win32u.c index aaa27361206..501938cd039 100644 --- a/dlls/win32u/tests/win32u.c +++ b/dlls/win32u/tests/win32u.c @@ -1386,9 +1386,31 @@ static void check_params( const struct lparam_hook_test *test, UINT message, ok ((LPARAM)&message < lparam && lparam < (LPARAM)NtCurrentTeb()->Tib.StackBase, "lparam is not on the stack\n"); - if (test->check_size) { - const void *expected = is_ret && test->change_lparam ? test->change_lparam : test->lparam; - ok( !memcmp( (const void *)lparam, expected, test->check_size ), "unexpected lparam content\n"); + switch (test->message) + { + case WM_COPYDATA: + { + const COPYDATASTRUCT *cds = (const COPYDATASTRUCT *)lparam; + const COPYDATASTRUCT *cds_in = (const COPYDATASTRUCT *)lparam_buffer; + ok( cds->dwData == cds_in->dwData, "cds->dwData != cds_in->dwData\n"); + ok( cds->cbData == cds_in->cbData, "cds->dwData != cds_in->dwData\n"); + if (cds_in->lpData) + { + ok( cds->lpData != cds_in->lpData, "cds->lpData == cds_in->lpData\n" ); + if (cds->cbData) + ok( !memcmp( cds->lpData, cds_in->lpData, cds->cbData ), "unexpected pvData %s\n", + wine_dbgstr_an( cds->lpData, cds->cbData )); + } + else + ok( !cds->lpData, "cds->lpData = %p\n", cds->lpData ); + } + break; + + default: + if (test->check_size) { + const void *expected = is_ret && test->change_lparam ? test->change_lparam : test->lparam; + ok( !memcmp( (const void *)lparam, expected, test->check_size ), "unexpected lparam content\n" ); + } } } @@ -1628,6 +1650,12 @@ static void test_wndproc_hook(void) static const COMPAREITEMSTRUCT cis_in = { .itemID1 = 1 }; static const WINDOWPOS winpos_in = { .x = 1, .cy = 2 }; static const WINDOWPOS winpos_out = { .x = 10, .cy = 22 }; + static const COPYDATASTRUCT cds_in = { .dwData = 1 }; + static WORD data_word = 3; + static const COPYDATASTRUCT cds2_in = { .cbData = 2, .lpData = &data_word }; + static const COPYDATASTRUCT cds3_in = { .dwData = 2, .lpData = (void *)0xdeadbeef }; + static const COPYDATASTRUCT cds4_in = { .cbData = 2 }; + static const COPYDATASTRUCT cds5_in = { .lpData = (void *)0xdeadbeef }; static const struct lparam_hook_test lparam_hook_tests[] = { @@ -1742,6 +1770,31 @@ static void test_wndproc_hook(void) .lparam_size = sizeof(WINDOWPOS), .lparam = &winpos_in, .poison_lparam = TRUE, .check_size = sizeof(WINDOWPOS), }, + { + "WM_COPYDATA", WM_COPYDATA, .wparam = 0xdeadbeef, + .lparam_size = sizeof(cds_in), .lparam = &cds_in, .poison_lparam = TRUE, + .check_size = sizeof(cds_in), + }, + { + "WM_COPYDATA-2", WM_COPYDATA, .wparam = 0xdeadbeef, + .lparam_size = sizeof(cds2_in), .lparam = &cds2_in, .poison_lparam = TRUE, + .check_size = sizeof(cds2_in), + }, + { + "WM_COPYDATA-3", WM_COPYDATA, .wparam = 0xdeadbeef, + .lparam_size = sizeof(cds3_in), .lparam = &cds3_in, .poison_lparam = TRUE, + .check_size = sizeof(cds3_in), + }, + { + "WM_COPYDATA-4", WM_COPYDATA, .wparam = 0xdeadbeef, + .lparam_size = sizeof(cds4_in), .lparam = &cds4_in, .poison_lparam = TRUE, + .check_size = sizeof(cds4_in), + }, + { + "WM_COPYDATA-5", WM_COPYDATA, .wparam = 0xdeadbeef, + .lparam_size = sizeof(cds5_in), .lparam = &cds5_in, .poison_lparam = TRUE, + .check_size = sizeof(cds5_in), + }, /* messages that don't change lparam */ { "WM_USER", WM_USER }, { "WM_NOTIFY", WM_NOTIFY }, diff --git a/dlls/wow64win/user.c b/dlls/wow64win/user.c index b2a7a3887cc..429854f1950 100644 --- a/dlls/wow64win/user.c +++ b/dlls/wow64win/user.c @@ -720,6 +720,20 @@ static size_t packed_message_64to32( UINT message, WPARAM wparam, case WM_WINDOWPOSCHANGED: winpos_64to32( params64, params32 ); return sizeof(WINDOWPOS32); + + case WM_COPYDATA: + { + COPYDATASTRUCT32 cds32; + const COPYDATASTRUCT *cds64 = params64; + + cds32.dwData = cds64->dwData; + cds32.cbData = cds64->cbData; + cds32.lpData = PtrToUlong( cds64->lpData ); + memcpy( params32, &cds32, sizeof(cds32) ); + size -= sizeof(cds32); + if (size) memmove( (char *)params32 + sizeof(cds32), cds64 + 1, size ); + return sizeof(cds32) + size; + } } memmove( params32, params64, size );

1 0

Jacek Caban : win32u: Use user message packing for WM_HELP.
by Alexandre Julliard July 31, 2023

July 31, 2023

Module: wine Branch: master Commit: e7aa8945550a00bcf3fe48d79dd61f06aab0eb9e URL: https://gitlab.winehq.org/wine/wine/-/commit/e7aa8945550a00bcf3fe48d79dd61f… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Tue Jul 18 12:41:46 2023 +0200 win32u: Use user message packing for WM_HELP. --- dlls/user32/winproc.c | 15 ++------------- dlls/win32u/message.c | 16 ++++++++++++++++ dlls/wow64win/user.c | 38 ++++++++++++++++++++++++++++++++++++++ 3 files changed, 56 insertions(+), 13 deletions(-) diff --git a/dlls/user32/winproc.c b/dlls/user32/winproc.c index 1363aad3340..8d6f89eaa6f 100644 --- a/dlls/user32/winproc.c +++ b/dlls/user32/winproc.c @@ -853,23 +853,11 @@ BOOL unpack_message( HWND hwnd, UINT message, WPARAM *wparam, LPARAM *lparam, case WM_COMPAREITEM: case WM_WINDOWPOSCHANGING: case WM_WINDOWPOSCHANGED: + case WM_HELP: break; case WM_NOTIFY: /* WM_NOTIFY cannot be sent across processes (MSDN) */ return FALSE; - case WM_HELP: - { - HELPINFO hi; - if (size < sizeof(ps->hi)) return FALSE; - hi.cbSize = sizeof(hi); - hi.iContextType = ps->hi.iContextType; - hi.iCtrlId = ps->hi.iCtrlId; - hi.hItemHandle = unpack_handle( ps->hi.hItemHandle ); - hi.dwContextId = (ULONG_PTR)unpack_ptr( ps->hi.dwContextId ); - hi.MousePos = ps->hi.MousePos; - memcpy( &ps->hi, &hi, sizeof(hi) ); - break; - } case WM_STYLECHANGING: case WM_STYLECHANGED: minsize = sizeof(STYLESTRUCT); @@ -1114,6 +1102,7 @@ BOOL WINAPI User32CallWindowProc( struct win_proc_params *params, ULONG size ) case WM_WINDOWPOSCHANGING: case WM_WINDOWPOSCHANGED: case WM_COPYDATA: + case WM_HELP: { LRESULT *result_ptr = (LRESULT *)buffer - 1; *result_ptr = result; diff --git a/dlls/win32u/message.c b/dlls/win32u/message.c index 5cadaa9e75b..7a3acf13ba5 100644 --- a/dlls/win32u/message.c +++ b/dlls/win32u/message.c @@ -523,6 +523,19 @@ static BOOL unpack_message( HWND hwnd, UINT message, WPARAM *wparam, LPARAM *lpa memcpy( &ps->cds, &cds, sizeof(cds) ); break; } + case WM_HELP: + { + HELPINFO hi; + if (size < sizeof(ps->hi)) return FALSE; + hi.cbSize = sizeof(hi); + hi.iContextType = ps->hi.iContextType; + hi.iCtrlId = ps->hi.iCtrlId; + hi.hItemHandle = wine_server_ptr_handle( ps->hi.hItemHandle ); + hi.dwContextId = (ULONG_PTR)unpack_ptr( ps->hi.dwContextId ); + hi.MousePos = ps->hi.MousePos; + memcpy( &ps->hi, &hi, sizeof(hi) ); + break; + } case WM_WINE_SETWINDOWPOS: { WINDOWPOS wp; @@ -1332,6 +1345,9 @@ size_t user_message_size( UINT message, WPARAM wparam, LPARAM lparam, BOOL other size = sizeof(*cds) + cds->cbData; break; } + case WM_HELP: + size = sizeof(HELPINFO); + break; } return size; diff --git a/dlls/wow64win/user.c b/dlls/wow64win/user.c index 429854f1950..5914a8d6d4e 100644 --- a/dlls/wow64win/user.c +++ b/dlls/wow64win/user.c @@ -218,6 +218,16 @@ typedef struct ULONG lpData; } COPYDATASTRUCT32; +typedef struct +{ + UINT cbSize; + INT iContextType; + INT iCtrlId; + ULONG hItemHandle; + DWORD dwContextId; + POINT MousePos; +} HELPINFO32; + typedef struct { UINT CtlType; @@ -734,6 +744,20 @@ static size_t packed_message_64to32( UINT message, WPARAM wparam, if (size) memmove( (char *)params32 + sizeof(cds32), cds64 + 1, size ); return sizeof(cds32) + size; } + case WM_HELP: + { + HELPINFO32 hi32; + const HELPINFO *hi64 = params64; + + hi32.cbSize = sizeof(hi32); + hi32.iContextType = hi64->iContextType; + hi32.iCtrlId = hi64->iCtrlId; + hi32.hItemHandle = HandleToLong( hi64->hItemHandle ); + hi32.dwContextId = hi64->dwContextId; + hi32.MousePos = hi64->MousePos; + memcpy( params32, &hi32, sizeof(hi32) ); + return sizeof(hi32); + } } memmove( params32, params64, size ); @@ -3280,6 +3304,20 @@ static LRESULT message_call_32to64( HWND hwnd, UINT msg, WPARAM wparam, LPARAM l return NtUserMessageCall( hwnd, msg, wparam, (LPARAM)&cds, result_info, type, ansi ); } + case WM_HELP: + { + HELPINFO32 *hi32 = (void *)lparam; + HELPINFO hi64; + + hi64.cbSize = sizeof(hi64); + hi64.iContextType = hi32->iContextType; + hi64.iCtrlId = hi32->iCtrlId; + hi64.hItemHandle = LongToHandle( hi32->hItemHandle ); + hi64.dwContextId = hi32->dwContextId; + hi64.MousePos = hi32->MousePos; + return NtUserMessageCall( hwnd, msg, wparam, (LPARAM)&hi64, result_info, type, ansi ); + } + case WM_GETDLGCODE: if (lparam) {

1 0

Paul Gofman : nsi: Add stub for NsiCancelChangeNotification.
by Alexandre Julliard July 31, 2023

July 31, 2023

Module: wine Branch: master Commit: e06093cbae775e165e53f4ea243ad8b9fdf90f5c URL: https://gitlab.winehq.org/wine/wine/-/commit/e06093cbae775e165e53f4ea243ad8… Author: Paul Gofman <pgofman(a)codeweavers.com> Date: Tue Jul 25 17:39:41 2023 -0600 nsi: Add stub for NsiCancelChangeNotification. --- dlls/nsi/nsi.c | 7 +++++++ dlls/nsi/nsi.spec | 2 +- include/wine/nsi.h | 1 + 3 files changed, 9 insertions(+), 1 deletion(-) diff --git a/dlls/nsi/nsi.c b/dlls/nsi/nsi.c index e20aa6ef21a..c00ec4f8684 100644 --- a/dlls/nsi/nsi.c +++ b/dlls/nsi/nsi.c @@ -88,6 +88,13 @@ err: return err; } +DWORD WINAPI NsiCancelChangeNotification( OVERLAPPED *ovr ) +{ + FIXME( "%p stub.\n", ovr ); + + return ERROR_NOT_SUPPORTED; +} + DWORD WINAPI NsiEnumerateObjectsAllParameters( DWORD unk, DWORD unk2, const NPI_MODULEID *module, DWORD table, void *key_data, DWORD key_size, void *rw_data, DWORD rw_size, void *dynamic_data, DWORD dynamic_size, void *static_data, DWORD static_size, diff --git a/dlls/nsi/nsi.spec b/dlls/nsi/nsi.spec index b343b228b9c..9f1a2472362 100644 --- a/dlls/nsi/nsi.spec +++ b/dlls/nsi/nsi.spec @@ -1,6 +1,6 @@ @ stub NsiAllocateAndGetPersistentDataWithMaskTable @ stdcall NsiAllocateAndGetTable(long ptr long ptr long ptr long ptr long ptr long ptr long) -@ stub NsiCancelChangeNotification +@ stdcall NsiCancelChangeNotification(ptr) @ stub NsiDeregisterChangeNotification @ stub NsiDeregisterChangeNotificationEx @ stdcall NsiEnumerateObjectsAllParameters(long long ptr long ptr long ptr long ptr long ptr long ptr) diff --git a/include/wine/nsi.h b/include/wine/nsi.h index 2f9fb0cb6e3..1a562c3c36d 100644 --- a/include/wine/nsi.h +++ b/include/wine/nsi.h @@ -504,6 +504,7 @@ struct nsi_request_change_notification_ex DWORD WINAPI NsiAllocateAndGetTable( DWORD unk, const NPI_MODULEID *module, DWORD table, void **key_data, DWORD key_size, void **rw_data, DWORD rw_size, void **dynamic_data, DWORD dynamic_size, void **static_data, DWORD static_size, DWORD *count, DWORD unk2 ); +DWORD WINAPI NsiCancelChangeNotification( OVERLAPPED *ovr ); DWORD WINAPI NsiEnumerateObjectsAllParameters( DWORD unk, DWORD unk2, const NPI_MODULEID *module, DWORD table, void *key_data, DWORD key_size, void *rw_data, DWORD rw_size, void *dynamic_data, DWORD dynamic_size, void *static_data, DWORD static_size,

1 0

Paul Gofman : nsi: Cache nsi device handle.
by Alexandre Julliard July 31, 2023

July 31, 2023

Module: wine Branch: master Commit: b8fa6de2c2df3526993673002c2a2453943c61a2 URL: https://gitlab.winehq.org/wine/wine/-/commit/b8fa6de2c2df3526993673002c2a24… Author: Paul Gofman <pgofman(a)codeweavers.com> Date: Tue Jul 25 17:49:07 2023 -0600 nsi: Cache nsi device handle. --- dlls/nsi/nsi.c | 43 ++++++++++++++++++++++++++++--------------- 1 file changed, 28 insertions(+), 15 deletions(-) diff --git a/dlls/nsi/nsi.c b/dlls/nsi/nsi.c index c00ec4f8684..c219d59ccf3 100644 --- a/dlls/nsi/nsi.c +++ b/dlls/nsi/nsi.c @@ -30,9 +30,34 @@ WINE_DEFAULT_DEBUG_CHANNEL(nsi); +static HANDLE nsi_device = INVALID_HANDLE_VALUE; + +BOOL WINAPI DllMain(HINSTANCE hinst, DWORD reason, void *reserved) +{ + switch (reason) + { + case DLL_PROCESS_ATTACH: + DisableThreadLibraryCalls( hinst ); + break; + case DLL_PROCESS_DETACH: + if (nsi_device != INVALID_HANDLE_VALUE) CloseHandle( nsi_device ); + break; + } + return TRUE; +} + static inline HANDLE get_nsi_device( void ) { - return CreateFileW( L"\\\\.\\Nsi", 0, FILE_SHARE_READ | FILE_SHARE_WRITE, NULL, OPEN_EXISTING, 0, NULL ); + HANDLE device; + + if (nsi_device == INVALID_HANDLE_VALUE) + { + device = CreateFileW( L"\\\\.\\Nsi", 0, FILE_SHARE_READ | FILE_SHARE_WRITE, NULL, OPEN_EXISTING, 0, NULL ); + if (device != INVALID_HANDLE_VALUE + && InterlockedCompareExchangePointer( &nsi_device, device, INVALID_HANDLE_VALUE ) != INVALID_HANDLE_VALUE) + CloseHandle( device ); + } + return nsi_device; } DWORD WINAPI NsiAllocateAndGetTable( DWORD unk, const NPI_MODULEID *module, DWORD table, void **key_data, DWORD key_size, @@ -140,11 +165,7 @@ DWORD WINAPI NsiEnumerateObjectsAllParametersEx( struct nsi_enumerate_all_ex *pa (params->key_size + params->rw_size + params->dynamic_size + params->static_size) * params->count; out = heap_alloc( out_size ); - if (!out) - { - CloseHandle( device ); - return ERROR_OUTOFMEMORY; - } + if (!out) return ERROR_OUTOFMEMORY; in.module = *params->module; in.first_arg = params->first_arg; @@ -172,7 +193,6 @@ DWORD WINAPI NsiEnumerateObjectsAllParametersEx( struct nsi_enumerate_all_ex *pa } heap_free( out ); - CloseHandle( device ); return err; } @@ -256,7 +276,6 @@ DWORD WINAPI NsiGetAllParametersEx( struct nsi_get_all_parameters_ex *params ) err: heap_free( out ); heap_free( in ); - CloseHandle( device ); return err; } @@ -293,11 +312,7 @@ DWORD WINAPI NsiGetParameterEx( struct nsi_get_parameter_ex *params ) if (device == INVALID_HANDLE_VALUE) return GetLastError(); in = heap_alloc( in_size ); - if (!in) - { - err = ERROR_OUTOFMEMORY; - goto err; - } + if (!in) return ERROR_OUTOFMEMORY; in->module = *params->module; in->first_arg = params->first_arg; in->table = params->table; @@ -309,9 +324,7 @@ DWORD WINAPI NsiGetParameterEx( struct nsi_get_parameter_ex *params ) if (!DeviceIoControl( device, IOCTL_NSIPROXY_WINE_GET_PARAMETER, in, in_size, params->data, params->data_size, &received, NULL )) err = GetLastError(); -err: heap_free( in ); - CloseHandle( device ); return err; }

1 0

Paul Gofman : nsi: Add stubs for NsiRequestChangeNotification[Ex].
by Alexandre Julliard July 31, 2023

July 31, 2023

Module: wine Branch: master Commit: 671412d3bb45f0fa639aefc031b523aa2edac3e8 URL: https://gitlab.winehq.org/wine/wine/-/commit/671412d3bb45f0fa639aefc031b523… Author: Paul Gofman <pgofman(a)codeweavers.com> Date: Tue Jul 25 17:37:12 2023 -0600 nsi: Add stubs for NsiRequestChangeNotification[Ex]. --- dlls/nsi/nsi.c | 22 ++++++++++++++++++++++ dlls/nsi/nsi.spec | 4 ++-- include/wine/nsi.h | 12 ++++++++++++ 3 files changed, 36 insertions(+), 2 deletions(-) diff --git a/dlls/nsi/nsi.c b/dlls/nsi/nsi.c index 3f324ef555b..e20aa6ef21a 100644 --- a/dlls/nsi/nsi.c +++ b/dlls/nsi/nsi.c @@ -307,3 +307,25 @@ err: CloseHandle( device ); return err; } + +DWORD WINAPI NsiRequestChangeNotification( DWORD unk, const NPI_MODULEID *module, DWORD table, OVERLAPPED *ovr, + HANDLE *handle ) +{ + struct nsi_request_change_notification_ex params; + + TRACE( "%lu %p %lu %p %p stub.\n", unk, module, table, ovr, handle ); + + params.unk = unk; + params.module = module; + params.table = table; + params.ovr = ovr; + params.handle = handle; + return NsiRequestChangeNotificationEx( &params ); +} + +DWORD WINAPI NsiRequestChangeNotificationEx( struct nsi_request_change_notification_ex *params ) +{ + FIXME( "%p stub.\n", params ); + + return ERROR_NOT_SUPPORTED; +} diff --git a/dlls/nsi/nsi.spec b/dlls/nsi/nsi.spec index ba326572fb8..b343b228b9c 100644 --- a/dlls/nsi/nsi.spec +++ b/dlls/nsi/nsi.spec @@ -16,8 +16,8 @@ @ stdcall NsiGetParameterEx(ptr) @ stub NsiRegisterChangeNotification @ stub NsiRegisterChangeNotificationEx -@ stub NsiRequestChangeNotification -@ stub NsiRequestChangeNotificationEx +@ stdcall NsiRequestChangeNotification(long ptr long ptr ptr) +@ stdcall NsiRequestChangeNotificationEx(ptr) @ stub NsiSetAllParameters @ stub NsiSetAllParametersEx @ stub NsiSetAllPersistentParametersWithMask diff --git a/include/wine/nsi.h b/include/wine/nsi.h index af35593b29c..2f9fb0cb6e3 100644 --- a/include/wine/nsi.h +++ b/include/wine/nsi.h @@ -492,6 +492,15 @@ struct nsi_get_parameter_ex UINT data_offset; }; +struct nsi_request_change_notification_ex +{ + DWORD unk; + const NPI_MODULEID *module; + UINT_PTR table; + OVERLAPPED *ovr; + HANDLE *handle; +}; + DWORD WINAPI NsiAllocateAndGetTable( DWORD unk, const NPI_MODULEID *module, DWORD table, void **key_data, DWORD key_size, void **rw_data, DWORD rw_size, void **dynamic_data, DWORD dynamic_size, void **static_data, DWORD static_size, DWORD *count, DWORD unk2 ); @@ -508,5 +517,8 @@ DWORD WINAPI NsiGetAllParametersEx( struct nsi_get_all_parameters_ex *params ); DWORD WINAPI NsiGetParameter( DWORD unk, const NPI_MODULEID *module, DWORD table, const void *key, DWORD key_size, DWORD param_type, void *data, DWORD data_size, DWORD data_offset ); DWORD WINAPI NsiGetParameterEx( struct nsi_get_parameter_ex *params ); +DWORD WINAPI NsiRequestChangeNotification( DWORD unk, const NPI_MODULEID *module, DWORD table, OVERLAPPED *ovr, + HANDLE *handle ); +DWORD WINAPI NsiRequestChangeNotificationEx( struct nsi_request_change_notification_ex *params ); #endif /* __WINE_NSI_H */

1 0

Paul Gofman : nsi/tests: Add test for change notifications.
by Alexandre Julliard July 31, 2023

July 31, 2023

Module: wine Branch: master Commit: 64c6819553b0bdb3f0a415558b833a07915999e0 URL: https://gitlab.winehq.org/wine/wine/-/commit/64c6819553b0bdb3f0a415558b833a… Author: Paul Gofman <pgofman(a)codeweavers.com> Date: Tue Jul 25 17:35:08 2023 -0600 nsi/tests: Add test for change notifications. --- dlls/nsi/tests/nsi.c | 64 ++++++++++++++++++++++++++++++++++++++++++++++++++++ 1 file changed, 64 insertions(+) diff --git a/dlls/nsi/tests/nsi.c b/dlls/nsi/tests/nsi.c index 4cf49993ea1..5b3a6775992 100644 --- a/dlls/nsi/tests/nsi.c +++ b/dlls/nsi/tests/nsi.c @@ -18,6 +18,8 @@ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA */ +#include "ntstatus.h" +#define WIN32_NO_STATUS #include "winsock2.h" #include "winternl.h" #include "ws2ipdef.h" @@ -1023,6 +1025,66 @@ static void test_udp_tables( int family ) winetest_pop_context(); } +void test_change_notifications(void) +{ + struct nsi_request_change_notification_ex params; + HANDLE handle, handle2; + OVERLAPPED ovr, ovr2; + DWORD bytes; + DWORD ret; + BOOL bret; + + memset( &ovr, 0, sizeof(ovr) ); + ovr.hEvent = CreateEventW( NULL, FALSE, FALSE, NULL ); + + handle = (HANDLE)0xdeadbeef; + ret = NsiRequestChangeNotification( 0, &NPI_MS_NDIS_MODULEID, NSI_NDIS_IFINFO_TABLE, &ovr, &handle ); + todo_wine ok( ret == ERROR_IO_PENDING, "got %lu.\n", ret ); + + memset( &params, 0, sizeof(params) ); + handle2 = (HANDLE)0xdeadbeef; + memset( &ovr2, 0, sizeof(ovr2) ); + params.module = &NPI_MS_NDIS_MODULEID; + params.table = NSI_NDIS_IFINFO_TABLE; + params.ovr = &ovr2; + params.handle = &handle2; + ret = NsiRequestChangeNotificationEx( &params ); + todo_wine ok( ret == ERROR_IO_PENDING, "got %lu.\n", ret ); + + ok( handle2 == handle, "got %p, %p.\n", handle, handle2 ); + bret = GetOverlappedResult( handle, &ovr, &bytes, FALSE ); + todo_wine ok( !bret && GetLastError() == ERROR_IO_INCOMPLETE, "got bret %d, err %lu.\n", bret, GetLastError() ); + + ret = NsiCancelChangeNotification( NULL ); + todo_wine ok( ret == ERROR_NOT_FOUND, "got %lu.\n", ret ); + + ret = NsiCancelChangeNotification( &ovr ); + todo_wine ok( !ret, "got %lu.\n", ret ); + + bytes = 0xdeadbeef; + bret = GetOverlappedResult( handle, &ovr, &bytes, FALSE ); + todo_wine ok( !bret && GetLastError() == ERROR_OPERATION_ABORTED, "got bret %d, err %lu.\n", bret, GetLastError() ); + todo_wine ok( ovr.Internal == (ULONG)STATUS_CANCELLED, "got %Ix.\n", ovr.Internal ); + ok( !bytes, "got %lu.\n", bytes ); + + bret = GetOverlappedResult( handle2, &ovr2, &bytes, FALSE ); + todo_wine ok( !bret && GetLastError() == ERROR_IO_INCOMPLETE, "got bret %d, err %lu.\n", bret, GetLastError() ); + ret = NsiCancelChangeNotification( &ovr2 ); + todo_wine ok( !ret, "got %lu.\n", ret ); + bret = GetOverlappedResult( handle, &ovr, &bytes, FALSE ); + todo_wine ok( !bret && GetLastError() == ERROR_OPERATION_ABORTED, "got bret %d, err %lu.\n", bret, GetLastError() ); + + ret = NsiRequestChangeNotification( 0, &NPI_MS_NDIS_MODULEID, NSI_NDIS_INDEX_LUID_TABLE, &ovr, &handle ); + todo_wine ok( ret == ERROR_INVALID_PARAMETER, "got %lu.\n", ret ); + + ret = NsiRequestChangeNotification( 0, &NPI_MS_IPV4_MODULEID, NSI_IP_FORWARD_TABLE, &ovr, &handle ); + todo_wine ok( ret == ERROR_IO_PENDING, "got %lu.\n", ret ); + ret = NsiCancelChangeNotification( &ovr ); + todo_wine ok( !ret, "got %lu.\n", ret ); + bret = GetOverlappedResult( handle, &ovr, &bytes, FALSE ); + todo_wine ok( !bret && GetLastError() == ERROR_OPERATION_ABORTED, "got bret %d, err %lu.\n", bret, GetLastError() ); +} + START_TEST( nsi ) { test_nsi_api(); @@ -1056,4 +1118,6 @@ START_TEST( nsi ) test_udp_stats( AF_INET6 ); test_udp_tables( AF_INET ); test_udp_tables( AF_INET6 ); + + test_change_notifications(); }

1 0