What the developer wrote
Authored by Tobin C. Harding
hashes: fmt skip a bunch of stuff
Add attribute to a bunch of stuff recently introduced in #5888.
What changed, and why it matters
This commit only adds `#[rustfmt::skip]` annotations to three places in a SHA-256 cryptographic implementation file. These annotations tell Rust's automatic code formatter to leave those blocks alone. There is no change to program logic, constants, memory handling, or any behavior that could affect security.
No action needed; this is a non-functional formatting change.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The diff adds #[rustfmt::skip] to an INIT constant array, a MIDS constant array, and a destructuring let binding in hashes/src/sha256/crypto.rs. The values and surrounding code are unchanged. This is purely a code-style/formatting commit with no functional, cryptographic, or security-relevant modifications.
Changed components
hashes/src/sha256/crypto.rsInspect captured patch +3 / −0
diff --git a/hashes/src/sha256/crypto.rs b/hashes/src/sha256/crypto.rs
index 9104856e..efe03045 100644
--- a/hashes/src/sha256/crypto.rs
+++ b/hashes/src/sha256/crypto.rs
@@ -835,6 +835,7 @@ impl HashEngine {
use core::arch::aarch64::vst1q_u8;
// initial state
+ #[rustfmt::skip]
const INIT: [u32; 8] = [
0x6a09e667, 0xbb67ae85, 0x3c6ef372, 0xa54ff53a,
0x510e527f, 0x9b05688c, 0x1f83d9ab, 0x5be0cd19
@@ -862,6 +863,7 @@ impl HashEngine {
];
// Precomputed W[i] + K[i] for the 2nd transform (padding block).
+ #[rustfmt::skip]
const MIDS: [u32; 64] = [
0xc28a2f98, 0x71374491, 0xb5c0fbcf, 0xe9b5dba5,
0x3956c25b, 0x59f111f1, 0x923f82a4, 0xab1c5ed5,
@@ -900,6 +902,7 @@ impl HashEngine {
let (mut state0_a, mut state0_b, mut state1_a, mut state1_b);
let (abcd_save_a, abcd_save_b, efgh_save_a, efgh_save_b);
+ #[rustfmt::skip]
let (mut msg0_a, mut msg0_b, mut msg1_a, mut msg1_b, mut msg2_a, mut msg2_b, mut msg3_a, mut msg3_b);
let (mut tmp0_a, mut tmp0_b, mut tmp2_a, mut tmp2_b, mut tmp);
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.