What the developer wrote
Authored by Jamil Lambert, PhD
Make siphash24::State private
The struct is not intended to be used directly by users of the library.
Make it private to conform to the rust API guidelines C-HIDDEN.
What changed, and why it matters
This commit simply hides an internal data structure (called State) from outside users of the library. It is an API-cleanup change, not a security fix. There is no patch for a vulnerability and no indication of any exploit or unsafe behavior.
No security action needed. Treat as a normal API-visibility cleanup commit.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The change removes the pub keyword from siphash24::State, making it crate-private. The struct represents the internal state of the SipHash-2-4 hash engine and was not intended for public use. This aligns with Rust API guideline C-HIDDEN (hide implementation details). The diff is one line and contains no logic changes, no memory-safety fixes, no cryptographic corrections, and no input-validation changes.
Changed components
hashes/src/siphash24/mod.rsInspect captured patch +1 / −1
diff --git a/hashes/src/siphash24/mod.rs b/hashes/src/siphash24/mod.rs
index 3f08e5d7..97eca32f 100644
--- a/hashes/src/siphash24/mod.rs
+++ b/hashes/src/siphash24/mod.rs
@@ -107,7 +107,7 @@ impl Hash {
/// Internal state of the [`HashEngine`].
#[derive(Debug, Clone)]
-pub struct State {
+struct State {
// v0, v2 and v1, v3 show up in pairs in the algorithm,
// and simd implementations of SipHash will use vectors
// of v02 and v13. By placing them in this order in the struct,
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.