core: Add comment on ZeroizeOnDrop

Resolves #244
dusk-network · Sep 18, 2024 · de5e503 · de5e503
1 parent 8d70fac
commit de5e503
Showing 1 changed file with 5 additions and 0 deletions.
diff --git a/core/src/keys/secret.rs b/core/src/keys/secret.rs
@@ -42,6 +42,11 @@ use subtle::{Choice, ConstantTimeEq};
 ///
 /// sk.zeroize();
 /// ```
+///
+/// # Note
+/// Implementing `ZeroizeOnDrop` seems like an excellent way to lift the burden
+/// of manually zeroizing after use off the user, but unfortunately it doesn't
+/// delete the memory reliably. See #244
 #[derive(Clone, Eq, Debug, Zeroize)]
 #[cfg_attr(
     feature = "rkyv-impl",