[PATCH v2 0/1] MR9220: bcrypt: key_export_ecc_public - Add missing break

[Alistair Leslie-Hughes (＠alesliehughes)]

--
  v2: bcrypt: key_export_ecc_public - Add missing break (Coverity)
 
https://gitlab.winehq.org/wine/wine/-/merge_requests/9220

[PATCH v2 1/1] bcrypt: key_export_ecc_public - Add missing break (Coverity)

[Alistair Leslie-Hughes]

From: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com>

---
 dlls/bcrypt/gnutls.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/dlls/bcrypt/gnutls.c b/dlls/bcrypt/gnutls.c
index 4ff8cff280f..e114003a3ee 100644
--- a/dlls/bcrypt/gnutls.c
+++ b/dlls/bcrypt/gnutls.c
@@ -879,6 +879,7 @@ static NTSTATUS key_export_ecc_public( struct key *key, UCHAR *buf, ULONG len, U
             FIXME( "unsupported curve %u\n", key->u.a.curve_id );
             return STATUS_NOT_IMPLEMENTED;
         }
+        break;
 
     case ALG_ID_ECDSA_P256:
         magic = BCRYPT_ECDSA_PUBLIC_P256_MAGIC;
-- 
GitLab

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

[Hans Leidekker (＠hans)]

Please fix the ALG_ID_ECDH case too, as well as key_export_ecc().

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/9220#note_119033