January 2024 - Wine-commits

Alexandre Julliard : user32: Return result through NtCallbackReturn for the load image callback.
by Alexandre Julliard 18 Jan '24

18 Jan '24

Module: wine Branch: master Commit: 33793446c438f99bfbe00602fb74e845a10a748c URL: https://gitlab.winehq.org/wine/wine/-/commit/33793446c438f99bfbe00602fb74e8… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Jan 18 15:00:24 2024 +0100 user32: Return result through NtCallbackReturn for the load image callback. --- dlls/user32/user_main.c | 6 ++++-- dlls/win32u/cursoricon.c | 3 ++- dlls/wow64win/user.c | 12 +++++++++++- 3 files changed, 17 insertions(+), 4 deletions(-) diff --git a/dlls/user32/user_main.c b/dlls/user32/user_main.c index efa33fb2760..79809835b91 100644 --- a/dlls/user32/user_main.c +++ b/dlls/user32/user_main.c @@ -145,11 +145,13 @@ static NTSTATUS WINAPI User32ImmTranslateMessage( const struct imm_translate_mes return ImmTranslateMessage( params->hwnd, params->msg, params->wparam, params->key_data ); } -static NTSTATUS WINAPI User32LoadImage( const struct load_image_params *params, ULONG size ) +static NTSTATUS WINAPI User32LoadImage( void *args, ULONG size ) { + const struct load_image_params *params = args; HANDLE ret = LoadImageW( params->hinst, params->name, params->type, params->dx, params->dy, params->flags ); - return HandleToUlong( ret ); + if (!ret) return STATUS_NO_MEMORY; + return NtCallbackReturn( &ret, sizeof(ret), STATUS_SUCCESS ); } static NTSTATUS WINAPI User32LoadSysMenu( const struct load_sys_menu_params *params, ULONG size ) diff --git a/dlls/win32u/cursoricon.c b/dlls/win32u/cursoricon.c index 5fe52fdb8b3..661ef34c2e0 100644 --- a/dlls/win32u/cursoricon.c +++ b/dlls/win32u/cursoricon.c @@ -740,5 +740,6 @@ HANDLE WINAPI LoadImageW( HINSTANCE hinst, const WCHAR *name, UINT type, return 0; } ret = KeUserModeCallback( NtUserLoadImage, &params, sizeof(params), &ret_ptr, &ret_len ); - return UlongToHandle( ret ); + if (!ret && ret_len == sizeof(HANDLE)) return *(HANDLE *)ret_ptr; + return 0; } diff --git a/dlls/wow64win/user.c b/dlls/wow64win/user.c index 219c48b754f..d010b826ba1 100644 --- a/dlls/wow64win/user.c +++ b/dlls/wow64win/user.c @@ -1343,6 +1343,9 @@ static NTSTATUS WINAPI wow64_NtUserLoadDriver( void *arg, ULONG size ) static NTSTATUS WINAPI wow64_NtUserLoadImage( void *arg, ULONG size ) { struct load_image_params *params = arg; + void *ret_ptr; + ULONG ret_len; + NTSTATUS status; struct { ULONG hinst; @@ -1359,7 +1362,14 @@ static NTSTATUS WINAPI wow64_NtUserLoadImage( void *arg, ULONG size ) params32.dx = params->dx; params32.dy = params->dy; params32.flags = params->flags; - return dispatch_callback( NtUserLoadImage, &params32, sizeof(params32) ); + status = Wow64KiUserCallbackDispatcher( NtUserLoadImage, &params32, sizeof(params32), + &ret_ptr, &ret_len ); + if (!status && ret_len == sizeof(ULONG)) + { + HANDLE handle = ULongToHandle( *(ULONG *)ret_ptr ); + return NtCallbackReturn( &handle, sizeof(handle), status ); + } + return status; } static NTSTATUS WINAPI wow64_NtUserLoadSysMenu( void *arg, ULONG size )

1 0

Alexandre Julliard : user32: Return result through NtCallbackReturn for the load sys menu callback.
by Alexandre Julliard 18 Jan '24

18 Jan '24

Module: wine Branch: master Commit: ee106102c571c146e587ef0adfb2e63017fad949 URL: https://gitlab.winehq.org/wine/wine/-/commit/ee106102c571c146e587ef0adfb2e6… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Jan 18 15:02:04 2024 +0100 user32: Return result through NtCallbackReturn for the load sys menu callback. --- dlls/user32/user_main.c | 6 ++++-- dlls/win32u/menu.c | 8 ++++---- dlls/wow64win/user.c | 12 +++++++++++- 3 files changed, 19 insertions(+), 7 deletions(-) diff --git a/dlls/user32/user_main.c b/dlls/user32/user_main.c index 79809835b91..045f95b3619 100644 --- a/dlls/user32/user_main.c +++ b/dlls/user32/user_main.c @@ -154,10 +154,12 @@ static NTSTATUS WINAPI User32LoadImage( void *args, ULONG size ) return NtCallbackReturn( &ret, sizeof(ret), STATUS_SUCCESS ); } -static NTSTATUS WINAPI User32LoadSysMenu( const struct load_sys_menu_params *params, ULONG size ) +static NTSTATUS WINAPI User32LoadSysMenu( void *args, ULONG size ) { + const struct load_sys_menu_params *params = args; HMENU ret = LoadMenuW( user32_module, params->mdi ? L"SYSMENUMDI" : L"SYSMENU" ); - return HandleToUlong( ret ); + if (!ret) return STATUS_NO_MEMORY; + return NtCallbackReturn( &ret, sizeof(ret), STATUS_SUCCESS ); } static NTSTATUS WINAPI User32FreeCachedClipboardData( const struct free_cached_data_params *params, diff --git a/dlls/win32u/menu.c b/dlls/win32u/menu.c index a6b7ec38003..6a8e8615945 100644 --- a/dlls/win32u/menu.c +++ b/dlls/win32u/menu.c @@ -1436,12 +1436,12 @@ static HMENU copy_sys_popup( BOOL mdi ) struct menu *menu; void *ret_ptr; ULONG ret_len; - HMENU handle; + NTSTATUS status; + HMENU handle = 0; params.mdi = mdi; - handle = UlongToHandle( KeUserModeCallback( NtUserLoadSysMenu, &params, sizeof(params), - &ret_ptr, &ret_len )); - + status = KeUserModeCallback( NtUserLoadSysMenu, &params, sizeof(params), &ret_ptr, &ret_len ); + if (!status && ret_len == sizeof(HMENU)) handle = *(HMENU *)ret_ptr; if (!handle || !(menu = grab_menu_ptr( handle ))) { ERR("Unable to load default system menu\n" ); diff --git a/dlls/wow64win/user.c b/dlls/wow64win/user.c index d010b826ba1..939155b7d41 100644 --- a/dlls/wow64win/user.c +++ b/dlls/wow64win/user.c @@ -1374,7 +1374,17 @@ static NTSTATUS WINAPI wow64_NtUserLoadImage( void *arg, ULONG size ) static NTSTATUS WINAPI wow64_NtUserLoadSysMenu( void *arg, ULONG size ) { - return dispatch_callback( NtUserLoadSysMenu, arg, size ); + void *ret_ptr; + ULONG ret_len; + NTSTATUS status; + + status = Wow64KiUserCallbackDispatcher( NtUserLoadSysMenu, arg, size, &ret_ptr, &ret_len ); + if (!status && ret_len == sizeof(ULONG)) + { + HMENU menu = ULongToHandle( *(ULONG *)ret_ptr ); + return NtCallbackReturn( &menu, sizeof(menu), status ); + } + return status; } static NTSTATUS WINAPI wow64_NtUserPostDDEMessage( void *arg, ULONG size )

1 0

Alexandre Julliard : user32: Return result through NtCallbackReturn for the draw text callback.
by Alexandre Julliard 18 Jan '24

18 Jan '24

Module: wine Branch: master Commit: a9e42169ca36e057bb75a65e573fd41e7fc10a7c URL: https://gitlab.winehq.org/wine/wine/-/commit/a9e42169ca36e057bb75a65e573fd4… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Jan 18 15:04:41 2024 +0100 user32: Return result through NtCallbackReturn for the draw text callback. --- dlls/user32/user_main.c | 10 ++++++---- dlls/win32u/font.c | 14 ++++++++++---- dlls/wow64win/user.c | 7 +------ include/ntuser.h | 5 +++++ 4 files changed, 22 insertions(+), 14 deletions(-) diff --git a/dlls/user32/user_main.c b/dlls/user32/user_main.c index 045f95b3619..f1b169a9741 100644 --- a/dlls/user32/user_main.c +++ b/dlls/user32/user_main.c @@ -125,13 +125,15 @@ static NTSTATUS WINAPI User32DrawScrollBar( const struct draw_scroll_bar_params return 0; } -static NTSTATUS WINAPI User32DrawText( struct draw_text_params *params, ULONG size ) +static NTSTATUS WINAPI User32DrawText( void *args, ULONG size ) { - int ret; + const struct draw_text_params *params = args; + struct draw_text_result result; size -= FIELD_OFFSET( struct draw_text_params, str ); - ret = DrawTextW( params->hdc, params->str, size / sizeof(WCHAR), &params->rect, params->flags ); - return NtCallbackReturn( &params->rect, sizeof(params->rect), ret ); + result.rect = params->rect; + result.height = DrawTextW( params->hdc, params->str, size / sizeof(WCHAR), &result.rect, params->flags ); + return NtCallbackReturn( &result, sizeof(result), STATUS_SUCCESS ); } static NTSTATUS WINAPI User32ImmProcessKey( const struct imm_process_key_params *params, ULONG size ) diff --git a/dlls/win32u/font.c b/dlls/win32u/font.c index af1a9b97f67..9ee89f1b5c4 100644 --- a/dlls/win32u/font.c +++ b/dlls/win32u/font.c @@ -7076,9 +7076,10 @@ BOOL WINAPI NtGdiGetCharWidthInfo( HDC hdc, struct char_width_info *info ) INT WINAPI DrawTextW( HDC hdc, const WCHAR *str, INT count, RECT *rect, UINT flags ) { struct draw_text_params *params; + struct draw_text_result *result; ULONG ret_len, size; - void *ret_ptr; - int ret; + NTSTATUS status; + int ret = 0; if (count == -1) count = wcslen( str ); size = FIELD_OFFSET( struct draw_text_params, str[count] ); @@ -7087,8 +7088,13 @@ INT WINAPI DrawTextW( HDC hdc, const WCHAR *str, INT count, RECT *rect, UINT fla params->rect = *rect; params->flags = flags; if (count) memcpy( params->str, str, count * sizeof(WCHAR) ); - ret = KeUserModeCallback( NtUserDrawText, params, size, &ret_ptr, &ret_len ); - if (ret_len == sizeof(*rect)) *rect = *(const RECT *)ret_ptr; + + status = KeUserModeCallback( NtUserDrawText, params, size, (void **)&result, &ret_len ); + if (!status && ret_len == sizeof(*result)) + { + ret = result->height; + *rect = result->rect; + } free( params ); return ret; } diff --git a/dlls/wow64win/user.c b/dlls/wow64win/user.c index 939155b7d41..d1f8a04e959 100644 --- a/dlls/wow64win/user.c +++ b/dlls/wow64win/user.c @@ -1266,18 +1266,13 @@ static NTSTATUS WINAPI wow64_NtUserDrawText( void *arg, ULONG size ) struct draw_text_params *params = arg; struct draw_text_params32 *params32; ULONG offset = offsetof( struct draw_text_params, str ) - offsetof( struct draw_text_params32, str ); - ULONG ret_len; - void *ret_ptr; - NTSTATUS ret; params32 = (struct draw_text_params32 *)((char *)params + offset); params32->flags = params->flags; params32->rect = params->rect; params32->count = params->count; params32->hdc = HandleToUlong( params->hdc ); - - ret = Wow64KiUserCallbackDispatcher( NtUserDrawText, params32, size - offset, &ret_ptr, &ret_len ); - return NtCallbackReturn( ret_ptr, ret_len, ret ); + return dispatch_callback( NtUserDrawText, params32, size - offset ); } static NTSTATUS WINAPI wow64_NtUserFreeCachedClipboardData( void *arg, ULONG size ) diff --git a/include/ntuser.h b/include/ntuser.h index e78ac3d03c2..31b93ef36e9 100644 --- a/include/ntuser.h +++ b/include/ntuser.h @@ -200,6 +200,11 @@ struct draw_text_params UINT flags; WCHAR str[1]; }; +struct draw_text_result +{ + int height; + RECT rect; +}; /* NtUserFreeCachedClipboardData params */ struct free_cached_data_params

1 0

Alexandre Julliard : user32: Return result through NtCallbackReturn for the enum monitors callback.
by Alexandre Julliard 18 Jan '24

18 Jan '24

Module: wine Branch: master Commit: 3f9de12980c65b5accf39483b703788fc040f307 URL: https://gitlab.winehq.org/wine/wine/-/commit/3f9de12980c65b5accf39483b70378… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Jan 18 15:07:39 2024 +0100 user32: Return result through NtCallbackReturn for the enum monitors callback. --- dlls/user32/sysparams.c | 13 +++++++++---- dlls/user32/user_private.h | 2 +- dlls/win32u/sysparams.c | 13 ++++++------- 3 files changed, 16 insertions(+), 12 deletions(-) diff --git a/dlls/user32/sysparams.c b/dlls/user32/sysparams.c index b941f126a58..18e02f1524b 100644 --- a/dlls/user32/sysparams.c +++ b/dlls/user32/sysparams.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 "user_private.h" #include "controls.h" #include "wine/asm.h" @@ -849,14 +851,17 @@ __ASM_GLOBAL_FUNC( enum_mon_callback_wrapper, "ret" ) #endif /* __i386__ */ -BOOL WINAPI User32CallEnumDisplayMonitor( struct enum_display_monitor_params *params, ULONG size ) +NTSTATUS WINAPI User32CallEnumDisplayMonitor( void *args, ULONG size ) { + struct enum_display_monitor_params *params = args; + BOOL ret; #ifdef __i386__ - return enum_mon_callback_wrapper( params->proc, params->monitor, params->hdc, - &params->rect, params->lparam ); + ret = enum_mon_callback_wrapper( params->proc, params->monitor, params->hdc, + &params->rect, params->lparam ); #else - return params->proc( params->monitor, params->hdc, &params->rect, params->lparam ); + ret = params->proc( params->monitor, params->hdc, &params->rect, params->lparam ); #endif + return NtCallbackReturn( &ret, sizeof(ret), STATUS_SUCCESS ); } /*********************************************************************** diff --git a/dlls/user32/user_private.h b/dlls/user32/user_private.h index 50d4005d973..9022e44b7d2 100644 --- a/dlls/user32/user_private.h +++ b/dlls/user32/user_private.h @@ -81,7 +81,7 @@ extern ATOM get_class_info( HINSTANCE instance, const WCHAR *name, WNDCLASSEXW * /* kernel callbacks */ -BOOL WINAPI User32CallEnumDisplayMonitor( struct enum_display_monitor_params *params, ULONG size ); +NTSTATUS WINAPI User32CallEnumDisplayMonitor( void *args, ULONG size ); BOOL WINAPI User32CallSendAsyncCallback( const struct send_async_params *params, ULONG size ); BOOL WINAPI User32CallWinEventHook( const struct win_event_hook_params *params, ULONG size ); BOOL WINAPI User32CallWindowProc( struct win_proc_params *params, ULONG size ); diff --git a/dlls/win32u/sysparams.c b/dlls/win32u/sysparams.c index a9563eaff84..e2c5b10da9e 100644 --- a/dlls/win32u/sysparams.c +++ b/dlls/win32u/sysparams.c @@ -3435,18 +3435,17 @@ BOOL WINAPI NtUserEnumDisplayMonitors( HDC hdc, RECT *rect, MONITORENUMPROC proc params.proc = proc; params.hdc = hdc; params.lparam = lparam; - for (i = 0; i < count; i++) + for (i = 0; i < count && ret; i++) { void *ret_ptr; ULONG ret_len; + NTSTATUS status; params.monitor = enum_info[i].handle; params.rect = enum_info[i].rect; - if (!KeUserModeCallback( NtUserCallEnumDisplayMonitor, &params, sizeof(params), - &ret_ptr, &ret_len )) - { - ret = FALSE; - break; - } + status = KeUserModeCallback( NtUserCallEnumDisplayMonitor, &params, sizeof(params), + &ret_ptr, &ret_len ); + if (!status && ret_len == sizeof(ret)) ret = *(BOOL *)ret_ptr; + else ret = FALSE; } if (enum_info != enum_buf) free( enum_info ); return ret;

1 0

Alexandre Julliard : user32: Return result through NtCallbackReturn for the window hook callback.
by Alexandre Julliard 18 Jan '24

18 Jan '24

Module: wine Branch: master Commit: 2c8b60af43fb8fee3b8485eb7b3374e494dfd6f4 URL: https://gitlab.winehq.org/wine/wine/-/commit/2c8b60af43fb8fee3b8485eb7b3374… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Jan 18 15:16:13 2024 +0100 user32: Return result through NtCallbackReturn for the window hook callback. --- dlls/user32/hook.c | 14 ++++++++++++-- dlls/user32/user_private.h | 2 +- dlls/win32u/hook.c | 11 ++++++++--- dlls/wow64win/user.c | 20 +++++++++++--------- 4 files changed, 32 insertions(+), 15 deletions(-) diff --git a/dlls/user32/hook.c b/dlls/user32/hook.c index 024bb5b7116..4f5b8d13973 100644 --- a/dlls/user32/hook.c +++ b/dlls/user32/hook.c @@ -62,6 +62,8 @@ * WH_MOUSE_LL Implemented but should use SendMessage instead */ +#include "ntstatus.h" +#define WIN32_NO_STATUS #include "user_private.h" #include "wine/asm.h" #include "wine/debug.h" @@ -452,8 +454,9 @@ BOOL WINAPI User32CallWinEventHook( const struct win_event_hook_params *params, return TRUE; } -BOOL WINAPI User32CallWindowsHook( struct win_hook_params *params, ULONG size ) +NTSTATUS WINAPI User32CallWindowsHook( void *args, ULONG size ) { + struct win_hook_params *params = args; HOOKPROC proc = params->proc; HMODULE free_module = 0; void *ret_ptr = NULL; @@ -514,7 +517,14 @@ BOOL WINAPI User32CallWindowsHook( struct win_hook_params *params, ULONG size ) params->prev_unicode, params->next_unicode ); if (free_module) FreeLibrary( free_module ); - return NtCallbackReturn( ret_ptr, ret_size, ret ); + + if (ret_size) + { + LRESULT *result_ptr = (LRESULT *)ret_ptr - 1; + *result_ptr = ret; + return NtCallbackReturn( result_ptr, sizeof(*result_ptr) + ret_size, STATUS_SUCCESS ); + } + return NtCallbackReturn( &ret, sizeof(ret), STATUS_SUCCESS ); } /*********************************************************************** diff --git a/dlls/user32/user_private.h b/dlls/user32/user_private.h index 9022e44b7d2..40d67ac468d 100644 --- a/dlls/user32/user_private.h +++ b/dlls/user32/user_private.h @@ -85,7 +85,7 @@ NTSTATUS WINAPI User32CallEnumDisplayMonitor( void *args, ULONG size ); BOOL WINAPI User32CallSendAsyncCallback( const struct send_async_params *params, ULONG size ); BOOL WINAPI User32CallWinEventHook( const struct win_event_hook_params *params, ULONG size ); BOOL WINAPI User32CallWindowProc( struct win_proc_params *params, ULONG size ); -BOOL WINAPI User32CallWindowsHook( struct win_hook_params *params, ULONG size ); +NTSTATUS WINAPI User32CallWindowsHook( void *args, ULONG size ); BOOL WINAPI User32InitBuiltinClasses( const struct win_hook_params *params, ULONG size ); /* message spy definitions */ diff --git a/dlls/win32u/hook.c b/dlls/win32u/hook.c index 378eda40abb..fab2b960fa5 100644 --- a/dlls/win32u/hook.c +++ b/dlls/win32u/hook.c @@ -330,9 +330,14 @@ static LRESULT call_hook( struct win_hook_params *info, const WCHAR *module, siz thread_info->hook = params->handle; thread_info->hook_unicode = params->next_unicode; thread_info->hook_call_depth++; - ret = KeUserModeCallback( NtUserCallWindowsHook, params, size, &ret_ptr, &ret_len ); - if (ret_len && ret_len == lparam_ret_size) - memcpy( (void *)params->lparam, ret_ptr, ret_len ); + if (!KeUserModeCallback( NtUserCallWindowsHook, params, size, &ret_ptr, &ret_len ) && + ret_len >= sizeof(ret)) + { + LRESULT *result_ptr = ret_ptr; + ret = *result_ptr; + if (ret_len == sizeof(ret) + lparam_ret_size) + memcpy( (void *)params->lparam, result_ptr + 1, ret_len - sizeof(ret) ); + } thread_info->hook = prev; thread_info->hook_unicode = prev_unicode; thread_info->hook_call_depth--; diff --git a/dlls/wow64win/user.c b/dlls/wow64win/user.c index d1f8a04e959..2dd811578f5 100644 --- a/dlls/wow64win/user.c +++ b/dlls/wow64win/user.c @@ -1113,8 +1113,9 @@ static NTSTATUS WINAPI wow64_NtUserCallWindowsHook( void *arg, ULONG size ) struct win_hook_params32 params32; UINT module_len, size32, offset; void *ret_ptr; - ULONG ret_len; - NTSTATUS ret; + LRESULT *result_ptr = arg; + ULONG ret_len, ret_size = 0; + NTSTATUS status; module_len = wcslen( params->module ); size32 = FIELD_OFFSET( struct win_hook_params32, module[module_len + 1] ); @@ -1142,22 +1143,23 @@ static NTSTATUS WINAPI wow64_NtUserCallWindowsHook( void *arg, ULONG size ) size - offset, (char *)arg + size32 ); } - ret = Wow64KiUserCallbackDispatcher( NtUserCallWindowsHook, arg, size32, &ret_ptr, &ret_len ); + status = Wow64KiUserCallbackDispatcher( NtUserCallWindowsHook, arg, size32, &ret_ptr, &ret_len ); + if (status || ret_len < sizeof(LONG)) return status; switch (params32.id) { case WH_SYSMSGFILTER: case WH_MSGFILTER: case WH_GETMESSAGE: - if (ret_len == sizeof(MSG32)) + if (ret_len == sizeof(MSG32) + sizeof(LONG)) { - MSG msg; - msg_32to64( &msg, ret_ptr ); - return NtCallbackReturn( &msg, sizeof(msg), ret ); + msg_32to64( (MSG *)(result_ptr + 1), (MSG32 *)((LONG *)ret_ptr + 1) ); + ret_size = sizeof(MSG); } + break; } - - return ret; + *result_ptr = *(LONG *)ret_ptr; + return NtCallbackReturn( result_ptr, sizeof(*result_ptr) + ret_size, status ); } static NTSTATUS WINAPI wow64_NtUserCopyImage( void *arg, ULONG size )

1 0

Hans Leidekker : bcrypt: Add support for setting DH parameters.
by Alexandre Julliard 18 Jan '24

18 Jan '24

Module: wine Branch: master Commit: a9193a200faeaaf9025506e07c003846b0343a7f URL: https://gitlab.winehq.org/wine/wine/-/commit/a9193a200faeaaf9025506e07c0038… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Wed Dec 6 19:59:45 2023 +0100 bcrypt: Add support for setting DH parameters. --- dlls/bcrypt/bcrypt_internal.h | 6 ++++-- dlls/bcrypt/bcrypt_main.c | 15 +++++++++++++++ dlls/bcrypt/gnutls.c | 41 ++++++++++++++++++++++++++++++++++++++--- include/bcrypt.h | 13 +++++++++++++ 4 files changed, 70 insertions(+), 5 deletions(-) diff --git a/dlls/bcrypt/bcrypt_internal.h b/dlls/bcrypt/bcrypt_internal.h index cfbc5d2ac83..a31d170c621 100644 --- a/dlls/bcrypt/bcrypt_internal.h +++ b/dlls/bcrypt/bcrypt_internal.h @@ -187,7 +187,7 @@ struct key { struct object hdr; enum alg_id alg_id; - UINT64 private[2]; /* private data for backend */ + UINT64 private[3]; /* private data for backend */ union { struct key_symmetric s; @@ -294,7 +294,9 @@ struct key_asymmetric_export_params ULONG *ret_len; }; -#define KEY_IMPORT_FLAG_PUBLIC 0x00000001 +#define KEY_IMPORT_FLAG_PUBLIC 0x00000001 +#define KEY_IMPORT_FLAG_DH_PARAMETERS 0x00000002 + struct key_asymmetric_import_params { struct key *key; diff --git a/dlls/bcrypt/bcrypt_main.c b/dlls/bcrypt/bcrypt_main.c index 52066a32c74..f58ddaf13aa 100644 --- a/dlls/bcrypt/bcrypt_main.c +++ b/dlls/bcrypt/bcrypt_main.c @@ -887,6 +887,21 @@ static NTSTATUS set_key_property( struct key *key, const WCHAR *prop, UCHAR *val key->u.a.bitlen = *(DWORD*)value; return STATUS_SUCCESS; } + else if (!wcscmp( prop, BCRYPT_DH_PARAMETERS )) + { + BCRYPT_DH_PARAMETER_HEADER *hdr = (BCRYPT_DH_PARAMETER_HEADER *)value; + struct key_asymmetric_import_params params; + + if (key->alg_id != ALG_ID_DH || size < sizeof(*hdr) || hdr->cbLength != size || + hdr->dwMagic != BCRYPT_DH_PARAMETERS_MAGIC || hdr->cbKeyLength != key->u.a.bitlen / 8) + return STATUS_INVALID_PARAMETER; + + params.key = key; + params.flags = KEY_IMPORT_FLAG_DH_PARAMETERS; + params.buf = value; + params.len = size; + return UNIX_CALL( key_asymmetric_import, &params ); + } FIXME( "unsupported key property %s\n", debugstr_w(prop) ); return STATUS_NOT_IMPLEMENTED; diff --git a/dlls/bcrypt/gnutls.c b/dlls/bcrypt/gnutls.c index 68f84a553d2..0d912e7a2d7 100644 --- a/dlls/bcrypt/gnutls.c +++ b/dlls/bcrypt/gnutls.c @@ -84,8 +84,9 @@ union key_data gnutls_cipher_hd_t cipher; struct { - gnutls_privkey_t privkey; - gnutls_pubkey_t pubkey; + gnutls_privkey_t privkey; + gnutls_pubkey_t pubkey; + gnutls_dh_params_t dh_params; } a; }; C_ASSERT( sizeof(union key_data) <= sizeof(((struct key *)0)->private) ); @@ -1833,6 +1834,36 @@ static NTSTATUS key_import_dh( struct key *key, UCHAR *buf, ULONG len ) return STATUS_SUCCESS; } +static NTSTATUS key_import_dh_params( struct key *key, UCHAR *buf, ULONG len ) +{ + BCRYPT_DH_PARAMETER_HEADER *dh_header = (BCRYPT_DH_PARAMETER_HEADER *)buf; + gnutls_dh_params_t params; + gnutls_datum_t p, g; + int ret; + + if ((ret = pgnutls_dh_params_init( &params ))) + { + pgnutls_perror( ret ); + return STATUS_INTERNAL_ERROR; + } + + p.data = (unsigned char *)(dh_header + 1); + p.size = dh_header->cbKeyLength; + g.data = p.data + dh_header->cbKeyLength; + g.size = dh_header->cbKeyLength; + + if ((ret = pgnutls_dh_params_import_raw( params, &p, &g ))) + { + pgnutls_perror( ret ); + pgnutls_dh_params_deinit( params ); + return STATUS_INTERNAL_ERROR; + } + + if (key_data(key)->a.dh_params) pgnutls_dh_params_deinit( key_data(key)->a.dh_params ); + key_data(key)->a.dh_params = params; + return STATUS_SUCCESS; +} + static NTSTATUS key_asymmetric_import( void *args ) { const struct key_asymmetric_import_params *params = args; @@ -1875,9 +1906,12 @@ static NTSTATUS key_asymmetric_import( void *args ) return STATUS_NOT_IMPLEMENTED; case ALG_ID_DH: + if (flags & KEY_IMPORT_FLAG_DH_PARAMETERS) + return key_import_dh_params( key, params->buf, params->len ); if (flags & KEY_IMPORT_FLAG_PUBLIC) return key_import_dh_public( key, params->buf, params->len ); - return key_import_dh( key, params->buf, params->len ); + ret = key_import_dh( key, params->buf, params->len ); + break; default: FIXME( "algorithm %u not yet supported\n", key->alg_id ); @@ -2300,6 +2334,7 @@ static NTSTATUS key_asymmetric_destroy( void *args ) if (key_data(key)->a.privkey) pgnutls_privkey_deinit( key_data(key)->a.privkey ); if (key_data(key)->a.pubkey) pgnutls_pubkey_deinit( key_data(key)->a.pubkey ); + if (key_data(key)->a.dh_params) pgnutls_dh_params_deinit( key_data(key)->a.dh_params ); return STATUS_SUCCESS; } diff --git a/include/bcrypt.h b/include/bcrypt.h index 7f768f61679..462c43a7021 100644 --- a/include/bcrypt.h +++ b/include/bcrypt.h @@ -118,6 +118,8 @@ typedef LONG NTSTATUS; #define BCRYPT_KDF_TLS_PRF L"TLS_PRF" #define BCRYPT_KDF_SP80056A_CONCAT L"SP800_56A_CONCAT" #define BCRYPT_KDF_RAW_SECRET L"TRUNCATE" + +#define BCRYPT_DH_PARAMETERS L"DHParameters" #else static const WCHAR BCRYPT_ALGORITHM_NAME[] = {'A','l','g','o','r','i','t','h','m','N','a','m','e',0}; static const WCHAR BCRYPT_AUTH_TAG_LENGTH[] = {'A','u','t','h','T','a','g','L','e','n','g','t','h',0}; @@ -198,6 +200,8 @@ static const WCHAR BCRYPT_KDF_HMAC[] = {'H','M','A','C',0}; static const WCHAR BCRYPT_KDF_TLS_PRF[] = {'T','L','S','_','P','R','F',0}; static const WCHAR BCRYPT_KDF_SP80056A_CONCAT[] = {'S','P','8','0','0','_','5','6','A','_','C','O','N','C','A','T',0}; static const WCHAR BCRYPT_KDF_RAW_SECRET[] = {'T','R','U','N','C','A','T','E',0}; + +static const WCHAR BCRYPT_DH_PARAMETERS[] = {'D','H','P','a','r','a','m','e','t','e','r','s',0}; #endif #define BCRYPT_ECDSA_PUBLIC_P256_MAGIC 0x31534345 @@ -363,6 +367,15 @@ typedef struct _BCRYPT_DH_KEY_BLOB ULONG cbKey; } BCRYPT_DH_KEY_BLOB, *PBCRYPT_DH_KEY_BLOB; +#define BCRYPT_DH_PARAMETERS_MAGIC 0x4d504844 + +typedef struct _BCRYPT_DH_PARAMETER_HEADER +{ + ULONG cbLength; + ULONG dwMagic; + ULONG cbKeyLength; +} BCRYPT_DH_PARAMETER_HEADER; + #define BCRYPT_AUTHENTICATED_CIPHER_MODE_INFO_VERSION 1 #define BCRYPT_AUTH_MODE_CHAIN_CALLS_FLAG 0x00000001

1 0

Hans Leidekker : bcrypt: Add support for retrieving DH parameters.
by Alexandre Julliard 18 Jan '24

18 Jan '24

Module: wine Branch: master Commit: 3bfccbc7e42c1edb54618ef29fa879e2fad36ed7 URL: https://gitlab.winehq.org/wine/wine/-/commit/3bfccbc7e42c1edb54618ef29fa879… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Wed Dec 6 20:23:53 2023 +0100 bcrypt: Add support for retrieving DH parameters. --- dlls/bcrypt/bcrypt_internal.h | 6 ++++-- dlls/bcrypt/bcrypt_main.c | 17 +++++++++++++++++ dlls/bcrypt/gnutls.c | 41 ++++++++++++++++++++++++++++++++++++++--- 3 files changed, 59 insertions(+), 5 deletions(-) diff --git a/dlls/bcrypt/bcrypt_internal.h b/dlls/bcrypt/bcrypt_internal.h index a31d170c621..a5dbeff4e8e 100644 --- a/dlls/bcrypt/bcrypt_internal.h +++ b/dlls/bcrypt/bcrypt_internal.h @@ -283,8 +283,10 @@ struct key_asymmetric_verify_params unsigned flags; }; -#define KEY_EXPORT_FLAG_PUBLIC 0x00000001 -#define KEY_EXPORT_FLAG_RSA_FULL 0x00000002 +#define KEY_EXPORT_FLAG_PUBLIC 0x00000001 +#define KEY_EXPORT_FLAG_RSA_FULL 0x00000002 +#define KEY_EXPORT_FLAG_DH_PARAMETERS 0x00000004 + struct key_asymmetric_export_params { struct key *key; diff --git a/dlls/bcrypt/bcrypt_main.c b/dlls/bcrypt/bcrypt_main.c index f58ddaf13aa..cce824ff602 100644 --- a/dlls/bcrypt/bcrypt_main.c +++ b/dlls/bcrypt/bcrypt_main.c @@ -917,6 +917,20 @@ static NTSTATUS get_hash_property( const struct hash *hash, const WCHAR *prop, U return status; } +static NTSTATUS get_dh_property( const struct key *key, const WCHAR *prop, UCHAR *buf, ULONG size, ULONG *ret_size ) +{ + struct key_asymmetric_export_params params; + + if (wcscmp( prop, BCRYPT_DH_PARAMETERS )) return STATUS_NOT_SUPPORTED; + + params.key = (struct key *)key; + params.flags = KEY_EXPORT_FLAG_DH_PARAMETERS; + params.buf = buf; + params.len = size; + params.ret_len = ret_size; + return UNIX_CALL( key_asymmetric_export, &params ); +} + static NTSTATUS get_key_property( const struct key *key, const WCHAR *prop, UCHAR *buf, ULONG size, ULONG *ret_size ) { if (!wcscmp( prop, BCRYPT_KEY_STRENGTH )) @@ -940,6 +954,9 @@ static NTSTATUS get_key_property( const struct key *key, const WCHAR *prop, UCHA if (!wcscmp( prop, BCRYPT_AUTH_TAG_LENGTH )) return STATUS_NOT_SUPPORTED; return get_aes_property( key->u.s.mode, prop, buf, size, ret_size ); + case ALG_ID_DH: + return get_dh_property( key, prop, buf, size, ret_size ); + default: FIXME( "unsupported algorithm %u\n", key->alg_id ); return STATUS_NOT_IMPLEMENTED; diff --git a/dlls/bcrypt/gnutls.c b/dlls/bcrypt/gnutls.c index 0d912e7a2d7..4183aad6cec 100644 --- a/dlls/bcrypt/gnutls.c +++ b/dlls/bcrypt/gnutls.c @@ -1651,8 +1651,7 @@ static NTSTATUS key_export_dh( struct key *key, UCHAR *buf, ULONG len, ULONG *re return STATUS_INTERNAL_ERROR; } - ret = pgnutls_privkey_export_dh_raw( key_data(key)->a.privkey, params, &y, &x, 0 ); - if (ret) + if ((ret = pgnutls_privkey_export_dh_raw( key_data(key)->a.privkey, params, &y, &x, 0 ))) { pgnutls_perror( ret ); pgnutls_dh_params_deinit( params ); @@ -1686,6 +1685,40 @@ static NTSTATUS key_export_dh( struct key *key, UCHAR *buf, ULONG len, ULONG *re return STATUS_SUCCESS; } +static NTSTATUS key_export_dh_params( struct key *key, UCHAR *buf, ULONG len, ULONG *ret_len ) +{ + BCRYPT_DH_PARAMETER_HEADER *hdr = (BCRYPT_DH_PARAMETER_HEADER *)buf; + unsigned int size = sizeof(*hdr) + key->u.a.bitlen / 8 * 2; + gnutls_datum_t p, g; + NTSTATUS status = STATUS_SUCCESS; + UCHAR *dst; + int ret; + + if (!key_data(key)->a.dh_params) return STATUS_INVALID_PARAMETER; + + if ((ret = pgnutls_dh_params_export_raw( key_data(key)->a.dh_params, &p, &g, NULL ))) + { + pgnutls_perror( ret ); + return STATUS_INTERNAL_ERROR; + } + + *ret_len = size; + if (len < size) status = STATUS_BUFFER_TOO_SMALL; + else if (buf) + { + hdr->cbLength = size; + hdr->dwMagic = BCRYPT_DH_PARAMETERS_MAGIC; + hdr->cbKeyLength = key->u.a.bitlen / 8; + + dst = (UCHAR *)(hdr + 1); + dst += export_gnutls_datum( dst, hdr->cbKeyLength, &p, 1 ); + dst += export_gnutls_datum( dst, hdr->cbKeyLength, &g, 1 ); + } + + free( p.data ); free( g.data ); + return status; +} + static NTSTATUS key_asymmetric_export( void *args ) { const struct key_asymmetric_export_params *params = args; @@ -1720,7 +1753,9 @@ static NTSTATUS key_asymmetric_export( void *args ) return STATUS_NOT_IMPLEMENTED; case ALG_ID_DH: - if (flags & KEY_EXPORT_FLAG_PUBLIC) + if (flags & KEY_EXPORT_FLAG_DH_PARAMETERS) + return key_export_dh_params( key, params->buf, params->len, params->ret_len ); + if (flags & KEY_EXPORT_FLAG_PUBLIC) return key_export_dh_public( key, params->buf, params->len, params->ret_len ); return key_export_dh( key, params->buf, params->len, params->ret_len );

1 0

Hans Leidekker : bcrypt: Allow or disallow some operations based on whether keys are finalized.
by Alexandre Julliard 18 Jan '24

18 Jan '24

Module: wine Branch: master Commit: b82faa2697ace903e3e627a182be0bf3289f6653 URL: https://gitlab.winehq.org/wine/wine/-/commit/b82faa2697ace903e3e627a182be0b… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Wed Dec 6 21:16:15 2023 +0100 bcrypt: Allow or disallow some operations based on whether keys are finalized. --- dlls/bcrypt/bcrypt_internal.h | 1 + dlls/bcrypt/bcrypt_main.c | 47 ++++++++++++++++--------------------------- dlls/bcrypt/gnutls.c | 2 ++ 3 files changed, 20 insertions(+), 30 deletions(-) diff --git a/dlls/bcrypt/bcrypt_internal.h b/dlls/bcrypt/bcrypt_internal.h index a5dbeff4e8e..f6556eb2aaa 100644 --- a/dlls/bcrypt/bcrypt_internal.h +++ b/dlls/bcrypt/bcrypt_internal.h @@ -175,6 +175,7 @@ struct key_symmetric }; #define KEY_FLAG_LEGACY_DSA_V2 0x00000001 +#define KEY_FLAG_FINALIZED 0x00000002 struct key_asymmetric { diff --git a/dlls/bcrypt/bcrypt_main.c b/dlls/bcrypt/bcrypt_main.c index cce824ff602..131fc2c2453 100644 --- a/dlls/bcrypt/bcrypt_main.c +++ b/dlls/bcrypt/bcrypt_main.c @@ -892,6 +892,7 @@ static NTSTATUS set_key_property( struct key *key, const WCHAR *prop, UCHAR *val BCRYPT_DH_PARAMETER_HEADER *hdr = (BCRYPT_DH_PARAMETER_HEADER *)value; struct key_asymmetric_import_params params; + if (key->u.a.flags & KEY_FLAG_FINALIZED) return STATUS_INVALID_HANDLE; if (key->alg_id != ALG_ID_DH || size < sizeof(*hdr) || hdr->cbLength != size || hdr->dwMagic != BCRYPT_DH_PARAMETERS_MAGIC || hdr->cbKeyLength != key->u.a.bitlen / 8) return STATUS_INVALID_PARAMETER; @@ -922,6 +923,7 @@ static NTSTATUS get_dh_property( const struct key *key, const WCHAR *prop, UCHAR struct key_asymmetric_export_params params; if (wcscmp( prop, BCRYPT_DH_PARAMETERS )) return STATUS_NOT_SUPPORTED; + if (!(key->u.a.flags & KEY_FLAG_FINALIZED)) return STATUS_INVALID_HANDLE; params.key = (struct key *)key; params.flags = KEY_EXPORT_FLAG_DH_PARAMETERS; @@ -1735,9 +1737,6 @@ static NTSTATUS key_import_pair( struct algorithm *alg, const WCHAR *type, BCRYP key_destroy( key ); return status; } - - *ret_key = key; - return STATUS_SUCCESS; } else if (!wcscmp( type, BCRYPT_ECCPRIVATE_BLOB )) { @@ -1782,9 +1781,6 @@ static NTSTATUS key_import_pair( struct algorithm *alg, const WCHAR *type, BCRYP key_destroy( key ); return status; } - - *ret_key = key; - return STATUS_SUCCESS; } else if (!wcscmp( type, BCRYPT_RSAPUBLIC_BLOB )) { @@ -1807,9 +1803,6 @@ static NTSTATUS key_import_pair( struct algorithm *alg, const WCHAR *type, BCRYP key_destroy( key ); return status; } - - *ret_key = key; - return STATUS_SUCCESS; } else if (!wcscmp( type, BCRYPT_RSAPRIVATE_BLOB ) || !wcscmp( type, BCRYPT_RSAFULLPRIVATE_BLOB )) { @@ -1829,9 +1822,6 @@ static NTSTATUS key_import_pair( struct algorithm *alg, const WCHAR *type, BCRYP key_destroy( key ); return status; } - - *ret_key = key; - return STATUS_SUCCESS; } else if (!wcscmp( type, LEGACY_RSAPRIVATE_BLOB )) { @@ -1864,9 +1854,6 @@ static NTSTATUS key_import_pair( struct algorithm *alg, const WCHAR *type, BCRYP key_destroy( key ); return status; } - - *ret_key = key; - return STATUS_SUCCESS; } else if (!wcscmp( type, LEGACY_DSA_V2_PRIVATE_BLOB )) { @@ -1904,9 +1891,6 @@ static NTSTATUS key_import_pair( struct algorithm *alg, const WCHAR *type, BCRYP key_destroy( key ); return status; } - - *ret_key = key; - return STATUS_SUCCESS; } else if (!wcscmp( type, LEGACY_DSA_V2_PUBLIC_BLOB )) /* not supported on native */ { @@ -1940,9 +1924,6 @@ static NTSTATUS key_import_pair( struct algorithm *alg, const WCHAR *type, BCRYP key_destroy( key ); return status; } - - *ret_key = key; - return STATUS_SUCCESS; } else if (!wcscmp( type, BCRYPT_DH_PRIVATE_BLOB )) { @@ -1962,9 +1943,6 @@ static NTSTATUS key_import_pair( struct algorithm *alg, const WCHAR *type, BCRYP key_destroy( key ); return status; } - - *ret_key = key; - return STATUS_SUCCESS; } else if (!wcscmp( type, BCRYPT_DH_PUBLIC_BLOB )) { @@ -1984,13 +1962,19 @@ static NTSTATUS key_import_pair( struct algorithm *alg, const WCHAR *type, BCRYP key_destroy( key ); return status; } + } + else + { + FIXME( "unsupported key type %s\n", debugstr_w(type) ); + return STATUS_NOT_SUPPORTED; + } + if (!status) + { + key->u.a.flags |= KEY_FLAG_FINALIZED; *ret_key = key; - return STATUS_SUCCESS; } - - FIXME( "unsupported key type %s\n", debugstr_w(type) ); - return STATUS_NOT_SUPPORTED; + return status; } NTSTATUS WINAPI BCryptGenerateSymmetricKey( BCRYPT_ALG_HANDLE handle, BCRYPT_KEY_HANDLE *ret_handle, @@ -2031,11 +2015,14 @@ NTSTATUS WINAPI BCryptGenerateKeyPair( BCRYPT_ALG_HANDLE handle, BCRYPT_KEY_HAND NTSTATUS WINAPI BCryptFinalizeKeyPair( BCRYPT_KEY_HANDLE handle, ULONG flags ) { struct key *key = get_key_object( handle ); + NTSTATUS ret; TRACE( "%p, %#lx\n", key, flags ); - if (!key) return STATUS_INVALID_HANDLE; - return UNIX_CALL( key_asymmetric_generate, key ); + if (!key || key->u.a.flags & KEY_FLAG_FINALIZED) return STATUS_INVALID_HANDLE; + + if (!(ret = UNIX_CALL( key_asymmetric_generate, key ))) key->u.a.flags |= KEY_FLAG_FINALIZED; + return ret; } NTSTATUS WINAPI BCryptImportKey( BCRYPT_ALG_HANDLE handle, BCRYPT_KEY_HANDLE decrypt_key, const WCHAR *type, diff --git a/dlls/bcrypt/gnutls.c b/dlls/bcrypt/gnutls.c index 4183aad6cec..171895e94f5 100644 --- a/dlls/bcrypt/gnutls.c +++ b/dlls/bcrypt/gnutls.c @@ -1725,6 +1725,8 @@ static NTSTATUS key_asymmetric_export( void *args ) struct key *key = params->key; unsigned flags = params->flags; + if (!(key->u.a.flags & KEY_FLAG_FINALIZED)) return STATUS_INVALID_HANDLE; + switch (key->alg_id) { case ALG_ID_ECDH_P256:

1 0

Hans Leidekker : bcrypt: Add helpers to create a public/private key pair.
by Alexandre Julliard 18 Jan '24

18 Jan '24

Module: wine Branch: master Commit: 4becbefe5d4b13f73aa0aa107fa7241acddfdc18 URL: https://gitlab.winehq.org/wine/wine/-/commit/4becbefe5d4b13f73aa0aa107fa724… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Thu Dec 7 10:00:42 2023 +0100 bcrypt: Add helpers to create a public/private key pair. --- dlls/bcrypt/gnutls.c | 67 ++++++++++++++++++++++++++++++++++------------------ 1 file changed, 44 insertions(+), 23 deletions(-) diff --git a/dlls/bcrypt/gnutls.c b/dlls/bcrypt/gnutls.c index 171895e94f5..dbcfd723687 100644 --- a/dlls/bcrypt/gnutls.c +++ b/dlls/bcrypt/gnutls.c @@ -1005,6 +1005,48 @@ done: return status; } +static gnutls_privkey_t create_privkey( gnutls_pk_algorithm_t pk_alg, unsigned int bitlen ) +{ + gnutls_privkey_t privkey; + int ret; + + if ((ret = pgnutls_privkey_init( &privkey ))) + { + pgnutls_perror( ret ); + return NULL; + } + + if ((ret = pgnutls_privkey_generate( privkey, pk_alg, bitlen, 0 ))) + { + pgnutls_perror( ret ); + pgnutls_privkey_deinit( privkey ); + return NULL; + } + + return privkey; +} + +static gnutls_pubkey_t create_pubkey_from_privkey( gnutls_privkey_t privkey ) +{ + gnutls_pubkey_t pubkey; + int ret; + + if ((ret = pgnutls_pubkey_init( &pubkey ))) + { + pgnutls_perror( ret ); + return NULL; + } + + if ((ret = pgnutls_pubkey_import_privkey( pubkey, privkey, 0, 0 ))) + { + pgnutls_perror( ret ); + pgnutls_pubkey_deinit( pubkey ); + return NULL; + } + + return pubkey; +} + static NTSTATUS key_asymmetric_generate( void *args ) { struct key *key = args; @@ -1012,7 +1054,6 @@ static NTSTATUS key_asymmetric_generate( void *args ) gnutls_privkey_t privkey; gnutls_pubkey_t pubkey; unsigned int bitlen; - int ret; if (!libgnutls_handle) return STATUS_INTERNAL_ERROR; if (key_data(key)->a.privkey) return STATUS_INVALID_HANDLE; @@ -1052,30 +1093,10 @@ static NTSTATUS key_asymmetric_generate( void *args ) return STATUS_NOT_SUPPORTED; } - if ((ret = pgnutls_privkey_init( &privkey ))) - { - pgnutls_perror( ret ); - return STATUS_INTERNAL_ERROR; - } - if ((ret = pgnutls_pubkey_init( &pubkey ))) - { - pgnutls_perror( ret ); - pgnutls_privkey_deinit( privkey ); - return STATUS_INTERNAL_ERROR; - } - - if ((ret = pgnutls_privkey_generate( privkey, pk_alg, bitlen, 0 ))) - { - pgnutls_perror( ret ); - pgnutls_privkey_deinit( privkey ); - pgnutls_pubkey_deinit( pubkey ); - return STATUS_INTERNAL_ERROR; - } - if ((ret = pgnutls_pubkey_import_privkey( pubkey, privkey, 0, 0 ))) + if (!(privkey = create_privkey( pk_alg, bitlen ))) return STATUS_INTERNAL_ERROR; + if (!(pubkey = create_pubkey_from_privkey( privkey ))) { - pgnutls_perror( ret ); pgnutls_privkey_deinit( privkey ); - pgnutls_pubkey_deinit( pubkey ); return STATUS_INTERNAL_ERROR; }

1 0

Hans Leidekker : bcrypt: Make DH blob size validation more strict in key_import_pair().
by Alexandre Julliard 18 Jan '24

18 Jan '24

Module: wine Branch: master Commit: 3ef933c08f44e8097f9abefede17f9171431a1c2 URL: https://gitlab.winehq.org/wine/wine/-/commit/3ef933c08f44e8097f9abefede17f9… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Thu Dec 7 12:30:08 2023 +0100 bcrypt: Make DH blob size validation more strict in key_import_pair(). --- dlls/bcrypt/bcrypt_main.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/bcrypt/bcrypt_main.c b/dlls/bcrypt/bcrypt_main.c index 131fc2c2453..0f2d94a418f 100644 --- a/dlls/bcrypt/bcrypt_main.c +++ b/dlls/bcrypt/bcrypt_main.c @@ -1929,7 +1929,7 @@ static NTSTATUS key_import_pair( struct algorithm *alg, const WCHAR *type, BCRYP { BCRYPT_DH_KEY_BLOB *dh_blob = (BCRYPT_DH_KEY_BLOB *)input; - if (input_len < sizeof(*dh_blob)) return STATUS_INVALID_PARAMETER; + if (input_len != sizeof(*dh_blob) + dh_blob->cbKey * 4) return STATUS_INVALID_PARAMETER; if (alg->id != ALG_ID_DH || dh_blob->dwMagic != BCRYPT_DH_PRIVATE_MAGIC) return STATUS_NOT_SUPPORTED; @@ -1948,7 +1948,7 @@ static NTSTATUS key_import_pair( struct algorithm *alg, const WCHAR *type, BCRYP { BCRYPT_DH_KEY_BLOB *dh_blob = (BCRYPT_DH_KEY_BLOB *)input; - if (input_len < sizeof(*dh_blob)) return STATUS_INVALID_PARAMETER; + if (input_len != sizeof(*dh_blob) + dh_blob->cbKey * 3) return STATUS_INVALID_PARAMETER; if (alg->id != ALG_ID_DH || dh_blob->dwMagic != BCRYPT_DH_PUBLIC_MAGIC) return STATUS_NOT_SUPPORTED;

1 0