Effectively zero TLS section on thread cleanup

Signed-off-by: Amaury Chamayou <[email protected]>
samuel-lee-msft · Nov 23, 2021 · 236e3a0 · 236e3a0
1 parent 264c967
commit 236e3a0
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/enclave/core/sgx/threadlocal.c b/enclave/core/sgx/threadlocal.c
@@ -420,7 +420,11 @@ oe_result_t oe_thread_local_cleanup(oe_sgx_td_t* td)
     // (i.e., tls_start is NULL when tdata and tbss are zero).
     oe_allocator_thread_cleanup();
     if (tls_start)
-        oe_memset_s(tls_start, (uint64_t)(fs - tls_start), 0, 0);
+        oe_memset_s(
+            tls_start,
+            (uint64_t)(fs - tls_start),
+            0,
+            (uint64_t)(fs - tls_start));
 
     return OE_OK;
 }