Hans Leidekker : bcrypt: Make sure key_asymmetric_derive_key() returns correct size.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: dfbbfd6f240992de21bec13200cca54ccb815f67
URL:    https://gitlab.winehq.org/wine/wine/-/commit/dfbbfd6f240992de21bec13200cca54...

Author: Hans Leidekker 
Date:   Fri Dec  8 12:13:12 2023 +0100

bcrypt: Make sure key_asymmetric_derive_key() returns correct size.

---

dlls/bcrypt/gnutls.c | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

diff --git a/dlls/bcrypt/gnutls.c b/dlls/bcrypt/gnutls.c
index 3592c685c56..1e5a81d9366 100644
--- a/dlls/bcrypt/gnutls.c
+++ b/dlls/bcrypt/gnutls.c
@@ -2671,6 +2671,7 @@ static NTSTATUS key_asymmetric_derive_key( void *args )
 {
     const struct key_asymmetric_derive_key_params *params = args;
     gnutls_datum_t s;
+    NTSTATUS status = STATUS_SUCCESS;
     int ret;

if ((ret = pgnutls_privkey_derive_secret( key_data(params->privkey)->a.privkey,
@@ -2680,15 +2681,15 @@ static NTSTATUS key_asymmetric_derive_key( void *args )
         return STATUS_INTERNAL_ERROR;
     }

-    if (!params->output) *params->ret_len = s.size;
-    else
+    *params->ret_len = EXPORT_SIZE( s, params->privkey->u.a.bitlen / 8, 1 );
+    if (params->output)
     {
-        *params->ret_len = min( params->output_len, s.size );
-        memcpy( params->output, s.data, *params->ret_len );
+        if (params->output_len < *params->ret_len) status = STATUS_BUFFER_TOO_SMALL;
+        else export_gnutls_datum( params->output, *params->ret_len, &s, 1 );
     }

free( s.data );
-    return STATUS_SUCCESS;
+    return status;
 }

const unixlib_entry_t __wine_unix_call_funcs[] =