Eric Pouech : dssenh: Use SecureZeroMemory to clear magic fields.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 2367415d323e497e16ece6e0ca753c7de1baeeae
URL:    https://gitlab.winehq.org/wine/wine/-/commit/2367415d323e497e16ece6e0ca753c7...

Author: Eric Pouech <epouech(a)codeweavers.com>
Date:   Tue Nov  7 19:10:13 2023 +0100

dssenh: Use SecureZeroMemory to clear magic fields.

Signed-off-by: Eric Pouech <epouech(a)codeweavers.com>

---

 dlls/dssenh/main.c | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/dlls/dssenh/main.c b/dlls/dssenh/main.c
index 887319d0a9d..6772d062fc5 100644
--- a/dlls/dssenh/main.c
+++ b/dlls/dssenh/main.c
@@ -162,7 +162,8 @@ static void destroy_key( struct key *key )
 {
     if (!key) return;
     BCryptDestroyKey( key->handle );
-    key->magic = 0;
+    /* Ensure compiler doesn't optimize out the assignment with 0. */
+    SecureZeroMemory( &key->magic, sizeof(key->magic) );
     free( key );
 }
 
@@ -213,7 +214,8 @@ static void destroy_container( struct container *container )
     if (!container) return;
     destroy_key( container->exch_key );
     destroy_key( container->sign_key );
-    container->magic = 0;
+    /* Ensure compiler doesn't optimize out the assignment with 0. */
+    SecureZeroMemory( &container->magic, sizeof(container->magic) );
     free( container );
 }
 
@@ -821,7 +823,8 @@ static void destroy_hash( struct hash *hash )
 {
     if (!hash) return;
     BCryptDestroyHash( hash->handle );
-    hash->magic = 0;
+    /* Ensure compiler doesn't optimize out the assignment with 0. */
+    SecureZeroMemory( &hash->magic, sizeof(hash->magic) );
     free( hash );
 }